Double stats on items appear with multiple mods.

"*Items with attack speed and attack damage modifiers now display the value of the modifier (i.e. the change that will be applied to those attributes) not the end result."
The conflict makes it so the vanilla stats and the new PlayerEx stats show at once:
https://i.imgur.com/ueCaGHV.png
Mods that I've found this problem with so far:
WTHIT/  HWYLA  - If "view mods" is turned on, this occurs.
Tiered - If you get an item that doesn't have any damage modifiers, then this occurs. When you get a damage modifier, it doesn't though. This mod combined with yours is absolutely perfect for making Minecraft into an RPG so compatibility between the two would be really nice for an RPG modpack.
sihywtcamd/ sihywtcamc - These mods also cause it, which are great mods for increasing MC difficulty, which is really nice with a mod like PlayerEx that makes you more powerful. This way the game doesn't become a cakewalk.
MC Dungeon Weapons/ Armor - Seems to cause issues alongside this too. Gets very confusing.
I can probably find more mods that cause this. I think this feature just doesn't play well with other mods.
I think the feature is a tad confusing even when it all works right, especially since your attack speed is always negative as punching is faster.
Is there anyway to revert stats to display as the original? Would be nice to be able to just turn this off and not have to worry about conflict.
*This mod also seems to stop weapons and armor from displaying the mod name at the bottom from WTHIT, though not a huge deal.

Hi there,
Thanks again for this in-depth bug report, I really appreciate it. I think I know what causes these issues, and maybe I'll add a config option to disable the change in item entity attribute modifier displays. The only reason why I changed it was because not changing it also results in the numbers being wrong, as good ol' vanilla hard codes them - meaning that if you actually deal 10 damage per hit, your sword may still say 6 or 7.
Thanks again, keep them coming!

Right, I get that the modified values are more accurate to what you're actually doing.
Either solution sounds good to me. If I run into anything else I'll post it. :)

Fixed with v2.0.2 (I ended up just removing the tooltip fix in favour of implementing a better one that I can spend more time thinking over later; for now weapons will show vanilla/whatever mod changed tooltips).


Update after testing for Tf2 "Using Time (C++)" and "Time travel (C++)"

. Updates code references to match the content create during previous tutorials.
. Add hints to build the package before executing the launch commands.
Found during testing. see (https://github.com/osrf/ros2_test_cases/issues/977) and (https://github.com/osrf/ros2_test_cases/issues/1009)

@BorisBoutillier can we also incorporate the necessary changes from https://github.com/osrf/ros2_test_cases/issues/1009 in this PR?

I have added a commit with similar changes for the Time Travel with Tf2 C++ tutorial page, next in the Tf2 tutorial series, as propoosed by @Yadunund


Doc/fixes

Small fixes on the doc:

add monad as a keyword for the examples
clarify the Read method

Coverage increased (+0.005%) to 63.645% when pulling 396d5d59628fc9afef16dcee82975a85b6814d2e on doc/fixes into 94b86be6140229be1ebbc8d59bc86e29826aec7d on master.

Coverage increased (+0.005%) to 63.645% when pulling 396d5d59628fc9afef16dcee82975a85b6814d2e on doc/fixes into 94b86be6140229be1ebbc8d59bc86e29826aec7d on master.


About-our-courses e2e tests

Issue link / number:
#1089
What changes did you make?
Created about-our-courses e2e test for english version
Why did you make the changes?
So the page is covered with test and help us to validate it is working when making changes or migrating it to app router
Did you run tests?
Yes

Sorry @boodland this test isn't necessary as this page is being retired


[SPARK-45192][UI] Fix overdue lineInterpolate parameter for graphviz edge

What changes were proposed in this pull request?

The edge.lineInterpolate no longer takes effect for drawing edges. It shall be replaced by d3.curve
Why are the changes needed?

trivial bugfix for ui.
Does this PR introduce any user-facing change?

no
How was this patch tested?

Before

After

Was this patch authored or co-authored using generative AI tooling?

no

cc @sarutak @HyukjinKwon @dongjoon-hyun thanks

Thank you @dongjoon-hyun

I like this!


ObjectPageSection without subSections is not visible

OpenUI5 version: 1.73.2
Browser/version (+device/version): Chrome 79.0.3945.117
Any other tested browsers/devices(OK/FAIL): no
URL (minimal example if possible): https://openui5.hana.ondemand.com/entity/sap.uxap.ObjectPageSection/sample/sap.uxap.sample.ObjectPageSection
Steps to reproduce the problem:

Create a ObjectPageLayout with a ObjectPageSection with only a Title and no subSections.

What is the expected result?
The ObjectPageSection should be visible.
What happens instead?
The ObjectPageSection is not visible
Any other information?
This issue is visible in the OpenUi5 sample. The Section 2 is missing.

Hi @radianer,
By design, empty sections are not displayed. This is because the purpose of the sections is to structure the app content, so sections have no meaning when no content.
This is documented in
https://sapui5.hana.ondemand.com/#/topic/2e61ab6c68a2480eb666c1927a707658,
section "Additional Rules for Displaying Sections and Subsections"
Thanks and regards,
Diana


orion for cryptographic functionality

This PR introduces several changes:

As previously discussed, this replaces the ring and chacha20-poly1305-aead dependencies with orion to provide XChaCha20Poly1305 and thereby also removes the internal HChaCha20 implementation.
Manual PartialEq implementation for the Branca type so that comparison on the secret key runs in constant-time.
Manual implementation of Debug to avoid the secret key leaking into logs.
Checking expired tokens (timestamp + ttl) is now done with checked_add to avoid a silent wrap-around on release builds.

is it okay to use orion given that on its page author recommends not to use for production ?

is it okay to use orion given that on its page author recommends not to use for production ?

That's a very valid concern. I am the author of orion. The warning about production use comes mainly from the fact, that there has not been a third-party security audit done on any current version of the library. It's also relatively new and has not seen that much use yet. If you're interested in the security and testing of it, I can recommend looking at the wiki.
I have been in touch with @return about using orion and we settled upon me opening this PR.

@charleschege As with several other Rust cryptography crates, I will have to put a warning notice up for using branca in a production environment since orion doesn't have a formal security audit yet. @brycx author aims for orion to have a security audit soon.
But in comparison to the previous crate that was used, orion is actively maintained and supports the full XChaCha20-Poly1305 construction in pure rust which was a hard requirement that was need in branca. Which was the reason why I chose to use orion.
@brycx Thanks for your pull request and I'll merge this in now.


micro_speech_lstm_test taking a very long time

The CI time for Xtensa has gone up by a lot (from ~10 mins to 20+ mins). The root cause is the micro_speech_lstm_test.
Running this locally:
make -f  tensorflow/lite/micro/tools/make/Makefile TARGET=xtensa TARGET_ARCH=hifi4 XTENSA_CORE=F1_190305_swupgrade OPTIMIZED_KERNEL_DIR=xtensa test_micro_speech_lstm_test -j8

confirms that that is the case.
Current workaround is to disable the test.
This slow-down may be related to #590.
tagging @cad-audio

@advaitjain  could you please add --turbo in "tensorflow/lite/micro/testing/test_xtensa_binary.sh" file
--- a/tensorflow/lite/micro/testing/test_xtensa_binary.sh
+++ b/tensorflow/lite/micro/testing/test_xtensa_binary.sh
@@ -26,7 +26,7 @@ declare -r MICRO_LOG_PATH=${TEST_TMPDIR}/$1
declare -r MICRO_LOG_FILENAME=${MICRO_LOG_PATH}/logs.txt
mkdir -p ${MICRO_LOG_PATH}
-xt-run $1 2>&1 | tee ${MICRO_LOG_FILENAME}
+xt-run --turbo $1 2>&1 | tee ${MICRO_LOG_FILENAME}
if [[ ${2} != "non_test_binary" ]]
then

@advaitjain  For HiFi4 we have not yet optimized ops, it is using C ref code and hence it is taking more time to run ms_lstm binary

@advaitjain we are fine to disable micro_speech_lstm_test example for HiFi4/HiFi3Z core.


Security Concerns with SDK V2

Description
The current implementation of the new SDK requires supplying API keys on the client-side using environment variables, especially in Next.js. This approach poses potential security risks since the keys are bundled with the JavaScript code during production, making them accessible to users.
Suggestions
I propose migrating the following processes to the server-side to enhance security and prevent exposure of sensitive information:
Bundler Migration: Move the bundler logic to the server-side to avoid exposing API keys to the client.
Paymaster Migration: Transfer the paymaster functionality to the server-side to ensure the secure handling of payment-related tasks.
Smart Account Creation on Server-Side: Implement smart account creation on the server-side to prevent the need for sending the JsonRpcSigner to the backend, which can introduce security vulnerabilities.
Current Limitations
At present, we encounter the following limitations:
JsonRpcSigner: The JsonRpcSigner cannot and should not be sent to the backend. As a result, creating the smart account does not work in this manner.
API Key Access: Even if we pass the paymaster and bundler from the server to the frontend, the API keys can still be accessed, posing a security risk.
Community Collaboration
I urge the community to come together and brainstorm novel solutions to address these security concerns and improve the overall safety and privacy of the SDK implementation.
Let's collaborate to find a more secure approach that ensures the confidentiality of sensitive data and mitigates potential risks associated with the current client-side handling of API keys.
Thank you for your attention to this matter. Your contributions and ideas are highly appreciated. Together, we can make the SDK safer and more robust.

A way to achieve this could be to have a Custom Bundler implemented with a custom api (to nextjs ssr or other backend) call instead of direct calls with bundled api key. This way the biconomy api call could be done in complete secret, without exposing it in Frontend.

I was just exploring this as well. I peeked into the source code for the SDK and figured out the paymaster object is only used in buildTokenPaymasterUserOp, so you can safely move the paymaster logic to the to the BE:

create userop on the FE
send the userop to the BE, attach payment master data and send it back to the FE
send userop from the FE

This works great for the paymaster part, except if you're using the buildTokenPaymasterUserOp
I'm not sure how to handle this part for the bundler yet, seems SmartAccount is using it, so would have to dig into it a bit more. My idea would be to refactor the SDK to make it so that the BiconomySmartAccount would not require the bundler and/or paymaster objects and be only responsible for creating userops which can then be bundled separately.


Specify cache directory for apt-get commands

Is it possible to specify a cache directory for the apt-get commands that crouton invokes, i.e. add -o dir::cache::archives="/path/to/somewhere" to the command line flags of apt-get?

You should be able to set preferences (such as cache directory) in the chroot's apt.conf, and crouton will respect it.


I need to run the testcase robotium from a java project

From<EMAIL_ADDRESS>on February 28, 2011 00:50:54
What steps will reproduce the problem? 1. 2. 3. What is the expected output? What do you see instead? What version of the product are you using? On what operating system? robotium 2.1 , windows Please provide any additional information below. My problem is to run my test case from a java project that is external to Android.
My problem is to run my test case from a java project that is external to Android.
If I can send running my test class robotium from a java project outside, I can restart the test and therefore I can restart my application.
My goal is always to be able to restart the application.
Original issue: http://code.google.com/p/robotium/issues/detail?id=78

From<EMAIL_ADDRESS>on February 28, 2011 01:18:21
It is currently not possible to run your Robotium test cases outside of the dalvik vm.
If you use a tearDown() with solo.finalize() your activities will be finished after each test case is finished and then the next test case in your ActivityInstrumentationTestCase2 will re-start your application. It will not resume it as finalize() has finished it.
Status: Invalid


Add quantization state reader and writer

Description
Adds a quantization state writer and reader for native engines
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.
[x] API changes companion pull request created.
[x] Commits are signed per the DCO using --signoff.
[x] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I still need to write unit tests, but raising now for initial feedback

I'll add end to end integration tests to fully test the quantization reader and writer once the integration is complete with indexing and query flow.  Additionally, I'll add another unit test for the second read method with the PR for integration to the query flow.

I see that writer is only added for flush and not for merge. Will there be separate PR for that?

I thought that the merge method eventually uses flush based on https://github.com/opensearch-project/k-NN/blob/main/src/main/java/org/opensearch/knn/index/codec/KNN990Codec/NativeEngines990KnnVectorsWriter.java#L90
If that is not the case, I can add in separate PR

I see that writer is only added for flush and not for merge. Will there be separate PR for that?

I thought that the merge method eventually uses flush based on https://github.com/opensearch-project/k-NN/blob/main/src/main/java/org/opensearch/knn/index/codec/KNN990Codec/NativeEngines990KnnVectorsWriter.java#L90
If that is not the case, I can add in separate PR

No it doesn't use same method. I would not recommend creating another PR. Please add it in 1 PR as this PR is very small and I would like to see both flush and merge for QS writer in 1 shot.

I see that writer is only added for flush and not for merge. Will there be separate PR for that?

I thought that the merge method eventually uses flush based on https://github.com/opensearch-project/k-NN/blob/main/src/main/java/org/opensearch/knn/index/codec/KNN990Codec/NativeEngines990KnnVectorsWriter.java#L90
If that is not the case, I can add in separate PR

No it doesn't use same method. I would not recommend creating another PR. Please add it in 1 PR as this PR is very small and I would like to see both flush and merge for QS writer in 1 shot.

With My CR lots of things has been changed with Flush and Merge , We don't have to save Quantization State at two places , it will be in one Function , We should even hold on Flush also .

Still need a unit test for the second read method now that it's fully implemented.

BWC failures are not due to this PR

Adding skip-changelog because entry is in the release notes instead

Merging the PR to unblock others. The test which is failing will be fixed in next iteration


Add 'main' branch as release PR target

add 'main' branch as release target
update CHANGELOG

@giantswarm/team-biscuit PTAL, this PR > new devctl release > devctl version bump in giantswarm/github > giantswarm/config release

@giantswarm/team-biscuit PTAL, this PR > new devctl release > devctl version bump in giantswarm/github > giantswarm/config release


Extra entries in Hosted zone records

Hi @williamsandonz ,
I went successfully through adding custom domain to our front-end using your serverless plugin and it works well. Thank you for that.
There was only one issue I had - domain only works with www. I went through Route 53 config and I can see A and AAAA entries created correctly for www, but nothing for without www - which would be the reason why DNS doesn't resolve address without www.

I am not sure if it's design or not, but in order to make it work without www prefix you need to manually add records in Route 53.
Would it be possible to extend/add functionality to have setting to include those entries? I can try to implement it and if you are ok with it and then add this to npm package.
I know you can do redirect in Edge Lambda, but I don't think there is always need for using it? Give me your thoughts :)
Thanks!
Pawel

Hey @pzawadzkipz . Sorry it's been a long time since I've lookjed at this plugin and I can see something aren't very clear about it how it works  @:(!
If you look at the updateDnsRecords function you can see it will only create the non WWW variant if both edgeLambda and edgeLambda.redirect and set in the config and if so it will use the value for config.edgeLambda.redirect.from. So basically you can only create both WWW and non WWW if you are using the edgeLambda redirect. I think I decided on this behaviour because assumed you'd always want one to redirect to the other (for Google indexing etc) which perhaps was bad foresight on my part. But if you do use that, it should work fine. If you're still having trouble, paste your config in here and I can take a look.

Ok! Will have a look at changing the config. Many thanks again :)

Closing as not an issue, it's just how plugin works.


[Solve] Input is focused when you creat new Service

I've worked on focusing input while you create new Service, by adding cdkFocusInitial attr to input and removing from button.
resolve Issue: #614

Thanks for your contribution!


Update for site launch

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: colinizer sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:x: Validation status: errors

File
Status
Preview URL
Details

rd/index.html.yml
:x:Error

Details

rd/index.yml
:x:Error

Details

rd/map.yml
:x:Error

Details

:x:Error

Details

rd/index.html.yml

[Error] Unable to load file: index.html.yml via processor: ProfileList: index.html.yml is not in supported format: (Line: 50, Col: 12, Idx: 1294) - (Line: 50, Col: 13, Idx: 1295): While scanning for the next token, find character that cannot start any token.

rd/index.yml

[Error] Unable to load file: index.yml via processor: ProfileList: index.yml is not in supported format: (Line: 50, Col: 12, Idx: 1294) - (Line: 50, Col: 13, Idx: 1295): While scanning for the next token, find character that cannot start any token.

rd/map.yml

[Error] Unable to load file: map.yml via processor: ProfileList: map.yml is not in supported format: (Line: 51, Col: 12, Idx: 1518) - (Line: 51, Col: 13, Idx: 1519): While scanning for the next token, find character that cannot start any token.

[Error] index.html.yml is not in supported format: (Line: 50, Col: 12, Idx: 1294) - (Line: 50, Col: 13, Idx: 1295): While scanning for the next token, find character that cannot start any token.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Must sign the CLA
The tagline words are too long
Separate tagline interests with a / instead of , to remain consistent with all other RDs


fix!: Change default balancing_mode to CONNECTION for backends

fix #146

/gcbrun

@graham-faulkner-geotab can u plz add docs upgrade to v7 in docs folder and mention that this release is backward incompatible and changes default value of balancing_mode?

/gcbrun

thank you @imrannayer


Add TSDF match cost function.

@wally-the-cartographer merge

Merge requested by authorized user kdaun. Merge queue now has a length of 1.


No intellisense in v.1.25.0 on Linux (downgrading to v1.24.4 resolves issue)

There's no intellisense on v1.25.0 on Linux with VSCode 1.68.0. Downgrading to v1.24.4 resolves the issue.
Here is a dump of dotnet --info, if this helps. I'm not sure what other information would help to diagnose the issue.
I'm happy with v1.24.4 but figured you should know about the behavior regression on linux.
.NET SDK (reflecting any global.json):
Version:   5.0.408
Commit:    da985e2a23
Runtime Environment:
OS Name:     debian
OS Version:  11
OS Platform: Linux
RID:         debian.11-x64
Base Path:   /usr/share/dotnet/sdk/5.0.408/
Host (useful for support):
Version: 5.0.17
Commit:  6a98414363
.NET SDKs installed:
3.0.103 [/usr/share/dotnet/sdk]
3.1.419 [/usr/share/dotnet/sdk]
5.0.408 [/usr/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.0.3 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.25 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.0.3 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.25 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

@elliottp6 Can you please share the OmniSharp Log from the VS Code Output pane?


Can't use npmUpgradeProvider with bun

Compiled package with dnt
npm upgrade provider works fine on node but on bun:
▶ bun /git/windmill/cli/npm/esm/main.js upgrade
87 |             }
88 |         });
89 |     }),
90 |     get writable() {
91 |         if (stdoutWritable == null) {
92 |             stdoutWritable = node_stream_1.default.Writable.toWeb(process.stdout);

                  ^
TypeError: undefined is not a function
      at node:stream:3045:33
      at writable (/git/windmill/cli/npm/node_modules/@deno/shim-deno/dist/deno/stable/variables/std.js:92:61)
      at start (/git/windmill/cli/npm/esm/deps/jsr.io/@cliffy/command/1.0.0-rc.5/upgrade/spinner.js:153:95)
      at /git/windmill/cli/npm/esm/deps/jsr.io/@cliffy/command/1.0.0-rc.5/upgrade/upgrade_command.js:86:22
      at /git/windmill/cli/npm/esm/deps/jsr.io/@cliffy/command/1.0.0-rc.5/upgrade/upgrade_command.js:77:30
      at /git/windmill/cli/npm/esm/deps/jsr.io/@cliffy/command/1.0.0-rc.5/command.js:970:2

and that's because bun is missing toWeb: https://github.com/oven-sh/bun/issues/3927
Is there a way to write the upgrade provider such that it doesn't rely on that api (after shimmed by dnt)

Commenting this line is enough:
https://github.com/c4spar/deno-cliffy/blob/main/command/upgrade/spinner.ts#L201
Maybe removing the spinner as an option would solve the issue for now?


tmux-mem-cpu-load: init at 3.4.0

Motivation for this change
Init tmux-mem-cpu-load.
Things done

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

[ ] NixOS
[ ] macOS
[X] other Linux distributions (Ubuntu 20.04)

[ ] 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 nixpkgs-review --run "nixpkgs-review wip"
[X] 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)
[N/A] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111609 run on x86_64-darwin 1

1 package built:

tmux-mem-cpu-load


[BUG] Error when inertia closing BottomSheet.

Describe the bug
When the BottomSheet is open and the keyboard is open in it, when trying to close the BottomSheet by inertia, an error occurs:
======== Exception caught by rendering library =====================================================
The following assertion was thrown during layout:
A RenderFlex overflowed by 21 pixels on the bottom.
The relevant error-causing widget was:
The overflowing RenderFlex has an orientation of Axis.vertical.
The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and black striped pattern. This is usually caused by the contents being too big for the RenderFlex.
Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the RenderFlex to fit within the available space instead of being sized to their natural size.
This is considered an error condition because it indicates that there is content that cannot be seen. If the content is legitimately bigger than the available space, consider clipping it with a ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex, like a ListView.
Steps to Reproduce
Open BottomSheet with TextField inside, give focus to the TextField, and try to close the BottomSheet by inertia.
What is the expected behavior?
There should be no error.
Additional context
Happens on android and on iOS.

https://github.com/surfstudio/flutter-bottom-sheet/pull/54 fixed


Please add zvariant crate to formats list

Hi, it'd be great if you could add zvariant to the advertised list of formats. Thanks.

@dtolnay wow, that was quick! Thanks.


Add artifact summary

It would be nice to be able to inspect an artifact from the command line, think of something similar to what was possible with biom summarize-table but generalized to any Artifact. For example qiime tools peek could produce this information:
(qiime2-2017.9) yovazquezbaeza:redacted$ qiime tools peek redacted/feature-table.qza
UUID:        1d4488e2-c62c-4df0-82c2-98e098ed4aea
Type:        FeatureTable[Frequency]
Data format: BIOMV210DirFmt
Summary:
Number of samples: 11
Number of features: 11,111
... other stats ...

Or alternatively:
(qiime2-2017.9) yovazquezbaeza:redacted$ qiime tools peek redacted/pcoa-results.qza
UUID:        1d4488e2-c62c-4df0-82c2-98e098ed4aea
Type:        PCoAResults
Data format: OrdinationDirectoryFormat
Summary:
Number of samples: 11

Duplicate of #197

🏖 🍻


Labels for sequence categories (breathing scanning deforming)

Hi, thanks for releasing the code and dataset.
Could you provide the labels for the sequence categories mentioned in the paper: breathing, scanning, and deforming? Thanks again.

Hello, have you run it successfully? Where can I download the dataset?


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app/src/atom.js:54:11)
Object.activate (/Users/danward/.atom/packages/todo-show/lib/show-todo.coffee:33:9)

Is this still present with 0.8.0?

Ah, it looks like for some reason Atom was not automatically updating from 0.6.0. So I missed the fix. Manually Uninstalled and Re-Installed now to fix. Thanks

I can confirm that this is present with 0.8.0. Here are my errors:

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/usr/share/atom/resources/app/src/atom.js:54:11)
ShowTodoView.openPath (/home/ben/.atom/packages/todo-show/lib/show-todo-view.coffee:260:9)

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/usr/share/atom/resources/app/src/atom.js:54:11)
ShowTodoView.moveCursorTo (/home/ben/.atom/packages/todo-show/lib/show-todo-view.coffee:269:25)

Closed by 0d966b6755f6cf01149502410828714119ff39a2.


[Feature] Graphical Way to Show/manipulate Jobs in Queue

Hey all,
I want to be able to show users what jobs are sitting in the queue, and possibly let them manipulate them there ("delete" before a job is run for example)
Has anyone created a way to do this yet? I'm sure it wouldn't be that hard since everything is sitting in the database, but I don't know much about it.
Ideally it would involve some kind of AJAX. A button could push something into the queue on the page, the queue would update (without a page reload) and then you could "delete" jobs there (again, without a page reload).
If it doesn't exist I'll try my hand at it, but I have zero experience with AJAX, and it took me days to figure out django-background-tasks in the first place ;)

Sorry for the late reply on this!
This is a great idea. @arteria/devs worked on something similar for internal use.  We can definitely contribute something to this use case.

Hey @philippeowagner glad to hear you like the idea! It would be great if this got published into the project. I have a use-case for Django-background-tasks where the end user needs to see the "status" of the job. it would be AMAZING to show some kind of "percent" of a job, but that I think is way more difficult (vs. just showing if a job is waiting, running, or complete)

hello @philippeowagner I was wondering if there's any progress in moving your internal use code to the project?

@JawnZ not really. The project name says it all … it's for background tasks.

Showing the status of tasks (Which you already have in the database) doesn't seem to violate the notion of background tasks.

No, it doesn't at all @JawnZ . Just saying that we didn't worked on the other module for at least 1.5 years and let it go based on the lake of usage. On the other side we use django-background-tasks in almost every project. We implement tasks based on priority and let everything work in the background.

makes sense


Support JDK 17

Why are the changes needed?

This PR aims to migrate the vanilla Zookeeper and Curator to the Kyuubi Shaded Zookeeper.
There is a known issue ZOOKEEPER-3779 that Zookeeper 3.4 client can not run on JDK 14 and above, in https://github.com/apache/kyuubi-shaded/pull/5, we fixed this issue by a surgical.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Seems build/dependency.sh still consumes the out-dated jars

Seems build/dependency.sh still consumes the out-dated jars @cfmcgrady @bowenliang123

Codecov Report

Merging #4849 (1e1d6bb) into master (b8ee006) will decrease coverage by 58.10%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master   #4849       +/-   ##
============================================
- Coverage     58.09%   0.00%   -58.10%
============================================
  Files           584     563       -21
  Lines         32593   31169     -1424
  Branches       4321    4058      -263
============================================
- Hits          18934       0    -18934
- Misses        11839   31169    +19330
+ Partials       1820       0     -1820

Impacted Files
Coverage Δ

.../main/scala/org/apache/kyuubi/util/SignUtils.scala
0.00% <0.00%> (-87.88%)
:arrow_down:

...a/org/apache/kyuubi/ha/client/DiscoveryPaths.scala
0.00% <ø> (-100.00%)
:arrow_down:

...ubi/ha/client/zookeeper/ZookeeperACLProvider.scala
0.00% <ø> (-85.00%)
:arrow_down:

.../ha/client/zookeeper/ZookeeperClientProvider.scala
0.00% <ø> (-73.92%)
:arrow_down:

...ha/client/zookeeper/ZookeeperDiscoveryClient.scala
0.00% <ø> (-73.58%)
:arrow_down:

...he/kyuubi/jdbc/hive/ZooKeeperHiveClientHelper.java
0.00% <ø> (-35.06%)
:arrow_down:

...rg/apache/kyuubi/server/api/v1/AdminResource.scala
0.00% <ø> (-85.10%)
:arrow_down:

...rg/apache/kyuubi/zookeeper/EmbeddedZookeeper.scala
0.00% <ø> (-86.67%)
:arrow_down:

... and 531 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Opened separated PRs for flaky tests, the rest of this PR is ready to go

Thanks, merged to master

Found the PR that broke the codecov @bowenliang123

Cool. How did these updates of java options break the codecov result?

@bowenliang123 I have no idea. Fortunately, the codecov has been restored

@bowenliang123 I have no idea. Fortunately, the codecov has been restored

Great!


ClockCustomizer - "Contestants: Do not edit this."

In the ClockCustomizer class are comments that says "Contestants: Do not edit this."
May I create and use my own ClockCustomizer to change the UI? (or update the ClockCustomizer?)

What would you like to change about the ClockCustomizer?
The same rules that apply to the clock (e.g. no network requests) also apply to the customizer.
Feel free to edit the default model values while you develop (e.g. update the default weather condition while you build UIs for each weather condition) but remember to revert your changes and submit the original ClockCustomizer with your project.

ClockCustomizer is provided, so that a user cycle through the various weather conditions, etc. to see how they are handled by the clock. Each submission should have this same set of conditions, which is one reason for not editing them. Please reopen if this still isn't clear.


ADDED : [LEARN REACT]

Added Bob Ziroll's React Course

Added C++ Foundational course from freeCodeCamp

Added C++ Foundational Course from freeCodeCamp

Added C++ Foundational Course from freeCodeCamp


Blank Dashboard and Discover Screen

My scenario is as follows.
I created 2 roles[role1, role2] , 2 users[user1, user2] and 2 tenants[tenant1, tenant2] and I mapped roles to users.
I loaded the data with two different index names[index_u1-, index_u2-]
user1 index_u1-* and user2 index_u2-* have read , write manage index permissions
when i sign in with a user name and password, i see a blank screen. The Dashboard and Discover screens are blank and I can't create anything.
if I map the role of kibana_user to users, users can create dashboards, but this time they can see someone else's dashboards (they can't see the data because the indexes are different)
if I map all_access role to users, users can create dashboards, but this time they can see someone else's dashboards and data (although the indexes are different)
how can i fix the problem?

@seanneumann  can this issue be looked upon by you?

created an issue on dashboard side https://github.com/opensearch-project/dashboards-anywhere/issues/140, closing issue here.


Update ExpOpConversionApprox

Our original implementation was ported from AMD, updating it to match the upstream implementation in this PR.

Rebase once PR #2981  lands

Can you add a description?

Can you add a description?

I think the abstract is descriptive enough.

Can you add a description?

I think the abstract is descriptive enough.

Actually the abstract is was stale. I have updated it to be more reflective of the change in the PR


Return custom error code

Hey
I'm trying to return a custom error code as shown here:
https://flask-restful.readthedocs.io/en/latest/extending.html?highlight=errors#define-custom-error-messages
At the bottom of at the page.
Here is a snippet from my code:
from flask import Flask, Response
from flask_restful import Resource, Api, reqparse, abort

class SamplesNotFound(HTTPException):
    pass

error_dict = {
        "SamplesNotFound": {
        'message': "No samples found for these features.",
        'status': 522,
    }
}

class Index(Resource):
   def get(self):
      abort(522)

app = Flask(__name__)
api = Api(app, errors=errors.error_dict)

api.add_resource(Index, '/index')

if __name__ == '__main__':
    app.run(host='<IP_ADDRESS>', port=5000, debug=False)

I get a 500 error code on the response instead of 522.
+
I want to get a json in the response and I get a string.
Please help
Best,
Roy

Use raise SamplesNotFound() in place of abort(522).
P.S. Make sure you're passing errors_dict to the Api constructor.  Your snippet has it written as errors.error_dict, which doesn't exist.


Hostagent logs are too noisy

Description
INFO level logs from "[hostagent"] contain many messages that could be debug level or duplicate messages.
Example log:
msg="[hostagent] Waiting for the essential requirement 1 of 2: \"ssh\""
msg="[hostagent] [VZ] - vm state change: running"
msg="[hostagent] Waiting for the essential requirement 1 of 2: \"ssh\""
msg="[hostagent] The essential requirement 1 of 2 is satisfied"
msg="[hostagent] Waiting for the essential requirement 2 of 2: \"user session is ready for ssh\""
msg="[hostagent] The essential requirement 2 of 2 is satisfied"
msg="[hostagent] Waiting for the optional requirement 1 of 6: \"systemd must be available\""
msg="[hostagent] Guest agent is running"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:53"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:53"
msg="[hostagent] The optional requirement 1 of 6 is satisfied"
msg="[hostagent] Not forwarding TCP [::]:22"
msg="[hostagent] Waiting for the optional requirement 2 of 6: \"containerd binaries to be installed\""
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:41899"
msg="[hostagent] The optional requirement 2 of 6 is satisfied"
msg="[hostagent] Waiting for the optional requirement 3 of 6: \"kubeadm to be installed\""
msg="[hostagent] The optional requirement 3 of 6 is satisfied"
msg="[hostagent] Waiting for the optional requirement 4 of 6: \"kubeadm to be completed\""
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:45865"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10248"
msg="[hostagent] Not forwarding TCP [::]:10250"
msg="[hostagent] The optional requirement 4 of 6 is satisfied"
msg="[hostagent] Waiting for the optional requirement 5 of 6: \"kubernetes cluster to be running\""
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2380"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2379"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2379"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2381"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10257"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10259"
msg="[hostagent] Not forwarding TCP [::]:6443"
msg="[hostagent] The optional requirement 5 of 6 is satisfied"
msg="[hostagent] Waiting for the optional requirement 6 of 6: \"coredns deployment to be running\""
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10249"
msg="[hostagent] Not forwarding TCP [::]:10256"
msg="[hostagent] The optional requirement 6 of 6 is satisfied"
msg="[hostagent] Waiting for the guest agent to be running"
msg="[hostagent] Waiting for the final requirement 1 of 1: \"boot scripts must have finished\""
msg="[hostagent] The final requirement 1 of 1 is satisfied"
msg="[hostagent] Copying config from /etc/kubernetes/admin.conf to /Users/nsoffer/.lima/cluster/copied-from-guest/kubeconfig.yaml"

Duplicate messages:
msg="[hostagent] Waiting for the essential requirement 1 of 2: \"ssh\""
msg="[hostagent] Waiting for the essential requirement 1 of 2: \"ssh\""

Debug level messages:
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:53"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:53"
msg="[hostagent] Not forwarding TCP [::]:22"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:41899"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:45865"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10248"
msg="[hostagent] Not forwarding TCP [::]:10250"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2380"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2379"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2379"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:2381"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10257"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10259"
msg="[hostagent] Not forwarding TCP [::]:6443"
msg="[hostagent] Not forwarding TCP <IP_ADDRESS>:10249"
msg="[hostagent] Not forwarding TCP [::]:10256"

Another logs that I see from time to time is:
msg="[hostagent] r.CreateEndpoint() = connection was refused"

I have working clusters (using complicated setup and running DR flows) so I guess this is not really an error and can reported as debug message, or maybe as a warning if this can help users to debug this issue.
If this is a real issue it need much more context in the error message (e.g. connection to which address? why the connection is needed?)

Another logs that I see from time to time is:
msg="[hostagent] r.CreateEndpoint() = connection was refused"

That is #1160.


Drag'n drop nodes into script broken

Godot version
v3.4.2.stable.mono.official [45eaa2daf]
System information
Windows 10, AMD, Nvidia
Issue description
When grabbing a node associated with an editor (e.g Spatial or CanvasItem) and dragging it into an open GDScript in the editor "script tab" the editor first time switches back to the scene view, then going back into the "scripting tab" dragging the node into the GDScript now correctly inserts the quoted relative node path.
While fixing this I've also realised that I use the $path/to/node.do_something() instead of get_node("path/to/node").do_something() so inserting the dollar-sign version would make the experience smoother, however not technically a bug per say, the enhancement is likely (without actually have looked at the source code) extremely tightly packed with the bug and could be ironed out like two birds with one stone. If you'd rather have this over at the feature suggestion rep let me know and I'll post it there instead.
(video):
https://user-images.githubusercontent.com/11806563/155209021-44e5821d-d49e-4fa5-870e-14d0856578f7.mp4
Steps to reproduce

Open project
new scene
new node
new child node
new GDScript connected to node
drag child node into script file.

Minimal reproduction project
N/a

The first part is duplicate of #33704 (in 3.5 you can mitigate it by holding Alt)
As for the second part, dollar operator supports quoted strings just fine, so you can do $"path/to/node". I once suggested that the quotes should be only inserted when needed: #33317


Filter not working properly

Discovered this while doing a portswigger lab called "Username enumeration via subtly different responses".
I was trying to filter the response body using flag -fr "/^Invalid username or password.$/"
Full command:
ffuf -w tmp_list_username  -X POST -d "username=FUZZ&password=asdasd" -u "https://ac291ffc1ec617b0c0e60cf300a600c2.web-security-academy.net/login" -c -fr "/^Invalid username or password.$/".
The expected output wouldn't come, even though the regex is correct, it can't filter the "." at the end.
What i had to do, was  to output all responses to a folder and use grep + find to search for a file that doesn't match this pattern "Invalid username or password." .

Hi @jgk0, do you have a sample of the website/api output?

The regex looks a bit strange, I mean it's a regex allright, but I assume the "/" are extra there. I suggest you to go to https://regex101.com/ and to select "Go" from the left menu and testing your regex out in order to ensure its correct.

Oh, and looks like you are also POSTing form data without the correct Content-Type header.


FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

Sandworm version
1.26.0
Describe the bug
<--- Last few GCs --->

[52485:0x130008000]   348912 ms: Mark-Compact 8016.7 (8230.0) -> 8003.9 (8233.0) MB, 7949.1 / 0.0 ms  (average mu = 0.104, current mu = 0.012) allocation failure; scavenge might not succeed
[52485:0x130008000]   357184 ms: Mark-Compact 8019.8 (8233.0) -> 8007.1 (8236.5) MB, 8194.0 / 0.0 ms  (average mu = 0.057, current mu = 0.009) allocation failure; scavenge might not succeed

<--- JS stacktrace --->

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory

To Reproduce
package.json
{
    "dependencies": {
        "@juggle/resize-observer": "^3.4.0",
        "@svgr/webpack": "^6.5.1",
        "axios": "^0.27.2",
        "axios-hooks": "^3.1.5",
        "dotenv": "^16.0.2",
        "formik": "^2.2.9",
        "gatsby": "^4.24.0-next.3",
        "gatsby-plugin-feed": "^4.23.0",
        "gatsby-plugin-gdpr-cookies": "^2.0.9",
        "gatsby-plugin-google-tagmanager": "^4.23.0",
        "gatsby-plugin-manifest": "^4.23.0",
        "gatsby-plugin-robots-txt": "^1.7.1",
        "gatsby-plugin-sass": "^5.23.0",
        "gatsby-plugin-schema-snapshot": "^4.6.0",
        "gatsby-plugin-sitemap": "^5.23.0",
        "gatsby-plugin-svgr": "^3.0.0-beta.0",
        "gatsby-plugin-use-query-params": "^1.0.1",
        "gatsby-source-strapi": "^2.0.0",
        "i18next": "^22.4.10",
        "i18next-http-backend": "^2.1.1",
        "jwt-decode": "^3.1.2",
        "qs": "^6.11.0",
        "react": "^18.2.0",
        "react-cookie-consent": "^8.0.1",
        "react-countup": "^6.3.1",
        "react-dom": "^18.2.0",
        "react-helmet": "^6.1.0",
        "react-i18next": "^12.1.4",
        "react-markdown": "^8.0.3",
        "react-redux": "^8.0.2",
        "react-responsive": "^9.0.0-beta.10",
        "react-scroll": "^1.8.7",
        "react-select": "^5.4.0",
        "react-slick": "^0.29.0",
        "react-syntax-highlighter": "^15.5.0",
        "react-tabs": "^5.1.0",
        "react-toastify": "^9.1.1",
        "react-visibility-sensor": "^5.1.1",
        "redux": "^4.2.0",
        "rehype-raw": "^6.1.1",
        "slick-carousel": "^1.8.1",
        "spinners-react": "^1.0.7",
        "use-query-params": "^1.2.3",
        "yup": "^0.32.11"
    },
    "devDependencies": {
        "@types/fs-extra": "^11.0.1",
<EMAIL_ADDRESS>"^4.0.7",
        "@types/qs": "^6.9.7",
        "@types/react": "^18.0.20",
        "@types/react-dom": "^18.0.6",
        "@types/react-helmet": "^6.1.5",
        "@types/react-redux": "^7.1.24",
        "@types/react-scroll": "^1.8.4",
        "@types/react-slick": "^0.23.10",
        "@types/react-syntax-highlighter": "^15.5.6",
        "@typescript-eslint/eslint-plugin": "^5.50.0",
        "@typescript-eslint/parser": "^5.50.0",
        "eslint": "^8.33.0",
        "eslint-config-prettier": "^8.6.0",
        "eslint-plugin-prettier": "^4.2.1",
        "eslint-plugin-react": "^7.32.2",
        "eslint-plugin-simple-import-sort": "^10.0.0",
        "prettier": "^2.8.3",
        "typescript": "^4.9.5"
    }
}

Expected behavior
I think this library should never take so much ram. Of course, after the transition from 8gb to 16gb, the build went through, but I imagine that with a larger package.json the problem will be much bigger.
export NODE_OPTIONS="--max-old-space-size=16384"

Node (please complete the following information):

Version 18.12.1 and 19.7.0
OS: macOS Ventura 13.2.1

From what I can see, the entire report is saved at once after its execution. I think that it would be much more optimal in terms of memory to divide the write to files into stages.

Thanks for reporting this, @xSyki!
We'll look into it asap.
cc @andreimarinescu

Hey, @xSyki. We've just released version 1.3.0 that should definitely help with the memory usage of the library. We still have some performance improvements in mind, however if you do have some time I'd be curious if things look better with this release. Overall, memory usage ought to have decreased and building the dependency graph should be faster.

@andreimarinescu After my tests in the new version, I didn't notice any difference. However, I found the biggest problem and put pr.

@xSyki thanks for the PR!
Reducing the max-depth is a good idea, but I feel truncating the tree chart further might make it less useful, especially for larger projects with deep trees - such as yours. I'd like to spend a bit more time looking into what's causing this bloat in the first place; your help would be very much appreciated 🙏 .
Here's what I did to test your setup:

I made a simple package.json based on your dependencies
used yarn 1.22.19 to install
set NODE_OPTIONS="--max-old-space-size=2048" to hard-limit my playground
used /usr/bin/time -l on OSX (that's -v on Linux) to run an audit using Sandworm v1.30.0

The audit completed successfully (around 1.5k dependencies scanned in total), and the time utility reported a peak memory usage of around 1.6GB.
To double-check, I also ran this via node --inspect-brk and Chrome's devtools - same outcome.
My first thought is: are you sure you're running v1.30.0? If so, sandworm -v or sandworm-audit -v should echo "1.30.0".

Yes of course I am using 1.30.0. Below are the logs from my scans, the first with npm, the second with yarn. It seems impossible that we have such different measurements with the same package.json.
I'm also wondering about the usability of svg, which is 190mb, my computer (macbook pro with m1) has a problem with opening such a file.
$ /usr/bin/time -l node ../sandworm-audit/src/cli/index.js
Sandworm 🪱
Security and License Compliance Audit
✔ Built dependency graph
✔ Got vulnerabilities
✔ Scanned licenses
✔ Scanned issues
⠋ Drawing tree chart...
<--- Last few GCs --->
[51874:0x118040000]   144201 ms: Mark-sweep 4006.8 (4140.9) -> 3996.0 (4143.9) MB, 3532.9 / 0.0 ms  (average mu = 0.122, current mu = 0.010) allocation failure; scavenge might not succeed
[51874:0x118040000]   147852 ms: Mark-sweep 4012.2 (4143.9) -> 4001.3 (4148.6) MB, 3629.5 / 0.0 ms  (average mu = 0.061, current mu = 0.006) allocation failure; scavenge might not succeed
<--- JS stacktrace --->
FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
time: command terminated abnormally
      148.14 real       119.93 user        21.11 sys
<PHONE_NUMBER>  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
             3649463  page reclaims
                   2  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
               14658  messages sent
               81095  messages received
                   2  signals received
                5590  voluntary context switches
              562054  involuntary context switches
<PHONE_NUMBER>376  instructions retired
<PHONE_NUMBER>31  cycles elapsed
<PHONE_NUMBER>  peak memory footprint
zsh: abort      /usr/bin/time -l node ../sandworm-audit/src/cli/index.js
$ node ../sandworm-audit/src/cli/index.js -v
1.30.0

$ /usr/bin/time -l node ../sandworm-audit/src/cli/index.js
Sandworm 🪱
Security and License Compliance Audit
✔ Built dependency graph
✔ Got vulnerabilities
✔ Scanned licenses
✔ Scanned issues
⠋ Drawing tree chart...
<--- Last few GCs --->
[52763:0x128040000]   179769 ms: Mark-sweep 4012.8 (4137.1) -> 4001.7 (4142.9) MB, 2852.2 / 0.0 ms  (average mu = 0.105, current mu = 0.012) allocation failure; scavenge might not succeed
[52763:0x128040000]   185148 ms: Mark-sweep 4017.8 (4142.9) -> 4007.0 (4147.1) MB, 5322.7 / 0.0 ms  (average mu = 0.046, current mu = 0.010) allocation failure; scavenge might not succeed
<--- JS stacktrace --->
FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
time: command terminated abnormally
      185.60 real       128.36 user        29.56 sys
<PHONE_NUMBER>  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
             5164678  page reclaims
                 183  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
               14985  messages sent
               77753  messages received
                   2  signals received
                6794  voluntary context switches
              587390  involuntary context switches
<PHONE_NUMBER>034  instructions retired
<PHONE_NUMBER>13  cycles elapsed
<PHONE_NUMBER>  peak memory footprint
zsh: abort      /usr/bin/time -l node ../sandworm-audit/src/cli/index.js
$ export NODE_OPTIONS="--max-old-space-size=2048"
$ /usr/bin/time -l node ../sandworm-audit/src/cli/index.js
Sandworm 🪱
Security and License Compliance Audit
✔ Built dependency graph
✔ Got vulnerabilities
✔ Scanned licenses
✔ Scanned issues
⠋ Drawing tree chart...
<--- Last few GCs --->
[54352:0x118040000]   211186 ms: Scavenge 2006.1 (2073.6) -> 2001.7 (2075.6) MB, 4.0 / 0.0 ms  (average mu = 0.539, current mu = 0.315) allocation failure;
[54352:0x118040000]   211197 ms: Scavenge 2007.9 (2075.6) -> 2003.5 (2092.6) MB, 5.4 / 0.0 ms  (average mu = 0.539, current mu = 0.315) allocation failure;
[54352:0x118040000]   212063 ms: Mark-sweep 2017.4 (2092.6) -> 2004.8 (2097.6) MB, 853.7 / 0.0 ms  (average mu = 0.302, current mu = 0.097) allocation failure; scavenge might not succeed
<--- JS stacktrace --->
FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
time: command terminated abnormally
      212.22 real        90.32 user         5.53 sys
<PHONE_NUMBER>  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
              350084  page reclaims
                  39  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
               15369  messages sent
               79211  messages received
                   2  signals received
                6487  voluntary context switches
              212925  involuntary context switches
<PHONE_NUMBER>616  instructions retired
<PHONE_NUMBER>94  cycles elapsed
<PHONE_NUMBER>  peak memory footprint
zsh: abort      /usr/bin/time -l node ../sandworm-audit/src/cli/index.js

Thank you, @xSyki!
This just keeps getting weirder.
My version of your tree SVG only has 12.3MB 🤔

Is it ok if I share my resulting files here, so we can compare and maybe understand what's going on?
Any peer or optional dependencies in your manifest?

Could you maybe also share your lockfile?

One note - I'm not able to install the deps above with npm v9.4.2, getting a number of ERESOLVE could not resolve errors. Yarn works just fine though.

And whats funny treemap shows "deps 91.61 GB".
Are you sure this function works well?
const humanFileSize = (size) => {
  const i = size === 0 ? 0 : Math.floor(Math.log(size) / Math.log(1024));
  return `${(size / 1024 ** i).toFixed(2) * 1} ${['B', 'kB', 'MB', 'GB', 'TB'][i]}`;
};

Ok I can replicate the problem using your zip file.
Seems like the lockfile might be the root cause here.

When starting from just package.json - no package-lock.json or yarn.lock - I can't install deps using npm i unless I specify --legacy-peer-deps
When using the --legacy-peer-deps option and then running the audit, I'm able to replicate the memory issue
When removing package-lock.json and installing with yarn, there's no more memory issue and the files are small in size (package-lock.json needs to be removed, though, otherwise Sandworm will default to using it & npm)
I'm assuming you've had this package-lock.json for a while, and just installing new things on top of it?

Next up, I'll start investigating the lockfile and the structure of dependencies. I feel there might be some deep, convoluted graph structures there that blow up when representing the tree. Will post any new findings here asap.

Updates:

The tree chart produced by npm for the test app's dependency structure contains 250k+ nodes.
It's way worse with pnpm, with roughly 800k nodes to represent.
I've been able to verify that the bulk of memory is being allocated by d3 for building the huge graph, and it's not Sandworm leaking or blowing up. We're stuck on the current d3 version because we use d3-node, but I'll add an item to the backlog to revisit possibly updating.
Yarn somehow gets it right and builds a dependency graph that audits without any issue within 2GB of memory, as in my initial tests.

To better address this:

I've added a --skip-tree command line option to skip generating the tree altogether #81
I've added a console warning for when trying to represent more than 100k nodes in a treemap chart #74

Your dependency tree has a very large number of nodes (286234).
Generating the tree chart might take a lot of time & memory.
If the process crashes you have several options:
- Allocate more memory to the node process by exporting `NODE_OPTIONS="--max-old-space-size=16384"`
- Reduce the depth of the tree represented by passing the `--max-depth` option to Sandworm'
- Use the `--skip-tree` option to skip building the tree
- Try another package manager

Closing this now. Thank you @xSyki !

@gabidobo Thanks for these fixes, I really like the warning message idea.


Android app

Can you Create  Android application for beatstreet

Not really. i am not into Android development but you can install this webapp on your phone too. its supports PWA.


return complex hidden features in inference

Return complex hidden features in inference so people can also use these features to do further prediction or other tasks.

Hi, @qwerthjkl45
Thank you for your great idea that complex hidden features could be returned for downstream tasks!
Considering that returning them is not 'very' necessary in docking, however, I close this issue and the PR . But you can still develop it by adding the features in your own fork, and that is well recommended!


sometimes compressed images are larger than the original

usually this happens with images of size ~500kb
kind of renders this package useless if "compressing" an image can end up making it larger...in fact the resulting image is often lower quality
after looking at this topic a bit more i realize that my above comment isnt valid.
Although it'd be appreciated if someone could share workarounds that they've tried to get around this issue as it does occur quite often

Any update on this one? recently i have faced and issue where after compressing, the images are heavier and i noticed that the image was modified on width and length and also in it's color depth (from 24 bits to 32). It appears to be happen randomly and i can't really replicate the issue

Hey, facing the same problem, any updates ?
I'm testing on iPhone X:

The first compressed image is larger by approximatively 5%
The second one by 7% of the first compressed image
After that, it takes each time 0.3%

On Android however, it looks like it is working properly

Same problem here. I've used all methods the package provides to compress images, but it didn't work. The "compressed" images still came larger than original.

are you providing a minHeight value? i removed the min values and that seemed to fix the problem

Same here, even without the minHeight value


More helpful failure message for noob developers

I've run into a situation at work where a less-experienced developer was trying to use a for in statement to loop over a standard array of strings, like so:
const arr = ["foo", "bar", "baz"];
for (item in arr) {
  //do stuff
}

Then TSLint would produce the message:

Do not use for in statements, use Object.keys instead: for (item in arr)

So the developer said "OK" and then went and changed the code to this:
const arr = ["foo", "bar", "baz"];
Object.keys(arr).forEach((key: number) => {
  //do stuff
});

:neutral_face:
Sooo... technically both ways actually do work for looping over flat arrays, but obviously they are not ideal.  I wanted to make the failure message for this rule a bit more explicit to better describe the available options.  Now the rule will produce this failure message:

Do not use the 'for in' statement: 'for (item in arr)'. If this is an object, use 'Object.keys' instead. If this is an array use a standard 'for' loop instead.

Great improvement. Feel free to do this as often as you can. Error messages are important.


Feat: Base Class Implemenations

base implemenation of some classes following code_structures.md

Why are you using enumerations and not dictionaries
class PlayerClasses(Enum):
    SCION = 0
    MARAUDER = 1
    RANGER = 2
    WITCH = 3
    DUELIST = 4
    TEMPLAR = 5
    SHADOW = 6

I setup the colorCodes as a dictionary
colorCodes = {
    "NORMAL": "#000000",
    "MAGIC": "#8888FF",
    "RARE": "#FFFF77",

I'm not sure of the advantages of either.
X

PEP 345 lists some reasons for using enums. Further advantages:

Enums have a distinct type
Dicts are stringly typed, so enums avoid typos
Enum variants are discoverable and you can auto-complete them
Nicer syntax: dot access vs key access; allows for automatic enumeration.
Stronger type guarantees for subtypes like enum.IntEnum (comparable to typing.TypedDict, but without the need for annotations)

PEP 345 lists some reasons for using enums. Further advantages:

Currently I'm filling a combobox with
        self.colour_combo_box.addItems(color_codes.keys())

Which is super convenient.

Whilst the color_codes naturally would be good as enums, what hoops to to go through to fill the combobox ? Or is there a different way to choose colours ?
x

You can iterate over an enum's variants and get their names:
self.colour_combo_box.addItems([colour.name for colour in ColourCodes])


Followed instructions to install USBIP server, got an error when listing USB devices

This is the error:
usbip: error: failed to open /usr/share/hwdata//usb.ids

busid 2-1.7 (054c:05c5)
unknown vendor : unknown product (054c:05c5)

Update, found different instructions online and managed to install USBIP and followed instruction server side, but now client side is having issues, error: vhci driver is not loaded

Never mind, I am an idiot, restarted pc and it works now

@Manic137
hello
Can you share how you dealt with this problem
I encountered the same issues, error: vhci driver is not loaded


feat(storage): implement {Get,Set}NativeBucketIamPolicy

This change is

This currently does not work. I ran into a snag with my munderstanding.The JSON API basically expects a Policy and that's how the emulator parses it. However, the proto seems to be in the form of:
    iam_request: {
      resource: "test-bucket-name"
      policy: {
        bindings: {
          role: "test-role"
          members<EMAIL_ADDRESS>        }
        etag: "test-etag"
        version: 3
      }
    }

Changing the emulator would break the bucket_integration_test, so I am missing some key element here.

This currently does not work. I ran into a snag with my misunderstanding.The JSON API basically expects a Policy and that's how the emulator parses it.

Yes. It seems the JSON API flattens the policy request, and lifts the fields inside Policy to the request body. Compare the proto definitions:
https://github.com/googleapis/googleapis/blob/c7b5032f6ee39de6c0359b912012f99dfccb49b5/google/iam/v1/iam_policy.proto#L98-L110
https://github.com/googleapis/googleapis/blob/c7b5032f6ee39de6c0359b912012f99dfccb49b5/google/iam/v1/policy.proto#L88-L123
with the JSON resource that you quoted:
https://cloud.google.com/storage/docs/json_api/v1/buckets/setIamPolicy#request-body

Changing the emulator would break the bucket_integration_test, so I am missing some key element here.

I think there needs to be a layer in the emulator that deals with these differences between the proto and JSON representations of the request. They are basically the same contents, with just one level of nesting removed for JSON.

I have zero idea why, but I'm getting Actual: Permanent error in GetNativeBucketIamPolicy: Method not implemented! [UNIMPLEMENTED] [UNIMPLEMENTED] for stub_->GetBucketIamPolicy. Perhaps this PR is a bit premature.

I figured out the issue. I embarrassingly named the functions incorrectly and was looking in the wrong locations while debugging.
This PR should be ready to review with one caveat: setting the etag seems to make the emulator unhappy (maybe the tag needs to be in a certain format?) and since it is set to the timestamp anyway, I remove it from the request. I'm also not sure what the best way of testing it has the correct value is since it returns the timestamp of the current date.


WebGL samples no longer work on Windows

Describe the bug
I tried to view a sample of Filament tutorial_triangle in Chrome 87.
However, an error appeared in the console and no triangles were displayed on the screen.
To Reproduce
Steps to reproduce the behavior:

Launch Chrome 87.
Go to the following page
https://google.github.io/filament/webgl/tutorial_triangle.html
Check the console.

The following error is output to the console and the triangle is not displayed on the screen.
GL_INVALID_FRAMEBUFFER_OPERATION: Framebuffer is incomplete.
GL_INVALID_FRAMEBUFFER_OPERATION: Draw framebuffer is incomplete.

Expected behavior
The assumed result is a triangle on the screen and no error output to the console.
Screenshots
If applicable, add screenshots to help explain your problem.

Chrome 87.0
Firefox 82.0

Desktop (please complete the following information):

OS: Windows 10 Version 1909
GPU: Intel UHD Graphics 617
Backend: ANGLE
Browser: Chrome 87.0.4280.66

I cannot reproduce. The samples work for me on Chrome 87 (macOS).

Most samples work, but the triangle demo does not, which is a dup of #3128.

The triangle works for me :)

I cannot reproduce. The samples work for me on Chrome 87 (macOS).

Yes. The problem seems to occur only in the Windows version of Chrome 87.
I also confirmed that the triangle appears in the case of Chrome 87 on macOS.

Most samples work, but the triangle demo does not, which is a dup of #3128.

Hmm. I presented the triangle sample as a representative example, but neither the other samples nor gltf-test worked in my Windows environment with Chrome 87.
I have three Windows 10 environments and all have had similar events. The environments I've tried are as follows.

MacBookAir + Windows 10 (BootCamp) + Chrome 87 + Intel UHD Graphics 617
ThinkPad X260 + Windows 10 + Chrome 87 + Intel HD Graphics 520
Desktop PC + Windows 10 + Chrome 87 + GeForce GTX 660

This problem seems to be specific to the Windows version of Chrome 87.
Should I report a bug to bugs.chromium.org?

I tried it out by embedding the debug output code into the following function in filament.js

_glClearBufferfv()
_glDrawBuffers()
_glDrawElements()

The embedded code is as follows:
console.log(_emscripten_glCheckFramebufferStatus(36160)); // 36160 is gl.FRAMEBUFFER

The results, traced in Spector.js, are as follows:

WebGL2 API
Chrome 87 (Windows)
Firefox 83 (Windows)

clearBufferfv
FRAMEBUFFER_INCOMPLETE_ATTACHMENT
FRAMEBUFFER_COMPLETE

drawBuffers
FRAMEBUFFER_INCOMPLETE_ATTACHMENT
FRAMEBUFFER_COMPLETE

drawElements
FRAMEBUFFER_INCOMPLETE_ATTACHMENT
FRAMEBUFFER_COMPLETE

I've embedded the trace output code in the sample below to see where the problem is happening.
https://cx20.github.io/webgl-test/examples/filament/square/index.html
Upon investigation, I found that the problem occurred at the point of the call to framebufferTexture2D().
Perhaps some parameter inconsistency was detected by the browser version upgrade, but the specific combination of parameters is not known.
The following case seems to cause problems when specifying RGBA16F in internalFormat, but as far as I can see from the webglreport, the EXT_color_buffer_half_float extension seems to be supported.

Chrome 87 (Windows)
Firefox 83 (Windows)

Thanks @cx20, I have a tentative fix that I'm going to push momentarily.  Would be great if you could test it out for us.

@prideout Thanks! I have confirmed that the latest version of Filament can now view glTF files again.
Filament v1.9.10rc + Duck.gltf result:

I tried tracing again on Chrome 87 and Firefox 83 just to be sure.
In both cases, RGBA16F was set and it was confirmed that the error was resolved.

internalFormat
Chrome 87 (Windows)
Firefox 83 (Windows)

RGBA16F

@prideout Previously, the Filament 1.9.10rc(Nightly Build) temporarily addressed this issue, but the release version is no longer working. Is there a version that works somewhere?

@cx20 It is known that MSAA does not work on WebGL with certain Chrome backends (#3128), we are working on a fix, this was never temporarily fixed (not intentionally anyway).  The triangle demo is the only demo that enables MSAA, so all our other web samples should work.  If the other samples do not work, please file a new bug.

@prideout Hmm. Currently, none of the Filament 1.9.10 samples in gltf-test work. Is it a usage problem?
Filament v1.9.10rc + Duck.gltf result:

Filament v1.9.10 + Duck.gltf result:

The following sample seems to have the same error.
https://google.github.io/filament/webgl/tutorial_redball.html
https://google.github.io/filament/webgl/tutorial_suzanne.html
https://google.github.io/filament/webgl/parquet.html
https://google.github.io/filament/webgl/suzanne.html
https://google.github.io/filament/webgl/helmet.html
The environment is MacBookAir + Windows 10(BootCamp) + Chrome88.0.4324.146.

Ah, right, apologies.  Indeed, we did not merge the RGB16F fix  (#3334) into our 1.9.10 release candidate, as it was not considered critical fix for an internal upgrade at Google.  However the fix will definitely be included in our release next week.  Thank you for being patient with us.  :-)

Oh man.  VSCode has updated itself to use the version of Chrome that exhibits this behavior, causing the whole Filament preview window for glTF to go dark and report these errors.  I'll be looking forward to next week's release.

Can someone tell me how to compile filament sample to wasm in Windows? The guide describes only for macos and linux

@amarav If it is the version of Nightly Build, you can get wasm from the following.
https://github.com/google/filament/actions?query=workflow%3AWeb

@emackey I have confirmed the same problem. The Filament sample seems to fail due to the VS Code chrome version being updated from 83 to 87.
I have confirmed that updating the version of Filament to 1.9.11rc of Nightly Build in my local environment resolves the error.

I too am looking forward to the next release.


Inject adapter ID into metadata and calculate submits per adapter

The problem
When we run gfxrecon-info tool on the trace that was captured on the system with 2 GPUs we do not have information which of the adapters were active.
The solution
To determine the active adapter, we can track the Object_id by reading the adapters that recorded during capture (based on 'capture_id'), mapping it based on D3D12CreateDevice, CreateCommandQueue, and finally counting with ExecuteCommandLists. The ID with the largest count would indicate the active adapter. The Object_id could be associated with the LUID that belongs to the active adapter.
Result and Test
Test on dual GPU:
before:

after:

CI gfxreconstruct build queued with queue ID 33192.

CI gfxreconstruct build # 2482 running.

CI gfxreconstruct build # 2482 failed.

CI gfxreconstruct build queued with queue ID 33360.

CI gfxreconstruct build # 2483 failed.

CI gfxreconstruct build queued with queue ID 33578.

CI gfxreconstruct build # 2489 running.

CI gfxreconstruct build # 2489 failed.

CI gfxreconstruct build queued with queue ID 39060.

This PR has changes that won't compile on Linux or Android.

CI gfxreconstruct build queued with queue ID 39098.

CI gfxreconstruct build # 2521 running.

CI gfxreconstruct build # 2521 failed.

CI gfxreconstruct build queued with queue ID 39197.

CI gfxreconstruct build queued with queue ID 39212.

CI gfxreconstruct build # 2525 running.

CI gfxreconstruct build # 2525 passed.

CI gfxreconstruct build queued with queue ID 5227.


Service Facebook

Service home website
https://www.facebook.com
Example link(s) and their coordinates

Example link
Coordinates

https://cs-cz.facebook.com/burgerzelva/
50.08724438,14.46923018

https://www.facebook.com/burgerzelva/
50.08724438,14.46923018

Example location link
N/A
Example drive link
N/A
Description (optional)
Coordinates might be gathered by:

HERE link in "Get Directions" (example)
static map image in "About" section (example)

Scraping via CURL is working good on local, but not on production server - facebook is redirecting to login page.
Probably it has to be done using official Facebook API.
tldr: I need to publish this app (BetterLocation) to review from Facebook staff to approve usage for public pages. Then I will be able to use permission to request public pages, like this: https://developers.facebook.com/tools/explorer/?method=GET&path=burgerzelva%3Ffields%3Did%2Cname_format&version=v9.0
Links to study:

https://developers.facebook.com/tools/explorer/
https://developers.facebook.com/docs/places/web
https://developers.facebook.com/tools/accesstoken/

Scraping via CURL is working good on local, but not on production server - facebook is redirecting to login page.
Probably it has to be done using official Facebook API.
tldr: I need to publish this app (BetterLocation) to review from Facebook staff to approve usage for public pages. Then I will be able to use permission to request public pages, like this: https://developers.facebook.com/tools/explorer/?method=GET&path=burgerzelva%3Ffields%3Did%2Cname_format&version=v9.0
Links to study:

https://developers.facebook.com/tools/explorer/
https://developers.facebook.com/docs/places/web
https://developers.facebook.com/tools/accesstoken/


Error: No module named 'download_from_gdrive'

Hi!
I got the following error while installing DAIN_4_Microscopy.ipynb notebook:
---------------------------------------------------------------------------

ModuleNotFoundError                       Traceback (most recent call last)

<ipython-input-3-09fd65144275> in <module>()
     52 _ = (sys.path.append("/usr/local/lib/python3.6/site-packages"))
     53 sys.path.insert(0,'/content/DAIN/load_functions')
---> 54 from download_from_gdrive import download_file_from_google_drive
     55 best_pretrained = "1EYgrdsaffVJuQlcBSK4dGqO2BXBkSlZf"
     56 if not os.path.exists("/content/DAIN/model_weights"):

ModuleNotFoundError: No module named 'download_from_gdrive'

Solved it with:
!pip install googledrivedownloader
from google_drive_downloader import GoogleDriveDownloader as gdd
gdd.download_file_from_google_drive(best_pretrained, file_path)

Hi!
Thank you for the alternative fix. When I run my notebook version on google colab and run the installation section I don't get this error since it should load the function correctly from the downloaded Git-repo.
Thank you anyways for the alternative fix if other people run into a similar problem.

I get the error during the installation. Colab has been updated recently. Maybe that's the reason, I don't know.
THank you!

Ok. I just tried it. But there could be many reasons why something breaks. If you run in to some more problems don't hesitate to ask. I am happy to help.


Nested results in typeahead

I need typeahead to show nested results (2 ng-repeats) with a custom template. Is there some way to do it?

One could implement this via overriding of the typeahead-popup template and creating a custom set of directives.
Otherwise, this is a little complex of a feature for this library to have baked in & not something likely to be used widely, thus we would not want to support this in the library itself, but allow users to extend it to support this.


Add support for accordion-flush

Bootstrap5 introduced "accordion-flush" html class, which basically removes borders and rounding from accordions.
This could be added by creating an AccordionFlush python class at bootstrap.py and an accordion-flush.html layout.
Alternatively, the Accordion python class could be modified in order to accept an optional "accordion-flush" css class on the template, since it's the only difference from a standard accordion. I think this solution might be the more elegant one.
I'll try to work it out and propose a PR

@JimiRecard thanks for the interest. Looking forward to seeing your proposal.

Thanks, glad to help.
I think I have a solution that adresses both accordion-flush and accordion always open (see https://getbootstrap.com/docs/5.0/components/accordion/#always-open)
The current behavior is actually to be always open, but I believe that should not be the default, and I don't think there's an option for the single-panel-open behavior
My solution involves changing bootstrap.py, which is a file on the main repository. Think we can manage that? I'll try to make a PR for the files on this repository soon

The use case I propose would be as follow:
self.helper.layout = Layout(             Accordion(                 AccordionGroup("First", always_open=True),                 AccordionGroup("Second", always_open=True),                 flush=True             )
I would need to add the kwargs always_open to the Container class and flush to the Accordion class for that to work

I made 2 PRs. Sorry about that, I struggle a lot with git and github.
Let me know what you think. I tested manually a few use cases and it seems to work fine, with the necessary changes do bootstrap.py. I'll see if I can propose the changes to the main repository as well.
Also, I believe bootstrap.py is on the main repository for legacy reasons, right? Maybe the best aproach would be for that file to be on the relevant template packs? Just a curiosity, I'm pretty new to this. Thanks for the help

I think if we need to make changes to bootstrap.py we should do that in this repo. We're trying to keep new template packs separate from core with a longer term aim of splitting out the existing template packs.
We should be able to inherit from class from core here and make any adjustments we need to here? (I can't quite remember off hand how complex those classes are!)

It's as I suspected, then. Thanks
Yeah, I believe we can inherit on the already existing bootstrap5.py file. I don't fully understand the classes but the changes I'm proposing are simple enough (famous last words). I'll test locally and then propose another PR for that


Add a compatibility table for VM vs Automation Account combinations

Hi Team, great article here
Given that there are 9 different combinations of using system-assigned managed identity with VM/AA or user-assigned managed identity with VM/AA, it would be much clearer if you could include a table detailling the combination and result.
Not quite sure exactly how you'd intend it to look, but I was thinking something like:

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

ID: a21ca143-2f33-5cea-94a8-ace7e9de5f9c
Version Independent ID: d7f2ef01-8c25-770e-dfd9-37b98dc7ba29
Content: Run Azure Automation runbooks on a Hybrid Runbook Worker
Content Source: articles/automation/automation-hrw-run-runbooks.md
Service: automation
Sub-service: process-automation
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

@SeniorConsulting
Thanks for your feedback! We will investigate and update as appropriate.

@SeniorConsulting, Thank you for the suggestion. I would like to add the following points to clarify the doubts. Please do let me know if you feel that the doc needs to be updated after this.
Azure Automation Authentication:

Without managed identity (user or system), Run As Account based authentication was used. This (including the managed identity) serves the purpose of authentication against the required Azure resources so that appropriate actions can be taken. For example, if the runbook is to start a VM, the Azure Automation account should have the required role (with permission) to perform the start of this VM.
Azure Automation Run As Account will retire on September 30, 2023 and it will be replaced by managed identity. Therefore, there are only 2 options available - user or system managed identity.

Regarding System OR User managed identity - the choice depends on the type of lifetime you are looking for said identity. While System identity is specifically allocated to the Automation Account, when the Automation Account is deleted, the identity goes away with it.
However, with user managed identity, you can use the same identity with multiple resources. Therefore, if you assign User Manager identity to Automation Account, and delete this Automation Account - the user identity still stays. This might be getting used with other resources. For more details. see Managed identies.
You can have scenarios without using identity if your runbooks do not need to authenticate against any resource (for example, a simple "Hello World!" runbook does not need any authentication to run.

Regarding VM/AA managed authentication - note that the requestor/client is the one which would need to authenticate. For example, in case of Azure Automation starting VM through runbook, the automation account needs to authenticate (as it is initiating the process). The managed identities on VM is not required for AA to function in this case.
However, if there are applications running on this VM which needs to connect to another resource (for example a SQL Server) then identity on that VM is required for authenticating with this SQL server.

Hope this helps.

@SeniorConsulting, We will now proceed to close this thread as we have not heard back from you. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.

Thanks for your response. I wasn't specifically looking for information on this, I had just thought a table would be a good way for people (at a glance) to know whether what they want to do is supported, and what the outcome would be (i.e. If they had both a machine/VM identity and an automation account identity, which one would win.
If you don't think that this is required, then that's fine. Your initial response seemed to indicate that you didn't believe that a table would add value to the article, so I'll close this by default after this comment.

@SeniorConsulting, thank you very much for the clarification. Let me share this feedback with the content team, to check the possibility of adding a matrix as mentioned in your question.


PubSub: correctly decrement the byte counter in drop()

I'm looking at this 2 files:

https://github.com/GoogleCloudPlatform/google-cloud-python/blob/5c010e3790c9067e6d7bf8ba37724ee5121eadb7/pubsub/google/cloud/pubsub_v1/subscriber/policy/base.py#L215
https://github.com/GoogleCloudPlatform/google-cloud-python/blob/5c010e3790c9067e6d7bf8ba37724ee5121eadb7/pubsub/tests/unit/pubsub_v1/subscriber/test_policy_base.py

Should the code in base.py line 215 be max instead of min?
self._bytes = min([self._bytes, 0])

Current code:

current state self._bytes = 50
then we call self._bytes -= 20
now self._bytes == 30
but then we do self._bytes = min([30, 0]) -> self._bytes = 0

Should be:

current state self._bytes = 50
then we call self._bytes -= 20
now self._bytes == 30
but then we do self._bytes = max([30, 0]) -> self._bytes = 30

even if we drop it below 0, we should use max:

self._bytes = 10
self._bytes -= 20
self._bytes == -10
min([-10, 0]) == -10 should be max([-10, 0]) == 0

Or is there something I miss?

Then I found #4514.
Hope it gets merged and released soon


Incorrect match for "LINK" anchor

Language is Lua.

Hello!
Sometimes the default configuration doesn't always work for each project or language. In this case I would suggest tweaking your global or workspace settings and alter the commentAnchors.tags.separators list to force anchors to be followed by a specific pattern, such as a colon.


Ensure auth on edit and update

Going to the edit page when not authenticated caused a runtime exception instead of the expected response of going to index and displaying 'Authentication Required'.

Thank you! 👍


Testing doc

Coverage remained the same at 95.019% when pulling 6f858e4139e9addcc7952d39d54b361536acda49 on testing-doc into 9bb036887a5a51b41e0a10733247886fb4f0471a on master.


Add existing VPC support

Add existing VPC support through:

A stub vpc-existing module
A set of variables for the stub module located in vpc-existing.tfvars that merges it into terraform.tfvars
An override of the root modules.tf file

@wellsie What did you think about this PR?

@mirthy - looks good. we just need to update the maps to use terraform 0.7.x syntax in the .tfvar file and '[]' for the maps in the .tf files. i'll take a swing at that over the next few days

@wellsie Took a crack at fixing the maps to terraform 0.7.x syntax

👍  ty


nuxt build fails with spa mode

Version
v2.10.0
Reproduction link
https://github.com/vinayakkulkarni/nuxt-2.10-bug
Steps to reproduce
npm install
npm run build
What is expected ?
Nuxt to build all routes and files post build success.
What is actually happening?
ℹ Generating pages                                                                                18:52:53

 ERROR   /                                                                                18:52:53

TypeError: Cannot read property 'renderRoute' of undefined
    at Generator.generateRoute (/Users/vinayak/Development/Personal/nuxt/nuxt-2.10-bug/node_modules/@nuxt/generator/dist/generator.js:222:42)
    at /Users/vinayak/Development/Personal/nuxt/nuxt-2.10-bug/node_modules/@nuxt/generator/dist/generator.js:120:24
    at async Promise.all (index 0)
    at async Generator.generateRoutes (/Users/vinayak/Development/Personal/nuxt/nuxt-2.10-bug/node_modules/@nuxt/generator/dist/generator.js:115:7)
    at async Generator.generate (/Users/vinayak/Development/Personal/nuxt/nuxt-2.10-bug/node_modules/@nuxt/generator/dist/generator.js:48:20)
    at async Object.run (/Users/vinayak/Development/Personal/nuxt/nuxt-2.10-bug/node_modules/@nuxt/cli/dist/cli-build.js:101:5)
    at async NuxtCommand.run (/Users/vinayak/Development/Personal/nuxt/nuxt-2.10-bug/node_modules/@nuxt/cli/dist/cli-command.js:2575:7)

 FATAL  Cannot read property 'renderRoute' of undefined                                                                                18:52:53

  at Generator.afterGenerate (node_modules/@nuxt/generator/dist/generator.js:168:43)
  at async Generator.generate (node_modules/@nuxt/generator/dist/generator.js:50:5)
  at async Object.run (node_modules/@nuxt/cli/dist/cli-build.js:101:5)
  at async NuxtCommand.run (node_modules/@nuxt/cli/dist/cli-command.js:2575:7)

   ╭────────────────────────────────────────────────────────────────╮
   │                                                                │
   │   ✖ Nuxt Fatal Error                                           │
   │                                                                │
   │   TypeError: Cannot read property 'renderRoute' of undefined   │
   │                                                                │
   ╰────────────────────────────────────────────────────────────────╯

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `nuxt build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/vinayak/.npm/_logs/2019-10-06T13_22_53_417Z-debug.log

Additional comments?
Related issues: #6500, #6513 & #6514

This bug report is available on Nuxt community (#c9867)

Instead 'npm run build' use 'npm run generate' - for me works fine

@kajcula it's not a real solution for the problem

its bug, i`l check what wrong

@aldarund  problem its here https://github.com/nuxt/nuxt.js/commit/95e043f1679f9d3bf1f6508282ac0c6ec468f4d8#diff-487be8dbf9747ee9db4e19a779085840R64

hmm

It affect not only spa mode.
I use universal mode and I have self made page level cache module where i use this.nuxt.renderer - in development everything is ok, but in prod i got TypeError

@cron13 ist different issue. You need to check for renderer to be exist in your module

it's appear only on linux.


[injected-v2.0.13-alpha.1] : fix - Liquality wallet injection update

Description
For some strange reason, the liquality wallet was being constrained to the arbitrum namespace. As a result, web3 onboard is not able to connect to liquality except for arbitrum.
Liquality supports regular window.ethereum with EIP1993.
This PR fixes this by using window.ethereum as injection namespace
Checklist

[ ] The version field in package.json is incremented following semantic versioning
[ ] The box that allows repo maintainers to update this PR is checked
[ ] I tested locally to make sure this feature/fix works
[ ] This PR passes the Circle CI checks

@monokh thanks for the PR! Could you please increment the version of the injected package within the package.json file to 2.0.13-alpha.1

@Adamj1232 Thanks, updated

Will want the version change within the injected package inside the package.json file and that will be set to 2.0.13-alpha.1

@monokh We will want the version change within the injected package inside the package.json file and that will be set to 2.0.13-alpha.1 This will allow us to publish an alpha release for testing of this specific package.

@monokh I cloned your fork and added the version update so we could get this into this weeks release. Thanks for your PR! And dont forget to pull if using this branch any further! 🥂

@monokh We will want the version change within the injected package inside the package.json file and that will be set to 2.0.13-alpha.1 This will allow us to publish an alpha release for testing of this specific package.

Sorry about that. Thanks for fixing it up

Sorry about that. Thanks for fixing it up
Not a problem at all! Thank you again for the PR! It will be released in a day or two after some testing in our staging but you can use with the current alpha tag if you like

@Adamj1232 Hello,If this fix is already in production ？i met this issue in https://passport.gitcoin.co/#/

@Adamj1232 Hello,If this fix is already in production ？i met this issue in https://passport.gitcoin.co/#/

@hellozeck yessir this is in production. What version of the injected package did you run into this issue?


Update README to include Java prerequisite

Further improvements to the README based on finds during local set up.

Adding the Java prerequisite as it is required for not just a developer set up.
Updating clone installation commands.

#98


Add release notes for 1.3.0

There weren't any urgent bugs and I was getting asked for a release but don't have time to fill in the release notes atm.
https://github.com/ScottLogic/d3fc/releases/tag/v1.3.0

Done. Nice commit hash!

Thanks


Fix unsaved changes alert from triggering when RTE is empty

The following changes are implemented
TODO: Summary
Changes in the user interface:
TODO: Add screenshots, recordings or remove this section
Checklist when submitting a final (!draft) PR

[ ] Commits are tidied up, squashed if needed and follow guidelines in CONTRIBUTING.md
[ ] Code builds
[ ] All existing tests pass
[ ] All new critical code is covered by tests
[ ] PR is linked to the relevant issue(s)
[ ] Rebased with the target branch


Bug: Tonic Response class sets two content-type headers

Tonic Response defaults to content-type = 'text/html' but $response->contentType = $mimetype; in Tonic Resource declares a new Content-Type header as well instead of overriding the response content-type, you end up with two content-type headers (different case).

The fix is just to change the Tonic Response class:
$headers = array('Content-Type' => 'text/html');

Great, thanks for finding this. Can you create a pull request?

Yes I'll create one

https://github.com/peej/tonic/pull/196


adding GOC db xrefs to 4 terms - easy

Hi,
Can you please add the GOC:BHF_telomere to these terms:
GO:1904356 regulation of telomere maintenance via telomere lengthening
GO:1904357 negative regulation of telomere maintenance via telomere lengthening
GO:1904358 positive regulation of telomere maintenance via telomere lengthening
GO:1990572 TERT-RMRP complex
I forgot to add when the terms were first created. Thanks

Done


timeseries in state??

Should timeseries be stored inside of "state" rather than being standalone? Not sure it matters much. Let me know thoughts - that came to mind when making parameter map because it seemed funny to have them separate.
@tbarchyn @KeeganShaw-GIS let me know

I found a problem that relates to this issue. I am integrating the agent-level scheduling and there is a wired error:
Traceback (most recent call last):
File "C:\Users\cookg.conda\envs\LDARSim\lib\multiprocessing\pool.py", line 121, in worker
result = (True, func(*args, **kwds))
File "C:\Users\cookg.conda\envs\LDARSim\lib\multiprocessing\pool.py", line 47, in starmapstar
return list(itertools.starmap(args[0], args[1]))
File "D:\LDAR_Sim-master\LDAR_sim_scheduling_0608\LDAR_Sim\model_code\ldar_sim_run.py", line 152, in ldar_sim_run
sim_summary = sim.finalize()
File "D:\LDAR_Sim-master\LDAR_sim_scheduling_0608\LDAR_Sim\model_code\ldar_sim.py", line 434, in finalize
time_df = pd.DataFrame(self.timeseries)
File "C:\Users\cookg.conda\envs\LDARSim\lib\site-packages\pandas\core\frame.py", line 529, in init
mgr = init_dict(data, index, columns, dtype=dtype)
File "C:\Users\cookg.conda\envs\LDARSim\lib\site-packages\pandas\core\internals\construction.py", line 287, in init_dict
return arrays_to_mgr(arrays, data_names, index, columns, dtype=dtype)
File "C:\Users\cookg.conda\envs\LDARSim\lib\site-packages\pandas\core\internals\construction.py", line 80, in arrays_to_mgr
index = extract_index(arrays)
File "C:\Users\cookg.conda\envs\LDARSim\lib\site-packages\pandas\core\internals\construction.py", line 401, in extract_index
raise ValueError("arrays must all be same length")
ValueError: arrays must all be same length
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "D:\LDAR_Sim-master\LDAR_sim_scheduling_0608\LDAR_Sim\model_code\ldar_sim_main.py", line 83, in
res = p.starmap(ldar_sim_run, simulations)
File "C:\Users\cookg.conda\envs\LDARSim\lib\multiprocessing\pool.py", line 276, in starmap
return self._map_async(func, iterable, starmapstar, chunksize).get()
File "C:\Users\cookg.conda\envs\LDARSim\lib\multiprocessing\pool.py", line 657, in get
raise self._value
ValueError: arrays must all be same length
**I suspect this is due to the scheduling and travel time calculation from euclidian distance limiting the agent's travelling. Therefore, fields in the timeseries
'active_leaks',
'new_leaks',
'n_tags',
'rolling_cost_estimate',
'cum_repaired_leaks',
'daily_emissions_kg',
'truck_prop_sites_avail'  are not updated at the end of the day since the agent did nothing except travel from one home base to another home base close to the next target facility. In other words, data recorded from state['t'] have different lengths to data stored in timeseries.  For now, this is fixed by automatically check the length of each field. However, to avoid similar issue we should merge timeseries into state. **

What's the status on this one?

The timeseries are still standalone. The bugs/errors of unequal timeseries length caused by route planning or rollover are already fixed.

So should we close this or do you think we should combine timeseries into state? Just leave it and close this?

I think we can leave it for now since it doesn't really affect any functionality, especially for LDAR Sim V2. If these two need to be combined in the future, we can always do it.


tpl/inflect: Add the SI function to format numbers with SI notation

https://en.wikipedia.org/wiki/Metric_prefix
One of use case of mine: a repository card.

Not sure if I put this function into the right place/package.

How would I use this with the lang.FormatSomething functions? Each of these functions returns a string. I imagine in some locales you might want 12,3 kb instead of 12.3 k.
Maybe this should be lang.FormatNumberSI, and possibly create derivatives of lang.FormatNumberCustom and lang.FormatCurrency as well.
I suggest closing this PR and opening a proposal to work out the details.

I imagine in some locales you might want 12,3 kb instead of 12.3 kb.

Haven't thought about this.
Done, https://discourse.gohugo.io/t/function-to-format-numbers-with-si-notation/50131, close for now.


Change GraphExecutionState's _get_next_node method to prioritize Iterate Nodes

Summary
This PR optimizes the graph execution order by prioritizing Iterate nodes and their children, ensuring the correct execution sequence. The changes made to the _get_next_node method within the GraphExecutionState class address the issue of random output order and execution of Iterate node children.
Key changes include:

Extracting Iterate nodes from the topological order and sorting them based on their index attribute.
Prioritizing the execution of Iterate nodes and their children, ensuring the correct order.
Falling back to the topological order if no Iterate node or its children are ready for execution.

These improvements provide a more predictable and intuitive behavior for Iterate nodes.
Related Issues / Discussions

QA Instructions

To test the changes in this PR:

Set up a workflow with one or more Iterate nodes.
Execute the workflow and inspect the output of the Iterate nodes (select the node in the workflow editor and inspecting the "Outputs" tab in the lower side panel)
Verify that the output order starts correctly with index 0 and that the item at index 0 is executed first.

Before this change, the index 0 item would be executed in a random order. With these changes, the execution order should be consistent and aligned with the index attribute of the Iterate nodes.
Merge Plan

Checklist

[X] The PR has a short but descriptive title, suitable for a changelog
[ ] Tests added / updated (if applicable)
[ ] Documentation added / updated (if applicable)

Thanks for fixing this very long-standing and impactful issue!
Here's my test setup:

I also added a test to the PR for execution order.