libvirt: darwin/linux fix dlopen("libjansson.so.4")

Motivation for this change
Commit 59027e28800dd5c2cd1c9d434c1a71d39be19266 broke libvirt
Revert "libvirt: fix dlopen("libjansson.so.4")"
broke libvirt/nixops on linux :/ I am not sure what's the proper fix but meanwhile could we wrap the change in an if darwin ?
cf
https://github.com/NixOS/nixpkgs/pull/44900
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0...
moving /nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/sbin/* to /nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0/bin
/nix/store/jcr9jlp7bxzmpvlmb3zf2wz7lcgvyb44-libvirt-4.6.0

Success on aarch64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0...
moving /nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/sbin/* to /nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0/bin
/nix/store/37cfpicx7axzs3c1y7lsnz69dx8p85jv-libvirt-4.6.0

@pikajude what do you think ? I don't have darwin either and it's a runtime issue so ofborg won't help.

https://github.com/NixOS/nixpkgs/pull/44900#issuecomment-413314095

@LnL7 Does that mean replace .so with targetPlatform.extensions.sharedLibrary ?

The commit #44900 was purely reverted so I assumed libjansson was not mandatory for darwin else I don't see the point in the revert it just broke libvirt on both platforms. Also don't assume everyone is familiar with every bit of nixpkgs it's the first time I come across ${targetPlatform.extensions.sharedLibrary} hence my question, I am just trying to help here (as you are).

Success on x86_64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0...
moving /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/sbin/* to /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/bin
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0

I pushed your fix, it is fine on linux. Hope some macos user can test it too

Success on aarch64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/x8npv3gr1rq1pf6gqpcdb9b68374pfd6-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0...
moving /nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/sbin/* to /nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0/bin
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0

@teto was able to build it on darwin, start libvirtd, and it kept running.
I was also able to connect via virsh to a remote system, and could instruct libvirt commands there - so nixopsing should work too after this is merged.
Note: I could connect to the locally started libvirtd as well, but wasn't able to define any domains - libvirt kept complaining it couldn't find capabilities for domaintype=qemu - unsure if running qemu with darwin as host ever worked, so this should be fine for now.

@teto I wasn't talking about the sharedLibrary helper specifically, if ... then "dylib" else "so", while not as nice will do the same thing. I'm just a bit confused why there's so much discussion about something that almost looks easier to fix then revert.
Just disabling the fix on darwin wouldn't break nixops anymore, but I assume (since it builds on darwin) you could run into the same problem there.

@GrahamcOfBorg build libvirt

Success on aarch64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
/nix/store/24y5k8gynpdxi0rk5qrf4wrvf59l43zy-libvirt-4.6.0

Success on x86_64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/79b6s2wql94lwk8dyib2jc4fq8ixl13f-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0...
moving /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/sbin/* to /nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0/bin
/nix/store/id6684w4qx8xblvwgisyi97b8rk4b5bq-libvirt-4.6.0

Success on x86_64-darwin (full log)
Attempted: libvirt
Partial log (click to expand)
Removing /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/share/gtk-doc
rmdir: failed to remove '/nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/share': Directory not empty
gzipping man pages under /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/share/man/
strip is /nix/store/bg39mxc401qqxgjasxfgardi6xl2ikmc-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/lib  /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/libexec  /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/bin  /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/sbin
patching script interpreter paths in /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0
/nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/avxlkpsr36xpnsy0byq2c4vxk6sw157a-bash-4.4-p23/bin/sh"
/nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/avxlkpsr36xpnsy0byq2c4vxk6sw157a-bash-4.4-p23/bin/sh"
moving /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/sbin/* to /nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0/bin
/nix/store/jqijf49nycl31xcjs42rnsirpphdhdwb-libvirt-4.6.0


Sentinel 2.0.1

How to make sure that when killing an npc does not drop a weapon (crackshot)

This happens by default. NPCs will only drop items when killed if you put them into /sentinel drops.
If drops are happening outside of this, you have some unrelated plugin forcing items to drop.


Gettables in pyworld

Description
Allows gettable objects in pyworld
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[x] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[x] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After

Codecov Report

Merging #1224 (d933f2a) into main (e79e320) will decrease coverage by 0.10%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##             main    #1224      +/-   ##
==========================================
- Coverage   34.02%   33.91%   -0.11%
==========================================
  Files         348      348
  Lines       34460    34533      +73
==========================================
- Hits        11726    11713      -13
- Misses      22734    22820      +86

Impacted Files
Coverage Δ

agents/locobot/locobot_agent.py
0.00% <0.00%> (ø)

...ts/craftassist/tests/base_craftassist_test_case.py
93.02% <33.33%> (ø)

agents/_agent.py
41.40% <0.00%> (-10.16%)
:arrow_down:

lowlevel/minecraft/pyworld/world.py
41.45% <0.00%> (-3.39%)
:arrow_down:

lowlevel/minecraft/pyworld/fake_mobs.py
88.13% <0.00%> (-0.85%)
:arrow_down:

lowlevel/minecraft/pyworld_mover.py
0.00% <0.00%> (ø)

lowlevel/minecraft/pyworld/run_world.py
0.00% <0.00%> (ø)

lowlevel/minecraft/pyworld/utils.py
48.45% <0.00%> (+0.53%)
:arrow_up:


1.6 Project’s Quality Attributes

Performance
The performance of the system must be fast and efficient. By being able to support at least 20 users simultaneously and has a fast load time (within 5 seconds). The system must also be simple to use so that the user will  be familiar with the system just by having 1 hour of guidance.
Security
To enhance the security of the RMMIS, we intend to implement features like information hiding (e.g., to display/show information to the necessary users only, like the higher-ups). This helps ensure that all the data is secure and prevents unauthorized users from accessing this information. Having a record of all the people who have viewed the forms will also be easier for the higher-ups to track. Additionally, as there are multiple users using this system, it is important to keep the app secure. Thus, we will implement encryption to all the passwords to keep is safe.
Scalability
The scalability for this part of the module will keep growing as time passes as it will continue to have more and more reports stored. With features like search and filter function, it can manage all the different forms, it will increase the ability to handle the increase in volume and, thus, contribute to the scalability of the system.
Usability
RMMIS needs to be user-friendly and easy to use. The system can become more usable by having features such as a user-friendly interface, intuitive navigation, and the ability to access relevant information quickly. It will be easy for the users to fill up the form and view the history. When the system is able to run on multiple modern web browsers (eg, Google, Firefox), it can help to make the system more usable as well.

Reviewed and Completed on Time: 15/1/2023


installation: add tls to caddy

Need to update setup.md document to explain the DOMAIN variable and development flag of the install-orchestrator.sh script


Question: Is there an Optional Authorized decorator?

How would u guys solve an endpoint that has an optional authorization (when there's no token or token is incorrect)?
Example:
...
@AuthorizedOptional() // how to do something like this?
async someMethod(@CurrentUser() authUser?: any) {}
...

your code without @Authorized()  means the auth is an optional authUser will be undefined.
if u want to force auth ,should add @Authorized() and  @CurrentUser({ required: true }).
but this will check token twice in "authorizationChecker" and "currentUserChecker"
@NoNameProvided
an other idea is
@Authorized('optianal')
and return true in authorizationChecker.

So you want to achieve that when no token is provided, it will skip authorization checking and @CurrentUser() will return undefined but when token is provided you want to validate it's roles in authorizationChecker? I can't imagine the use case like this 😜
If you just need to have current user object, don't mark the method as @Authorized, just inject @CurrentUser into action handler params with required: false 😉


binaries too big

Enhancement
All versions of kclvm release larger than 150MB, smaller binaries are always welcome.

I personally recommend to consider the static linking of LLVM at the same time. By statically linking the packaging of libraries such as lld into kclvm_cli, the binary size can be effectively reduced.


3sumclosest file added

Please add the link to the code, the video link and the question link in README.md


[MUXC, WUXC] Memory leak in NavigationView

Describe the bug

Memory leak in NavigationView, both WUXC and MUXC.
Leaked NavigationView itself, all children, and navView.DataContext even if was inherited from parent control rather than being directly set as a property of navView.
The reason is four Bindings created in code of NavigationView that creates circular reference from part of NavigationView control template to NavigationView itself. Usually bindings from template to control are created in XAML as TemplateBinding, and do not cause strong reference to control, current code of NavigationView was not so careful.
Note, that issue is present in both WUXC and MUXC versions of NavigationView control.
Steps to reproduce the bug

Steps to reproduce the behavior:
NavViewLeakTest1.zip

Unzip, build and run attached NavViewLeakTest project
Click on 'Navigate Forward from MainPage to TestPage' button.
Debug output will contain such strings:
NavViewLeakTest::LifeTimeProbe_DataContext::LifeTimeProbe_DataContext
NavViewLeakTest::LifeTimeProbe_TestPage::LifeTimeProbe_TestPage
NavViewLeakTest::LifeTimeProbe_NavView::LifeTimeProbe_NavView
NavViewLeakTest::TestPage::TestPage
NavViewLeakTest::MainPage::~MainPage
Click on 'Navigate Backward from TestPage to MainPage' button.
Debug output will contain such strings:
NavViewLeakTest::MainPage::MainPage
NavViewLeakTest::TestPage::~TestPage
NavViewLeakTest::LifeTimeProbe_TestPage::~LifeTimeProbe_TestPage
note the absence of
NavViewLeakTest::LifeTimeProbe_NavView::~LifeTimeProbe_NavView
NavViewLeakTest::LifeTimeProbe_DataContext::~LifeTimeProbe_DataContext

Expected behavior

After the step 3 debug output should also contain evidence of NavigationView and DataContext destruction:
NavViewLeakTest::LifeTimeProbe_NavView::~LifeTimeProbe_NavView
NavViewLeakTest::LifeTimeProbe_DataContext::~LifeTimeProbe_DataContext
Screenshots

Version Info

NuGet package version:

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2019 Update (18362)
Yes

October 2018 Update (17763)
Yes

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Note, that probes are destructed if NavigationView is replaced by Any other control.
Memory leak is also gone if style for NavigationView is tweaked to hide from code parts of template whose content was bound to control.

Issue could be fixed by replacing four guilty Bindings by five property changed callbacks on NavigationView.{PaneHeader, PaneCustomContent, PaneFooter, AutoSuggestBox, PaneDisplayMode} properties

Great bug report! We'll have to take a look. I wonder if there's an easy way to make the Binding target a weak reference instead?
@licanhua maybe you have time to investigate?


ArgusWeb can't login

Hi,
I want to launch the Argus platform without ldap support. I did this by adding
service.binding.auth=com.salesforce.dva.argus.service.auth.NoAuthService

to the argus.properties file. For the argus.properties I used the blueprint in the Getting Started Section for the integration tests. I hope that's ok. When I open the website a login prompt shows up.
Is this behavior correct? Which credentials do I have to enter?
If I enter some credentials and hit the submit button a toast message 'Login failed' shows up.
Confusing to me is that the only request which is sent after hitting the submit button is an OPTIONS request, which gets a response code of 200 and returns the verbs POST and OPTIONS.
I would expect a POST request, which should trigger a message like 'Login failed'.
Basically I have two questions.
How do I enable the NoAuthService and how can I check that it is working?
Is the login behavior described above as expected?
Thanks for your help!

Hey @numbnut, Apologies for the delayed response, but we've just been a tad busy.  Argus will always require a login via the /auth webservice endpoint.  The default UI is wired to that endpoint.  If you're accessing it through the web UI, you will always get a login form.
Now, you can effectively turn off authentication, by binding the NoAuthService implementation.  You'll still get the login form, but you can type in any username and password.  To enable the NoAuthService implementation use the following configuration:
In the main configuration file
service.binding.auth=com.salesforce.dva.argus.service.auth.NoAuthService
service.config.auth=<path/to/auth/service/specific/configuration/file>
In the auth specific configuration file add this property
If set to true, all users will get privileged access.  If omitted, it defaults to 'false'
service.property.auth.noauthservice.privileged=false
A couple of things to consider:

If you are deploying over https, you should build Argus so that it requires secure cookies.  You do that by setting the build.property.secure.cookies=true in argus-build.properties and rebuild.
If you're hosting the UI and web services on separate hosts, make sure your web service hosts allow cross site traffic.  E.g. for Tomcat, make sure you have the CORS filter installed and configured to allow cross site POST, PUT, DELETE, GET & OPTIONS.

http://stackoverflow.com/questions/16296145/set-cors-header-in-tomcat

Hi @saaja-sfdc, thanks for your response. Just to let you know the context. I am only trying to setup a non production system. Security related topics are not relevant for me at the moment, but I will keep them in mind.
I adapted the configuration as you mentioned. According to the getting started documentation I did it in one configuration file and not as you mentioned a second one for the auth service. I hope that's ok.
The behavior of the application is still the same and I am still wondering why the frontend app only sends an Http Options query on pushing the Login button. The following image shows the behavior.

There is no Post request to the /auth endpoint, which I don't understand. Can you explain that?
Is there somewhere a documentation about the REST Api?

@numbnut
From your screenshot basically there are some possibilities. You may want to rule out one by one. (I just ran through a similar issues)
1, Seeing ArgusUI login page doesn't mean ArgusWS is up. Go to  http://localhost:8080/argusws/help
see if you are able to see a json return. If not, go check tomcat start logs, there might be a error there.

CROS is usually a cause. But it should showed up on your browser console.

@aertoria thanks for your reply! I will check that on Monday, because I can't access my development environment at the moment.
I think 1. can be excluded since I checked that, but I will check it again. 2. is a good point. I excluded it because the REST service and the client app are both deployed on the same host, but they are listening on different ports. Maybe CORS is the problem. I will share the results of my further investigation.

Thank you very much! It was the CORS problem. Now everything works as expected.


Would it help if I loan (well, temporarily purchase) you an RX 6400?

Hello @Rem0o
I really love Fan Control, and I really dislike that due to AMDs API changes, it works poorly with Fan Control and needs an iffy plugin.
You mentioned that its really hard to debug without acces to an AMD GPU.
Would it help if I bought a cheap RX 6400, had it shipped to you, and after you're done debugging you just ship it back to me (or sell it off and transfer me the money, if shipping costs would be egregious)?

Would be better of with an RDNA 3 card, as this is what ADLX was designed for in the first place.
See https://gpuopen.com/introducing-adlx/
The reality of it is that for this to work there need to be a bit of a community effort. I made a big first step/push but haven't seen much interest nor expertise being added in.  I got a bit too much on my plate right now to dig in this specific project.
Also manual control seems to be just straight broken in the driver, or badly supported. The only workaround is editing the built-in graph, which is costly resource-wise and not viable (this is what the plugin does right now).  Haven't seen much development on the binding side either from AMD.   Until there is some kind of breakthrough or news from AMD, I won't put time onto this project.

Ah, too bad.. at any rate, thanks for the consideration and the effort put forth so far!


Could not find com.google.android.gms:play-services-iid:16.1.0.

In commit ea570d2bbe26b631681bfcc0572d71cc14a7c186, the gradle dependency play-services-gcm was swapped to play-services-iid but still uses googlePlayServicesVersion to determine the package version.
This causes a problem play-services-gcm and play-services-iid versions differ.
See extract below from https://dl.google.com/dl/android/maven2/index.html:
play-services-gcm
16.0.0
16.1.0
17.0.0

play-services-iid
16.0.0
16.0.1
17.0.0

16.1.0 in play-services-gcm, but 16.0.1 in play-services-iid.
So by setting googlePlayServicesVersion to 16.1.0 for pre-AndroidX (as per the README), it actually breaks the Android build with Could not find com.google.android.gms:play-services-iid:16.1.0.

As a workaround I am now setting googlePlayServicesVersion to 16.0.0

Very interesting - a more precise patch (which I'll commit later) will be to first check for googlePlayServicesIidVersion first (from the ext{} block in your android/build.gradle) and then fall back to googlePlayServicesVersion.
That will allow more precise control but not break installed base
Sorry it caused you to step back in versions

@Johan-dutoit I'm going to do a 4.0.2 that has a fix for this once you've settled #801

Okay the PR is ready - should merge soon. The chunk you want (if you want to use patch-package until it's merged+released is https://github.com/react-native-community/react-native-device-info/pull/804/files#diff-7ae5a9093507568eabbf35c3b0665732R46 )

We upgraded to react native 0.60.5 and updated this package from 0.21.5 to 3.1.4 and we are getting this "Could not find com.google.android.gms:play-services-iid:16.1.0" error.
Any update on when this fix will be officially released?

Sorry - we're blocked on release for a moment while we ingest a conversion to typescript #144 and work through #776 - this one is a really trivial patch and I would use patch-package without delay so you are unblocked

released as @next on NPM or 4.0.1-beta.2

I'm getting this also on 5.3.0, is this expected?

For posterity, adding googlePlayServicesIidVersion = "16.0.1" in android/build.gradle seems to have fixed it:
buildscript {
    ext {
        buildToolsVersion = "28.0.3"
        minSdkVersion = 16
        compileSdkVersion = 28
        targetSdkVersion = 28

        googlePlayServicesVersion = "16.1.0" // default: "+"
        googlePlayServicesIidVersion = "16.0.1"
        firebaseVersion = "15.0.2" // default: "+"

    }
    ...
}


Add non-code example to agent-chat quickstart

Like we had in 0.2, we should provide an example that does not require setting up docker or running code locally - see here: https://microsoft.github.io/autogen/0.2/docs/Getting-Started

Closed by #3849


brings react card into react example repo

Updates the react-example to use the react card for development. We had a lengthier method to build and test; hopefully this should ease some pain in developing on the sdk card.

Oh actually, receiving an error:
errors
:
[{message: "The current user is not authorized to access this resource.",…}]
0
:
{message: "The current user is not authorized to access this resource.",…}
extensions
:
{code: "AUTH_NOT_AUTHENTICATED"}
locations
:
[{line: 99, column: 3}]
message
:
"The current user is not authorized to access this resource."
path
:
["notificationHistory"]

It appears that an auth token isn't being sent for notification history until after submitting the user login info. any thoughts?

we can fix the notificationHistory issue separately.
Just a heads up @marukohao this PR will probably conflict with yours (#126 )


please make a fix for no scroll bar or custom scroll bar css looks ugly

@thatbeautifuldream this project is closed a long time ago. still thankyou for the contribution buddy !


integer out of range when pk is uuid

get error when pk is uuid.
ERROR 2021-03-28 17:29:30,102 log 1274593<PHONE_NUMBER>10016 Internal Server Error: /en-us/manager/trades/order/f1f522e9-4e00-4401-8700-0ce6ba6879db/change/
Traceback (most recent call last):
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/fsm_admin/mixins.py", line 164, in _do_transition
    trans_func(request=request, by=request.user)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm/__init__.py", line 520, in _change_state
    return fsm_meta.field.change_state(instance, func, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm/__init__.py", line 326, in change_state
    result = method(instance, *args, **kwargs)
TypeError: obsolete() got an unexpected keyword argument 'request'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/fsm_admin/mixins.py", line 168, in _do_transition
    trans_func(by=request.user)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm/__init__.py", line 520, in _change_state
    return fsm_meta.field.change_state(instance, func, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm/__init__.py", line 326, in change_state
    result = method(instance, *args, **kwargs)
TypeError: obsolete() got an unexpected keyword argument 'by'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
psycopg2.errors.NumericValueOutOfRange: integer out of range
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/core/handlers/base.py", line 181, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/lib/python3.9/contextlib.py", line 79, in inner
    return func(*args, **kwds)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/contrib/admin/options.py", line 614, in wrapper
    return self.admin_site.admin_view(view)(*args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/utils/decorators.py", line 130, in _wrapped_view
    response = view_func(request, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/views/decorators/cache.py", line 44, in _wrapped_view_func
    response = view_func(request, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/contrib/admin/sites.py", line 233, in inner
    return view(request, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/contrib/admin/options.py", line 1656, in change_view
    return self.changeform_view(request, object_id, form_url, extra_context)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/utils/decorators.py", line 43, in _wrapper
    return bound_method(*args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/utils/decorators.py", line 130, in _wrapped_view
    response = view_func(request, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/contrib/admin/options.py", line 1534, in changeform_view
    return self._changeform_view(request, object_id, form_url, extra_context)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/contrib/admin/options.py", line 1580, in _changeform_view
    self.save_model(request, new_object, form, not add)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/fsm_admin/mixins.py", line 188, in save_model
    self._do_transition(transition, request, obj, form, fsm_field)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/fsm_admin/mixins.py", line 171, in _do_transition
    trans_func()
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm/__init__.py", line 520, in _change_state
    return fsm_meta.field.change_state(instance, func, *args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm/__init__.py", line 345, in change_state
    post_transition.send(**signal_kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/dispatch/dispatcher.py", line 177, in send
    return [
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/dispatch/dispatcher.py", line 178, in <listcomp>
    (receiver, receiver(signal=self, sender=sender, **named))
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm_log/backends.py", line 82, in post_transition_callback
    return _pre_transition_callback(sender, instance, name, source, target, StateLog.objects, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django_fsm_log/backends.py", line 28, in _pre_transition_callback
    manager.create(**values)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/query.py", line 447, in create
    obj.save(force_insert=True, using=self.db)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/base.py", line 753, in save
    self.save_base(using=using, force_insert=force_insert,
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/base.py", line 790, in save_base
    updated = self._save_table(
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/base.py", line 895, in _save_table
    results = self._do_insert(cls._base_manager, using, fields, returning_fields, raw)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/base.py", line 933, in _do_insert
    return manager._insert(
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/query.py", line 1254, in _insert
    return query.get_compiler(using=using).execute_sql(returning_fields)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/models/sql/compiler.py", line 1397, in execute_sql
    cursor.execute(sql, params)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/debug_toolbar/panels/sql/tracking.py", line 198, in execute
    return self._record(self.cursor.execute, sql, params)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/debug_toolbar/panels/sql/tracking.py", line 133, in _record
    return method(sql, params)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 98, in execute
    return super().execute(sql, params)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 66, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/home/yswtrue/Code/seatower_erp/.venv/lib/python3.9/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.DataError: integer out of range

Hi @yswtrue
Indeed the code is pretty deeply tied to pk being an integer. It won't be a small change, but if you need support for it, feel free to submit a PR :-)

Is this issue fixed? I ran to this issue again today.

Is this issue fixed? I ran to this issue again today.

I faced this issue. It's still not fixed...


RxMobius: Adds support for RxMobius.loopFrom to accept starting effects

@pettermahlen I made the changes, + added tests + port the changes over to rx2

@togi or @pettermahlen can you merge this PR, I cant.


Understanding the isExhausted formula is too exhausting

Hi Rickard,
We made a change to ScalaTest that has caused some pain for users using our 3.0 milestone releases, and we think it may best be addressed by revisiting this line of code in ScalaCheck:
https://github.com/rickynils/scalacheck/blob/master/src/main/scala/org/scalacheck/Test.scala#L292
What I wanted to do in ScalaTest in 3.0 was dial down the default minSuccessfulTests from 100 to 10. (So we did that.) The reason that I thought that will be better is that when people use ScalaTest, they run tests over and over again both on developer desktops and on continuous integration servers. Once freshly written code is working, those tests are often run for years to catch regressions, and they almost never fail. So the default of 100 ends up just wasting a lot time, electricity, and money, releasing greenhouse gasses that warm the planet with no benefit to show of improved software quality. In other words, even with a default minSuccessfulTests of 10, you'll get 100 trials after running your test suite 10 times, which is often done in just one day. I think that's fast enough for regression testing, and what I felt is that the defaults in ScalaTest should be ones that make sense for that kind of regression testing.
Ok, so what that changes is that sometimes people use a generator that discards quite a large percentage of the generated values, because it is quicker for them to do that than write a new generator that is more accurate. And sometimes, the first 11 values generated turn out to be all discarded, whereas one of the first 101 values would likely not have been. The formula ScalaCheck uses for determining exhaustedness seems to short circuit in that situation. If we set minSuccessfulTests to 10 and maxDiscardRatio to 5.0, then users expect it would not give up until at least 50 evaluations were discarded, but it gives up after 11. To see an exact use case with numbers (based on a user issue), see the bottom of this issue:
https://github.com/scalatest/scalatest/issues/661
I would be happy to suggest a change to the formula with a PR, but I don't know exactly why it is the way it is now. It may be as simple as adding one more clause to the Boolean expression that says that plain old d is greater than minSuccessfulTests * maxDiscardRatio.
n+d > minSuccessfulTests && 1+workers*maxDiscardRatio*n < d && (d > minSuccessfulTests * maxDiscardRatio)

But I'm not sure. I'd be interested in understanding the formula better.
Thanks.
Bill

The scaladocs on maxDiscardRatio say this:
The maximum ratio between discarded and passed tests allowed before ScalaCheck gives up and discards the property. At least minSuccesfulTests will always be run, though.

The problem you have is that the current implementation checks if maxDiscardRatio has been violated after each test run. If the very first test is discarded, we already have an infinitely high discard ratio. The n+d > minSuccessfulTests condition is there to protect against such cases. However, when minSuccessfulTests is small (as in your case), it doesn't help all that much (as you've described above).
To me, it is not immediately clear that minSuccessfulTests=10 and maxDiscardRatio=5.0 should allow up to 60 evaluations (I mean, we could even continue testing, maybe sometime we find enough passing tests to get the discard ratio down...). But maybe it is the reasonable thing to do.
So, I think we could switch isExhausted to just your condition:
d > minSuccessfulTests * maxDiscardRatio

Now, that d is per worker, so the above condition will probably create extra discard evaluations (beyond maxDiscardRatio) when workers > 1. A less excessive implementation would be:
d * workers > minSuccessfulTests * maxDiscardRatio

But that assumes all workers have the same pass/discard ratio, which is not a safe assumption.
I have implemented this in c358554385362f0f774409d7d69e8cb3874fcab5. If you think it looks OK (especially if the changed scaladoc makes sense), I'll merge it to master.
A side-note: The test for this (in https://github.com/rickynils/scalacheck/blob/c358554385362f0f774409d7d69e8cb3874fcab5/jvm/src/test/scala/org/scalacheck/TestSpecification.scala#L65) is an example of when ScalaCheck really shines. I tested d * workers > minSuccessfulTests * maxDiscardRatio and other variations, and ScalaCheck immediately came up with weird combinations of workers, minSuccessfulTests and maxDiscardRatio that falsified the property.


Merge tabs across windows into one

Check for existing issues

[X] Completed

Describe the feature
As a Zed lover, I would like to have the option of merging tabs across all windows into one.
The above feature may be further sophisticated by asking the user which windows they want to merge (by showing a pop up of Zed windows to choose from)
While this is generally useful feature, one that is supported by many apps (not just editors), it would help deal with this particular problem: Using Open With in Finder menu to open files in Zed ends up one file per window. Yes, you can select multiple files and open it in Zed. But that would work only if you know the set of files beforehand. In times of troubleshooting and such, it is more ad hoc and on-demand way of opening files (at random).
If applicable, add mockups / screenshots to help present your vision of the feature
No response

Probably blocked by this:

https://github.com/zed-industries/community/issues/493


Remove metrics arg from service manifest used to install Piped on Cloud Run

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/changelog


The ClusterInfo may support a group id, the iterator should be able to call group data provider to expand the group id.

false

https://github.com/project-chip/connectedhomeip/blob/f0f5de0d79ae1c51d7fddbf70900ebac0acd899a/src/app/AttributePathExpandIterator.h#L67-L72

This issue was generated by todo-issue based on a TODO comment in f0f5de0d79ae1c51d7fddbf70900ebac0acd899a.

@rcasallas-silabs ?

It looks like WriteHandler already got updated to expand group paths, but that logic wasn't added to AttributePathExpandIterator.
The cleaner option here might be to move it to the expand iterator as the final home for that logic, but this is just purely a clean-up exercise.
Punting out of 1.0.


Generate bindings for IEEE802.15.4 radio on ESP32-C6/H2

As previously discussed, we will make esp-ieee802154 depend on esp-wifi-sys so that we can move it into esp-hal and keep that repo blob-free.
I've already confirmed I can build esp-ieee802154 using esp-wifi-sys including these changes.

@bjoernQ do you track changes for esp-wifi-sys in the CHANGELOG here? Happy to add an entry, just want to ask first 😅

@bjoernQ do you track changes for esp-wifi-sys in the CHANGELOG here? Happy to add an entry, just want to ask first 😅

I didn't think we need to add a channeling entry for this. Thanks


(fleet provisioning): Feature to change the certificate path

Feature Description
In Greengrass Easy setup, the -cert-path option allows you to specify the location where the certificate generated during setup is stored, although this is not shown in the documentation github issue & github pr.
Use Case
The ability to specify a storage location for certificates obtained through fleet provisioning as well would make certificate management more flexible and easier to deal with when a device is reset.
Proposed Solution
Add an optional YAML configuration parameter such as certRootPath and if this parameter is set, the location of the certificate file is certRootPath instead of rootPath.
Other
N/A

[x] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

Hi
Thanks for reaching out to us and apologies for the radio silence on this issue. For this issue, I would recommend installing Greengrass using an initial configuration and setting the root CA path in that file. Please find an example of the same on this installation page.

The method you described is the path where the root CA is placed, and I believe that the certificates generated after provisioning are unconditionally placed directly under rootPath in the current implementation. The intention is to control this placement position.

Yep, certs are placed relative to rootPath currently.
I noticed you checked the box for implementing, if you'd like to contribute a PR, I'd be happy to review: should be a matter of adding an optional configuration and then overriding the rootPath config later on.

@jcosentino11 I apologise for not responding to your comments here for a long period of time. I had been able to fix the problem in my local environment and had confirmed that it was working, but I missed the timing to PR it. The fix now seems to be in progress and I will wait with anticipation for this merge. Thank you!

Glad to hear you're unblocked! PR has been merged and will need to go through testing before release


added human readable version of newly added classification from #61

This adds the human version of what has been merged in https://github.com/enisaeu/Reference-Security-Incident-Taxonomy-Task-Force/pull/61


Scripting the VI Includes Calls This Module an 'Alarm Module'

This doesn't affect execution (it still runs as the Tag Select), but its really confusing when you look in the Includes VI and see this class named Alarms.

Yes not sure where its getting the alarms module or exactly where to change it. If you right click on the node and select use long name it will use tag select module.
I created the module using the alarms as a starting point.

I think you change it in the runtime class properties under Documentation in the 'Localized Name' field

Yes it was that property but It was the one in configuration class

The runtime class was misnamed in that property also. Did you change that one as well? We drop the runtime class in the VI includes, not the config.

Yes updated both of them


Fix compile errors with latest bevy

Also get rid of the now-redundant .system() calls

Ooops, didn't see the other pr (#102 )... Well, fwiw this one doesn't touch cargo.toml and has a few extra fixes

Thanks @johanhelsing. I'm updating the upstream bevy_mod_raycast now.


need install pytorch in Xavier ?

I watched  pytorch is installed in docker
but NVIDIA Jetson AGX Xavier have no  pytorch
need install pytorch in Xavier ?

https://docs.nvidia.com/deeplearning/frameworks/install-pytorch-jetson-platform/index.html


V9 scene import does not nest folders

On V9, when importing scenes, the folder structure created in the scene sidebar is not nested, resulting in a Moulinette folder that is empty, and then empty subfolders beneath it.

Thanks for reporting it!
Fixed in moulinette-scenes-3.0.1 and moulinette-tiles-4.0.1.


Added the sinon and phantomjs in devDependencies

If you don't have phantomjs and sinon globally installed when you run the unit tests it gives an error.

Thank you! Great catch.


Fix #20 for branch 3.5

Coverage remained the same at 79.07% when pulling e805426727fd2a4d10050ac4514b3868a41be600 on SalomonBrys:3.5.1 into 0fbdb0914110c08f42c83e7ce0605a6e2805886c on wupdigital:support/3.5.x.


Update braces to 2.3.1

Update braces to 2.3.1
Braces has 2.3.0 has a ReDos vulnerability: https://snyk.io/vuln/npm:braces:20180219
This is fixed 2.3.1.

Coverage remained the same at 98.498% when pulling 987d5b596ac1b6b3ef625621a7885e0428c6e1a8 on maxcbc:bump_braces_to_2.3.1 into a95a1f83b57fdb88889f9cbe4299cf52899cecad on paulmillr:master.

This is unnecessary. The fix is covered in the semver range, and chokidar is used in locally-run build tools where I'm not very concerned about users trying to ReDos themselves.


Some GTFS fields are optional, the types should mirror that

Would it be desirable for the types to add ? next to optional fields? Right now, typescript gives errors when assigning types to objects that do not contain those optional fields.
And thanks for this package, it's really helpful!

thanks, this was released in v2.0.0


Current Default limit for storage accounts per region per subscription is 250 ?

>Storage
>There is a limit on the number of storage accounts per region as well in a given subscription. The default limit is 200 and >includes both Standard and Premium Storage accounts. If you require more than 200 storage accounts in a given region, >make a request through Azure Support. The Azure Storage team will review your business case and may approve up to 250 >storage accounts for a given region.
the default limit is 250 on other page and on my own subscriptions,.
https://docs.microsoft.com/en-us/azure/azure-subscription-service-limits#storage-limits
Resource	Default limit
Number of storage accounts per region per subscription, including both standard and premium accounts	250

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

ID: 8ea6aa65-3cbb-83d5-0f16-23b58c52cb88
Version Independent ID: 91fa8a56-cde6-06e2-b1bb-d521a7819c30
Content: Manage and request resource quotas - Azure Machine Learning
Content Source: articles/machine-learning/service/how-to-manage-quotas.md
Service: machine-learning
Sub-service: core
GitHub Login: @nishankgu
Microsoft Alias: nigup

@gutzeit1106 Thanks for the feedback. We are investigating into the issue and will update you shortly.

@gutzeit1106  Thanks for pointing this out. @nishankgu The document needs an update with the storage limits as pointed out in Storage service documentation and as seen under the subscription limits in azure portal..

fixed, thanks for reporting.

@gutzeit1106 thanks for reporting. We have fixed the issue and will now proceed to close this issue. You'll see the change live in a few hours. #please-close


Website restored to initial parameters!!!

I used BetterCMS Demo with source code; after publishing the project, the next day all the content has been restored to the default parameters.
How to avoid this, thank you for your cooperation.

Question answered here: http://www.bettercms.com/support/using-bettercms/bettercms-demo-website-restored-to-intial-content/


Tidal range display

0.7 introduced the display of the tidal range neap/spring/normal which is very nice :)
Checking the values however, I was wondering how spring/neap is done. Usually, I would expect spring around full/new moon, neaps around first and last quarter with a period of ~4 days for each phase (normal is sometimes shorted to 3 days due to solar vs. lunar days).
In Trail Sense I see e.g. normal for 3rd and 4th of September 2020, while the official tables give a spring from 2nd  (full moon) till the 5th, then a normal from 6th..9th and neap from 10th (3rd quarter)..13th so I think the spring/neap period in Trail Sense is shorter?
Note that I use tables for the German Bight. In this areas we see a delay of tides of some days as the North Sea is a bit special compared to the free oceans. See e.g. Tide timing
isRelatedTo: #125

Thank you, maybe I was a bit too ambitious with this feature - I didn't consider how local variations could affect this as well. The algorithm is really simple: is the moon is full/new then it is a spring tide and same for neap tides and the quarter moons.
The algorithm should really be if the moon was full in the last ~80-90 hours, then spring. I'll try to address this and compare against some tide tables.
With that said, I may mark this feature as experimental under the newly added setting in 0.8.0

is the moon is full/new then it is a spring tide and same for neap tides and the quarter moons.

BTW: this isn't too bad, especially if you allow for a spring delay of n days. One would need to look up the n required for the current place, but they are actually tabulated or you can derive it from an old fashioned printed or electronic tide table by checking the lowest low / highest high next to full/new moon.


Refactor of content on Choosing your courses page

Context
Update of content as per new content design
Changes proposed in this pull request
Before:

After:

Guidance to review
Review against design template: https://docs.google.com/document/d/1zjQYNBrEggefjuwdBtNoCH7k9xeZlHAQ8I1CSvqwH8I/edit#
Link to Trello card
https://trello.com/c/Prml0X76/1841-dev-content-changes-on-choosing-your-courses-page
Things to check

[x] This code doesn't rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Thanks @george-bit !
Just noticed a few things improve on the content - see here:
https://docs.google.com/document/d/1zjQYNBrEggefjuwdBtNoCH7k9xeZlHAQ8I1CSvqwH8I/edit?usp=sharing
Sorry and thanks :))

@EmmaFrith - No worries I've amended based on the new design. Please review.

thanks @george-bit - perfect!


lazyload the inline style="background-image:url('file.jpg')"

Dear,
I was wondering if the plugin can lazyload the inline background images added directly to the html markup (style="background-image:url('file.jpg')")!
I am using a bespoke platform that I can not change the HTML markup and BFG image is added as an inline style tag!
Thank @verlok

Sorry for the delay, I thought I answered 2 days ago but my message wasn't actually sent.
If you really can't change the HTML markup, there's nothing you can do.


Add Dom1n1cL's file

I added some code, to make the second slide more readable, let mew know what you think

Looks good, happy to proceed


Does not work

I have tried following the instructions in the README as well as manipulating both the client connection and server connection and nothing seems to work.
When it does not blow up with a problem, the best I can get is a constant reconnection and the
Socket.IO server version: 4.6.1
@skgdev/socket.io-msgpack-javascript version: 1.2.0
Server
import { Server } from "socket.io";
import { build } from<EMAIL_ADDRESS>
const io = new Server(3000, {
  serveClient: false,
  parser: customParser,
  cors: environment.cors,
  maxHttpBufferSize: 3.2e8,
});

io.on("connection", (socket) => {
  console.log('connected:', socket.id);
  setTimeout(() =>
    socket.emit('hello', { say: 'hello world!'}),
    2000,
  );
  socket.on('error', err => console.error('socket:', err));
  socket.on('disconnecting', (reason, desc) => console.log({ reason, desc }));
  socket.on('disconnect', (reason, desc) => console.log('disconnect:', { reason, desc }));
});
io.on('error', console.error);

Socket.IO client version: 4.6.1
@skgdev/socket.io-msgpack-javascript version: 1.2.0
Client
import { io } from "socket.io-client";
import { build } from<EMAIL_ADDRESS>
const socket = io("ws://localhost:3000/", {
  withCredentials: true,
  parser: customParser,
  // transports: ["websocket"] // uncomment for websocket-only transport
});

socket.on("connect", () => {
  console.log(`connect ${socket.id}`);
});

socket.on('disconnect', (reason, desc) => console.log({ reason, desc }));

Expected behavior
Whether specifying transports option or not on the client, the socket will connect, perform the handshake successfully and messages will be passed back and forth in binary encoding.
Platform:

Device: MacBook Pro 2020 M1
OS: MacOS Ventura 13.2.1
Browser: Chrome

Additional Context
I tried using the example code from the README for the client which is:
const socket = ioc('ws://localhost:' + PORT, {
  parser: customParser
});

which just bricks b/c socket.io complains that customParser.Encoder is not a function.
I tried marking the package as external for webpack and esbuild.
I tried importing using require() instead and no difference.
I tried importing as:
import customParser from<EMAIL_ADDRESS>
instead of just importing build and no difference.
I looked at the code and it appears to have the exact same implementation and interface as the msgpack parser maintained by the socket.io team so I don't understand what is wrong.

I tried to reproduce the issue locally (by copying your code sample) and my client kept attempting to reconnect.
The on-disconnect error was websocket connection closed.
I managed to establish a connection by including the ignoreUndefined option in the encoder. However, I am not sure why this is happening. Needs more investigation.
import msgPackParser from<EMAIL_ADDRESS>msgPackParser.build({ encoder: { ignoreUndefined: true }, decoder: {} });

Additionally, the ES module file is not listed in the files property of the package.json file. So, if you use the es module instead of the common js, you may experience issues.
We will include the 'es' module file in the next release.

@eudaimos Did you try @georgiosgiatsidis suggestion?
I will give it a go this week most probably and let you know.

thanks @georgiosgiatsidis - I tried your solution and it works.
AND I was able to import using the following both server and client and it works:
import { build } from<EMAIL_ADDRESS>

Probably the new client of socket.io by default includes data in the handshake that come with undefined values and that's something that msgpack only supports via the ignoreUndefined flag. I think we can close this issue for now.


Revisit BSA 804 and 805 Logic as the address being accessed might be not safe

As offline discussion within the team, the arbitrary read/writes to device NP memory in BSA 804 is not necessarily a legal thing to do.
It is possible to do these accesses, but you would need to know what addresses would be valid to access without side effects. Unfortunately, this really depends on the device you are accessing, so the safe address may change based on the device, and one of those addresses being accessed could be a magic “start on fire” register.
Before we find a SAFE address or some other way to check PCI_IN_13 rule, we might need to disable/remove BSA 804.

Thanks @sunnywang-arm for raising the ticket.
Tests 804 and 805 performs P/NP BAR memory access checks. The tests perform

BAR memory  in-range address access
BAR memory out of range address access

As you also highlighted access to a not a KNOWN address might lead to side-effects on systems. After further internal discussion, we have decided to split the test cases into two parts.

In-range address access will be covered for baremetal run of tests
Out of range address access will be covered for both baremetal and certification run of tests.

We are planning to upstream the changes by next week.
Thanks,
ACS team

Hi @sunnywang-arm,
Required changes for the issue are merged in the ACS.
Thanks,
ACS team

Hello @sunnywang-arm,
Changes for the issue are merged in the mainline.
Thanks,
ACS team


SparseDataFrame does not allow single value data

You can create a normal DataFrame with, e.g., pandas.DataFrame(0, index=[1, 2, 3], columns=["A", "B", "C"]).  However, this fails with a SparseDataFrame:
>>> x = pandas.SparseDataFrame(0, index=[1, 2, 3], columns=["A", "B", "C"])
Traceback (most recent call last):
  File "<pyshell#97>", line 1, in <module>
    x = pandas.SparseDataFrame(0, index=[1, 2, 3], columns=["A", "B", "C"])
  File "c:\users\brenbarn\documents\python\extensions\pandas\pandas\sparse\frame.py", line 123, in __init__
    NDFrame.__init__(self, mgr)
UnboundLocalError: local variable 'mgr' referenced before assignment

It looks like the code in SparseDataFrame.__init__ is not handling the case where the data is a single value.  It has a bunch of if statements to handle different kinds of data, but falls through without creating mgr if none of them are met.
This makes it difficult to create an "empty" SparseDataFrame whose default fill value is something other than nan, e.g., pandas.SparseDataFrame(0, index=[1, 2, 3], columns=["A", "B", "C"], default_fill_value=0).  In fact, I would suggest that if a default_fill_value is provided but no data is provided, the SparseDataFrame should be filled with the fill value (not nan), but perhaps that warrants a separate issue.

# "real world" data that can be used to demonstrate this behavior
#http://archive.ics.uci.edu/ml/datasets/URL+Reputation
# http://scikit-learn.org/stable/modules/generated/sklearn.datasets.load_svmlight_file.html
import pandas as pd
from sklearn.datasets import load_svmlight_file
X,y = load_svmlight_file('Day0.svm')
dataframe=pd.SparseDataFrame(X)

Traceback (most recent call last):
File "", line 1, in
File "C:\Anaconda\lib\site-packages\pandas\sparse\frame.py", line 122, in init
NDFrame.init(self, mgr)
UnboundLocalError: local variable 'mgr' referenced before assignment

looks like this is not tested, so pull-requests are welcome.
this is a nice work-around actually.
In [28]: pd.SparseDataFrame( index=[1, 2, 3], columns=["A", "B", "C"]).fillna(0)
Out[28]:
     A    B    C
1  0.0  0.0  0.0
2  0.0  0.0  0.0
3  0.0  0.0  0.0


Bundle is invalid when used via global export

There are two problems with https://cdnjs.cloudflare.com/ajax/libs/react-highcharts/6.0.0/highcharts.src.js and https://cdnjs.cloudflare.com/ajax/libs/react-highcharts/6.0.0/highstock.src.js:

root["Highcharts"] = factory(root["react"]) should be root["Highcharts"] = factory(root["React"]).
root["Highcharts"] = factory(root["React"]) should be root["ReactHighcharts"] = factory(root["React"]).

These both apply to the highstock module as well, I'm not sure about the others.

Thanks for the issue, I'll take a look when I have some time, or feel free to make a PR.

Should be fixed by #90, feel free to reopen


Implement FABRIK

Expected Behavior

FABRIK IK function
http://www.andreasaristidou.com/FABRIK.html

@article{AristidouFABRIK,
	author    = {Aristidou, Andreas and Lasenby, Joan },
	title     = {\textsc{FABRIK}: \textsc{A} fast, iterative solver for the Inverse Kinematics problem},
	issn 	  = {1524-0703},
	doi 	  = {DOI: 10.1016/j.gmod.2011.05.003},
	journal   = {Graphical Models},
	number    = {5},
	pages     = {243-260},
	publisher = {Elsevier},
	volume    = {73},
	year      = {2011}
}

Pseudo code: https://medium.com/unity3danimation/create-your-own-ik-in-unity3d-989debd86770

Closing stale issue.


[Select] Make SelectInput's ref contain the DOM node

[x] I have followed (at least) the PR section of the contributing guide.

fix #32898

It would be great if you could add a test to verify if the new behavior works correctly.

Removing node would be a breaking change at this point, so we need to keep both for now. Let's mark it with // TODO v6: Remove to get rid of it in the next major.

I browsed through the existing tests, and I found one that is somewhat relevant to my change: https://github.com/mui/material-ui/blob/master/packages/mui-material/src/Select/Select.test.js#L1103

You can create a new test based on this one that would verify that expect(ref.current).to.have.tagName('input');

While trying to write the test, I found out that my proposed fix does not actually work. Here's what happens when spreading:

Closing this because of https://github.com/mui/material-ui/pull/32983#issuecomment-1145927726.


Make db as exportable

Since new clients can now specify where they want to connect to,
either nb db or sb db, make db identifier as exportable so that
when used in other packages, it can be available to use as is.

Thanks guys for quick review. Merging it as got 2 lgtms.


Product demo

Hi, is there any ready demo that would allow me to test end to end the whole Courier notification experience?

hi,the issues previously weren't being monitored.  i know its been a minute but do you still nee assistance?


worker_groups_launch_template failed to update asg_desired_capacity number

I have issues
I'm submitting a...

[x] bug report

What is the current behavior?
I updated asg_desired_capacity from 2 to 4, and asg_max_size from 4 to 5, eventually only asg_max_size got updated into Spot ASG, the number of asg_desired_capacity stays the same 2
$ vi terraform-aws-modules\terraform-aws-eks\examples\eks_test_fixture\main.tf
...
  worker_groups_launch_template = [
    {
      # This will launch an autoscaling group with only Spot Fleet instances
      name                                     = "workers_group_launch_template_spot"
      instance_type                            = "m4.large"
      override_instance_type                   = "t2.large"
      asg_desired_capacity                     = "4"
      asg_max_size                             = "5"
      asg_min_size                             = "1"
...

$ terraform plan
  ~ update in-place
Terraform will perform the following actions:
  ~ module.eks.aws_autoscaling_group.workers_launch_template
      max_size: "4" => "5"
Plan: 0 to add, 1 to change, 0 to destroy.

$ terraform apply
module.eks.aws_autoscaling_group.workers_launch_template: Modifying... (ID: test-eks-AbEd-workers_group_launch_template_spot20190222043445948300000014)
  max_size: "4" => "5"
module.eks.aws_autoscaling_group.workers_launch_template: Modifications complete after 0s (ID: test-eks-AbEd-workers_group_launch_template_spot20190222043445948300000014)

What's the expected behavior?
Should've implemented asg_desired_capacity 4, and asg_max_size 5 in AWS.
Any tips?
Are you able to fix this problem and submit a PR? Link here if you have already.
Environment details

Affected module version: v2.2.1
OS: Ubuntu 16.04.3 LTS
Terraform version: Terraform v0.11.11

Any other relevant info
Thanks!

@confiq Anything I missed?

Working as intended. Desired capacity isn't changed after initial creation because it would conflict with runtime autoscalers like cluster-autoscaler.

@confiq Anything I missed?

I would not call this a bug. It might be it's not documented well but not a bug...

Thank you! Not yet installed cluster-autoscaler, will do soon.

@brant4test I ended up appending the capacity to the name so that a change in desired capacity could force a re-create. This in combination with asg_force_delete on the worker group I can force a group to 0 which will remove the nodes, has proved useful for my blue/green worker groups so that I can force a group to spin down after I have drained and moved pods from one group to another and ensure that the group is no longer visible to the autoscaler.
"name", "foo-${var.desired_capacity}",

So there is no way to resize using Terraform? I understand the issues for autoscaler, but it should be optional to ignore or not desired_capacity.
We are trying to have some blue/green setup which implies having alwas 2 ASGs running, one with zero size and the other with the desired size and everytime we need to update the AMI we just move things to the other ASG, which needs resize on both groups.

So there is no way to resize using Terraform?

Correct.
Currently it's not possible to support both ways as lifecycle rules in Terraform don't support interpolation. Therefore we support the most common and correct method, which is to use the cluster-autoscaler.
Also worth noting that scaling down the ASG is usually a manual process anyway. To do this safely you need to drain the node and then terminate it. If you are scaling the cluster down solely via changing the ASG min/max/desired, then the ASG will just terminate some instance without draining, which in many situations is not desirable. But I understand in your usecase it might be OK.
Hope that makes sense!


Add error logs in scrollable div

Hello,
The error logs are harder to track as the component height increases on each error appended. It would be good to have a scrollable window for error logs.

A scrollable window in a fixed height div.


Remove obsolete usage of party repository

This change removes a property referring to the party repository which
is not used anyway and requires the Party package to be installed
(which is not declared as a dependency in composer.json).

Thx for the fix :)


launcher.launchUrl() always returns false on Flutter web

Steps to reproduce

Just call launcher.launchUrl() in an web application.

Expected results
launcher.launchUrl() should return true and launch the URL in any type of application if the URL is valid.
Actual results
launcher.launchUrl() always return false in a web application, even when the URL is valid. It works as expected on Linux and macOS.
Code sample
Here is a simple test application, showing the issue. You can read the code, run the application. Please see additional comments.
Screenshots or Video
N/A
Logs
N/A
Flutter Doctor output
[✓] Flutter (Channel stable, 3.22.3, on macOS 14.6 23G80 darwin-arm64, locale en-US)
    • Flutter version 3.22.3 on channel stable at /Users/milke/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (2 weeks ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] Connected device (3 available)
    • macOS (desktop)                 • macos                 • darwin-arm64   • macOS 14.6 23G80 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad • darwin         • macOS 14.6 23G80 darwin-arm64
    • Chrome (web)                    • chrome                • web-javascript • Google Chrome 127.0.6533.89

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Closing this as a duplicate of https://github.com/flutter/flutter/issues/139783.


Scrolling to last item is position-dependant

http://jsfiddle.net/ww8Lkpqs/2/ - there, first two case shows work of "scrollTail" function (first on div with static positioning, second on div with relative positioning), and second - work of "scroll" function. Due to list.position calls in scrollTail function - it works different.
While adding "position: relative" to your jcarousel containers is not a big deal, i still think that this should at least be reflected somewhere in documentation

It is actually reflected, isn't it?
From jCaousel installation doc:
/*
This is the visible area of you carousel.
Set a width here to define how much items are visible.
The width can be either fixed in px or flexible in %.
Position must be relative!
*/
.jcarousel {
    position: relative;
    overflow: hidden;
}
[...]

oops.. sorry(


Add "Change Password" function to GUI

The CLI allows you to change your account password, but the GUI does not offer a similar function. It should be fairly easy to add, since a REST endpoint likely exists to provide this functionality to the CLI.

https://opensource.ncsa.illinois.edu/jira/browse/NDS-185

Resolved as part of #104


fix: constraint containerType outboundnode boundary. Fixes #12997

Fixes #12997
Motivation
Root cause:
say we have dag
task: A(containerset)
task: B(containerset), depends: A
task: C(containerset), depends: A

if task B is processed and started before the controller can process C,
C will find the deepest node in A without considering boundaries and mistakenly attach to B
Modifications
if a node(containerType) child node do not have the same boundyID, we return the current node as the deepest outbound node.
Verification
Tested:
Before code change

ran the workflow from the issue ticket, failed: incorrect dependency and controller panic
ran new unit test without change, failed as expected

After code change

ran the workflow from the issue ticket, succeeded with the correct dependencies
ran unit test with the code change, succeed

Merging as this has been approved and has no conflicts & passes tests.


Model Builder Error in when Training

System Information (please complete the following information):

Model Builder Version: <IP_ADDRESS>7905
Visual Studio Version 16.7.3

Describe the bug
Model Builder fails in the "Train" phase with the error message:
Specified argument was out of the range of valid values.
Parameter name: index
  at System.Linq.Enumerable.ElementAt[TSource](IEnumerable`1 source, Int32 index)
   at Microsoft.ML.ModelBuilder.AutoMLService.Experiments.AutoMLExperiment`3.<ExecuteAsync>d__21.MoveNext() in /_/src/Microsoft.ML.ModelBuilder.AutoMLService/Experiments/AutoMLExperiment.cs:line 136
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ML.ModelBuilder.AutoMLEngine.<StartTrainingAsync>d__30.MoveNext() in /_/src/Microsoft.ML.ModelBuilder.AutoMLService/AutoMLEngineService/AutoMLEngine.cs:line 147

To Reproduce
Steps to reproduce the behavior:

Start Model Builder
Choose scenario Value prediction
Continue to Data tab.
Pick a file, select the result column and click train.
Wait until several Trainer are explored and notice the exception happening.

Expected behavior
Taining step completes without error popups.

Thanks for reporting this issue. In order to help,  could you provide the dataset you used?

Hi,
I saw that there was a new version of the model builder available and updated it - now training works, I guess it was fixed in the newer version.
Take care,
Martin


Is not working

i get this when executed   :
Exception in thread "main" java.lang.IllegalStateException: Target host is null
at org.apache.http.util.Asserts.notNull(Asserts.java:46)
at org.apache.http.impl.client.InternalHttpClient.determineRoute(InternalHttpClient.java:125)
at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:185)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:115)
at ch.boye.GDriveUpload.DriveResumableUpload.getCurrentByte(DriveResumableUpload.java:101)
at ch.boye.GDriveUpload.GDriveUpload.uploadFile(GDriveUpload.java:67)
at ch.boye.GDriveUpload.GDriveUpload.main(GDriveUpload.java:28)

Could you please show the full execution command line?
Did you create the credentials.json file?

Hello. i create GDriveUpload.json in ~/.GDriveUpload/ downloaded from https://console.developers.google.com/project/etc..........
And  i used this command:
./GDriveUpload -upload ~/2gdrive.tar.gz
And  i Get:
GDriveUpload
Got refresh_token from file.
Updating access_token from Google
Creating resumable upload...
Exception in thread "main" java.lang.IllegalStateException: Target host is null
at org.apache.http.util.Asserts.notNull(Asserts.java:46)
at org.apache.http.impl.client.InternalHttpClient.determineRoute(InternalHttpClient.java:125)
at org.apache.http.impl.client.InternalHttpClient.doExecute(InternalHttpClient.java:185)
at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:115)
at ch.boye.GDriveUpload.DriveResumableUpload.getCurrentByte(DriveResumableUpload.java:101)
at ch.boye.GDriveUpload.GDriveUpload.uploadFile(GDriveUpload.java:67)
at ch.boye.GDriveUpload.GDriveUpload.main(GDriveUpload.java:28)
Thankx for the fast answer


finished switching out emojis for icons

finished switching out emojis for icons in At a Glance and Associated Materials sections. Also added icons for the Explore Featured Gardens and Explore Other Gardens headers. Looked around the website to find any places to add icons but I personally thought it looks better/more minimalistic as is~!

Looks good to me! merging into staging


Update README.md

When reviewing the README I noticed two things:

We should verify that the whitespace for the hostapd vlan file is correct and make people aware that whitespace is important. I think it has to be tab separated. That could be the issue rubens encountered.
Maybe we should update the local build instructions and tell people to create a build directory first and then run cmake in there. I personally really like it when I can just delete the build directory afterward. And we can just add it to the .gitignore

I checked the hostapd source code and verified that spaces or tabs are fine in hostapd.vlan.
The build directory seems to be mostly personal preference. Since the build works right now, I would leave it as-is.

One more thing: We wanted to add a reference to the existing issue in FRR, didn't we?


hoodie does not install with pnpm package manager

Hi there hoodie people,
Just wanting to let you know that I tried to take hoodie for a spin for the first time using your quickstart instructions but using pnpm instead of installing with npm and I was out of luck.
I have raised an issue on pnpm over here. @zkochan, the developer of pnpm has offered a solution for the problem which can easily be fixed at your end apparently.
Regards, Justin Johansson

Hoodie is mostly dead. Sorry.

No worries. Thanks for reply.


Adding KNN Classifier

First try. I haven't figure out already about output kinds.

@kirtsar Hi there. I wondered if you would consider revisiting this PR. The MLJ model API has simplified a bit since and I think your implementation was pretty close to finished.

This PR is now made obsolete by #47, now merged onto #dev
Thanks to @kirtsar for his work on this PR.


Add server support for null values

Currently, convert_cell only has support for np.nan, which corresponds to NaN in query responses. This PR adds support for pd.NA (which crops up in columns with nullable or pyarrow dtypes), which should correspond to NULL in responses.

Codecov Report

Merging #1062 (551711b) into main (afbe4df) will decrease coverage by 0.41%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1062      +/-   ##
==========================================
- Coverage   81.30%   80.89%   -0.41%
==========================================
  Files          78       78
  Lines        4375     4376       +1
  Branches      786      786
==========================================
- Hits         3557     3540      -17
- Misses        645      655      +10
- Partials      173      181       +8

Impacted Files
Coverage Δ

dask_sql/physical/rel/custom/analyze_table.py
89.28% <ø> (ø)

dask_sql/physical/rel/custom/show_columns.py
86.36% <ø> (ø)

dask_sql/server/responses.py
97.89% <100.00%> (+0.02%)
:arrow_up:

dask_sql/integrations/fugue.py
84.44% <0.00%> (-13.34%)
:arrow_down:

dask_sql/physical/rex/core/literal.py
58.09% <0.00%> (-2.86%)
:arrow_down:

dask_sql/physical/rel/custom/export_model.py
89.18% <0.00%> (-2.71%)
:arrow_down:

dask_sql/physical/rel/logical/window.py
93.56% <0.00%> (-1.76%)
:arrow_down:

dask_sql/physical/utils/statistics.py
89.41% <0.00%> (-1.18%)
:arrow_down:

dask_sql/_version.py
32.76% <0.00%> (-1.13%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


os/user: TestGroupIds fails on windows-arm

Our windows-arm builder fails https://build.golang.org/log/024228bdaccb0c0f92bc505bdfc1f2287b9ef818 in
--- FAIL: TestGroupIds (0.02s)
    user_test.go:141: &{Uid:S-1-5-18 Gid:S-1-5-18 Username:NT AUTHORITY\SYSTEM Name:SYSTEM HomeDir:C:\Data\Users\System}.GroupIds(): The user name could not be found.
FAIL
FAIL	os/user	0.614s

I looked at os/user.TestGroupIds windows implementation.
It appears that os/user.Current() returns NT AUTHORITY\SYSTEM. @jordanrh1 why? I expect it return some real user. Are you running builder as a service or something?
Also GroupIds() function fails with The user name could not be found. error. This is too general error to guess what happened. @jordanrh1 do you mind debugging to see what happened there?
Thank you.
Alex

Hi Alex,
The builder currently runs as System account. I'll look into running it under a normal user account.
I'll debug the GroupIds function. Could be related to running as System account.
Jordan

I restarted the builder under DefaultAccount (a standard user account), and the os/user tests are now passing :)
ok  	os/user	0.655s

I restarted the builder under DefaultAccount (a standard user account), and the os/user tests are now passing :)
ok  	os/user	0.655s

Indeed they are. Thank you very much. Closing this now.
Alex


Integration with Sublime 3

Hi !
on https://github.com/kiteco/plugins/tree/master/sublimetext3 the link is not working and the instruction are not really clear. I tried to install it using the Package Control but it doesnt seem to find it.

Hi @Wonay, thank you for the report!  Indeed those instructions were outdated.  I updated that README.md to point to kite.com which is currently the recommended way to get set up with Kite.

okay thanks

So there is no more integration with sublime ?

Kite supports Sublime Text 3.  You should see Kite detect your Sublime Text 3 installation during the setup flow, and it's one click to install the Kite plugin, along with the other software needed to use it (the native Kite app).

Oh ok I see. Thanks


Update one unclear method name

the changed method is used to get the good QR code pics which regarded as the name. But the original name getKeyName is not clear what is the function of this method. I just want to change the name a little bit to make it clear.

这是机器人还是智能AI，自动向开源项目提交方法改名的commit。

目的是什么？
明明都是国内中文项目，却都用英文写issue。


skipped example. Why?

why is not PorterDuff example for chapter 11 published?

The code that generated the example PorterDuff images from Chapter 11 is remarkably similar to what is built in much more detail in Chapter 12, so I didn't include both versions.


Fix assertion when inputs, targets, or weights are not necessary

If the model does not has any targets or sample_weights, the corresponding lists hold None values.
While None values still contribute to the length of the tensors variable, assert len(val_data) == len(tensors) asserts.
This faulty behavior is circumvented by refining the tensors variable so that it holds no more None values.
Signed-off-by: Timo Korthals<EMAIL_ADDRESS>

Summary
see above
Related Issues
None
PR Overview

[n] This PR requires new unit tests [y/n] (make sure tests are included)
[n] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[y] This PR is backwards compatible [y/n]
[n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Any update (unit test)?

hey, will be back in ~3 weeks. So I can provide a unit test by then.
Am So., 29. Sept. 2019 um 22:01 Uhr schrieb François Chollet <
<EMAIL_ADDRESS>
Any update (unit test)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keras-team/keras/pull/13333?email_source=notifications&email_token=AAZ35V4RB6XWYRBDNW4IEXLQMECSNA5CNFSM4IXXGWFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD735LCA#issuecomment-536335752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZ35V27K5Q5AV46AG2YDTLQMECSNANCNFSM4IXXGWFA
.