Update README.md - “i/o timeout”

下载 Go 依赖包并构建项目常常会因为速度过慢而报错“i/o timeout”后终止，可以可以设置环境变量 GOPROXY 使用 Go 官方提供的代理来加速下载过程

@Huangshan16 thanks for the suggestion.
But I think this part only needs to be in  the cn versioned readme. Don't you agree?

okk, I c

I've added this part to #131, @Huangshan16 feel free to take a look.

#131 merged into main, closing this ticket.


HDDS-8531. Remove outdated/stale code around TrashRecoverRequest.

What changes were proposed in this pull request?
Remove unused OMTrashRecoverRequest and OMTrashRecoverResponse.
What is the link to the Apache JIRA
HDDS-8531
How was this patch tested?
NA

@smengcl , thanks a lot for reviewing this!


Directories with only .gitkeep file within a plugin are not downloaded to pub cache

Steps to reproduce

Create a flutter app
Add a dependency for video_player_avfoundation 2.6.0
flutter config --enable-swift-package-manager
flutter build ios

Actual results
Will error with
xcodebuild: error: Could not resolve package dependencies:
  public headers ("include") directory path for 'video_player_avfoundation_ios' is invalid or not contained in the target

Logs
For example, this path should contain an include/.gitkeep directory
ls /path/to/.pub-cache/hosted/pub.dev/video_player_avfoundation-2.6.0/darwin/video_player_avfoundation/Sources/video_player_avfoundation_ios

https://github.com/flutter/packages/tree/a9f14bc819e635f43f1861b8da807ce03b7ce013/packages/video_player/video_player_avfoundation/darwin/video_player_avfoundation/Sources/video_player_avfoundation_ios/include
Flutter Doctor output

Doctor output

<!-- Paste your output here -->

Probably worth filing this at https://github.com/dart-lang/pub to explain that it's causing issues for plugins.

Probably worth filing this at https://github.com/dart-lang/pub to explain that it's causing issues for plugins.

Done: https://github.com/dart-lang/pub/issues/4262

Was able to replicate this on master.

 master flutter doctor -v

[!] Flutter (Channel master, 3.22.0-28.0.pre.17, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.22.0-28.0.pre.17 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 972dcd7c65 (2 hours ago), 2024-05-08 22:41:26 -0400
    • Engine revision 69f2d9610a
    • Dart version 3.5.0 (build 3.5.0-138.0.dev)
    • DevTools version 2.36.0-dev.5
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

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

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

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

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

As a temporary solution, we'll be advising plugin authors to add !.gitkeep to their .gitignore, which flutter pub publish respects

@vashworth is this issue tracking adding that documentation? Or is there something else the tooling needs to do, other than what pub needs to do in https://github.com/dart-lang/pub/issues/4262 ?

@vashworth is this issue tracking adding that documentation? Or is there something else the tooling needs to do, other than what pub needs to do in dart-lang/pub#4262 ?

No, we can track using https://github.com/dart-lang/pub/issues/4262


CUDA out of memory only when GFPGAN is selected for a batch

Describe your environment

GPU: NVIDIA 1070Ti
VRAM: 8GB
CPU arch: Intel Core i7 5930K @ 3.50GHz
OS: Windows 10
Python: Anaconda
Branch: 2.0.0 release
Commit:

Describe the bug
Brand new fresh installation of 2.0.0, starting from git clone and env create.
When Restore Face is selected in the Web UI for a large batch, the batch eventually encounters CUDA out of memory errors.
K_lms, k_euler, k_euler_a were used, at 512X512 and 512X768, and 15 steps and 50 steps; all combination exhibits the same behavior.
To Reproduce

Using the web UI
Set Images to a high number
Select Restore Face
Enter any prompt,
Press Invoke

Expected behavior
All images would be generated.
Additional context: full trace from a fresh start
(invokeai) C:\stable-diffusion\InvokeAI>python scripts\invoke.py --web

Initializing, be patient...

GFPGAN Initialized
CodeFormer Initialized
ESRGAN Initialized
Loading model from models/ldm/stable-diffusion-v1/model.ckpt
LatentDiffusion: Running in eps-prediction mode
DiffusionWrapper has 859.52 M params.
making attention of type 'vanilla' with 512 in_channels
Working with z of shape (1, 4, 32, 32) = 4096 dimensions.
making attention of type 'vanilla' with 512 in_channels
NOTE: Redirects are currently not supported in Windows or MacOs.
Using faster float16 precision
Model loaded in 20.05s
Max VRAM used to load the model: 2.17G
Current VRAM usage:2.17G
Setting Sampler to k_lms

--web was specified, starting web server...

Started Invoke AI Web Server!
Default host address now <IP_ADDRESS> (localhost). Use --host <IP_ADDRESS> to bind any address.
Point your browser at http://<IP_ADDRESS>:9090
System config requested
System config requested
System config requested
Image generation requested: {'prompt': 'Cat', 'iterations': 50, 'steps': 15, 'cfg_scale': 7.5, 'threshold': 0, 'perlin': 0, 'height': 512, 'width': 512, 'sampler_name': 'k_lms', 'seed':<PHONE_NUMBER>, 'seamless': False, 'progress_images': False, 'variation_amount': 0}
ESRGAN parameters: False
GFPGAN parameters: {'strength': 0.8}
Generating:   0%|                                                                               | 0/50 [00:00<?, ?it/s]>> Sampling with k_lms starting at step 0 of 15 (15 new sampling steps)
100%|██████████████████████████████████████████████████████████████████████████████████| 15/15 [00:07<00:00,  2.04it/s]
GFPGAN - Restoring Faces for image seed:3950314710██████████████████████████████████| 15/15 [00:07<00:00,  2.12it/s]
Image generated: "C:\stable-diffusion\InvokeAI\outputs\img-samples\000026.3950314710.postprocessed.png"

[snip... for brevity]
Generating:  68%|███████████████████████████████████████████████▌                      | 34/50 [05:54<02:51, 10.70s/it]>> Sampling with k_lms starting at step 0 of 15 (15 new sampling steps)
100%|██████████████████████████████████████████████████████████████████████████████████| 15/15 [00:07<00:00,  2.13it/s]

GFPGAN - Restoring Faces for image seed:3378879192██████████████████████████████████| 15/15 [00:07<00:00,  2.02it/s]
Failed inference for GFPGAN: CUDA out of memory. Tried to allocate 128.00 MiB (GPU 0; 8.00 GiB total capacity; 7.02 GiB already allocated; 0 bytes free; 7.25 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF.
Image generated: "C:\stable-diffusion\InvokeAI\outputs\img-samples\000060.3378879192.postprocessed.png"
Generating:  70%|█████████████████████████████████████████████████                     | 35/50 [06:05<02:43, 10.89s/it]>> Sampling with k_lms starting at step 0 of 15 (15 new sampling steps)
100%|██████████████████████████████████████████████████████████████████████████████████| 15/15 [00:07<00:00,  2.13it/s]
Generating:  70%|█████████████████████████████████████████████████                     | 35/50 [06:13<02:40, 10.68s/it]
Traceback (most recent call last):
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\generate.py", line 370, in prompt2image
results = generator.generate(
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\invoke\generator\base.py", line 78, in generate
image = make_image(x_T)
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\autograd\grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\invoke\generator\txt2img.py", line 54, in make_image
return self.sample_to_image(samples)
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\invoke\generator\base.py", line 90, in sample_to_image
x_samples = self.model.decode_first_stage(samples)
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\autograd\grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\models\diffusion\ddpm.py", line 1117, in decode_first_stage
return self.first_stage_model.decode(z)
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\models\autoencoder.py", line 421, in decode
dec = self.decoder(z)
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\nn\modules\module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\modules\diffusionmodules\model.py", line 599, in forward
h = self.up[i_level].block[i_block](h, temb)
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\nn\modules\module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "d:\dropbox\programming\ai\stable-diffusion\invokeai\ldm\modules\diffusionmodules\model.py", line 120, in forward
h = self.norm1(x)
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\nn\modules\module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\nn\modules\normalization.py", line 268, in forward
return F.group_norm(
File "C:\Users\khain\anaconda3\envs\invokeai\lib\site-packages\torch\nn\functional.py", line 2499, in group_norm
return torch.group_norm(input, num_groups, weight, bias, eps, torch.backends.cudnn.enabled)
RuntimeError: CUDA out of memory. Tried to allocate 256.00 MiB (GPU 0; 8.00 GiB total capacity; 6.70 GiB already allocated; 0 bytes free; 7.26 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Could not generate image.
Usage stats:
0 image(s) generated in 374.29s
Max VRAM used for this generation: 7.61G. Current VRAM utilization: 2.21G
Max VRAM used since script start:  7.61G

Could you please try to use option --free_gpu_mem and check if it makes difference?

Could you please try to use option --free_gpu_mem and check if it makes difference?

Using "invoke.py --free_gpu_mem --web"  produces the same result.
"an old woman" 512X512, k_euler, cfg 7.5, Restore Face:
5 steps -- fails at around 30 images consistently
50 steps -- fails at around 50 images consistently
I don't know if --free_gpu_mem is being processed when --web is also specified, but I see no change in behavior.

This sounds like it may be a memory leak. To help isolate whether it is in the Web GUI or the underlying libraries, could you please try your generation with the command-line interface?
python scripts\invoke.py
invoke> an old woman -s5 -n50 -G0.8 -Ak_euler

This should generate 50 face reconstructed images using 5 steps each

python scripts\invoke.py
invoke> an old woman -s5 -n50 -G0.8 -Ak_euler

It completed all 50, and applied to GFPGAN to all 50 with no error.

Usage stats:
50 image(s) generated in 283.57s
Max VRAM used for this generation: 3.90G. Current VRAM utilization: 2.21G
Max VRAM used since script start:  4.72G

The code path is different.  The Web UI applies GFPGAN after each image is generated before the next image begins.  The CI waits until all images are done and does a batch operation of GFPGAN on all the images.
I've noticed that during the GFPGAN batch operation through the CI the memory usage starts at 4.1GB goes up to 5.1GB for each image, and always returned to 4.1GB after each image was processed.  The GFPGAN through web UI does not behave this cleanly, I can see it slowly retaining more and more memory until it runs out.

I have the exact same issue also with a fresh installation, here's a screenshot of my vram, each image adds about 200mb (I restarted the server at the end).

In comparison here is with restore faces disabled

Thanks for the investigation, we will need to dig in to the web UI server code to fix this.

Basically just need to apply the same solution I did for the CLI when I added face restoration.
No matter how optimized our SD generator has become, this will be an issue with batch processing. I thought that maybe with all optmizations we can run post processing right after the image is generated but this shows that there will eventually be a memory overload. And it is always the most ideal solution to do post processing after the batch has been fully generated,

Maybe we make a setting to do post-processing in a batch or individually. Would require a bit of fanagling of the server bc it always calls postprocessing individually.
I'll have a look at the restoration code at some point to see the solution

We are transitioning to a new server and backend, where this should no longer be a problem.


REFACTOR: Make API Gateway closable.

This cleans up server shutdown handling and makes it easy to close.
@cpacia Please review.

Coverage remained the same at 61.367% when pulling 7bb5c282ffe48c36baf9152e76204245a38fc241 on TS_shutdown_gateway into c68913644f3d7c29475c44a8e1f8f6ea8f253e03 on master.


[BUG] State "roborock.0.HomeData" has no existing object

Bug Description
Folgende Log-/Warn Meldungen werden minütlich generiert.
State "roborock.0.HomeData" has no existing object, this might lead to an error in future versions
Steps to Reproduce
.
Expected Behavior
No Warn
Debug Log

Adapter Version
0.6.18
JS-Controller Version
7.0.3
Node.js Version
20.18.1
Operating System
linux
Additional Context
No response

Thanks. Fixed via https://github.com/copystring/ioBroker.roborock/commit/478ff52eb6e1ae2776c92f9c3ba4da3480a4996f. Coming in next release soon.


refactor: de-react-ify

Closes #
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-08-15).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog
Some components seem to be React when they don't have to be.

Screenshots
[Screenshots]
💯

should i try porting the TOC too? then it's only the typewriter left but i think that's fine unless we just go for a gif or smth? @nexxeln

should i try porting the TOC too? then it's only the typewriter left but i think that's fine unless we just go for a gif or smth? @nexxeln

Yeah sure

@nexxeln I also made the change so that the H1 is "rendered" in the TOC without having to be explicitely defined as H1 and H2


enum CompInterType: make a real enum

this PR looks big but most lines are because I needed to reverse two branches so that
if comp_type.is_none() { ... } else { ... }

can turn into
if let Some(comp_type) = comp_type { ... }  else { ... }

the first commit is most changes, the second one just the switching of branches and using the variable bound by the if-let

yes, that and it removes these lines
// TODO use if let in the surrounding if structure
let comp_inter_type = b.c2rust_unnamed.c2rust_unnamed_0.comp_type.unwrap();

because that variable is now bound by the if let


Prevent default clicking on markdown preview links when using openLinksInNewTab

Describe your changes here.
Closes https://github.com/primer/react/issues/2717
MarkdownViewer accepts an openLinksInNewTab prop to ensure link previews open in a new tab. Because default is not prevented, navigation is also happening in the working tab.
I've also added gone ahead and added noopener noreferrer to the call to window.open s/o @mattcosta7
Screenshots
Before:
https://user-images.githubusercontent.com/5373900/208768433-ca2095a9-9965-42eb-860c-bb27a2667727.mov
After:
https://user-images.githubusercontent.com/5373900/208768491-87f39bc8-43af-4f18-b00f-92d62903adb6.mov
Merge checklist

[x] Added/updated tests
[ ] Added/updated documentation
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

@joshblack are you able to reenable auto-merge? pushed out a linting fix and I don't have access to merge 🙏

@marywhite on it!


Who to contact for security issues

Hello 👋
I run a security community that finds and fixes vulnerabilities in OSS. A researcher (@vvxhid) has found a potential issue, which I would be eager to share with you.
Could you add a SECURITY.md file with an e-mail address for me to send further details to? GitHub recommends a security policy to ensure issues are responsibly disclosed, and it would help direct researchers in the future.
Looking forward to hearing from you 👍
(cc @huntr-helper)

Create a security document in root. The proper email is<EMAIL_ADDRESS>and I believe we did get an email from them, but it did not go into detail on the issue - only on the proper channel for reporting which is that email. 👍


@JsonClass annotation is not generated

First of all thank you very much for the plugin!
About issue, created dummy project to try: https://github.com/Andre777/test_swagger and noticed that @JsonClass(generateAdapter) annotation is not generated in the models.
Please, could you look what I'm doing wrong.

Hi @Andre777
That is expected as the @JsonClass(generateAdapter) annotation is currently on master but unreleased. Will be available in the upcoming version (1.4.0) that will be released in the next days.

The new version has been release, hence closing this.
Feel free to re-open this if you are still experiencing the same problem.


Minor: tweak docuement of sort enforcement optimizer rule

Which issue does this PR close?

Closes #.
Rationale for this change
Some struct referenced in sort_enforcement optimizer rule are not linked
What changes are included in this PR?
Some minor style changes that

add a new line under mod comment
add link to referenced struct

Are these changes tested?

Are there any user-facing changes?

Thanks @waynexia


[ENG-5836] add --message param to eas build

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
We want to allow users to attach messages to builds.
How

Add --message / -m param to eas build.
Add message to metadata.

Test Plan
I ran eas build -m "test message" and checked that the message was saved to metadata.

/changelog-entry new-feature Add support for attaching messages to builds.

@dsokal Hey, quick question about this one, the message field is not being replicated to the App payload manifest yet right? Is something that we could expect to happen or perhaps need a hand putting that change up?

@MorenoMdz - i believe you are interested in the message param for eas update, right? this pr is about the a message param for eas build. we do not add this to the manifest, no.

@MorenoMdz - i believe you are interested in the message param for eas update, right? this pr is about the a message param for eas build. we do not add this to the manifest, no.

Exactly, I'm curious if we could have this message replicated in the manifest extra, this could lead to a few interesting flows in the app


Add IHG.com

IHG Rewards accounts are protected with a four digit PIN.
https://www.ihg.com/rewardsclub/us/en/join/register

Thank you. I will happily take PRs, if you’d like to submit one.


Add unlimited local avatar favs

Implement unlimited local avatar favs with VRC+ requirement.

wait why vrc+ requirement if it's local only and for you to change into them you gotta completely reboot the game?

wait why vrc+ requirement if it's local database only and for you to change into them you gotta completely reboot the game?

Because making it non VRCPlus would make VRCPlus pointless thus interfering with VRChats source of income, VRChats has made it very clear in the past they don't want this to happen (such as with EMMvrc the mod for VRChat having to lock unlimited avatars behind VRCPlus ownership)

wait why vrc+ requirement if it's local only and for you to change into them you gotta completely reboot the game?

You don't have to restart your game to switch into the Avatars, I do not know where you got that from.

yeah for clarification changing avatar from the website or VRCX is not a VRCPlus requirement (unless vrchat changes their mind), only local favorites are locked behind owning VRCPlus for the above reasons.

wait why vrc+ requirement if it's local database only and for you to change into them you gotta completely reboot the game?

Because making it non VRCPlus would make VRCPlus pointless thus interfering with VRChats source of income, VRChats has made it very clear in the past they don't want this to happen (such as with EMMvrc the mod for VRChat having to lock unlimited avatars behind VRCPlus ownership)

i believe having it non vrc+ would make people annoyed enough for having to mess around with their favorites all the time to actively want to buy vrc+, sounds counter intuitive to me if the idea is to get people to want to buy it (considering how to get people to actively want to engage with your product)
although i guess it's also fair to just be safe enough, since the community is not responsible for how they plan out the game's finance (and we've had issues with stuff like that with the API before as well, not only on VRCX)

wait why vrc+ requirement if it's local only and for you to change into them you gotta completely reboot the game?

You don't have to restart your game to switch into the Avatars, I do not know where you got that from.

If you don't have to restart anymore that's new to me, I got so used to not being able I just gave up entirely on even bothering with it

If you don't have to restart anymore that's new to me, I got so used to not being able I just gave up entirely on even bothering with it

yep the change was made 4 months ago now as apart of an open beta update before it was pushed to live
https://discord.com/channels/189511567539306508/503009489486872583/1228107514399293440

translation is wrong till latest version.
https://github.com/vrcx-team/VRCX/pull/845/files#diff-af9254c4cffd2f02ec309237af67573be0b6f9d22069aab754ee5768a678f7a8R769
to above:
this is open-source software no idea why you guys complain.


Note on mixing network_mode with links in Docker compose file

I just encountered this little gotcha today coming from https://github.com/moby/moby/issues/15086 :)
This might save other people some time :tada:

Thanks, @orangecms :)


Results Map Improvements

Ability to

zoom to item (add map search if not selected?)
just show the N items on the map, and then highlight when an item is moused over.
zoom to aggregated items show in map  when clicked

on 1, there are some fixes recently checked in to improve the zoom to function.
2 &  3 not implemented, it would be an enhancement for the future.


Extend process section

The [process:*] section needs additional configuration values.

chown=user:group
norestart=<true|false>
flush=<number><s|m|h>

Related to old braum project.


Added new Kubernetes tutorial

What is the purpose of the change

I'm adding another resource for K8s basic concepts.

(E.g.: This pull request improves documentation of area A by adding ....)

@all-contributors add @mariuszmichalowski as code


Downpin OpenMMForcefields to avoid AmberTools/OpenMM/OpenMMForceFields incompatibility

/regenerate-cache

/regenerate-cache

OK looks like the remaining failure is a different issue so I'll merge this! Thanks @j-wags!


hotreload not working in Win10 and Ubuntu

@HeaTTheatR
hotreload not working in Win10 and Ubuntu ( py3.10, kivy2.1.0, kivymd1.0.0&1.0.1)
have to focus in window and press CTRL+R every time
Versions

OS: Win + Ububntu
Python: 3.10 + 3.9
Kivy: 2.1.0
KivyMD: 1.0.0 + 1.0.1

https://user-images.githubusercontent.com/14234237/184272230-3df47fc6-76e3-4efe-adea-6401d0c4bcdf.mp4

In theory, this is how it should work, but in a good way we need to do something about it.

@Neizvestnyj Enable autosave and there will be no problems.


chain.call_cache will cache the data whose return_value='' is empty

Do you want to request a feature or report a bug?
bug
What is the current behavior?
chain.call_cache will cache the data whose return_value='' is empty, causing the subsequent returned data to be wrong
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
select count(*) from chain1.call_cache where return_value='';
{
"errors": [
{
"message": "indexing_error"
}
]
}
What is the expected behavior?
Don't cache empty data
chain/ethereum/src/ethereum_adapter.rs
if !for_cache.is_empty() {

Re-upping this issue, as I just stumbled into it.
Following an outage of an RPC node, graph-node was restarted while the RPC node was still recovering:
2024-11-22 07:46:57.644	Nov 22 12:46:57.644 DEBG Trying again after eth_call RPC call for block #64589850 (c8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6) failed (attempt #4) with result Err(Web3Error(Rpc(Error { code: InternalError, message: "hash c8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6 is not currently canonical", data: None }))), provider: dshackle-polygon-ing, sgd: 1854, subgraph_id: QmQEYSGSD8t7jTw4gS2dwC4DLvyZceR9fYQ432Ff1hZpCp, component: SubgraphInstanceManager
2024-11-22 07:47:06.335	Nov 22 12:47:06.335 DEBG Contract call finished, source: rpc, block_number: 64589850, block_hash: c8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6, result: none, time_ms: 9348ms, args: [], signature: totalFeeGrowth0Token():(uint256), contract: Pool, address: 0xac4494e30a85369e332bdb5230d6d694d4259dbc, sgd: 1854, subgraph_id: QmQEYSGSD8t7jTw4gS2dwC4DLvyZceR9fYQ432Ff1hZpCp, component: SubgraphInstanceManager
2024-11-22 07:47:06.338	Nov 22 12:47:06.338 INFO Applying 3 entity operation(s), block_hash: 0xc8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6, block_number: 64589850, sgd: 1854, subgraph_id: QmQEYSGSD8t7jTw4gS2dwC4DLvyZceR9fYQ432Ff1hZpCp, component: SubgraphInstanceManager
2024-11-22 07:47:06.338	Nov 22 12:47:06.338 ERRO Subgraph error 1/1, code: SubgraphSyncingFailure, error: transaction b82f3c8b76070668d8472d7df5c913b90722d00eb1ddec4f657743fccc469256: error while executing at wasm backtrace:	    0: 0x5b9c -<EMAIL_ADDRESS>   1: 0x8ac9 - <unknown>!src/mappings/core/handleSwap: Mapping aborted at<EMAIL_ADDRESS>line 440, column 7, with message: Call reverted, probably because an `assert` or `require` in the contract failed, consider using `try_totalFeeGrowth0Token` to handle this in the mapping. in handler `handleSwap` at block #64589850 (c8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6), block_hash: 0xc8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6, block_number: 64589850, sgd: 1854, subgraph_id: QmQEYSGSD8t7jTw4gS2dwC4DLvyZceR9fYQ432Ff1hZpCp, component: SubgraphInstanceManager
2024-11-22 07:47:06.470	Nov 22 12:47:06.470 ERRO Subgraph instance failed to run: transaction b82f3c8b76070668d8472d7df5c913b90722d00eb1ddec4f657743fccc469256: error while executing at wasm backtrace:	    0: 0x5b9c -<EMAIL_ADDRESS>   1: 0x8ac9 - <unknown>!src/mappings/core/handleSwap: Mapping aborted at<EMAIL_ADDRESS>line 440, column 7, with message: Call reverted, probably because an `assert` or `require` in the contract failed, consider using `try_totalFeeGrowth0Token` to handle this in the mapping. in handler `handleSwap` at block #64589850 (c8fb260eaf4269ea0a17bd687ef52ca1f876c8e239fb618965a57beb3f50bba6), code: SubgraphSyncingFailure, sgd: 1854, subgraph_id: QmQEYSGSD8t7jTw4gS2dwC4DLvyZceR9fYQ432Ff1hZpCp, component: SubgraphInstanceManager
2024-11-22 07:48:06.507	Nov 22 12:48:06.507 INFO Committed write batch, time_ms: 167, weight: 1204, entities: 1, block_count: 1, block_number: 64589850, sgd: 1854, subgraph_id: QmQEYSGSD8t7jTw4gS2dwC4DLvyZceR9fYQ432Ff1hZpCp, component: SubgraphInstanceManager

The RPC node was unable to process the eth_call, and as a result the subgraph failed, and graph-node store an empty value inside the call cache.
"encode","decoded_return_value","decoded_contract_address","block_number"
"f28849358f09ea47707fb7e8510006a40b322ee138c23f572d4a926e033b2807","","ac4494e30a85369e332bdb5230d6d694d4259dbc",64589850
"c86ea13e8fd3e535c02a4db3edea8d17c15e0ca3ee9cb76a19045431e29864df","","4c580fc9761a89a578ee03817e1ed49b5b96a6f0",64589850
"3642ec82b0ebebed7b53345def71071c5764834a8c66d8251eec70de498b358f","","4c580fc9761a89a578ee03817e1ed49b5b96a6f0",64589850

As a result, all subsequent run of the subgraph failed at the same point, despite the fact that the RPC node had since recovered. Clearing the block cache for the specific block and restarting graph-node resolved the issue.


[schema-registry] Hide _response and reduce interface count

Reduced the number of interfaces in the schema-registry API in two ways:

Removed _response from typing, eliminating the Response interface and each of the XxxResponse types which are now just Xxx.

Inlined content property into SchemaDescription and Schema, eliminating SchemaDefinition. There is no customer use case for a type that holds only the content. More likely, they'd just pass 'string' around for that.

It's also possible to further collapse SchemaId and Schema, and that is being proposed for the Java API. The reason it is possible is that they differ only by content , but when we return a SchemaId, we are passed the content so we could hand that back. However, I haven't done this yet as I'm not yet convinced it's helpful. I think it would actually make the purpose each API less clear. That said, I'll do it if the group feels strongly about it after we discuss the Java proposal.

Incorporating this into another PR that will implment schema registry-based avro serializer


stats.js: Turn messages sent by client into a horizontal bar chart.

A typical org will have 5-15 clients that will be interesting, so ideally the bar chart would look good in those ranges. It would also be nice to cut it off at 12 clients (with the rest going in "Other"), and have a "show all" button that toggles showing all the clients vs "top 12 + other".
Some notes:

Probably no changes need to be made outside of static/js/stats/stats.js. If you do need to make a css change, the relevant file is static/styles/stats.css.
Few changes need to be made to the existing code in populate_messages_sent_by_client. Most of the action is in compute_pie_chart_data (which will need to be renamed) and in adding a new trace configuration.
I'd make x the values, the y the labels with %ages (e.g. "Website (34%)"). The hover should work as it does in the other bar charts. (Something like "Website: 123,456" in the lower left corner).
I'd increase the width to whatever the messages sent over time graph is (750px? Though there also seems to be a margin).
Remove the Total messages:  line.
https://plot.ly/javascript/horizontal-bar-charts/ might be helpful.

To get to the /stats page:

Run ./manage.py populate_analytics_db.
Restart the dev server (tools/run-dev.py)
Log in as Shylock from localhost:9991/login
Go to localhost:9991/stats

This would be a great project for someone interested in frontend development or visual design. It requires little knowledge or interaction with Zulip backend code.

I'd like to work on this task.

Thanks! Though let's hold off on this for a bit, actually; we're about to merge something that likely resolves this. Closing this for now; will reopen if things change.
Sorry about that, and we should be posting other stats-page related tasks in the upcoming days.

@rishig Thanks for the info!


'PROBLEMS Window' doesn't report any errors or warnings when imported file is empty

Environment data
Build of Visual Studio Code - Insiders: 1.81.0-insider
Build of python-insider: v2023.13.12071009
Build of pylance-insider: v2023.7.41
Repro Steps

Open a folder and create two python files
Open test.py and define a method in it
Open file1.py and import the method from test.py
Go back to test.py and clear the method
Save all files

Expected behavior
PROBLEMS Window should report errors or warnings
Actual behavior
PROBLEMS Window doesn't report any errors or warnings

It appears that you have python.analysis.typeCheckingMode set to off. The error that you are expecting to see is disabled by default when typeCheckingMode is off. You'll get the behavior that you expect if you set it to either basic or strict.
Clicking on the curly braces in the VS Code status bar when a Python file is open is a quick way to toggle typeCheckingMode between off and basic:


not closing client

process.on('SIGINT', function() {
    consumer && consumer.close();
    client && client.close();
});

My node process won't close when using ctrl+c
This unit test is also failing:
./node_modules/.bin/mocha test/manual.gracefulexit.js

Client #close should close gracefully:
Error: timeout of 4000ms exceeded. Ensure the done() callback is being called in this test.

Closing the consumer will close the client.

You should have a handler in your consumer.close call that calls process.exit();


F5: function to log user in

Log user into application and allow them access to functions through UI

"Create Function:
Function signInWithEmailAndPassword should log the user in, else show a text displaying input was incorrect
Checks:

Ensure the text is displayed if user input is incorrect
Success: Log the User In
Un-Successful: Display incorrect login details text"


On some sites, the fill function does not work correctly

On some sites, the fill function does not work correctly.
Characters are not fully entered.
Running in python venv (Win10x64) Python 3.6
Installed packages in venv:
splinter==0.10.0
from splinter import Browser
.fill("123123")
output:
131
For example site http://npchk.nalog.ru/
browser.find_by_xpath('//*[@id="inn"]').fill('5249155837')

This site in particular is using some nasty javascript to validate the form value. This isn't something selenium or splinter can solve for you.


Diags collection script

I got tired of collecting the wrong diags for issue #67, so I wrote a shell script to do it for me.
I'm not claiming this is perfect, but it gives us something to add to whenever diags are insufficient. :)

First I think we should code manage this somewhere, as that will help further review and enhancement.  I suggest creating a 'tools' or 'utils' subdir in the Calico repo, and adding this script in there.
Secondly I wonder if we can rely on iptables-save, which will do a more complete job than the current iptables invocations in this script.  I think we can, on the grounds that we can ensure that iptables-save is available on any of our test machines, and document the requirement for others.
That's all the script comments I have for now.  I guess the next thing after that would be automation so that it actually runs, and so that the logs are made available via the Jenkins UI.

This was merged in as part of #125, so this can be closed.
However, Nick Bartos has provided some good enhancements: see the linked issue to track those.


Pomotroid failing to launch

uname -r : 5.16.10-arch1-1
Using SwayWM with Wayland.
When launching pomotroid from the command line, this error is showing up:
[25143:0220/150706.660543:FATAL:gpu_data_manager_impl_private.cc(439)] GPU process isn't usable. Goodbye.
If there's any other info needed, please let me know.

I should mention, reinstalling Pomotroid did not work.

Same problem here, I'm using KDE with X11, installing from  aur/pomotroid-bin
[48732:0221/162157.138893:FATAL:gpu_data_manager_impl_private.cc(439)] GPU process isn't usable. Goodbye.
yafish: Job 1, 'pomotroid' terminated by signal SIGTRAP (Trace or breakpoint trap)

Same here, using i3 with Xorg.

Same here

For me, Pomotroid-bin works again with --no-sandbox.

--disable-gpu-sandbox did the trick for me. Seems to be a electron bug

For me, Pomotroid-bin works again with --no-sandbox.
Update: Related link in r/archlinux.
Thanks a lot! It's working on my machine again

I can verify that this bug happens on Wayland too, in both AppImage and compiled binary versions.
Passing --no-sandbox to the AppImage solves the problem as well.

For those on Arch, after installing from AUR pomotroid-bin you can modify Pomotroid's .desktop file by adding the --disable-gpu-sandbox flag to the Exec config property. Here's a imdepotent command you can use to apply the fix:
sudo sed -i 's|Exec=/usr/bin/pomotroid|Exec=/usr/bin/pomotroid --disable-gpu-sandbox|g' /usr/share/applications/pomotroid.desktop

For those on Arch, after installing from AUR pomotroid-bin you can modify Pomotroid's .desktop file by adding the --disable-gpu-sandbox flag to the Exec config property. Here's an [idempotent] command you can use to apply the fix:
sudo sed -i 's|Exec=/usr/bin/pomotroid|Exec=/usr/bin/pomotroid --disable-gpu-sandbox|g' /usr/share/applications/pomotroid.desktop

Depending on your setup, you could also first copy the file to your home directory if you want to avoid modifying the system desktop entry:
cp /usr/share/applications/pomotroid.desktop ~/.local/share/applications/
sed -i -r 's|Exec=(/usr/bin/pomotroid)|Exec=\1 --disable-gpu-sandbox|' ~/.local/share/applications/pomotroid.desktop

this is still an issue on linux mint 21.2

For those on Arch, after installing from AUR pomotroid-bin you can modify Pomotroid's .desktop file by adding the --disable-gpu-sandbox flag to the Exec config property. Here's an [idempotent] command you can use to apply the fix:
sudo sed -i 's|Exec=/usr/bin/pomotroid|Exec=/usr/bin/pomotroid --disable-gpu-sandbox|g' /usr/share/applications/pomotroid.desktop

Depending on your setup, you could also first copy the file to your home directory if you want to avoid modifying the system desktop entry:
cp /usr/share/applications/pomotroid.desktop ~/.local/share/applications/
sed -i -r 's|Exec=(/usr/bin/pomotroid)|Exec=\1 --disable-gpu-sandbox|' ~/.local/share/applications/pomotroid.desktop

Btw, these are not strictly idempotent as they keep adding the flag to the start of the command if run multiple times on the same file. But having multiples of the same flag is probably not an issue unless it's a crazy amount of times.

I update the file as you suggested
when I try to open pomotroid again, using special_key+p on my awesome, the app stills goes down.
what to do next?

I update the file as you suggested
when I try to open pomotroid again, using special_key+p on my awesome, the app stills goes down.
what to do next?

You can give https://github.com/vjousse/pomodorolm a try if you want, it's a pomotroid clone. The archlinux package name is pomodorolm-bin. I wrote it to have an up to date version of pomotroid running on my system and compatible with Wayland as pomotroid seems unmaintained.


Change preview box filename when previewed file is changed

Steps to reproduce:
1- visit a record page with multiple attachments
2- click to preview a different file
expect
3- previewed filename to be showed at top of box
get
3- initially previewed file's name (it doesn't change)

I have (possibly in error) raised this here too https://github.com/inveniosoftware/invenio-previewer/issues/110


[Release Issue] Deploy step not executed

The current Travis configuration (I think) did not trigger the deployement to jazzband pypi server.
https://travis-ci.org/jazzband/pip-tools/jobs/483642337
@atugushev @jezdez Any idea of recent change that may be in cause?

This seems to have happened again. Was notified about a release on Github yesterday, still do not see the release on PyPI.

@SpotlightKid package has been uploaded successfully, but release not approved yet, see https://github.com/jazzband/roadies/issues/141.

Ah, I see.

@SpotlightKid FYI 3.6.0 is released


[CS/EDJE] Change router to add inference

Change router to contain inference pages

:octocat: cibot: Thank you for posting issue #571. The person in charge will reply soon.

resolved with #580


Projects API: Change urlencode to rawurlencode

Trying to search a project like this: "Test project" it gets converted to "Test+Project", instead RedmineAPI expect the space encoded as "%20", which is correctly handled by rawurlencode function.

Thanks for this @paolomainardi. I will not be merging this as you've only updated the Project endpoints but I shall make sure the library is updated to use rawurlencode() in the next release.


Possible Install Improvements

Great work you're doing here in the absence of official Intune integration right now. I've made a few function additions to your install script and thought I'd share. I don't profess to be a scripter, so apologies for any unnecessary lines.
For installs to devices where the inclusion or version of Winget might not be guaranteed, but enforced during installation:
function Install-WinGet{
        #Download WinGet MSIXBundle
        Write-Host "Downloading WinGet..."
        $WinGetURL = "https://aka.ms/getwinget"
        $WebClient=New-Object System.Net.WebClient
        $WebClient.DownloadFile($WinGetURL, "$PSScriptRoot\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle")

        #Install WinGet MSIXBundle
        Write-host "Installing MSIXBundle for App Installer..."
        Add-AppxProvisionedPackage -Online -PackagePath "$PSScriptRoot\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle" -SkipLicense

        #Check Package Install
        Write-Host "Checking Package Install"
        $TestWinGet = Get-AppxProvisionedPackage -Online | Where-Object {$_.DisplayName -eq “Microsoft.DesktopAppInstaller”}
            If($TestWinGet.DisplayName) {
                Write-Host "WinGet Installed"
                Remove-Item -Path "$PSScriptRoot\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle" -Force -ErrorAction Continue
                Return $True
            }
            Else {
                Write-Host "WinGet Not Installed"
                Return $False
            }
}

And for trying to configure high-priority notifications out the gate, which caters for being delivered to existing devices, or as part of a device-targeted Win32 deployment during Autopilot:
function Set-WAUNotificationPriority{
    $LoggedInUser = Get-WMIObject -class Win32_ComputerSystem | Select-Object -ExpandProperty username
    If($LoggedInUser -contains "defaultuser0") {
        Write-Host "Autopilot deployment defaultuser0 detected. Loading Default User registry hive."
        reg load HKU\Default C:\Users\Default\NTUSER.DAT
        Write-Host "Creating default user notification registry keys"
        New-Item -Path "HKU:\Default\SOFTWARE\Microsoft\Windows\CurrentVersion\Notifications\Settings\Windows.SystemToast.WAU.Notification"
        reg unload HKU\Default
    }
    Else {
        $objUser = New-Object System.Security.Principal.NTAccount($LoggedInUser)
        $strSID = $objUser.Translate([System.Security.Principal.SecurityIdentifier])
        Write-Host "Standard deployment detected. Adding registry keys to logged-in user hive."
        New-PSDrive -Name "HKU" -PSProvider "Registry" -Root "HKEY_USERS"
        Set-Location -Path "HKU:\$strSID\Software\Microsoft\Windows\CurrentVersion\Notifications\Settings"
        New-Item -Path "HKU:\$strSID\Software\Microsoft\Windows\CurrentVersion\Notifications\Settings" -Name "Windows.SystemToast.WAU.Notification"
        New-ItemProperty "HKU:\$strSID\Software\Microsoft\Windows\CurrentVersion\Notifications\Settings\Windows.SystemToast.WAU.Notification" -Name "Rank" -Value "99" -PropertyType Dword
        Remove-PSDrive -Name "HKU"
    }
}

Hi, Thanks for your comment and your help ;)
I'll try to have a look on your work when I have time. Seems great.

by the way, don't hesitate to fork and add your modifications

Hi, Winget install has been implemented. But could you give us more info about the second part of your work (Set-WAUNotificationPriority function)?

Hi! So I haven't tested this on Windows 10, only Windows 11, but what I've been trying to do is configure the notification to have Top priority (if you go to System>Notifications and choose a specific notification). The logic around the "defaultuser0" section is to do with Intune Autopilot deployments, and then adding the registry key for the updater into the Default user hive for when the user actually logs in, otherwise it should find who the current logged-in user is and add the relevant keys.
When priority is set to "High" the Rank registry key is created and is set to a value of 1, setting it to "Top" changes the Rank to 99.


[#PLAT-542]Update Identifier creation to no longer create ARKs

Purpose
Update the identifier process, where we allow users to click 'Create DOI/ARK' to only create DOIs going forward.
Continue to display ARKs that were previously created
Update the UI so that users no longer see the option to Create ARKs, and only have the option to get a DOI for their content.

Changes
Now you should only see 'Create DOI' for public projects we are admin on. For all new created DOI, it will only have DOI. For all existing identifier, it will show DOI and ARK.

QA Notes

go to a public project you have already created DOI/ ARK, everything should be the same.
create a new public project
You should see ' Create DOI' instead of 'Create DOI/ ARK'
After clicking that link and DOI is created, you should only see 'DOI: ........' without any 'ARK:.......'

Side Effects

Ticket
https://openscience.atlassian.net/browse/PLAT-542

Merged into release/next-platform. 🚢 🇮🇹


Contribution to "Incorrect Permission Assignment for Critical Resource in Node"

Updates

Affected products
Description
Source code location

Hey @jbmagination, sorry for the delay on getting to this. This CVE is about the node.js runtime which has a source code repo here
https://github.com/nodejs/node
The node package on the npm registry is a different beast which does look related, but also unaffiliated and potentially different source code altogether. Sources here
https://www.npmjs.com/package/node and https://github.com/aredridel/node-bin-gen
We can only curate on packages which exist in our ecosystems which I know we don't have docs up for yet and that's on us. Thank you for the contribution and the effort, but I will need to reject this one. I will make sure to get some docs up on how our ecosystems work asap. If you don't see anything on them by the end of next week please open an issue and badger us :)

@darakian Okay wait, then what about https://github.com/advisories/GHSA-gqhp-5j32-xwmm? It's a CVE about the runtime yet references the package node. I referenced it whenever I was making this contribution

That would be an error on our part. Thanks for pointing that out too 👍


Add a paradigm for gettable protocol getters

Best reviewed in Files Changed view.
This PR builds on https://github.com/fdiaz/SwiftInspector/pull/110 to add a paradigm for properties in protocols.
We start by documenting the current behavior.
Then we add ways to strongly encode these cases into PropertyInfo.Paradigm.

var foo: Foo { get } -> PropertyInfo.Paradigm.protocolGetter
var foo: Foo { get set } -> PropertyInfo.Paradigm.protocolGetterAndSetter

var foo: Foo { set } is not valid Swift code.
@fdiaz cc: @dfed

I'm still very perplexed on Code Coverage. I made a new PR (to see if somehow CodeSense is corrupted on this PR) and we see a similar result. https://github.com/fdiaz/SwiftInspector/pull/114

The issue with code coverage was that I had converted one of the tests from it to fit and checked in that change.

@fdiaz this PR is now ready for review!


Change JDK distribution, add build for JDK 21

@ctubbsii -  Should we change the JDK distribution from adopt to temurin? There is a note at setup-java that says:

NOTE: AdoptOpenJDK got moved to Eclipse Temurin and won't be updated anymore. It is highly recommended to migrate workflows from adopt and adopt-openj9, to temurin and semeru respectively, to keep receiving software and security updates. See more details in the Good-bye AdoptOpenJDK post.

Adding the distribution and JDK21 are fine, but the compiler properties should be set in the POM, along with maven.compiler.release to ensure Java 11 compliance... until the project is ready to require something newer.

Why wouldn't we want to check compliance for other JDK versions? This project is intended to be used outside of Accumulo by other projects, which may be using a different version of the JDK.

Adding the distribution and JDK21 are fine, but the compiler properties should be set in the POM, along with maven.compiler.release to ensure Java 11 compliance... until the project is ready to require something newer.

Why wouldn't we want to check compliance for other JDK versions? This project is intended to be used outside of Accumulo by other projects, which may be using a different version of the JDK.

I'm not sure what you're trying to test by adding this. Do you anticipate Java will at some point in the future be changed so it can no longer read or execute Java 11 byte code? The whole point of the maven.compiler.release is to ensure that the byte code that is produced will properly and strictly comply with the requirements of a particular Java byte code version, so that future JDK versions can execute it compliant with that older byte code version without suprises. I don't really understand what you're testing if you're trying to build it with different compiler compliance levels.

I'm not sure what you're trying to test by adding this.

I was under the impression that compiling with a newer JDK might produce different or more optimized byte code. I did some reading about this last night, and there are only a few cases where this is the case. So, generally, my impression was incorrect. I'll remove the compiler flags from the flags from the command line args.


python311Packages.trytond: 7.0.5 -> 7.0.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.trytond is: The server of the Tryton application platform
meta.homepage for python311Packages.trytond is: http://www.tryton.org/
meta.changelog for python311Packages.trytond is: https://hg.tryton.org/trytond/file/7.0.6/CHANGELOG
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7.0.6 with grep in /nix/store/4raqrxzi0jrn0wvbmrxjqxganalg0mxc-python3.11-trytond-7.0.6
found 7.0.6 in filename of file in /nix/store/4raqrxzi0jrn0wvbmrxjqxganalg0mxc-python3.11-trytond-7.0.6

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.trytond
python312Packages.trytond
trytond

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4raqrxzi0jrn0wvbmrxjqxganalg0mxc-python3.11-trytond-7.0.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/ff943286f14689be52abcda872cc35dbaf5dfacc#python311Packages.trytond

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4raqrxzi0jrn0wvbmrxjqxganalg0mxc-python3.11-trytond-7.0.6
ls -la /nix/store/4raqrxzi0jrn0wvbmrxjqxganalg0mxc-python3.11-trytond-7.0.6/bin

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

1 package marked as broken and skipped:

python312Packages.trytond

1 package built:

trytond (python311Packages.trytond)

Maintainer pings
cc @udono @johbo for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.trytond is: The server of the Tryton application platform
meta.homepage for python311Packages.trytond is: http://www.tryton.org/
meta.changelog for python311Packages.trytond is: https://hg.tryton.org/trytond/file/7.0.8/CHANGELOG
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7.0.8 with grep in /nix/store/nghs0481d1l2vmbwvvfwkysqj7bi0h1k-python3.11-trytond-7.0.8
found 7.0.8 in filename of file in /nix/store/nghs0481d1l2vmbwvvfwkysqj7bi0h1k-python3.11-trytond-7.0.8

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.trytond
python312Packages.trytond
trytond

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nghs0481d1l2vmbwvvfwkysqj7bi0h1k-python3.11-trytond-7.0.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/1c98b586e46de75eb1c07e05f912d734f2b9771c#python311Packages.trytond

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nghs0481d1l2vmbwvvfwkysqj7bi0h1k-python3.11-trytond-7.0.8
ls -la /nix/store/nghs0481d1l2vmbwvvfwkysqj7bi0h1k-python3.11-trytond-7.0.8/bin

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

4 packages built:

trytond (python311Packages.trytond)
trytond.dist (python311Packages.trytond.dist)
python312Packages.trytond
python312Packages.trytond.dist

Maintainer pings
cc @udono @johbo for testing.

@ofborg build python311Packages.trytond


Repeated searches leaves orphaned, unkillable rg processes

Type: Bug
On all modern versions I could test from 1.80 to 1.83-Insider, on Windows 11, VSCode searches launch rg.exe (ripgrep) processes in parallel with what seems to be a 1:1 mapping to the number of top-level folders in the workspace without pooling/parallel limits (at least none that I could see with up to ~50 folders).
With any number of folders however (including just 1) repeated searches will eventually leave one or more rg.exe processes hanging.
When the workspace or VSCode is closed (or even the relevant folder removed from the workspace) this rg.exe becomes orphaned and is completely unkillable through any method I could find: not task manager, process explorer, pskill, taskkill, or wmic. Attempting to takeown and icacls the workspace also has no effect, nor killing every other non-core process in Windows, they're uniquely untouchable and only a reboot evicts them.
Needless to say, this isn't ideal, and also mucks up any update process.
With a large enough workspace (but nothing over the top, a couple of dozen folders representing different repos) it's fairly easy to trigger this even just by typing with a cadence of ~0.5s per keystoke, which seems to be enough time for the automatic search to kick in, but not enough time for it to complete, which seems to be the most reliable reproducer (and perhaps only method of reproducing).
The greater the folders/size/search result set, the easier it seems to be to achieve this state, but if you're aggressively searching, even one single large folder experiences this with typing egg<Enter>g<Enter>g<Enter>... quickly enough.
I've tried this with standalone installs from the VSCode site and Scoop, both regular and insider releases.
May also be related to other issues like #192454 but they've not been closed with any resolution and report wasn't detailed enough to confirm.
How it manifests in process explorer
When they first hang, they appear like this as child processes

and when the parent processes are eventually killed, they hang around orphaned like this

I don't know what feature makes these orphans so invincible, there's one Deny flag I can see but a quick scan of other processes shows this isn't unique.

System details
VS Code version: Code - Insiders 1.83.0-insider (c72447e8d8aaa7497c9a4bd68bc4301584b92beb, 2023-09-22T09:29:47.641Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 9 5900X 12-Core Processor             (24 x 3693)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.91GB (39.23GB free)

Process Argv
--crash-reporter-id 3b777b0c-6c5f-41e2-aaec-e70f53b1003c

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythontb:30258533
pythonptprofiler:30281269
vshan820:30294714
vscod805cf:30301675
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
py29gd2263:30784851
vsclangdf:30492506
c4g48928:30535728
dsvsc012:30540252
pynewext54:30618038
vscrpc:30624061
showlangstatbar:30737417
ecj1e332:30687743
pythonfmttext:30716741
fixshowwlkth:30771523
showindicator:30805243
pythongtdpath:30726887
i26e3531:30792625
welcomedialog:30812478
pythonnosmt12:30779711
pythonidxpt:30768918
pythonnoceb:30776497
copilotsettingc:30808720
dsvsc013:30777762
dsvsc014:30777825
diffeditorv1:30786205
pythonlinttypecf:30823782
pythonmpsinfo:30842935
dsvsc015:30821418
pythontestfixt:30826906
pythonfb280951:30830809
pythonregdiag:30842812

I've tried this with standalone installs from the VSCode site and Scoop, both regular and insider releases.

I assume this means that you tried without any extensions? I've tried to repro, with large directories, but I can't find the orphaned rg processes. What do you have for search settings and your query? Do you have a multi-root workspace, or just a single root?

Yes, exactly; literally just download VSCode, start it up, add a folder to
the new workspace and do repeated searches on slightly different strings.
So no extensions (I think the auto env description at the bottom shows that
too), though yes the first observation was with a bunch of different ones,
it happens both with all disabled and with none even installed.
No particular search settings, just the default it opens up with, and no
particular search query. No correlation I could really see between whether
the query text was actually in the results or not. I could search for
"export" on Bash scripts or "egggggg" and it seemed to have about the same
incidence.
I replicated it both with adding Git repo folders to the workspace and
adding a single parent folder (that contained those Git repos), so I'm not
sure it was related. I also waited for all the git.exe processes to
complete doing anything before searching in case there was some initial
race condition.
I'll see if I can repro on another OS I have access to if you've given it a
shot without seeing it, thanks.
On Mon, 25 Sept 2023, 19:34 Andrea Mah, @.***> wrote:

I've tried this with standalone installs from the VSCode site and Scoop,
both regular and insider releases.
I assume this means that you tried without any extensions? I've tried to
repro, with large directories, but I can't find the orphaned rg processes.
What do you have for search settings and your query? Do you have a
multi-root workspace, or just a single root?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/193927#issuecomment-1734268786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSMYI6ETA2P3MDVDGMVTALX4HFEXANCNFSM6AAAAAA5FB3O2I
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
IB BOOST LTD is a limited liability company registered in England and Wales
(No. 07027475). Registered address 86-90 Paul Street, London, EC2A 4NE, UK.
This message is intended solely for the addressee and may contain
confidential information. If you are not a named addressee to this e-mail
it may be unlawful for you to read, copy, distribute, disclose or otherwise
use the information contained in it. If you have received this message in
error, please send it back to us, and immediately and permanently delete
it. IB BOOST LTD will not be liable for direct, special, indirect or
consequential damages arising from the alteration of the contents of this
e-mail by any third party or as a result of any virus being passed on. Our
privacy policy is available here
https://ibboost.com/privacy-and-cookie-policy.

Hmm. I was also trying on a windows machine, so I expected the same results. I do wonder if you can repro on a fresh install on a VM perhaps.

We are a big group working on shared RHEL8 servers with remote-SSH. This phenomena seems to happens quite frequently, and it will crash the whole server.
The larger the scope of the search, the more likely is it to go rogue.

Sorry for the lack of update - it's sent me down a time-consuming rabbit hole :-)
I've tried on Windows server 2019, and set up an equivalent machine with Windows 11 Dev preview more-or-less mirroring my primary device's setup and not had this replicate in either, with either "installed" or portable (scoop-shimmed) VSCode.
To your specific question I've tried running ripgrep manually with one of the hung commands and also not replicated that way yet though I need to do this more to be sure and probably programmatically.
However I also use ripgrep as my primary search in both Git Bash and WSL and I've never seen this. So I don't think it's the binary specifically. Also I both brought my .ripgreprc to the other device but it didn't induce the hang, and renamed on the main (replicating) device and it didn't stop the hang, so I don't think - at this stage - it's related to my non-VSCode ripgrep installation.
I'll continue to try to narrow down differences, just might take some time.

I have this happening on macOS.

I can pinpoint it down to two contributing factors:

large monorepo open in vscode
doing git operations on the monorepo in terminal that may change arbitrary parts of the tree

Still don't get what part of this would spawn dozens of rg processes exactly…

This is also happening to me in MacOS, latest version (1.83.1), medium monorepo. I do believe it's regression as this wasn't a problem in the past (been using it for years).

Okay, I have some conclusions after quite a lot of investigation. In short it seems to relate to having a network drive location (Google Drive mount) on the PATH, ripgrep's compressed file approach, and something with the context in which VSCode runs the process.
Workaround
In case someone sees this for the same reasons as me, and wants an easy workaround on Windows:  ensure that you don't have a network drive path on your PATH, OR ensure that you have ALL of the compression utilities ripgrep searches for on your PATH before the network drive entry.
Git Bash/mingw provides most of these; zstd and lz4 are available on Scoop, for example, as the two others ripgrep looks for, so having this in the PATH seems to short-circuit the conditions needed.
Replication
Firstly, replication: as mentioned it's indeed a function of how many searches are done. The most conssitent way I've found in the smallest repro is a workspace containing one folder containing two modestly-sized Git repos. I choose a search term that returns no results, like "egggggggg", then go into the search field, and just hold down the Enter key. This spawns many rg processes a second but they all complete relatively quickly which ensures a fair level of contention without locking cores forever.
However if you have a replicating environment with this setup, pushing enter 5-10 times is usually enough to create the conditions if your workspace has double-digit folders. It looks like this below, with the rg processes above the highlighted line, with no CPU hung (and permanently unkillable).

Investigation
I used Process Monitor and Process Explorer to identify when rg processes stopped doing any work (CPU cycles remain constant). In Process Explorer they appear with a thread waiting with Wait Reason Executive.

Then, with a few samples of these I went through the thousands of events for each rg process. From there I noticed the last entries were ALWAYS CreateFile events on a particular path in my PATH immediately before a network drive (mounted Google Drive Filestream drive, specifically).
Rearranging my PATH to something like this

we can see that the last non-ThreadExit event is always a CreateFile on C:\Windows\System32\wbem - being the last path before my S:\ network PATH entry.

Notice that these are different PIDs that locked on a different path each time, one looking for xz.exe and one for uncompress.exe. And this is consistent with the ~proportional nature of the hung processes - there are some amount of what would be attempted calls to a network drive that are failing or never return.
Also note though that Process Monitor doesn't even show any attempt to CreateFile in the network drive (S:) where it hasn't hung. For example in the second screenshot, zstd checks the C:\Windows paths then the next entry is the Git paths in this contrived PATH setup.
With this contrived setup I can replicate in seconds because ripgrep is looking for a bunch of compression utilities (zstd, gzip, uncompress, lz4, bzip, brotli...) and in this PATH it will check the network drive for all of them before the locations they actually exist. With my regular PATH it happened far less frequently because I had the network paths at the end and so it was only reaching them for lz4 as I had the rest of them in other PATH entries along the way.
Related issues
With that knowledge in hand I found this issue in ripgrep https://github.com/BurntSushi/ripgrep/issues/2111 and this brought me back to another VSCode issue that I hadn't seen when first searching (as it manifests very differently) https://github.com/microsoft/vscode/issues/184934.
The point being in the ripgrep ticket that even without having compressed file searching enabled ripgrep is always finding where those utilities are on the PATH for each launched process. Normally this takes ~100ms to complete per rg process on a working thread on my (high-spec, but largish PATH'd) machine so I'll add a comment there at the least as that one seems at an impasse on real impact. It takes about 15ms per binary it doesn't find, judging by the calls made to check for each binary, so on a clean Windows system without Git Bash/mingw on the path where many of these are found, it can add a fair amount of time.
But although the execution of these PATH checks (though not needed in this usage case) is a necessary condition, I can't say how this is definitely a ripgrep problem because I've been unable to replicate it using the identical commands when run massively in parallel (in neither mingw nor - to account for PATH manipulation in Git Bash - the Command Prompt).
I tried to replicate in CMD by running a script that runs multiple checks using the same command line as used in VSCode - running 5 in parallel every 2 seconds on a path that takes about half a second to search - and I could not get it to replicate at least over a few thousand launches.
@echo off
:loop
start /b cmd /c<EMAIL_ADDRESS>--hidden --no-require-git --ignore-case --no-ignore-parent --follow --crlf --fixed-strings --no-config --no-ignore-global --json  -g "!**/.git" -g "!**/.svn" -g "!**/.hg" -g "!**/CVS" -g "!**/.DS_Store" -g "!**/Thumbs.db" -g "!**/node_modules" -g "!**/bower_components" -g "!**/*.code-search" -- egggg . > nul 2>&1
start /b cmd /c<EMAIL_ADDRESS>--hidden --no-require-git --ignore-case --no-ignore-parent --follow --crlf --fixed-strings --no-config --no-ignore-global --json  -g "!**/.git" -g "!**/.svn" -g "!**/.hg" -g "!**/CVS" -g "!**/.DS_Store" -g "!**/Thumbs.db" -g "!**/node_modules" -g "!**/bower_components" -g "!**/*.code-search" -- egggg . > nul 2>&1
start /b cmd /c<EMAIL_ADDRESS>--hidden --no-require-git --ignore-case --no-ignore-parent --follow --crlf --fixed-strings --no-config --no-ignore-global --json  -g "!**/.git" -g "!**/.svn" -g "!**/.hg" -g "!**/CVS" -g "!**/.DS_Store" -g "!**/Thumbs.db" -g "!**/node_modules" -g "!**/bower_components" -g "!**/*.code-search" -- egggg . > nul 2>&1
start /b cmd /c<EMAIL_ADDRESS>--hidden --no-require-git --ignore-case --no-ignore-parent --follow --crlf --fixed-strings --no-config --no-ignore-global --json  -g "!**/.git" -g "!**/.svn" -g "!**/.hg" -g "!**/CVS" -g "!**/.DS_Store" -g "!**/Thumbs.db" -g "!**/node_modules" -g "!**/bower_components" -g "!**/*.code-search" -- egggg . > nul 2>&1
start /b cmd /c<EMAIL_ADDRESS>--hidden --no-require-git --ignore-case --no-ignore-parent --follow --crlf --fixed-strings --no-config --no-ignore-global --json  -g "!**/.git" -g "!**/.svn" -g "!**/.hg" -g "!**/CVS" -g "!**/.DS_Store" -g "!**/Thumbs.db" -g "!**/node_modules" -g "!**/bower_components" -g "!**/*.code-search" -- egggg . > nul 2>&1
timeout /t 2 /nobreak >nul
goto loop

I can replicate it on a single repo in a single workspace folder by hammering the Enter button (which generates much fewer occurences than just holding Enter down - it can take minutes of hammering on a single folder, compared to a few seconds just holding down Enter or with a workspace with many folders). According to https://cps-check.com/keyboard-cps I can do around 8 per second, but scripting even double that rate can max out my PC resources and not manage to recreate the problem even once.
So failing to replicate in this manner I have to conclude there's something about how VSCode is running this binary that contributes to the problem. Maybe there's some kind of sandboxing that is the cause of the issue with network drives and it just happens to be ripgrep's over exuberant repeated path searching that provokes it?
Others conditions considered
I'm not sure if this holds in MacOS and it's the same issue or not. But I could replicate on both Windows 11 and Windows 10 when I had this setup (and not on other Windows machines where I didn't).
To the point above that it's recent, actually I tried every 10th version from 1.20 to 1.80, and it happens back in 1.60 (so two years old) and that was the first of those versions that seemed to use ripgrep at all judging by spawned processes, so I don't believe it's new.
Happens in both system/user installs and portable install, on Insider and standard release channels.
Finally, worth saying in respect of the last comment, @rdsedmundo, you seem to have very long CPU Times being recorded, at least if I read this screenshot correctly, suggesting you just have lots of very long-running ripgrep searches, which is different to my issue where they get orphaned after a few seconds and never exceed 0.1s of CPU time that I've seen.

Hi, in case this helps, here's another way to reproduce the problem:

Clone https://github.com/withastro/astro and open in VSCode
Run $ pnpm i
Reload VSCode window, then occur rg processes 100% CPU usage non-stop

i've been having this issue on/off for quite a while now, it's getting quite annoying.. i dont think i ever search for files inside vscode so not sure what's triggering it.
i even disabled the search in the treeview.. only search stuff inside my files.
i did create a dump file of the rg.exe
EXCEPTION_RECORD:  (.exr -1)
ExceptionAddress:<PHONE_NUMBER>000000
   ExceptionCode: 80000003 (Break instruction exception)
  ExceptionFlags: 00000000
NumberParameters: 0

FAULTING_THREAD:  00009304
PROCESS_NAME:  rg.exe
ERROR_CODE: (NTSTATUS) 0x80000003 - {EXCEPTION}  Breakpoint  A breakpoint has been reached.
EXCEPTION_CODE_STR:  80000003

STACK_TEXT:
ntdll!NtOpenFile+0x14
KERNELBASE!InternalFindFirstFileExW+0x1b6
KERNELBASE!FindFirstFileW+0x2c
rg+0x282166
rg+0x199bdc
rg+0x1b5451
rg+0x1abe7d
rg+0x19d45b
rg+0x28bf2c
kernel32!BaseThreadInitThunk+0x1d
ntdll!RtlUserThreadStart+0x28

STACK_COMMAND:  ~0s; .ecxr ; kb
SYMBOL_NAME:  rg+282166
MODULE_NAME: rg
IMAGE_NAME:  rg.exe
FAILURE_BUCKET_ID:  BREAKPOINT_80000003_rg.exe!Unknown
OS_VERSION:  10.0.22621.1
BUILDLAB_STR:  ni_release
OSPLATFORM_TYPE:  x64
OSNAME:  Windows 10

FAILURE_ID_HASH:  {21a7de60-ca4c-e593-a792-c562b346c4fb}

I do not have any networked drives in my Path, everything in my path is in C:
I do have Google Drive.
it's stuck running with this cmdline,
"c:\Users\<username>\AppData\Local\Programs\Microsoft VS Code<EMAIL_ADDRESS>--files --hidden --case-sensitive --no-require-git -g **/*.{cshtml,razor} -g !**/.git -g !**/.svn -g !**/.hg -g !**/CVS -g !**/.DS_Store -g !**/Thumbs.db --no-ignore --follow --no-config --no-ignore-global

And as far as i can tell it's completely un-killable which is quite curious..

I'm having the same/similar problem, only started happening with the latest version of VS Code, rg.exe that never ends, happens even if I don't search for anything, only on fresh start. I need to kill the process twice for it to go away.
"c:\Users\<username>\AppData\Local\Programs\Microsoft VS<EMAIL_ADDRESS>--files --hidden --case-sensitive --no-require-git -g **/.copilotignore -g !**/.git -g !**/.svn -g !**/.hg -g !**/CVS -g !**/.DS_Store -g !**/Thumbs.db -g !**/.linaria-cache -g !**/.next --no-ignore --follow --no-config --no-ignore-global

I had locked-in my VSCode version to 1.82.x for a while to avoid this, tried to upgrade today and VSCode still has this bug, I'm very surprised more people are not seeing it.

I tried setting Follow Symlinks to false but it didn't work. I had two folders on my locally mapped Google drive and two folders on the C drive. The Google drive folder "My Files" was the issue. I removed the folder from my workspace and the issue of rg.exe not closing disappeared. The other Google drive folder (one shared to me) doesn't seem to affect it. I'm not sure what the exact issue was but whatever the cause was, removing that Google drive folder from my workspace resolved the issue for me.

@nikgoodley-ibboost indeed you were right that we have different problems. I ended doing an extension bisect and realized that my problem in particular is actually coming from the Version Lens extension, with it enabled after 1.82.3 I see the several rg processes consuming all my CPU, disabling it, it works fine in the latest VSCode. I'll raise an issue there, but leaving this in case it helps anyone searching for something similar.

I'm also having this issue. I'm using VSCode in Mac and connection to CENTOS 7.
It's causing my VM to crash once per day, because somehow the connection between the VM and the network disk is becoming permanently broken. I can see 20+ ripgrep processes launched when this connection breaks

I had to disable the Remote Development, WSL and Dev Containers extensions. It seems the open any folder part of those extensions is launching ripgrep to do fulltext searching.
Microsoft said in 2022 that this problem is caused by a faulty driver
Huh...
My configuration is VScode and Google Drive. It worked successfully before VScode 1.80. Not having Remote Dev is a non-starter.
I added Files:Exclude **/desktop.ini and it appears to help a bit with speeding up the fulltext search.
What do those three extensions need a ripgrep fulltext search for every time VScode is opened?
Would enabling Search > Search Editor: Reuse Prior Search Configuration solve this problem with ripgrep?

I'm also experiencing this on MacOS with the latest version of Code (1.90.1). It's not strictly search related. Happens several times a day and I have to quite/reopen Code to make it stop. Tons of rg processes shown in activity monitor.
I opened Code's process explorer and grabbed a screenshot today. See the window process is using 435% of my CPU and 2.6gb ram!

@andreamah if you need to see this in action to figure it out, I'm happy to screenshare with you. I think I can reproduce it.

Is this happening solely when connected to a remote?

Does this happen when all extensions are disabled for you?

If you download the insiders version of vscode and set search.ripgrep.maxThreads, I believe that this will consume less CPU, but will cause a longer extension initialization. Can you try that?

Version: 1.91.0-insider (user setup)
Commit: 0da59bdde2339d6ba632524dc8f3fb6f4b8a2bcb
Date: 2024-06-14T10:07:53.538Z
Electron: 29.4.0
ElectronBuildId: 9593362
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Windows_NT x64 10.0.19045

Opened vscode-1.91.0-insider with search.ripgrep.maxthreads: 0 Opened a .ps1 file. No rg.exe.
Added and enabled the Remote Dev extension.
PS> get-process -name rg

 NPM(K)    PM(M)      WS(M)     CPU(s)      Id  SI ProcessName
 ------    -----      -----     ------      --  -- -----------
      7     2.34       7.01       0.30   22080   2 rg

Set "search.ripgrep.maxThreads": 1
Reboot
Opened vscode-1.91.0-insider
PS> while ($true) { get-process -name rg; Start-Sleep -Seconds 5 }

Ran it for several minutes. No rg.exe process.
Please integrate whatever is in the insider Code base into the GA distribution of Code.

The release of Code 1.91.1 appears to have resolved the reported problem. The default setting is
search.ripgrep.maxthreads: 0

Correction. I had to set search.ripgrep.maxthreads: 1 to make it stop.

I've already given pretty thorough description of the problem and how it relates to both inherent ripgrep compression searching (see https://github.com/BurntSushi/ripgrep/issues/2111#issuecomment-1793793703), and PATH setup, don't think there's more I can add. At least two of the people above mention having Google Drive too. The "solutions" mentioned above of dropping threads to one to me are just masking the problem by serialising the execution however the context of how it runs through VSCode seems to matter and that's what I can't really test since it works on command line. Since altering my path I've not seen the issue.

Based on my results, this issue can be closed. @andreamah appears to have integrated the code change to stabilize the reported problem I documented on 2024-07-19. Version: 1.93.1 is stable.

I'm not sure it should be closed really, though having changed my PATH config I haven't had the issue, and perhaps no one else commenting in the last few months does mean it's solved more properly? But to be clear the subsequent issues raised in this thread are not precisely the same as I reported and detailed, which was unrelated to remote connections (at least in the context of VSCode). If the solution is forcing what ought to be a parallelised search to a single-thread to me that's more a workaround.
From what I showed there was pretty clear evidence it was related to the PATH using network drives and some ripgrep default behaviour for compression utilities that was apparent in the way VSCode invokes the binary. Given it feels "racey" I would have thought even something like a mode to stagger synchronised invocations might be sufficient. Maybe that's been done but there's been no reference to an explicit commit to address so I'm speculating.
Depends whether the goal of the issue is to solve it properly for all users, regardless of context of use, which I would have thought was the goal.


Fix parsing http://

Building #39 with go 1.4.1 produces "addrs declared and not used".

The project has moved to https://github.com/rakyll/hey. Closing the PRs on this legacy repo.


Feature request: Install DynaKube using a Helm Chart in the same manner that the Operator is installed via Helm.

Installing Dynatrace Operator currently involves installing Operator via Helm chart and then installing DynaKube via kubectl.
Why DynaKube cannot be installed via Helm Chart in the same manner that the Operator is installed via Helm.

Hi, the problem is that helm v3 has some downsides, when it comes down to handling and maintaining CRDs. The problem is that you basically have 2 different options to install CRDs.

Place it in the crds folder, this folder will be applied before all other ressources, however this folder will only be applied during the first install, not during upgrades, which does not allow updates to CRDs

Have it as a normal manifests -> this allows upgrades to CRDs, however this approach only applies the CRDs during the process of applying all other manifests aswell -> If you know have a custom ressource (instance of this CRD) and also want to apply it -> it may fail as you cannot predict which manifest will be applied first

As we opted for the 2nd approach, we will not be able to also deploy a DynaKube CR in the same step.
More info: https://helm.sh/docs/chart_best_practices/custom_resource_definitions/
Thanks,
Lukas


Run snappy-header target to generate snappy-stubs-public.h

See #186.
I think these changes are needed on any platform.  The reason you may not have seen this problem on your platform is probably that your system has the snappy header files installed and the compiler refers not to target/snappy-1.1.7-*/snappy-stubs-public.h but to /usr/include/snappy-stubs-public.h.

Thanks for fixing. Actually I ran cmake inside target/snappy-1.1.7 folder (before fixing Makefile) and it seems some build was using that file...


Update composer.json

Composer dependencies message:
guzzlehttp/guzzle suggests installing psr/log (Required for using the Log middleware)
aws/aws-sdk-php suggests installing aws/aws-php-sns-message-validator (To validate incoming SNS notifications)
aws/aws-sdk-php suggests installing doctrine/cache (To use the DoctrineCacheAdapter)

I think that its simply a value/policy call on whether we want to load all possible suggestions.

What would be a valid reason to use in Vokuro the declared dependencies in your composer and omit the suggestions? I think that if I am not going to use a dependency I remove it (if this is not critical for the project), but just if I know and understand well the implications of installing or not installing dependencies, then I should install them.
In this case, I would remove the aws/aws-sdk-php dependency or the other dependencies from composer and in the documentation I would make a reference to support and add the suggestions, this makes sense that what I expect in console is what the documentation warns/suggests.

@alejosv I like the scenario with documentation

@sergeyklay the next week I'll send a pull request with a proposal.


Remove unnecessary null pointer checks

An extra null pointer check is not needed in functions like the following.

ParameterOptimizer
StorageEngine
Vector

OK， we will check and fix them


Add a description to the metadata.rb

As a user I would expect pcb to create a good metadata.rb so I can share my cookbook to supermarket without failures.
Mixlib::ShellOut::ShellCommandFailed
   ------------------------------------
   Expected process to exit with [0], but received '1'
   ---- Begin output of knife supermarket share delivery-truck --config /var/opt/delivery/workspace/.chef/knife.rb --supermarket-site https://ip-10-15-240-16.ec2.internal --cookbook-path /var/opt/delivery/workspace/<IP_ADDRESS>/corp/corp/delivery-truck/master/build/publish/cache/cookbook-share ----
   STDOUT:
   STDERR: Generating metadata for delivery-truck from /tmp/chef-delivery-truck-build20150930-28814-ar6kmd/delivery-truck/metadata.rb
   Making tarball delivery-truck.tgz
   ERROR: Description can't be blank
   ---- End output of knife supermarket share delivery-truck --config /var/opt/delivery/workspace/.chef/knife.rb --supermarket-site https://ip-10-15-240-16.ec2.internal --cookbook-path /var/opt/delivery/workspace/<IP_ADDRESS>/corp/corp/delivery-truck/master/build/publish/cache/cookbook-share ----
   Ran knife supermarket share delivery-truck --config /var/opt/delivery/workspace/.chef/knife.rb --supermarket-site https://ip-10-15-240-16.ec2.internal --cookbook-path /var/opt/delivery/workspace/<IP_ADDRESS>/corp/corp/delivery-truck/master/build/publish/cache/cookbook-share returned 1

DEPRECATED: Delivery CLI now simply uses chef generate cookbook -d <cookbook_name> to generate a delivery-ready cookbook. This is no longer used.


Update ringbuf to 0.2

I'm planning to use ringbuf in cubeb-coreaudio-rs, and I felt like updating this, and using 0.2 also there, to avoid duplicates when vendored in Gecko.
This also fixes a warning that probably appeared with an updated rustc version.

@kinetiknz can you have a look at this? Also I don't know why but I can't event ask for a review on this repo.


Package Create overwrites/appends to existing files

I mixed up my commands and ran functionary package create -l python kitchen_sink when I already had a kitchen_sink directory. The create command overwrote my functions.py and appended a new yaml to the end of package.yaml. The command should be consistent in what it does, and either prompt to overwrite or fail and force the command to run again with a -f flag to overwrite what's there.

I think have a -f to force the overwrite seems sensible, but I also kind of wonder what's the use case for when you'd use that?  And what's the expected behavior?  Does it wipe everything in the directory and then start you totally fresh?  Or does it leave any additional files you've added and just overwrite the ones that it would normally write (functions.py, package.yaml, etc.)?
I could also see taking the easy way out and just refusing to do anything if the directory isn't empty.  How would people feel about that?

I think it's also worth noting that it would be good to decide what the behavior should be if the files in your directory are say in python, but you run the create command again and declare the package is now in javascript. Current behavior is that a new functions.js file will be created and the package.yaml will be overwritten with the new language, but the functions.py file will remain. As it stands now, files that are not function.py/js and package.yaml are not overwritten.
Not doing anything to the directory if it already exists and isn't empty would definitely be the easiest solution. Otherwise I would be inclined to leave it as package.yaml and functions.py/js get overwritten if the user uses -f and everything else remains

In discussion, Scott and I have decided to go with his suggestion of just not doing anything when the dictionary isn't empty for right now


Add translations for CC-BY-NC-SA

After #10 is merged, we would need to add the translated versions of the CC-BY-NC-SA license to Spanish and German.

I can do this if I find some time for it.
Greetings

Awesome @cyb3rko !
Let's wait until #10 is merged and then you can start. Do you agree?

Do you need Italian translation?

Hi @Magicianred !
I've started this repo for personal usage, but it ended up getting a lot of attention.
@cyb3rko opened a PR for adding a German translation and I thought that would be awesome to have.
So, if you want to add an Italian translation, feel free to open a PR! I'll be glad to have that!
(Would be awesome to try to remember the little Italian I learned at school!)
I'll open a new Issue for adding an Italian translation, so we can discuss there instead of here.
Thanks for reaching out!

@santisoler
It is about another language,
can I add Japanese and Korean translations this repo?
I'm a native Korean speaker and I live in Japan.

Hi @vaporwavy! Sure, feel free to add them. As I mention before, this repo started just as a personal deposit for adding CC Licenses to my works, but as it caught attention, we started adding as many translations as needed by other contributors.
So feel free to add any translation you might want to use.


Where is the final model?

Hi there,
I'd like to reproduce your amazing work but I can only find the pretrained models and not the final fine-tuned model. Am I correct?
Could you please upload the final model?
Thank you.

Yes. Can you please share the final trained model?

Hi there,
I'd like to reproduce your amazing work but I can only find the pretrained models and not the final fine-tuned model. Am I correct?
Could you please upload the final model?
Thank you.

你好，请问可以分享预训练模型吗


Feature Request: Allow alternative Port options for Home assistant Login Details

**Is your feature request related to a problem? Please describe.yes
I have 2 cameras setup at another location and both need to be port forwarded to access but of course only one can have the port 554 as the unique Port , the other camera has to have a * Public * Facing port E.g 5544 which internally forwards to 554 at the other cameras ip address  and of course the login details on the HA setup will only look for port 554 by default so i would liking to see if we could add an option to change the port under an advanced menu option when popping in credentials because when i pop in the address with :5544 at the end the link shows cant connect yet it works and i have tested this working as the camera works normally in the rtsp stream on HA i just cant setup the functions to access the reboot commands and such via HA because of the port limitation with login details
Describe the solution you'd like
Potentially add / change for an option to change the Port upon login authentication entry screen
Describe alternatives you've considered
any way where we can set the facing port to another port while it still retains the 554 in the background

https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/375

Port forwarding the camera is also a terrible idea and you will get hacked eventually.


Remove dependency on stefandoorn/sitemap-plugin, move to suggest & describe in docs

Q
A

Bug fix?
no

New feature?
no

BC breaks?
yes (although.. not it depends on how you see it)

Deprecations?
no

Related tickets
#311, #287

License
MIT

Thanks for the PR, could you please check the behat tests? unfortunately, they do not pass.

Fixed @patrick477.
Adjusted it so the Sitemap Plugin gets installed in CI only for these specific tests.

It will be necessary to remove the dependencies to the Sitemap plugin, otherwise, everyone will get an error after installing the plugin.
PHP Fatal error: Uncaught Symfony\Component\Debug\Exception\ClassNotFoundException: Attempted to load class "SitemapPlugin" from namespace "SitemapPlugin".


Make --ingressendpoint configurable via env

Changes

Check an environment variable $INGRESS_ENDPOINT for the test option --ingressendpoint before the override check at test/ingress/ingress.go

/kind enhancement

Fixes https://github.com/knative/serving/issues/11294

Release Note

Docs

/ok-to-test
Just wanted to add some reasons why this change could be worth.

--ingressendpoint has to be added every single go test like https://github.com/knative/serving/issues/11294#issue-875327563 currently.
GATEWAY_OVERRIDE and GATEWAY_NAMESPACE_OVERRIDE are supported by env values but --ingressendpoint is command option, which is not consistent.


Dartmouth - making a user an owner creates a duplicate user

As the designated "owner" of our organization, I went into the users page (https://pod.stanford.edu/organizations/dartmouth/users) to designate another user. After designating Eric Bivona as an owner, I noticed that the list of users now shows Eric twice and me twice. For each of us, we have one line that indicates being an owner, and another line indicating not being an owner:

This doesn't look right to me - it seems like giving the "owner" status to a user should just update the existing line for them on the list of users.
Take care,
Joe Montibello
<EMAIL_ADDRESS>

Closed via #700


Error on CPU with enet_b2_8_best model

There is following error while running enet_b2_8_best model on CPU with the latest git hse package:
...
    fer = HSEmotionRecognizer(model_name = model_name)
  File "/home/build/.local/lib/python3.10/site-packages/hsemotion/facial_emotions.py", line 49, in __init__
    model=torch.load(path)
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 712, in load
    return _load(opened_zipfile, map_location, pickle_module, **pickle_load_args)
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 1049, in _load
    result = unpickler.load()
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 1019, in persistent_load
    load_tensor(dtype, nbytes, key, _maybe_decode_ascii(location))
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 1001, in load_tensor
    wrap_storage=restore_location(storage, location),
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 175, in default_restore_location
    result = fn(storage, location)
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 152, in _cuda_deserialize
    device = validate_cuda_device(location)
  File "/home/build/.local/lib/python3.10/site-packages/torch/serialization.py", line 136, in validate_cuda_device
    raise RuntimeError('Attempting to deserialize object on a CUDA '
RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU.

At that, model enet_b0_8_best_afew works fine with the same script on the same CPU.

Thanks for letting me know. I fixed the issue and updated the latest code in the main branch. Please, kindly verify that it works now

Fix works, thank you!