test/utils/github_spec: filter further to fix test failure

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Thanks @cho-m!


DOC: Document code contributors on website

This gets us started on keeping track of code contributions to MNE, the first part of #11605.

Create a "Contributors" page, and link to it from the main landing page

Add a "Code Credit" heading on that page, mentioning eventually we'll add other stuff.

Populate it with code credit based on lines changed in commits in MNE-Python:

Overall changes to MNE-Python
submodule-by-submodule credit

We have a lot of contributors and a lot of modules, so it's tough to get all the information in there. There are lots of potential ways to do this, what to include, how to show it -- and thus a lot of potential bike shedding, so we should probably just try to make sure what's here is an improvement over the author lists we currently use. Let's not shoot for perfection here but just "good enough"!
Still some stuff to do:

[x] Manually run and create the page
[x] Converge on the idea it's actually better
[x] Make the page rebuild, likely by running on CircleCI (or opt-in locally), perhaps in a directive rather than writing an RST file
[x] Ensure that the line counts are actually correct / reasonable, considering the squash+merge (new) vs standard merge (old) ways of doing things. I think it's probably not correct yet, but I'll think on it. (I started writing something with the gh command-line tool, but querying every merged PR and get files changed and lines changed in each file takes forever. Getting the merge commit -- so that we could then use the git history to figure things out -- sometimes returns commit hashes that don't exist in our codebase. So it's not trivial.)
[ ] Get reviews
[ ] Unzip prs.zip
[ ] Merge
[ ] Remove author names from mne/ files in separate PR

Frontpage

Links to

https://output.circle-artifacts.com/output/job/1d4da1a2-6f08-4643-b1f9-16582ba5a3df/artifacts/0/html/credit.html

looks nice !

Message ID: @.***>

Okay @drammock I think I'm ready for review here!
What I came up with was:

Having separate doc/sphinxext/prs/<pr_num>.json files, one per PR, with the info we need to come up with code credit. Eventually if we want we can add other stuff like PR comment counts to those, and maybe issue comment counts in issues/<issue_num>.json or whatever. So I think the format is pretty flexible.
Getting the change info is done using PyGithub in tools/dev/update_credit_json.py so we stay Pythonic. Only downside is that the initial gathering of data is quite slow. But incremental monthly updates should be fast, and I've added a GitHub action that will eventually run monthly to do it for us. This ultimately seemed preferable over using, say, git diff locally because PyGithub gives us easier access to more info and future extensibility, and also the files-changed-in-PR is something we get directly there, and something that was really quite difficult using git locally (for me at least!).
Mapping the change counts to a rst file is done in tools/dev/update_credit_rst.py, and consumes the .json files. This does all sorts of accounting stuff, like mapping between author names, grouping into module types, etc. And then it takes that info and decides how to present it, which is as author badges embeddeding in module cards. Lots of opinionated choices here that we can tweak (maybe mostly in follow-up PRs?) but it's hopefully better than what we have in main already.

For ease of reviewing, I zipped the json files to a single zip, as I assumed that without doing so the Files page on this PR was going to be unusable. So once we agree this is ready for merge, I'll go back to individual .json files, make sure it still looks okay, and merge.

Nice, I like it!
One nitpick - if I convert to greyscale, the two colors for the top 10% contributors and rest are not distinguishable anymore:

Can we pick colors that are more easily distinguishable (assuming this would be an easy fix)?

Okay changed to something with more contrast that looks okay in grayscale, will merge once I manually check by eye it rendered correctly!


Enhancement: Change default mouse actions

Default mouse action should be Drag.
Shift modifier could be used for selection

Take a look at what tools like Miro use.
Optionally, make these settings configurable.

Thank you @mohdali for your contributions!


modified government artifact function definition per 504

Fixes #504.
I went with the bare 'Domain' option rather than the others mentioned, but I'm willing to change it.

@gregfowlerphd See comment on issue ticket. Since we are moving into some releases, I'll close for now, but if we can work out new candidate wording on the issue, would look forward to seeing this resolved.


Fully customizable sidebar

There have been requests for putting the weather/calendar/battery at different positions in the sidebar. It could be possible to change the settings such that there are three different items in the sidebar, which could be rearranged and turned on/off at will.

Completed in 73ddff7ca7e014bf20d97baefc36cbc1c131c1a3


Switching tabs causes strange query plan rendering

Azure Data Studio Version: master

Steps to Reproduce:

Run a query with Explain
Switch tabs then switch back

Expected: Rendering is the same.
Actual: Arrows disappear from plan and a strange element is rendered.  I think this may be the missing arrows?

This is likely fixed in the new execution plan feature


(Feature request) Visualize color data inside Texture3D

hey @Looooong  - check out https://github.com/UoA-eResearch/DICOM_viewer/commit/cce182b9d47af348b7b7dea4d465c3c1da4e80e2 - I was able to tweak this shader to display color data with fairly minimal changes

Thanks for the reference. I didn't figure out the alpha calculation step :sweat_smile:


Windows build fails

            .-/+oossssoo+/-.               roalyr@roalyr-VirtualBox
        `:+ssssssssssssssssss+:`           ------------------------
      -+ssssssssssssssssssyyssss+-         OS: Ubuntu 18.04.6 LTS x86_64
    .ossssssssssssssssssdMMMNysssso.       Host: VirtualBox 1.2
   /ssssssssssshdmmNNmmyNMMMMhssssss/      Kernel: 5.4.0-122-generic
  +ssssssssshmydMMMMMMMNddddyssssssss+     Uptime: 2 days, 55 mins
 /sssssssshNMMMyhhyyyyhmNMMMNhssssssss/    Packages: 1766
.ssssssssdMMMNhsssssssssshNMMMdssssssss.   Shell: bash 4.4.20
+sssshhhyNMMNyssssssssssssyNMMMysssssss+   Resolution: 1920x977
ossyNMMMNyMMhsssssssssssssshmmmhssssssso   DE: Xfce
ossyNMMMNyMMhsssssssssssssshmmmhssssssso   WM: Xfwm4
+sssshhhyNMMNyssssssssssssyNMMMysssssss+   WM Theme: OneColor XFCE orange
.ssssssssdMMMNhsssssssssshNMMMdssssssss.   Theme: Flat-Remix-GTK-Orange-Light [GTK2]
 /sssssssshNMMMyhhyyyyhdNMMMNhssssssss/    Icons: Elementary-xfce [GTK2], Elementary
  +sssssssssdmydMMMMMMMMddddyssssssss+     Terminal: xfce4-terminal
   /ssssssssssshdmNNNNmyNMMMMhssssss/      Terminal Font: DejaVu Sans Mono 14
    .ossssssssssssssssssdMMMNysssso.       CPU: AMD 3020e with Radeon Graphics (1) @
      -+sssssssssssssssssyyyssss+-         GPU: VMware SVGA II Adapter
        `:+ssssssssssssssssss+:`           Memory: 1297MiB / 1983MiB
            .-/+oossssoo+/-.

roalyr@roalyr-VirtualBox /m/r/G/godot-for-3d-open-worlds> ./rebuild_windows.sh
i686-w64-mingw32-gcc (GCC) 7.3-win32 20180904
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

x86_64-w64-mingw32-gcc (GCC) 7.3-posix 20180904
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

tee: ./logs/scons_windows_tools_build.log: No such file or directory
scons: Reading SConscript files ...
Configuring for Windows: target=release_debug, bits=64
Using MinGW
Scons cache enabled... (path: './scons_cache_windows')
Checking for C header file mntent.h... (cached) no
scons: done reading SConscript files.
scons: Building targets ...
Compiling ==> platform/windows/os_windows.cpp
In file included from platform/windows/os_windows.h:46:0,
                 from platform/windows/os_windows.cpp:31:
./servers/visual/rasterizer.h:128:56: warning: 'RasterizerScene::InstanceBase::interpolation_method' is too small to hold all values of 'enum TransformInterpolator::Method'
   TransformInterpolator::Method interpolation_method : 3;
                                                        ^
In file included from ./scene/main/viewport.h:35:0,
                 from ./servers/arvr/arvr_interface.h:36,
                 from ./servers/visual/visual_server_viewport.h:36,
                 from ./servers/visual/visual_server_canvas.h:35,
                 from ./servers/visual/visual_server_raster.h:37,
                 from platform/windows/os_windows.cpp:45:
./scene/main/node.h:138:26: warning: 'Node::Data::pause_mode' is too small to hold all values of 'enum Node::PauseMode'
   PauseMode pause_mode : 2;
                          ^
./scene/main/node.h:139:57: warning: 'Node::Data::physics_interpolation_mode' is too small to hold all values of 'enum Node::PhysicsInterpolationMode'
   PhysicsInterpolationMode physics_interpolation_mode : 2;
                                                         ^
In file included from ./servers/visual/visual_server_raster.h:39:0,
                 from platform/windows/os_windows.cpp:45:
./servers/visual/visual_server_scene.h:90:56: warning: 'VisualServerScene::Camera::interpolation_method' is too small to hold all values of 'enum TransformInterpolator::Method'
   TransformInterpolator::Method interpolation_method : 3;
                                                        ^
platform/windows/os_windows.cpp: In constructor 'OS_Windows::OS_Windows(HINSTANCE)':
platform/windows/os_windows.cpp:4009:13: error: 'ENABLE_VIRTUAL_TERMINAL_PROCESSING' was not declared in this scope
  outMode |= ENABLE_VIRTUAL_TERMINAL_PROCESSING;
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
platform/windows/os_windows.cpp:4009:13: note: suggested alternative: 'MM_SPECTRUM_SIGNAL_PROCESSING'
  outMode |= ENABLE_VIRTUAL_TERMINAL_PROCESSING;
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
             MM_SPECTRUM_SIGNAL_PROCESSING
scons: *** [platform/windows/os_windows.windows.opt.tools.64.o] Error 1
scons: building terminated because of errors.
[Time elapsed: 00:00:36.268]
tee: ./logs/scons_windows_debug_build.log: No such file or directory
scons: Reading SConscript files ...
Configuring for Windows: target=release_debug, bits=64
Using MinGW
Scons cache enabled... (path: './scons_cache_windows')
Checking for C header file mntent.h... (cached) no
scons: done reading SConscript files.
scons: Building targets ...
Compiling ==> platform/windows/os_windows.cpp
In file included from platform/windows/os_windows.h:46:0,
                 from platform/windows/os_windows.cpp:31:
./servers/visual/rasterizer.h:128:56: warning: 'RasterizerScene::InstanceBase::interpolation_method' is too small to hold all values of 'enum TransformInterpolator::Method'
   TransformInterpolator::Method interpolation_method : 3;
                                                        ^
In file included from ./scene/main/viewport.h:35:0,
                 from ./servers/arvr/arvr_interface.h:36,
                 from ./servers/visual/visual_server_viewport.h:36,
                 from ./servers/visual/visual_server_canvas.h:35,
                 from ./servers/visual/visual_server_raster.h:37,
                 from platform/windows/os_windows.cpp:45:
./scene/main/node.h:138:26: warning: 'Node::Data::pause_mode' is too small to hold all values of 'enum Node::PauseMode'
   PauseMode pause_mode : 2;
                          ^
./scene/main/node.h:139:57: warning: 'Node::Data::physics_interpolation_mode' is too small to hold all values of 'enum Node::PhysicsInterpolationMode'
   PhysicsInterpolationMode physics_interpolation_mode : 2;
                                                         ^
In file included from ./servers/visual/visual_server_raster.h:39:0,
                 from platform/windows/os_windows.cpp:45:
./servers/visual/visual_server_scene.h:90:56: warning: 'VisualServerScene::Camera::interpolation_method' is too small to hold all values of 'enum TransformInterpolator::Method'
   TransformInterpolator::Method interpolation_method : 3;
                                                        ^
platform/windows/os_windows.cpp: In constructor 'OS_Windows::OS_Windows(HINSTANCE)':
platform/windows/os_windows.cpp:4009:13: error: 'ENABLE_VIRTUAL_TERMINAL_PROCESSING' was not declared in this scope
  outMode |= ENABLE_VIRTUAL_TERMINAL_PROCESSING;
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
platform/windows/os_windows.cpp:4009:13: note: suggested alternative: 'MM_SPECTRUM_SIGNAL_PROCESSING'
  outMode |= ENABLE_VIRTUAL_TERMINAL_PROCESSING;
             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
             MM_SPECTRUM_SIGNAL_PROCESSING
scons: *** [platform/windows/os_windows.windows.opt.debug.64.o] Error 1
scons: building terminated because of errors.
[Time elapsed: 00:00:26.282]
strip: './bin/godot.windows.*': No such file

https://github.com/roalyr/godot-for-3d-open-worlds/commit/7f560a2833ff78ce337e9fcbdda636644d499cee

@akien-mga


Local RHEL Image Builder for Qemu

Description of changes:
This PR adds a make target to build a RHEL based qcow2 image for kvm/qemu locally. Also adds readme instructions for setting up the builder machine, and running the image build process.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test imagebuilder-presubmit


Circular Progress Bar for Calorie Tracker

Implement a Circular Progress bar for Calorie and Protein progress for the user on the calorie tracker page

Pivot to Django's progress bar implementation, ensuring compatibility with the UI layout.


RuntimeError: language "de" is not supported by the espeak backend

I try to train a new mulitilingual model, it contains three languages(english, german and spanish),  I got an error when trying to prepare_fastspeech_corpus for these three languages:
RuntimeError: language "de" is not supported by the espeak backend
but when i Individual run $python Preprocessing/TextFrontend.py, it works well !
My phonemizer version is :
$phonemize --version
phonemizer-3.2.1
available backends: espeak-ng-1.51, segments-2.2.1
uninstalled backends: espeak-mbrola, festival
My FastSpeech2_MetaCheckpoint.py:
    english_datasets = list()
    german_datasets = list()
    spanish_datasets = list()

    english_datasets.append(prepare_fastspeech_corpus(transcript_dict=build_path_to_transcript_dict_libritts(),
                                                      corpus_dir=os.path.join(PREPROCESSING_DIR,"libri_train_clean_100"),
                                                      lang="en"))

    german_datasets.append(prepare_fastspeech_corpus(transcript_dict=build_path_to_transcript_dict_mailabs_de(),
                                                     corpus_dir=os.path.join(PREPROCESSING_DIR, "deu_mailabs"),
                                                     lang="de"))

    spanish_datasets.append(prepare_fastspeech_corpus(transcript_dict=build_path_to_transcript_dict_mailabs_esp(),
                                                      corpus_dir=os.path.join(PREPROCESSING_DIR, "esp_mailabs"),
                                                      lang="es"))

My build_path_to_transcript function:

PS:each line of the wavs file is the path of the audio and the trans file is the corresponding transcript

def build_path_to_transcript_dict_mailabs_de():
    root = f"IMS-Toucan/eng-deu-esp/deu"
    path_to_transcript = dict()
    with open(os.path.join(root, "trans"), "r", encoding="utf8") as f1:
        text_lookup = f1.read().split("\n")
    with open(os.path.join(root, "wavs"), "r", encoding="utf8") as f2:
        wav_lookup = f2.read().split("\n")
    for t, w in zip(text_lookup, wav_lookup):
        path_to_transcript[w] = t

    return limit_to_n(path_to_transcript)

Can you give me some suggestions? Thank you very much!

It seems you haven't added the language you are using in TextFrontend.py
https://github.com/DigitalPhonetics/IMS-Toucan/issues/45#issuecomment-1283163131

All of the languaghes you are using (en, de and es) are already included in the TextFrontend, so that should not be a problem.
Someone else has in the past reported that they got this error, even though it worked when they ran the tests in the text frontend on their own. But I never heard back from them again, so I'm not sure if they solved it or they gave up. Unfortunately I cannot reproduce the problem, so I cannot really help. I tried the versions that you specified and even used the mailabs data, but it worked for me under both Windows and Linux. It might be something strange like the order of the imports or so, because the phonemizer library is only an interface to espeak and does not actually carry out the functionality of espeak. So maybe there is a weird interaction between multiprocessing and espeak being executed in the background. If you find a solution, please let me know.

All of the languaghes you are using (en, de and es) are already included in the TextFrontend, so that should not be a problem.
Someone else has in the past reported that they got this error, even though it worked when they ran the tests in the text frontend on their own. But I never heard back from them again, so I'm not sure if they solved it or they gave up. Unfortunately I cannot reproduce the problem, so I cannot really help. I tried the versions that you specified and even used the mailabs data, but it worked for me under both Windows and Linux. It might be something strange like the order of the imports or so, because the phonemizer library is only an interface to espeak and does not actually carry out the functionality of espeak. So maybe there is a weird interaction between multiprocessing and espeak being executed in the background. If you find a solution, please let me know.

Thank you for your response. I discovered another person who had the same issue(https://github.com/espeak-ng/espeak-ng/issues/1580) and who fixed it by switching from Ubuntu to Windows 11. The problem persists despite my attempts to reinstall the espeak package and establish a fresh Python environment on my Linux system, therefore I want to do some experiments using the Google Colab. Once more, I appreciate your response.


gutenberg block work flow?

is there a recommended work flow when working with gutenberg blocks?
Is it better to render individual block assets (block.js/block.css) and enqueue them  in register_block_type or just have it all included in one main.js?

Hi,
I will create a workflow guide when I get some time. The concept is to think gutenberg blocks as any react app. So have one main.js file enqueued through gutenberg hooks. That main.js file would instantiate your blocks as needed.

Hi @swashata, just wondering if you've had time to create a guide for your Gutenberg workflow yet. I have a fairly well working method currently, but I'd love to have a more "official" way to hook things up. I'm currently running into issues trying to take advantage of the 5.8 changes to how block assets can be more efficiently loaded. Thanks!

Having some direction on this would be really great.
I've been using wpack for the frontend build so I thought I'd give it a go with the bundle for Gutenberg blocks as well. It almost works, but something's not right.
Here's my setup:
wpackio.project.js
files: [
    {
      name: 'blocks',
      entry: {
        main: ['./src/js/blocks.js'],
      },
    },
  ],

That blocks.js entrypoint calls the individual blocks:
import '../../blocks/test';

The entry point for each block is simply running the registerBlockType function passing the barebones for a custom block:
import { registerBlockType } from '@wordpress/blocks';
import './styles.scss';

import metadata from './block.json';
const { name, title } = metadata;

const blockData = { ...metadata };

export const settings = {
  ...blockData,
  edit(props) {
    return (

Edit test block.

    );
  },
  save(props) {
    return (

Save test block

    );
  },
};

registerBlockType(name, settings);

With this setup, any edits to the edit method inside the settings object causes a full reload. However if I abstract the edit property to another file, as typically you'll want to do for organization, you'll end up with something like:
import { registerBlockType } from '@wordpress/blocks';
import './styles.scss';

import metadata from './block.json';
const { name, title } = metadata;

import edit from './edit';

const blockData = { ...metadata };

export const settings = {
  ...blockData,
  edit,
  save(props) {
    return (

Save test block

    );
  },
};

registerBlockType(name, settings);

In this scenario edit is exporting a standard React component:
import { Panel, PanelBody, TextControl } from '@wordpress/components';

const Edit = (props) => {
  const { attributes, setAttributes } = props;
  const { gwm_test } = attributes;
  // console.log(attributes);
  return (

Edit Test Block..

      <Panel>
        <PanelBody title={'example meta box'} icon="admin-plugins">
          <TextControl
            label={'Example Meta'}
            help={'This is an example meta field.'}
            onChange={(text) => setAttributes({ gwm_test: text })}
            value={gwm_test}
          />
        </PanelBody>
      </Panel>

  );
};

export default Edit;

However, in this scenario any changes to edit.js produce no effect in the browser, despite console showing some awareness to the changes: https://share.cleanshot.com/Eey5Fp
Had anyone successfully got this working?


problem with googletrans and 'the cat sat on the mat'

after installing with 'pip install googletrans' to the anaconda env for pytorch
the line:
sentences = ['The cat sat on the mat']
translations_fr = translator.translate(sentences, dest='fr')
throws an error:
File "myuser\anaconda3\envs\PyTc\lib\site-packages\googletrans\gtoken.py", line 62, in _update
code = self.RE_TKK.search(r.text).group(1).replace('var ', '')
AttributeError: 'NoneType' object has no attribute 'group'
how can i make this nice tool work?!

That error is currently under review, see the package's open issue. I'll open a PR.
--> Please try googletrans==3.1.0a0 instead. It's a temporary fix, but its effectiveness seems to depend on your local environment. Feedback is welcome!

I haven't tried nor tested it, but I've also encountered an alternative (for non-commmercial usage) called google_trans_new: https://github.com/lushan88a/google_trans_new


⚠ Bofrost.lu has degraded performance

In b225bf3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1370 ms

Resolved: Bofrost.lu performance has improved in fa9fb10 after 6 minutes.


ksmbd-tools: 3.4.5 -> 3.4.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cifsd-team/ksmbd-tools/releases.
meta.description for ksmbd-tools is: "Userspace utilities for the ksmbd kernel SMB server"
meta.homepage for ksmbd-tools is: "https://www.kernel.org/doc/html/latest/filesystems/cifs/ksmbd.html"
meta.changelog for ksmbd-tools is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.4.6 with grep in /nix/store/cbb30fwqhfi804qxj4c7p2a3hnx34654-ksmbd-tools-3.4.6
found 3.4.6 in filename of file in /nix/store/cbb30fwqhfi804qxj4c7p2a3hnx34654-ksmbd-tools-3.4.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ksmbd-tools

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cbb30fwqhfi804qxj4c7p2a3hnx34654-ksmbd-tools-3.4.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ksmbd-tools https://github.com/r-ryantm/nixpkgs/archive/4c65cfcf285e25a6d96925b0ec07b0457537bf59.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cbb30fwqhfi804qxj4c7p2a3hnx34654-ksmbd-tools-3.4.6
ls -la /nix/store/cbb30fwqhfi804qxj4c7p2a3hnx34654-ksmbd-tools-3.4.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ksmbd-tools

Maintainer pings
cc @elohmeier for testing.

Might want to switch to meson

@ofborg eval


Validations broken with remote migrations

When doing remote migrations, the schema is serialized. For models with validations, the RegExp is not exported and is lost.
RegExp needs to be converted to a string which JSON.stringify does not do.

Fixed in 1.0.3


[cxx-interop][5.9] fix windows ABI convention for methods returning trivial records indirectly

Fixes https://github.com/apple/swift/issues/67674
Explanation:
Windows MSVC ABI had additional problem with IRGen for C++ methods in Swift: Some methods could return loadable records indirectly, leading us to performing an emitToMemory call emission with wrong order of arguments between this, and the temporary allocated return value storage. This fixes the argument order in this case for the MSVC ABI (Itanium ABI is  already correct), and also ensures that we don't try to pass undef as the pointer to the indirect return value in such cases.

Scope: C++ interop, IRGen.
Risk: Low, the affected code changes only affect the MSVC windows target, and only affect C++ imported types, not C.
Testing: Unit tests, manual IR inspection.
Original PR: https://github.com/apple/swift/pull/67685

@swift-ci please test windows

@swift-ci please test

@swift-ci please test source compatibility


Check exit code of apt-install before proceeding

https://github.com/tfischer4765/mm-install/blob/a04dc38827d58eb029f09b3b1d44d13f486773eb/mm-install.sh#L252
got a weird error while apt was running and it crashed early. lots of the subsequent steps were left half-complete as a result.
Err:33 http://deb.debian.org/debian bookworm/main arm64 libcups2 arm64 2.4.2-3+deb12u5
  404  Not Found [IP: <IP_ADDRESS> 80]
  <snip>
Get:97 http://deb.debian.org/debian bookworm/main arm64 libegl1 arm64 1.6.0-1 [33.1 kB]
Fetched 74.4 MB in 10s (7,135 kB/s)
E: Failed to fetch http://deb.debian.org/debian/pool/main/c/cups/libcups2_2.4.2-3%2bdeb12u5_arm64.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
Created symlink /etc/systemd/system/multi-user.target.wants/xserver.service → /usr/local/share/mm-support/xserver.service.
Xserver has been installed and configured to run at startup

This should in theory already be happening. Unfortunately apt seems to return inconsistent and pretty useless return codes.
Note to self: check if there is a better way to discern between complete success and partial success


High Peak Memory with CUDAGraphTransform

Peak Memory is very high when CUDAGraphTransform is used.
# With CUDAGraphTransform - 27517.101568
# Without CUDAGraphTransform - 11917.129728

Example -
import torch
import thunder
import litgpt
from torch.testing import make_tensor
from functools import partial
from thunder.dynamo import ThunderCompiler
from thunder.transforms.cudagraph import CUDAGraphTransform

device = torch.device("cuda")

cfg = litgpt.Config.from_name("open_llama_3b", n_layer=10)
with device:
    make = partial(make_tensor, low=0, high=255, device=device, dtype=torch.long, requires_grad=False)
    shape = (1,) + (cfg.block_size,)

    x = make(shape)
    m = litgpt.GPT(cfg)

# m = thunder.jit(m)
m = thunder.jit(m, transforms=[CUDAGraphTransform()])

o = m(x)
o.sum().backward()

# With CUDAGraphTransform - 27517.101568
# Without CUDAGraphTransform - 11917.129728
print(torch.cuda.max_memory_allocated() / 1e6)

Tested with internal image dated 20241209 on RTX 6000 Ada.

This is likely an issue with the tag for marking parameters / buffers as fixed in memory (so they can be fed into the CUDAGraph directly rather than through a fixed-location buffer) not being correctly propagated.

So there are two things actually:

the input buffers in the backward. This will improve on that:

class MyCUDAGraphTransform(CUDAGraphTransform):
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self.outputs_from_forward = None

    def transform_trace_post_optimization(self, trace, **kwargs):
        if trace.siginfo().name == 'backward_fn':
            # todo: have a backward tag
            assert self.outputs_from_forward is not None, "called on backward without forward before"
            # make this more generic or have an utility?
            assert len(trace.bound_symbols[2].args) == 2 and trace.bound_symbols[2].args[0].name == 'saved_for_backward'
            assert trace.bound_symbols[8].sym.name == 'unpack_sequence' and trace.bound_symbols[8].args[0] is trace.bound_symbols[2].output[0]
            saved_for_backwards_unpacked = trace.bound_symbols[8].output
            assert len(saved_for_backwards_unpacked) == len(self.outputs_from_forward)
            for (_, is_static), p_bw in zip(self.outputs_from_forward, saved_for_backwards_unpacked):
                if is_static:
                    p_bw.tags.add(thunder.core.proxies.ProxyTag.STATIC_MEMORY_LOCATION)
            self.outputs_from_forward = None

        new_trace = super().transform_trace_post_optimization(trace, **kwargs)

        if thunder.core.trace.TraceTag.AUGMENTED_FORWARD in new_trace.tags:
            assert self.outputs_from_forward is None, "called on augmented forward twice without backward in between"
            # apparently, it is safer to go by name than assume we have the same proxies here. :(
            cudagraph_output_names = set()
            for bsym in new_trace.bound_symbols:
                if bsym.sym.name.startswith('CUDAGraph'):
                    for o in bsym.flat_proxy_outs:
                        cudagraph_output_names.add(o.name)
            saved_for_backward = thunder.core.vjp_utils.get_saved_for_backward_tensors(new_trace)
            self.outputs_from_forward = [(o.name, o.name in cudagraph_output_names or thunder.core.proxies.ProxyTag.STATIC_MEMORY_LOCATION in o.tags) for o in saved_for_backward]
        return new_trace

If we decide the sharing is the right thing, we should do that.
The other part is that the memory for the graphs are not currently shared.
I thought it would be as easy as initializing self.pool to None and then changing
https://github.com/Lightning-AI/lightning-thunder/blob/d3b22764def5acab1c27f9155e0c9f85a2f00cdf/thunder/transforms/cudagraph.py#L98-L100
to
        graph = torch.cuda.CUDAGraph()
        with torch.cuda.graph(graph, pool=self.pool, stream=stream):
            static_outputs = fn(*static_inputs)
        if self.pool is None:
          self.pool = graph.pool()

but this does not do it (and makes things worse instead?)...
I would highly appreciate any hint how this should work.


Allow binding to full keyboard

Resolves https://github.com/cjquines/qboard/issues/120
Does not resolve these points; the first should go in a separate issue:

Make the localStorage format backwards and forwards compatible
#114

I am not updating the styles for the buttons under the assumption that you'd want to do that. The important ones are button.key.modifier:hover and button.key.modifier.active:hover, but you might also wish to distinguish button.key.modifier from button.key.
Currently shows ? for the / key, which is slightly inaccurate. May wish to change. Also, I cannot override the ? behavior.
Commit 59adc93529e426644ad326ea33f611a474625ad0 is large and does a few very different things. I can break it up if desired
Potential improvement:
When the shift modifier is active, may wish to replace the symbols for "`", /, and 1...0 in the UI

Regarding layout, you may wish to

Get rid of "", replace with esc`
Use the freed spot "in the capslock location" to add the alt key, possibly moving around other modifier keys

Also may want to do something about the text Press ? to show or hide this screen. since this isn't necessarily true anymore.

tysm avi!

should we expand default keys to include left-handed bindings? unclear to me.
possible to make keys even narrower? not sure about this
probably update the default keybindings in readme

should we expand default keys to include left-handed bindings? unclear to me.

i'm good either way. now that they don't necessarily need to be mirrored, we can actually engineer the bindings on the right half. i think that's a good route

possible to make keys even narrower? not sure about this

this looks a bit odd to my eyes, but i'm good with shrinking both dimensions (and actually prefer it that way, so that the header text actually feels like a header. right now, the keys have more prominence over the text)

if we do that, that would increase density even more. might be worth it to unbold the keys or do something else to decrease visual density, like make the borders lighter

I agree, even now, the font weight of the actions for bound keys seems too much. i do think that's the only thing that needs to be reduced, though.

probably update the default keybindings in readme

sounds good

alright, how does this look like? it's the smallest i can think of without making the actual font size smaller which i dont really want to do :/
any thoughts on right half bindings? honestly im thinking of just mirroring it but im not sure. also now thinking of right half bindings, its possible that there are people who use , and .
one thing we could do is to move ctrl and alt below as another modifier row. then put the current text along the bottom along that row too, to the right of ctrl and alt.

i don't think the keys need to be this small. also, there is a collision here
 between text and the border

any thoughts on right half bindings? honestly im thinking of just mirroring it but im not sure. also now thinking of right half bindings, its possible that there are people who use , and .

L for line (unless it is needed for pagination), P for draw (paint), O for ellipse, I for either eraser (location-wise) or laser (people might also think "eye" which is a reasonable mnemonic)

one thing we could do is to move ctrl and alt below as another modifier row. then put the current text along the bottom along that row too, to the right of ctrl and alt.

i want to be mindful of height. on mobile, even our original 3 rows were too much to fit. five would make the bindings box practically unusable

Can button.key.modifier:hover and button.key.modifier.active:hover be made different? button.key.modifier.active is actually darker than button.key.modifier.active:hover

Can button.key.modifier:hover and button.key.modifier.active:hover be made different? button.key.modifier.active is actually darker than button.key.modifier.active:hover

oh hm yeah, can resolve this

Also, I cannot override the ? behavior.

interesting :/ uh ? is not ? but shift + / in keyboard js

Commit 59adc93 is large and does a few very different things. I can break it up if desired

would be great if you could split 59adc93529e426644ad326ea33f611a474625ad0

When the shift modifier is active, may wish to replace the symbols for "`", /, and 1...0 in the UI

not sure if this is desirable? think it makes more sense to just keep everything constant

Also, I cannot override the ? behavior.

interesting :/ uh ? is not ? but shift + / in keyboard js

i believe it works now, not sure what the issue was

would be great if you could split 59adc93

will do

When the shift modifier is active, may wish to replace the symbols for "`", /, and 1...0 in the UI

not sure if this is desirable? think it makes more sense to just keep everything constant

👍

see 3bf3a0c5aec5f5f82be781fc4cd290839f7c447e for some changes

Modulo discussion about this, the commit looks good :)

Current version (8c2aea8) is hosted at https://pihart.github.io/qboard/dev/135/

(do note that you pushed to your own branch)

yeah for some reason i dont have write access any more?

that's odd, try now? (and pls don't override my merge commit ty)

alright. one last bit of discussion an dthen we can merge mod conflicts xp


EnumRequest - Table 112 Sales Invoice Header - Field - Bal. Account Type

EnumRequest - Table 112 Sales Invoice Header - Field - Bal. Account Type

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


fix: [#4684] ESLint issues in botbuilder-dialogs-adaptive-runtime

Addresses # 4684
#minor
Description
This PR fixes the new ESLint errors and warnings that occur after upgrading the ESLint packages in the botbuilder-dialogs-adaptive-runtime project.
Specific Changes

Removed the 'only-warnings' plugin from botbuilder-dialogs-adaptive-runtime/eslint.config.cjs' file and the related dependency.
Applied the auto fix flag for most issues (trailing commas and removal of unnecessary eslint exceptions).
Applied manual fixes for new rules like 'import/no-named-as-default-member'.

Testing
The following images show the number of issues fixed in the library and the tests passing after the changes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 84.355%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.3%

Covered Lines:
20433

Relevant Lines:
23064

💛  - Coveralls

/promoted 4810


event list activity

needs custom text view input followed by list/recycler view of preset events (stored in res/arrays.xml possibly?)
usage we agreed on (to my recollection, correct me if wrong): the event is chosen either on a) a tap from the preset list or b) after the user dismisses the keyboard and the title is whatever is the active text inputted upon dismissal. the activity stack then auto pops back with the title now set.

Plan:

Add arrays.xml to res/values/

Add event array to the xml file with names of the pre-set events

Create a layout file for event list

A text view input, pre-populated with something like "Add custom event..."
A list view, to be populated with the values from the array

Add logic

Clicking a text view input will automatically bring up the keyboard. When the user dismisses the keyboard, return the entered value to the previous activity so that the title can be filled in
set the list view item's onClick to return the value (event name) of that item to the previous activity so that the title can be filled in


Use PLRAM as exchange memory

Currently we have a small exchange memory mapped both in the Microblaze and host address spaces to hold configuration data. The size of this memory is limiting, e.g. some users might need much more memory than others. To enable user-configurable and potentially very large exchange memory, we could read configuration from PLRAM. Performance implications must be considered.

From Vitis documentation:
The READ_LATENCY [...] attribute [...] sets the number of pipeline stages between memories cascaded in depth [...] A good rule of thumb is to pick a read latency of depth/2 + 2.
The depth is calculated from the total PLRAM memory depth divided by the primitive depth (URAM or BRAM). So for a 8K-deep memory in URAM, READ_LATENCY should be 3.

right now to go back and forth to the exchange meme implemented in BRAM
to enque 16 spares it takes 3430 clock cycles. 3 reads and 2 write x16 = 42 cc each access on average


mark-as-read icon not showing upon clicking the button

For some reasons, either using customized CSS or default plugin style, I still get that error

My setting is
  - mark-as-read:
      read_icon: simple-ticktick
      updated_icon: fontawesome-solid-circle-exclamation
      include:
        - /wiki/*

Cannot read properties of null error caused by this.readMarkIcon.cloneNode(true) line seems like readMarkIcon variable couldn't be assigned properly. Normally it should be assigned to the element with "mark-as-read-read-icon" id. Plugin adds this element by extending your markdown content.
Can you check if an element exist with that id by running document.getElementById("mark-as-read-read-icon"); command in console and share the output?

Probably not related to the error but I noticed that the regex you set under the "include" may be wrong. If you want to enable plugin on every path starts with /wiki/ you should set regex as /wiki/.* not /wiki/*.


Enhancement: Create a compiler for the SQL Server Bulk Operations Set Identities

This user story is an enhancement on the existing functionality introduced for SQL Server Bulk Operations.
Currently, the way we set the identity is through System.Reflection in which slower than a pre-compiled IL/Expression.
Ensure that this line of code is a pre-compiled IL/Expression.

This is now available at RepoDb.SqlServer.BulkOperations >= v1.0.11.


Automatically bump foraging_gui version

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

Increments the version of the foraging gui package with every merge to main. This means each week's release will have a unique tag. Right now we have 0.0.0, this will increment to 0.0.1, then 0.0.2 and so on. I propose we manually bump the minor version (eg 0.1.0) when we have updates that requirement manual intervention
Then in python foraging_gui.__version__ will reflect this version number. This will improve clarity of metadata and issue tracking
Logs this version in the GUI log, and in session metadata

What issues or discussions does this update address?

resolves #218

Describe the expected change in behavior from the perspective of the experimenter

none

Describe any manual update steps for task computers

none

Was this update tested in 446/447?

@hanhou @XX-Yin This is ready for review

Should we start to add release type to the commit message?
https://github.com/AllenNeuralDynamics/aind-library-template?tab=readme-ov-file#semantic-release

@hanhou I'm not sure how to integrate that into the automatic PRs for the weekly merges to main

You are calling tag.yml in aind-github-actions, which in turn uses<EMAIL_ADDRESS>(this line), right?
I guess how it works is that it will analyze the type prefixes of all commit messages inside a PR and determine the highest level of change:

If there are any commits with BREAKING CHANGE, it triggers a major version bump. (0.0.0 -> 1.0.0)
If there are any feat commits but no BREAKING CHANGE commits, it triggers a minor version bump. (0.0.0 -> 0.1.0)
If there are only fix commits and no feat or BREAKING CHANGE commits, it triggers a patch version bump. (0.0.0 -> 0.0.1)
If we don't use any type prefix before, it will always trigger a patch version bump by default. (0.0.0 -> 0.0.1)

So there are two options for us:
Option 1 (what you described): we don't use type prefix in the commit message as before. In this case, weekly PRs always bump the patch version number by default. When we
Option 2: We adopt the Angular commit message conventions. In this case, if we want to indicate the requirement of manual intervention, we can add "BREAKING CHANGE:" to the footer of the commit message to bump the major version number.
I think Option 2 may be a better practice in the long run.

@hanhou Thanks for explaining that! Option 2 seems to be great.

Do we need to make any changes to this PR? I think we just need to enforce standards on commit messages


neptune-tensorboard doesn't work with Tensorflow 2.0

@jakubczakon thankss. Already passed it to dev team

It is now fixed (#21 ). Closing this.


test: make tests independent from working directory

Closes #66.
Summary of Changes
Make tests independent from working directory, so pytest can easily be invoked in any subdirectory of the tests folder.

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
3
3
0
0.97s

✅ PYTHON
flake8
3

0
0.38s

✅ PYTHON
isort
3
3
0
0.25s

✅ PYTHON
mypy
3

0
1.73s

✅ PYTHON
pylint
3

0
2.62s

✅ REPOSITORY
git_diff
yes

no
0.04s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by


Create Rahul-gupta.md

Notes for Reviewers

This PR fixes #

Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 2aefad28f68e0c7b7b085df36060fcd209d445e5 at: https://67360e37ccd38d00de9c2d9b--meshery-docs-preview.netlify.app


Expose Milli.milli to let Zinc be more efficient

It can avoid the allocation of the Path => File conversion
and also save extra calls to readAttribute when we are using
JavaMillis.

To validate this pull request against other sbt modules, please visit this link.

To enable a change like: https://github.com/sbt/zinc/compare/develop...retronym:topic/classpath-cache-overhead?expand=1


Added anchor to webpages, fixed issue #87

[x] Added the anchor to webpage\
[x] Changed logo of some pages that only had texts to make it consistent
[x] read contributing rules
[x] solves issue #87

https://github.com/user-attachments/assets/d0893758-9175-4870-9f64-2300648a3c42
@Tejashri-Taral  Hii I have done the changes, it would be great if you merge this under GSSOC extd, Level 2 , Hacktoberfest accepted .
Thank you so much !

@Tejashri-Taral  please merge this

Add it into footer pages also

Done the changes required.

@shimmer12 as per the guideline I can't increase level because you have just add link instead of word which is already present in the code


Update README.md

Hello - request review and approval on the inclusion of Explore Edge Computing on your list. It would be a good addition under TOOLS section of this article. Would you like to include it in your list? We'd love to hear your feedback. Thanks!

Thanks for your contribution!
Explore Edge Computing looks like an awesome list. I merged the pull request and will move it to Section "Other awesome list".
Thanks!

Hello - request review and approval on the inclusion of Explore Edge Computing on your list. It would be a good addition under TOOLS section of this article. Would you like to include it in your list? We'd love to hear your feedback. Thanks!

Thanks for your contribution!
Explore Edge Computing looks like an awesome list. I merged the pull request and will move it to Section "Other awesome list".
Thanks!


Build error

Not sure if this is a clobber issue or due to me running it on Catalina.

(  \ ( )    (  _  )(  _\ (  \ (  _\ |  `
| ( ()| |    | ( ) || () )| () )| (()| () )
| |  _ | |  _ | | | ||  _ <'|  _ <'|  ) | ,  /
| (( )| |( )| () || () )| () )| (( )| |\
(/'(/'(_____)(/'(/'(_/'() ()
v0.3.3 by @Dids
✔ Verifying folder structure
✔ Verifying folder structure
✔ Verifying EDK is up to date
✔ Verifying Clover is up to date
✔ Building base tools
✔ Setting up EDK
◌ Linking nasm ⠼
Error: Failure detected, aborting
See the log file for more details:
/Users/shane/.clobber/logs/clobber.log
clobber.log

Hmm, almost sounds like nasm is bundled with Catalina, unless it was installed by something else you use, like a different Clover build script or something?
Either way, I should be able to check for nasm before attempting to install it from Homebrew.

@SavageAUS Can you try the following and let me know if it fixes it.
Uninstall clobber:

brew remove clobber --force

Install clobber (bleeding edge):

brew install clobber --HEAD

Run clobber again:

clobber

Thanks @Dids that worked for me yesterday, just didn't have time to post here.
I will run again today and make sure all is good.


[feature request] add to queue/play next

I would be awesome to have the ability to select track from album to play as next, or add the album to queue.

Hey! Thanks for the input. Selecting a track from an album to play next is already possible through long-pressing the track in question. Adding an entire album to the back of the queue isn't though so I'll leave this issue open. The recommended (but admittedly convoluted) way of solving this right now is creating a playlist in Jellyfin web and playing back the playlist.


Ideas for output data of a pipeline

1. Output JSON file
In order for pipeline results to be loaded by IRIDA Next, an output.json (or output.json.gz) file should be produced that informs IRIDA Next of which data/metadata to store.
This file can be structured like the following:
{
    "files": { ... },
    "metadata": {
        "samples": {
            ...
        },
    },
}

1.1. Output files
In order to store output files within IRIDA Next, they should be listed in the files section as key/value pairs using the following structure.
{
    "files": {
        "summary": {
            "phylogenetic-tree": "tree.nwk",
        },
        "samples": {
            "SampleA": {
                "assembly_contigs": "output_file.fasta.gz",
                "assembly_qualty": "quality_report.zip",
            },
        },
    },
}

The summary keyword lists output files related to all samples/data in the pipeline (e.g., a phylogenetic tree). The samples keyword lists output files associated with a particular sample (e.g., an assembled genome, etc).
Within each of these sections, there are key-value pairs which will allow access of files for an analysis by the key (e.g., SampleA.assembly_contigs returns output_file.fasta.gz).
1.2. Sample metadata
In order to store sample metadata, it should be structured by the pipeline like the following:
{
    "metadata": {
        "samples": {
            "SampleA": {
                "key1": "value1",
                "key2": {"subkey1": "value1", "subkey2": "value2"},
            },
            "SampleB": {
                "key1": "value2",
            },
        },
    },
}

2. Storage of data in IRIDA Next
2.1. Metadata
Metadata will be stored in IRIDA Next by loading the output.json file and looking for the metadata.samples section. It will store the information associated with each respective sample (e.g., the "SampleA" part below will be used to lookup "SampleA" in IRIDA Next, and the contents of the JSON dictionary will be merged with any existing metadata for  SampleA).
"SampleA": {
    "key1": "value1",
    "key2": {"subkey1": "value1", "subkey2": "value2"},
},
"SampleB": {
    "key1": "value2",
},

There will be a parallel table which stores metadata about the source of each above field:
"SampleA": {
    "key1": {
        "source": "analysis",
        "source_id": "1234",
    },
    "key2": {
        "source": "analysis",
        "source_id": "1234",
    },
},
"SampleB": {
    "key1": {
        "source": "analysis",
        "source_id": "1234",
    },
},

Idea for defining output files
One idea for defining output files listed in the files section of the output.json is to provide support for Nextflow tower's tower.yml file defining reports to visualize: https://help.tower.nf/22.2/reports/overview/#limitations

1.1. Output files (comment)
One comment on output files, some pipelines may want output files associated with other divisions than samples. For example, wg/cgMLST clustering may want to associate output clusters (a tree and other information) with each MLST scheme passed to the pipeline. As in this pipeline https://github.com/phac-nml/nf-pipelines/issues/1, which breaks up analysis results by MLST profile.
One option here is to just reserve the words "summary" and "samples", but provide any other divisions here. For example:
{
    "files": {
        "summary": { ... },
        "samples": { ... },
        "mlst_profiles": {
            "listeria_cgmlst": {
                "clusters": "clusters.text.gz"
            }
        }
    }
}

Added this in #7


fix: update Codecov action file parameter to 'files' for compatibility

Changed the parameter from 'file' to 'files' in the Codecov action configuration within the GitHub Actions workflow.
This update ensures compatibility with the latest Codecov action requirements, improving the reliability of coverage report uploads.

Huly®: TAGFL-24

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Group zaps by author

🚀 Goal
Issue: https://github.com/vitorpamplona/amethyst/issues/159 Combine the zaps from the same user into one list entry
📚 Description
First group all zaps by author, and then aggregate the zaps amount for each author, cloning the original notes to avoid altering the original data.
🖼 Screenshots
BEFORE

Or even this annoying "multiple 1 sats"...

AFTER
Now it's grouped by user and sorted accordingly 🚀

💭 Extra...
I wrote the logic in UserZaps in an imperative approach, although I am sure it could be written in a more functional "kotlin way". I am open for suggestions, refactoring or any kind of improvement to the current logic 🧠

Good work. My only issue is with the idea of cloning a Note as opposed to creating a new UI object (similar to how we do the Notification Cards) to keep the Zap information.
The issue with cloning notes is that, throughout the entire application, there is only one Note object per event_id. LocalCache enforces that. It makes it easy to deal with sets and maps because the same object references mean the same underlying event.id without having to compare the id string or the whole object.
Here we should create a new UI Card object and make the Zap FeedView, model and filters return this new object instead of the clone of note.

Thanks for the feedback! I am not 100% sure how to do what you are saying, so I see two possible solutions:
1.- We can merge this PR and you can finalise it on your own, because you seems to know what is left.
2.- We can work on it together doing pair-programming; tomorrow evening (27), for example, or the Wed(29) or Thur(30) next week evening would work as well for me. I would love to do it as driver myself with your help navigating the solution.
What do you say, @vitorpamplona?

Note: Although I prefer the 2nd (pair-prog), if you choose the 1st (you do it) I will still learn form your diff-changes.

Closing this idea for now. We may remove zaps from profiles altogether.


BetterDiscord still hasn't loaded

My friend sent me a thing on discord to download BetterDiscord. It worked, but it isn't loading. Is there a way to fix this, its been loading for the past day now and I've tried refreshing discord and reinstalling BetterDiscord.

Try using this https://0x71.cc/bd/troubleshoot


Why does it just show one object in the pic with more than one person when using "example" part,  although the result seems right using "Panoptic" or "Text" with 'person' ?

Why does it just show one person in the pic with actually more than one person when using "example" part,  although the result seems right using "Panoptic" or "Text" with 'person' ?    If there is any other setting needed to be changed, please tell me, thx!

Sorry, I am not quite understand the question, could you please provide an example to your argument? Thanks!

I believe the question was about the second person not being detected when using "text", but being detected when using "panoptic".


NullPointerException thrown when publication contains nested levels of chunking

Expected Behavior
The XHTML transformation should work without errors when the publication contains nested levels of chunking.
The DITA Map used to reproduce the problem has the next simplified structure:
main.ditamap
<mapref href="ditamap_level2.ditamap" format="ditamap" type="map"/>
---
**  ditamap_level2.ditamap
**  <map chunk="to-content
[test_map.zip](https://github.com/dita-ot/dita-ot/files/1474190/test_map.zip)
">
**    <title>DITA MAp Level 2</title>
**    <mapref href="ditamap_level3.ditamap" format="ditamap" type="map"/>
**  </map>
---
****    ditamap_level3.ditamap
****	<map chunk="to-content">
****      <title>DITA Map 3</title>
****      <topicref href="Topics/test.dita" type="concept"/>
****    </map>

Actual Behavior
The XHTML transformation fails with java.lang.NullPointerException in chunk processing.
chunk:
    [chunk] Processing file:/D:/test-oxygen/dita-ot-chunk-issue/temp/xhtml/oxygen_dita_temp/main.ditamap
    [chunk] Processing file:/D:/test-oxygen/dita-ot-chunk-issue/temp/xhtml/oxygen_dita_temp/Chunk1025069204.dita
    [chunk] Processing file:/D:/test-oxygen/dita-ot-chunk-issue/temp/xhtml/oxygen_dita_temp/Topics/test.dita
    [chunk] Writing file:/D:/test-oxygen/dita-ot-chunk-issue/temp/xhtml/oxygen_dita_temp/Chunk456609017.dita

BUILD FAILED
D:\Program Files\Oxygen XML Editor 19.1\frameworks\dita\DITA-OT2.x\build.xml:45: The following error occurred while executing this line:
D:\Program Files\Oxygen XML Editor 19.1\frameworks\dita\DITA-OT2.x\plugins\org.dita.base\build_preprocess.xml:323: java.lang.NullPointerException
	at org.dita.dost.writer.ChunkTopicParser.processChunk(ChunkTopicParser.java:239)
	at org.dita.dost.writer.ChunkTopicParser.write(ChunkTopicParser.java:61)
	at org.dita.dost.reader.ChunkMapReader.processCombineChunk(ChunkMapReader.java:542)
	at org.dita.dost.reader.ChunkMapReader.processTopicref(ChunkMapReader.java:347)
	at org.dita.dost.reader.ChunkMapReader.process(ChunkMapReader.java:143)
	at org.dita.dost.writer.AbstractDomFilter.read(AbstractDomFilter.java:55)
	at org.dita.dost.reader.ChunkMapReader.read(ChunkMapReader.java:119)
	at org.dita.dost.module.ChunkModule.execute(ChunkModule.java:80)
	at org.dita.dost.pipeline.PipelineFacade.execute(PipelineFacade.java:80)
	at org.dita.dost.invoker.ExtensibleAntInvoker.execute(ExtensibleAntInvoker.java:230)
	at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:293)
	at sun.reflect.GeneratedMethodAccessor6.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at org.apache.tools.ant.dispatch.DispatchUtils.execute(DispatchUtils.java:106)
	at org.apache.tools.ant.Task.perform(Task.java:348)
	at org.apache.tools.ant.Target.execute(Target.java:435)
	at org.apache.tools.ant.Target.performTasks(Target.java:456)
	at org.apache.tools.ant.Project.executeSortedTargets(Project.java:1405)
	at org.apache.tools.ant.helper.SingleCheckExecutor.executeTargets(SingleCheckExecutor.java:38)

Steps to Reproduce

Run the XHTML transformation for the attached DITA Map.

Environment

DITA-OT version: 2.5.2
Operating system and version:  Windows
How did you run DITA-OT? oXygen
Transformation type: XHTML

I have attached the test DITA Map.
test_map.zip

Reproduced with the sample files in 3.0, using both xhtml and pdf transform types (so it's not fixed in preprocess2.

Just encountered another simpler variant of this that may show the likely exception - I think this is caused by a chunked branch that has a chunked topicgroup inside. The sub-map is converted to a topicgroup container when the map references are resolved, and chunk="to-content" from the submap is preserved on that group; this led me to try the following map, which also ends in a null pointer error. The two topics are very basic, just a title and paragraph:
<!DOCTYPE map PUBLIC "-//OASIS//DTD DITA Map//EN" "ibm-map.dtd">
<map xml:lang="en-us"><title>Feeling chunky</title>
<topicref href="container.dita" chunk="to-content">
<topicgroup chunk="to-content">
<topicref href="fromsub.dita"/>
</topicgroup>
</topicref>
</map>

jira129.zip

@robander are these two other issues duplicates with this one?
https://github.com/dita-ot/dita-ot/issues/2730
https://github.com/dita-ot/dita-ot/issues/2428

@raducoravu definitely a duplicate of #2730 but I think #2428 is different.

Updated test case:
jira129.zip
Now includes:

Container topic with chunk="to-content"
nested topicgroup with chunk="to-content" (currently causes NPE); ideally this would create a new chunk with generated name, but not add in an extra generated title
nested topichead with chunk="to-content" (also causes NPE); ideally this would create a new chunk with generated name, including the title as the root "topic"
nested topicref with chunk="to-content" and no title + no href, but includes shortdesc (also causes NPE); edge case, it should not crash, should instead create new chunk with generated name, empty title but use the short description

Fixed with #3216.


Feature Request: "Remote" Groups

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
Right now, functionality of Groups is somewhat hampered by the fact that they are tied to local system groups. This means group names have length limits and can't accommodate all characters used in LDAP groups (esp. spaces).
Additionally, it requires synchronizing groups and importing users to the local database in order to grant GUI access. This requires some level of manual intervention that is generally not desired when using a centralized user database like LDAP.
This seems to have come up a lot recently: #5295 #5116 #4660 #4963
My proposed solution would address all of these
Describe the solution you like
"Remote Groups" would be a new type of group that can be used for anything not related to OS-level groups, such as GUI privileges, and can bypass naming restrictions.
pfSense implements this and I think their description is useful:

Groups from remote sources, such as authentication servers (RADIUS or LDAP). These groups are not exposed to the operating system, and thus are only available for use in the GUI and other similar uses not involving the operating system layer. This scope has relaxed name restrictions, for example, group names may be longer and may contain spaces.

pfSense Docs - User Management
Describe alternatives you considered
An alternative could be group aliases, as described here, but I think Remote Groups is better because it addresses other issues with group sync and user importing.
Additional context
I posted a thread in the forums about my experience setting up LDAP, which was frustrating compared to the ease of the rest of OPNsense setup: https://forum.opnsense.org/index.php?topic=25234.0
For the use case of using LDAP and existing users/groups to grant GUI access, Remote Groups would make the process much easier and completely automatic.
Example implementation from pfSense interface:

When testing LDAP integration with OPNSense we also wondered how the deletion of group memberships on the domain is handled. Currently it seems that the user is not revoked from the group, when I remove the user from the domain group.
Synchronization only adds people to the local group but does not revoke the access.
At least this feature should be implemented in order to make sure no access is left when e.g. the Active Directory team does remove people from e.g. a ProxyUser group.

@cmprmsd ldap group membership is enforced upon login when "Synchronize groups" is used (https://docs.opnsense.org/manual/how-tos/user-ldap.html#step-1-add-new-ldap-server).

We use synchronize groups. But when the user is removed from the group the user is not removed from the local opnsense group.
I only tested it with the "test user" functionality.
Maybe that's a workflow issue with the test user menu?
The user should be removed from the local group when the user is removed in active directory right?

when the user is successfully logged in, the groups will be synced first. cleaning up (removing) users automatically is only offered in the Business Edition (as mentioned in the docs)

Oh yeah, there it is:

As of version business edition 21.10, the system will automatically query the ldap servers and remove unexisting users. (not available in the community version of OPNsense)

Sad. Didn't know there are features that are available only in a paid version. 😿
Thanks for the clarification though!

You can also delete users manually on periodic bases, if there's no user in LDAP, it can't login anyway so usually there's no real rush... but on the other end, it's not that odd paying a few bucks for software your using every day ;)

That is true. The pricing seems reasonable :)
Have a great new year 😊


Modified Armstrong number function into Library function

Related Issue

Info about Issue or bug

fix #5321

Converted Armstrong number into a library function so that we are not dependent to get an element by a particular if

A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot
updated screenshot

@sairish2001 please review it


Add some diagrams!

There is a lot of class names and constructs, most of which are very similar. One or two diagrams showing the architecture would help tremendously!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c3460b0d-9c60-3a06-ac63-18cf990e53c3
Version Independent ID: b2dece1f-15b8-03af-6d52-7cc070432b4b
Content: Background tasks with hosted services in ASP.NET Core
Content Source: aspnetcore/fundamentals/host/hosted-services.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @qlxh ... We won't be able to provide diagrams to go with this topic. Diagrams are difficult for us to maintain due to the high volume of priority issues. When the framework changes, images change, and they're not nearly as easy to update as document text and code examples. As you can see, we have close to 600 issues here ... over 400 topics ... and over 100 sample apps. We just don't have the staffing to support images broadly in the docs at this time. We'll keep an :ear: open for more feedback. If there's feedback from the community that this topic is confusing without visual representations, we'll consider adding diagrams to it. Alternatively, we're open to accepting a submission from the community.


Make maxDuration configureable

Let's do it like the expected latency (i.e., make it configurable in settings protobuf file).

Originally posted by @freedomtan in https://github.com/mlcommons/mobile_app_open/pull/791#discussion_r1336642313

Where should we place the max_duration setting?

In the task configuration, near the min_duration setting
https://github.com/mlcommons/mobile_app_open/blob/2d8a9b2a94282383b4b9830da62ebe244d2253e8/flutter/cpp/proto/mlperf_task.proto#L43-L44
In the backend configuration > same max_duration for all delegates
In the delegate configuration > each delegate has its own max_duration


BUG: Page reloads when switching screens in send funds modal

This PR:

Wrap modal screens which are lazy loaded with Suspense component with fallback to displaying loading spinner

Travis automatic deployment:
https://pr191--safereact.review.gnosisdev.com
Storybook book automatic deployment:
https://pr191--safereactstorybook.review.gnosisdev.com


Add production cli (hcp) to dockerfile

I'm transitioning the KubeVirt platform OpenShift CI lanes to exercise the new production cli (hcp). However, it appears that we aren't building that tool in the hypershift-operator image.
my rehearse runs in this ci pr fail due to no hcp cli being present.
/bin/bash: line 13: /usr/bin/hcp: No such file or directory

I think copying the hcp tool into the hypershift-operator container using the default Dockerfile will resolve this.

/area ci-tooling


SCB-1879 Support to disable the TCC feature from configuration

Coverage decreased (-0.03%) to 80.654% when pulling 5ab99bb1ed42fd4db6f4051491d46446a60138df on SCB-1879 into a0e5cfe9b1bd3ee2f0a056806b2cc249879b587a on master.


[413] Use shared colours for standard UI components

Implements #413
Changes

Update onTintColor of all UISwitches to Green.medium
Update tintColor of all UIAlertControllers to Blue.medium
Update preferredControlTintColor of all SFSafariViewControllers to Blue.medium

Notes
I created an SFSafariViewController factory which performs the HTTP scheme check, and also applies the preferredControlTintColor. I’ve updated every initializer of SFSafariViewController to use this factory instead, so that the styling only needs to be set in one place. In doing so, I’ve had to force unwrap in CreateProfileViewController and NotificationNavigation to preserve the method signatures, but let me know if you want this handled better.
I have also created a similar factory which configures the tintColor for UIAlertController.
I’ve created these factories in their respective UIViewController extension classes since I didn’t see a clear place for them in the project structure, but this isn't ideal so let me know if you want them moved.
Attachments

Before
After

It's beautiful!!


Automatically add creators gpg key to recipients

Summary
I just created a mount/repo, added another recipient, synced, and had the other person run gopass recipients.
It says that my gpg key is missing and they can not create secrets without it.
Manually running gopass recipients add --store foo my_gpg worked around this issue, adding my key as a recipient.
Expected behavior
My gpg key should be automatically included in the recipients upon initialization of the store.
Environment

OS: Arch and OSX
gopass Version: gopass 1.12.5 (2021-03-29 17:27:06) go1.16.2 linux amd64
Installation method: from the respective package managers

Additional context

I have tested this and couldn't fully reproduce it. But I found some related useability issues that I'm trying to fix. Please let me know if the original issue is still reproducible from HEAD and provided detailed instructions how to reproduce.

Were you testing creating a main repo, or a sub-mount?


Heavy CPU load on running scripts

A lot of CPU cycles on cores are used when its running the scripts. Possible root cause is the https://github.com/pkg/sftp package which is not aimed for performance. Have to dig into the code though to confirm if it's the package or the use of it.

https://github.com/pkg/sftp/pull/482


No traceback on error: TypeError: 'str' object does not support item assignment

$ dephell deps convert --traceback --level=DEBUG --from=pyproject.toml --to pip --to-path dev-requirements.txt
ERROR TypeError: 'str' object does not support item assignment

Config:
[tool.poetry.dependencies]
python = ">=3.6"

[tool.poetry.dev-dependencies]
pytest = "*"

[tool.dephell.main]
from = {format = "poetry", path = "pyproject.toml"}
to = {format = "setuppy", path = "setup.py"}

dephell inspect self output:
{
  "cache": "44.09Mb",
  "path": "/Users/rocky/dev/py/dephell/dephell",
  "python": "/Library/Frameworks/Python.framework/Versions/3.6/bin/python3",
  "version": "0.7.4",
  "versions": {}
}

Is dev-requirements.txt already exists? If you delete dev-requirements.txt, will you have the same error?

Yes, it exists. No, if I delete it, issue remain the same.

Please, use --to-format instead of --to here.
Missed traceback fixed in #202.

Thank you for your feedback :)


Changelog to display readable guide based changes

Is your feature request related to a problem? Please describe.
I think it is difficult and in parts challenging to keep up to date with Guide based changes, where the changelog goes directly to Github commits. While I work with Github and understand how to use it (but potentially majority does not) it still feels excessive having to read commits to see guide based changes.
To be clear this is only in relation to game based content, not the framework or technicals behind the Compendium itself.
Describe the solution you'd like
I would like the Changelog to be a section that includes some brief release notes indicating changes.
EG:
02/06/2024

Trinket X has been removed from the trinket section as this was discovered as a simulation issue
Ring Y has been added to Gearing as it [insert reason]

10/07/2024

Updated rotation section to remove [spell] as it was identified [reason]

Describe alternatives you've considered
I feel having the Changelog being a section where there is a brief release note style structure could help users quickly identify changes that have occurred.
Could be possible News section is repurposed to house this changelog.
Could also be possible to create a text based file in the repo, and have the Changelog link go to there instead of the commit page. This text based file could be a release note style file where easily consumed release/changes can be read.
Additional context
To be clear I feel this is only needed for content related to the guide itself or in game content. Changes/commits done for the framework of the page (CSS, alignments, coding etc.) does not need to be included.

Hey, thanks for the suggestion. To have a seamless deploy we ended up using multiple repositories and unfortunately some links are not yet updated. This repository will be closed in the following days. Could you please reopen the same issue on the main dreamgrove repo: https://github.com/dreamgrove/dreamgrove/issues
Thanks!

Hey, thanks for the suggestion. To have a seamless deploy we ended up using multiple repositories and unfortunately some links are not yet updated. This repository will be closed in the following days. Could you please reopen the same issue on the main dreamgrove repo: https://github.com/dreamgrove/dreamgrove/issues
Thanks!

Took the liberty to move it: https://github.com/dreamgrove/dreamgrove/issues/545


Support for the experimental syntax 'classProperties' isn't currently enabled

I've just got this error after installing react-plaid-link package in my React.js project.
I bootstrapped it with create-react-app which means adding babel plugins is a pain.
I'm creating this issue because I assume there are many developers doing their react applications by using the very popular Facebook's create-react-app tool and having to install a plugin is so weird.
Would love to know if I'm doing something wrong or if there's anyone else experiencing the same issue.

So sorry about this, just noticed that WebStorm's autocomplete imported it like this:
import PlaidLink from "react-plaid-link/src";
and not as it should be:
import PlaidLink from "react-plaid-link";


[v0.14.0.x] Bump version to <IP_ADDRESS> and draft release notes

Also added a first draft for "Notable Changes"

Should also mention SPORK_16_INSTANTSEND_AUTOLOCKS deprecation we backported earlier https://github.com/dashpay/dash/commit/2e7ec2369381bccfa243ae3998f7f1f795e0fd64


Invalid hook call. Hooks can only be called inside of the body of a function component.

Describe the bug
we are call useQuery inside usecallback and giving this error
useFocusEffect(
useCallback(() => {
callQuary()//calling useQuery
}, [])
);
Your minimal, reproducible example
vsCode
Steps to reproduce
make a function which includes useQuary
call inside useEffect or useCallback
Expected behavior
As a user useQuary also work inside useEffect or usecallback
How often does this bug happen?
No response
Screenshots or Videos

Platform
MacBook Pro (13-inch, 2019, Two Thunderbolt 3 ports)
Processor 1.4 GHz Quad-Core Intel Core i5
Memory 8 GB 2133 MHz LPDDR3
react-query version
3.26.0
TypeScript version
4.8.4
Additional context
No response

this is just how hook work: you cannot invoke them in callbacks.

what's the alternative when useState needs to be called inside something like setTimeout?
This is likely the wrong question to ask. What's the use-case you're trying to solve, then we can figure out what the solution is. "I want to call the hook in setTimeout" is not a question, because the only possible answer is: you can't. But I'm quite certain there is a solution to the actual problem...


Fixing the Python benchmark graph Layout

The layout of the python benchmark graph is not properly align. It is overflowing to the other component.
Fixing issue #4170
Issue

Expected Behavior

@cartermp @chalin I have created the PR please review and let me know thoughts . Thanks


Maybe take a look into using swc instead of webpack

swc is a super-fast typescript / javascript compiler written in rust.

swc is a webpack replacement written in Rust that is way faster than webpack: https://github.com/swc-project/swc
I don't know if we could use it: we currently use a lot of webpack plugins that may or may not work with or exist for swc. But making these rebuilds way faster might be worth looking into it.

Webpack 5 got also recently released, and is supposedly faster, due to better caching and things like that. We might also migrate there and see whether things are already fast enough. x)

Somehow I confused stuff. swc is mainly a replacement for babel, not webpack. It also has "spack", which is a bundler, but yeah, the main project is about babel-like transformations. The transformation actually supports everything we currently use and can read browserlist automatically. It's pretty sweet. It reduces the webpack build times from 3.9s to 3.4s on my machine.
However, the generated code errors in the browser with "React is not defined". Potentially fixable, but already not a good sign if switching from babel to swc results in broken code. Further, running webpack --mode=production leads to swc panicking... badly:
thread '<unnamed>' panicked at 'called `Option::unwrap()` on a `None` value', ecmascript/transforms/compat/src/es2015/regenerator/case.rs:873:49
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5
Aborted (core dumped)

So, let's not switch to SWC for now :D


fix: move encrypted icon to excalidraw-app

An attempt to fix https://github.com/excalidraw/excalidraw/issues/2899
When restoring zenmode the encrypted icon since now is rendered from excal-app via renderFooter prop hence gets pushed from right (due to translate) which might look jerky.

Let's take the opportunity a clean it up a bit:

make .App-menu_bottom grid-template-columns into min-content auto min-content
use the center column only for user custom footer; move the language picker + help into the right-most column
remove all the position: absolute from the footer elements

use the center column only for user custom footer; move the language picker + help button into the right-most column

The language picker belongs to the custom footer as well, If we split it into 3 cols and use the center column for the custom footer then in zen mode won't it become tricky to handle the animation?

The language picker belongs to the custom footer as well

Oh, forgot. Good point.

If we split it into 3 cols and use the center column for the custom footer then in zen mode won't it become tricky to handle the animation?

Isn't it tricky to handle it even now with respect to the shield icon? It has the same issue. So if we want to solve it from the animation perspective, we can either split the custom footer into renderFooterLeft & renderFooterRight, or do the animation differently — e.g. animate the center footer column downwards: I think that's solution I'd go wit.

Isn't it tricky to handle it even now with respect to the shield icon? It has the same issue. So if we want to solve it from the animation perspective, we can either split the custom footer into renderFooterLeft & renderFooterRight, or do the animation differently — e.g. animate the center footer column downwards: I think that's solution I'd go wit.

Yeah separate animation for center column sounds good 👍🏻

Looking :fire:


accessing data files from monomer examples

In order to make deployment a little more portable, can we use the following APIs provided by cabal?
https://cabal.readthedocs.io/en/3.4/cabal-package.html#accessing-data-files-from-package-code
Without that, running the example binaries (/my/very/path/opengl for instance) fail to find the fonts and the vertex files.

Renderer: NVIDIA GeForce GTX 770/PCIe/SSE2
Version: 3.2.0 NVIDIA 470.86
Failed to load font: Regular
Could not find any valid fonts. Text size calculations will fail.

@fjvallarino let me know if you'd like me to push a PR

@smunix if possible I'd like to avoid using that API, since (as far as I understand) it forces the user to list all the resource files in data-files, which can be annoying (although I think adding only the base directory is an option too). The idea is that the resource files are relative to where the application is executed, which works well on Windows, Linux, and macOS.
I wonder if this issue is related to how the examples are invoked using Nix. Is it possible to somehow set the working directory when running the examples using Nix?

Nix uses cabal to deploy files indeed; so it won't touch these files in asset unless specified in cabal. I'm afraid setting the CWD won't help after the package has been deployed. How do you deploy monomer apps on other Linux flavors or macOS? I could try  to replicate with Nix.
I guess the alternative is to copy these files manually wherever the binary located, for the following to work.
https://github.com/fjvallarino/monomer/blob/ad16b56f04a50edf332ade4a2076fcf965bf3c28/examples/tutorial/Tutorial01_Basics.hs#L66

At least on macOS, you just need to have the assets folder inside the current working directory. For example, you can create a new directory nested inside the monomer-starter (or any other) project's folder, cd into nested and try running:
../.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/build/app/app.
Since the program can't find the assets, it will show errors regarding missing fonts. If you then run:
cp -r ../assets .
and try again, it will work fine.
I tested on Windows a while ago and, beyond needing to copy the .dlls into a visible directory, it worked too.

I don't think anything is pending here, so I'll close the issue for the time being. Please re-open if needed. Thanks!


Bump 7-Zip Beta to 21.03

Changes

Changed URLs from MSI to EXE for a slightly smaller file size.
The unzipped contents appear to be identical.
License(s) updated. Previously lacked the BSD 3-Clause (due to code derived from Apple's "LZFSE compression library") and UnRAR licenses.

Questions

Are MSIs preferred when the assembly is unsigned?
I find it a little strange the auto-updater didn't catch the latest beta release. That may need to be investigated.

Notes

URLs pulled from https://7-Zip.org/Download.html and https://sourceforge.net/p/sevenzip/discussion/45797/thread/9f5b067368/
I was also going to add the URL and SHA256 for the ARM64 release, but Scoop's ARM64 specs haven't been finalized just yet.

@BinToss I believe, the checkver/autoupdate parts of manifest from @Ash258's bucket may be reused here. https://github.com/Ash258/Shovel-Ash258/blob/main/bucket/7zip-alpha.json


Update kerberos-authentication-portal.md

Kerberos can also be used for premium SKU. Replacing Basic and standard with "Basic SKU tier or higher for Azure Bastion"

@cherylmc
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[BUG] Failed to load formio.component.html in Jest test

Environment
Please provide as many details as you can:

Hosting type

[ ] Form.io
[x] Local deployment

Version: 5.0.2

Formio.js version: 4.12.7
Frontend framework: Angular 11.2.4
Browser:
Browser version:

Steps to Reproduce

import { FormioModule } from '@formio/angular';
expect(component).toBeTruthy();
Test fails

Expected behavior
Test passes
Observed behavior
Test failure
Stacktrace:
Uncaught (in promise): Failed to load formio.component.html

      at resolvePromise (node_modules/zone.js/dist/zone.js:832:39)
      at resolvePromise (node_modules/zone.js/dist/zone.js:784:21)
      at node_modules/zone.js/dist/zone.js:894:21
      at ZoneDelegate.Object.<anonymous>.ZoneDelegate.invokeTask (node_modules/zone.js/dist/zone.js:421:35)
      at AsyncTestZoneSpec.Object.<anonymous>.AsyncTestZoneSpec.onInvokeTask (node_modules/zone.js/dist/async-test.js:86:33)
      at ProxyZoneSpec.Object.<anonymous>.ProxyZoneSpec.onInvokeTask (node_modules/zone.js/dist/proxy.js:145:43)
      at ZoneDelegate.Object.<anonymous>.ZoneDelegate.invokeTask (node_modules/zone.js/dist/zone.js:420:40)
      at Zone.Object.<anonymous>.Zone.runTask (node_modules/zone.js/dist/zone.js:188:51)
      at drainMicroTaskQueue (node_modules/zone.js/dist/zone.js:601:39)
      at ZoneTask.Object.<anonymous>.ZoneTask.invokeTask [as invoke] (node_modules/zone.js/dist/zone.js:507:25)
      at invokeTask (node_modules/zone.js/dist/zone.js:1671:18)
      at XMLHttpRequest.globalZoneAwareCallback (node_modules/zone.js/dist/zone.js:1708:25)
      at XMLHttpRequest.callTheUserObjectsOperation (node_modules/jsdom/lib/jsdom/living/generated/EventListener.js:26:30)
      at innerInvokeEventListeners (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:318:25)
      at invokeEventListeners (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:274:3)
      at XMLHttpRequestImpl._dispatch (node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:221:9)
      at fireAnEvent (node_modules/jsdom/lib/jsdom/living/helpers/events.js:18:36)
      at Request.<anonymous> (node_modules/jsdom/lib/jsdom/living/xhr/XMLHttpRequest-impl.js:889:5)
      at Gunzip.<anonymous> (node_modules/request/request.js:1076:12)

Example
Trying to write a test for a component that has a formio component.
For code or form JSON, please enclose in a code block:
beforeEach(
        waitForAsync(() => {
            TestBed.configureTestingModule({
                declarations: [
                    SlamCrudFormComponent,
                ],
                imports: [
                    FormioModule,
                ],
            }).compileComponents().then(() => {
                fixture = TestBed.createComponent(SlamCrudFormComponent);
                component = fixture.componentInstance;
            });
        })
    );

Update:
Adding CommonModule to the imports, change the error to:
Uncaught (in promise): Failed to load formio.alerts.component.html

Hello @michealharrington, are you still experiencing this issue? If so,  can you try upgrading to the latest version of angular and see if you can still replicate the issue? The latest version of Angular is 5.2.2

My co-worker recently went through an updated all our formio versions and everything seems to be working now. We are running 5.2.2 now.


Is it possible to retrieve the statistics section?

In the "Statistics" tab, you can find data like for example "PER - Price Earnings Ratio". Can't see this in the usage examples - is that possible?

I think it will be possible soon. Please see #903. IIRC PER is close to the PegRatio, we resolved there. (Or you can simply write your own parser snippet for that , based on the script in that issue.

Indeed the infrastructure provided by @eabase opened up many missing parameters - we started with the trailingPegRatio and in the future I’ll add the quarterlyPegRatio as per his example resolution

Related to issue #568, both fixed by fetching key-statistics.


Remove any final attribute from classes to make more extensible

This makes the package a little more extensible

@phanan Thoughts?

I don't really see why people would need to extend these classes, but guess it doesn't hurt. Thanks for the PR!


unable to submit request to api server over localhost

When attempting to submit api requests from the socket.io server, the below error is thrown:
$ npm run server-dev

><EMAIL_ADDRESS>server-dev F:\Code\stryve
> SET PROD_ENV=0 & node src/server.js

Listening on *:3333

User connected and ready.
A user has connected to the server.
{ [Error: connect ECONNREFUSED <IP_ADDRESS>:8181]
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 8181 }
{ errorCode: 0,
  errorMessage: 'connect ECONNREFUSED <IP_ADDRESS>:8181' }

Can do this:
$ curl http://localhost:8181/  # Returns successful response

but if I replace localhost with 127.0.01 it fails:
$ curl http://<IP_ADDRESS>:8181/  # Failed to connect to <IP_ADDRESS> port 8181: Connection refused

In addition to my previous post, although localhost is explicitly listed in the code, it seem node translates the hostname to an IP address.

I don't get a refused connection when requesting <IP_ADDRESS>:8181 (linux)

@CknDev
May be my local dev environment that is screwing with something
Are you able to run everything without error using the Getting Started guide?
I have remove node-webkit as a dependency, it must now be installed globally. Updated the readme and Getting Started tutorial with respect to this change.

Made a fresh install with upstream (apart from the db step (1));
Everything is running (register; login; create server; create channel; send message)

looks to be an issue just with my dev environment. closing for now.


Wrong height in decompressed DXT5 images

Here is an image as it appears on the Dota 2 CDN at http://cdn.dota2.com/apps/570/icons/econ/leagues/subscriptions_gpsea.c08851767ea252d6fcd993f3b0411c2f81f1d2d8.png

Here is that same image extracted from the Dota 2 VPK at pak01_dir.vpk/panorama/images/econ/leagues/subscriptions_gpsea_png.vtex_c

You can see the height is off and the texture is stretched to compensate. I see this occurring on quite a few DXT5 compressed textures after they are decompressed.

That's what the size indicates, I believe this used to be an issue in source 1 games too?

Oh. I see now that some are stretched in the width direction, so the issue title is not entirely correct. Regardless, if the data indicates a wrong dimension, I'm not sure what we can do. Should I close the issue?

It might be worth investigating if it's possible to fix it, if the game somehow knows how to render it correctly, although it will be related to the fact that they're rendered as textures, and the size is known separately.

Since it's a texture, there's no way to cut it based on the texture alone.


uwp: when will we have .netcore runtime 3.1 ? [question]

uwp: when will we have .netcore runtime 3.1 ?
Currently, UWP apps use .netcore 2.2 runtime. It seems that UWP is always the last to get updates, and they are the buggiest.
When will we have support for .netcore runtime 3.1 in UWP?

@tommcdon @jashook can you please take a look?

Thank you for the feedback.  UWP issues are tracked as Azure DevOps issue.  If you are running into .NET native issues, please see https://docs.microsoft.com/en-us/visualstudio/ide/feedback-options.

@tommcdon I don't understand. It's useless for me to ask this question on the VS2019 app. Where I am I supposed to ask it, if not here?


plugin doesn't track a thing (view)

When the app is loaded, the initial pageView analytics request:
GET https://www.google-analytics.com/r/collect?v=1&_v=j79&a=1885621779&t=pageview&_s=1&...

is issued. All the subsequent trackView requests are ignored.
Using<EMAIL_ADDRESS>Registration:
Vue.use(VueGtm, { id: "GTM-1234567" });

Tracking view in the router's afterEach guard:
router.afterEach(to => {
	const baseUrl = router.options.base || "";
	Vue.gtm.trackView(to.name, `${baseUrl}${to.fullPath}`);
});

If I enable debugging, the console contains log record, for instance:
VueGtm : Dispatching TrackView {screenName: "Home", path: "/"}

but there is no analytics request in the network tab.

seeing something similar with trackEvent called and logging (when debug enabled), but no collect?=v.... is visible in the network tab and event isn't counted in gtm web panel

Same issue any idea ?

same here

Is this still an issue?

Yes, I'm still having the issue.

@jippy89 it's not an issue, it's rather a misunderstanding of how the plugin and google tag manager works. Have a look at my answer here: https://github.com/mib200/vue-gtm/issues/66 - it works perfectly.

As @deanbrophy  and @simplenotezy  explained, you need to publish the container in GTM and setup the tracking layer correctly.


Previous Activity gets recreated by UP navigation

If the previous activity exists, is it supposed to "pop up"?
I thought according to Principles of Navigation, "if the previous destination exists on the stack, then UP should behave as BACK navigation".

Up and Back are equivalent within your app's task
When the system Back button would not exit your app, such as when you are on your own task and not on the start destination, the Up button should function identically to the system Back button.

That's correct - this is a bug.  The Up button on the plant detail screen should behave like the Back button.

Also see the related feature request on the Navigation library to have Navigation handle the activity up button as well

If you guys ever add a pending intent that opens the detail view directly, then that up navigation won't behave as expected.

Absolutely.  Adding deep links will require synthesizing a new back stack.
For more on this topic, check out https://developer.android.com/training/implementing-navigation/temporal#SynthesizeBackStack


correct label for Maximum Zoom for Polygon Label?

@ilyankou - In the Polygons tab, we describe Maximum Zoom for Polygon Label
Is that the correct description? I'm wondering if we mean Minimum Zoom for Polygon Label, since labels disappear when the zoom falls below a certain level
Or maybe I misunderstood?

I'll rewrite the description (and possibly label, and also constants.js) to describe the concept of threshold (labels show here and above)


Review and create db indexes

Since we launched prod, need to review and create missed db indexes to avoid slow queries (e.g.: reports)

[ ] add unique indexes only first

Mithril

client_types:

name

roles:

name

Il

declaration_requests

declaration_id is not null

Mpi

persons:

first_name, last_name, second_name, tax_id, birth_date

Ops

declarations

person_id, status = 'active'
declaration_request_id

medication_dispenses

medication_request_id, status = 'PROCESSED'

medication_requests

request_number
medication_request_requests_id

Prm

black_list_users

tax_id, is_active = true

employees

legal_entity_id, is_active = true, employee_type = 'OWNER'

global_parameters

parameter

innms

sctid, is_active = true

parties

first_name, last_name, second_name, tax_id, birth_date
drop index by tax_id, birth_date

Uaddresses

districts

name, region_id

settlements

district_id, region_id, name

streets

settlement_id, name, type

Verification

verifications

phone_number, active = true

Remove the following UNIQUE indexes:

Uaddresses

settlements

district_id, region_id, name

streets

settlement_id, name, type

There are duplicated settlements in one district

Do not replace parties_tax_id_birth_date_index - it makes no sense

innms
sctid not null, is_active = true

how about name?

innms
sctid not null, is_active = true

how about name?

Our implementation allows us to create two records with the same name


Add Kotlin Support

Adds Kotlin support using Kotlin script per #51.
The -script argument allows executing a .kts (Kotlin script) file without generating a compiled JVM .class file.

Looks great!

Released in version 0.14.0


When talking about RCF, should revin be included?

The results of ablating cyclenet's rcf (only etth1 and electricity were run) were almost unchanged. Then I found that the results of Table 6 in the paper should be the result of removing revin as well. . It is equivalent to only comparing +revin+rcf and ordinary linear in the article, and also comparing +revin+rcf and +rcf. There is no real comparison of rcf (if there is any possibility that I have missed it, please point out.

The results I reproduced show that the results of rcf are not that much improved.