Got permission related issue in Get sources task

Having issue with YAML?
Please log an issue at Azure-Pipelines-YAML. Over there we discuss YAML templates, samples for Azure Pipelines, and designs for upcoming YAML features. Also a place for the community to share best practices, ideas, and so on. File suggestions and issues here if they're specific to YAML pipelines.
Having issue with Tasks?
Pipeline run link: https://dev.azure.com/aiinfra/PyTorch/_build/results?buildId=199641&view=logs&j=e762304e-b169-5963-0b3c-83ff68dfbe29&t=1ad14994-d397-5d20-7571-3051451bd627
2022-03-23T13:38:07.9902847Z ##[section]Starting: Checkout ptebic@xiaoyu/shiproom to s
2022-03-23T13:38:08.0082056Z ==============================================================================
2022-03-23T13:38:08.0082624Z Task         : Get sources
2022-03-23T13:38:08.0083086Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.
2022-03-23T13:38:08.0083362Z Version      : 1.0.0
2022-03-23T13:38:08.0083715Z Author       : Microsoft
2022-03-23T13:38:08.0084173Z Help         : [More Information](https://go.microsoft.com/fwlink/?LinkId=798199)
2022-03-23T13:38:08.0084523Z ==============================================================================
2022-03-23T13:38:08.3179214Z Syncing repository: ptebic (Git)
2022-03-23T13:38:08.3397362Z ##[command]git version
2022-03-23T13:38:08.3825874Z git version 2.25.1
2022-03-23T13:38:08.3895840Z ##[command]git config --get remote.origin.url
2022-03-23T13:38:08.4021024Z ##[command]git clean -ffdx
2022-03-23T13:38:08.4146389Z warning: failed to remove test/downloads/test_ort/super_resolution.onnx: Permission denied
2022-03-23T13:38:08.4147793Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/objects/info: Permission denied
2022-03-23T13:38:08.4149987Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/objects/pack/pack-729563fda50ac285d17e4868f25bff4768faf4e9.idx: Permission denied
2022-03-23T13:38:08.4152106Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/objects/pack/pack-729563fda50ac285d17e4868f25bff4768faf4e9.pack: Permission denied
2022-03-23T13:38:08.4153504Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/branches: Permission denied
2022-03-23T13:38:08.4154599Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/HEAD: Permission denied
2022-03-23T13:38:08.4155745Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/info/exclude: Permission denied
2022-03-23T13:38:08.4157509Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/pre-applypatch.sample: Permission denied
2022-03-23T13:38:08.4159224Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/fsmonitor-watchman.sample: Permission denied
2022-03-23T13:38:08.4160809Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/pre-rebase.sample: Permission denied
2022-03-23T13:38:08.4162530Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/applypatch-msg.sample: Permission denied
2022-03-23T13:38:08.4164071Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/pre-push.sample: Permission denied
2022-03-23T13:38:08.4165594Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/pre-receive.sample: Permission denied
2022-03-23T13:38:08.4167206Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/pre-commit.sample: Permission denied
2022-03-23T13:38:08.4168770Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/post-update.sample: Permission denied
2022-03-23T13:38:08.4170612Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/prepare-commit-msg.sample: Permission denied
2022-03-23T13:38:08.4172579Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/commit-msg.sample: Permission denied
2022-03-23T13:38:08.4174208Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/pre-merge-commit.sample: Permission denied
2022-03-23T13:38:08.4175411Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/hooks/update.sample: Permission denied
2022-03-23T13:38:08.4176512Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/index: Permission denied
2022-03-23T13:38:08.4177681Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/refs/tags: Permission denied
2022-03-23T13:38:08.4179390Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/refs/heads/pte/bic-testing: Permission denied
2022-03-23T13:38:08.4180653Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/refs/remotes/origin/HEAD: Permission denied
2022-03-23T13:38:08.4181798Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/logs/HEAD: Permission denied
2022-03-23T13:38:08.4183311Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/logs/refs/heads/pte/bic-testing: Permission denied
2022-03-23T13:38:08.4184586Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/logs/refs/remotes/origin/HEAD: Permission denied
2022-03-23T13:38:08.4185713Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/config: Permission denied
2022-03-23T13:38:08.4187200Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/packed-refs: Permission denied
2022-03-23T13:38:08.4188309Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/.git/description: Permission denied
2022-03-23T13:38:08.4190014Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/blog/hola.md: Permission denied
2022-03-23T13:38:08.4192025Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/blog/2019-05-29-hello-world.md: Permission denied
2022-03-23T13:38:08.4193648Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/blog/2019-05-30-welcome.md: Permission denied
2022-03-23T13:38:08.4195014Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/plugins/appinsights/index.js: Permission denied
2022-03-23T13:38:08.4196209Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/plugins/readme.md: Permission denied
2022-03-23T13:38:08.4197380Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/docusaurus.config.js: Permission denied
2022-03-23T13:38:08.4198641Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/.gitignore: Permission denied
2022-03-23T13:38:08.4200237Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/docs/babel-library/turing-ulr-v5.md: Permission denied
2022-03-23T13:38:08.4202003Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/docs/babel-library/turing-nlr-v5.md: Permission denied
2022-03-23T13:38:08.4203653Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/docs/babel-library/getting-started.md: Permission denied
2022-03-23T13:38:08.4205298Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/docs/babel-library/turing-nlr-v3.md: Permission denied
2022-03-23T13:38:08.4206540Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/webpage/docs/babel-library/babel-assistant.md: Permission denied
(skip same logs here...)
2022-03-23T13:38:08.4599525Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/src/turing/tnlgv1/finetune/TNLG/evaluate.py: Permission denied
2022-03-23T13:38:08.4600056Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/src/.azureml/config.json: Permission denied
2022-03-23T13:38:08.4600573Z warning: failed to remove test/downloads/test_babel_tnlr_training/babel/src/test.ipynb: Permission denied
2022-03-23T13:38:08.4601244Z warning: failed to remove test/downloads/test_hf_training/hf-pytorch-training.log: Permission denied
2022-03-23T13:38:08.4601919Z warning: failed to remove test/downloads/test_hf_training/hf-ds-training.log: Permission denied
2022-03-23T13:38:08.4602602Z warning: failed to remove test/downloads/test_hf_training/hf-ds-ort-training.log: Permission denied
2022-03-23T13:38:08.4603210Z warning: failed to remove test/downloads/test_hf_training/hf-ort-training.log: Permission denied
2022-03-23T13:38:08.4603890Z warning: failed to remove test/test_fairscale/golden_configs/__pycache__/oss_mnist.cpython-38.pyc: Permission denied
2022-03-23T13:38:08.4604559Z warning: failed to remove test/test_fairscale/golden_configs/__pycache__/__init__.cpython-38.pyc: Permission denied
2022-03-23T13:38:08.4627102Z ##[warning]Unable to run "git clean -ffdx" and "git reset --hard HEAD" successfully, delete source folder instead.
2022-03-23T13:38:08.4679480Z ##[error]One or more errors occurred. (One or more errors occurred. (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/src/turing/tnlgv1/finetune/TNLG/turingnlg/detokenizer.py' is denied.)) (One or more errors occurred. (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/src/turing/tnlgv1/finetune/TNLG/turingnlg/gpt2_data_loader.py' is denied.)) (One or more errors occurred. (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/src/turing/tnlgv1/finetune/TNLG/dataset/deepvox_test.json' is denied.)) (One or more errors occurred. (Access to the path
'/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/src/turing/tnlgv1/finetune/TNLG/turingnlg/utils.py' is denied.) (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/src/test.ipynb' is denied.) (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_hf_training/hf-ds-training.log' is denied.) (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/.gitignore' is denied.) (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_hf_training/hf-ort-training.log' is denied.) (Access to the path '/home/ciagent/gpu2/_work/5/s/test/downloads/test_hf_training/hf-ds-ort-training.log' is denied.)
2022-03-23T13:38:08.4733258Z ##[section]Finishing: Checkout ptebic@xiaoyu/shiproom to s

Having issue with software on Hosted Agent?
Log an issue at Hosted Agent Image Repository. It contains the VM image used in the Azure Pipelines Hosted Agent Pool. If you're having Build/Release failures that seems like they are related to software installed on the Hosted Agent (e.g. the dotnet SDK is missing or the Azure SDK is not on the latest version) please log an issue there.
Having generic issue with Azure-Pipelines/VSTS/TFS?
Please report it on Developer Community
Have you tried troubleshooting?
Yes
Agent Version and Platform
2.192.0
OS of the machine running the agent? OSX/Windows/Linux/...
Linux
Azure DevOps Type and Version
dev.azure.com (formerly visualstudio.com) or on-premises TFS/Azure DevOps Server?
If on-premises, which release? 2015.0, 2017.1, 2019 RC2, etc.
If dev.azure.com, what is your organization name?
https://dev.azure.com/aiinfra
What's not working?
This task should checkout to my branch and run the code in my branch. I just got this error at some point since yesterday. No config change from my end. Looks like the pipeline user is missing some permissions here. Only happens to our rocm machine. Other machine works fine.
The succeed log should looks like:
2022-03-22T16:23:58.7054716Z ##[section]Starting: Checkout ptebic@xiaoyu/shiproom to s
2022-03-22T16:23:58.7243400Z ==============================================================================
2022-03-22T16:23:58.7243916Z Task         : Get sources
2022-03-22T16:23:58.7244444Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.
2022-03-22T16:23:58.7244754Z Version      : 1.0.0
2022-03-22T16:23:58.7245054Z Author       : Microsoft
2022-03-22T16:23:58.7245517Z Help         : [More Information](https://go.microsoft.com/fwlink/?LinkId=798199)
2022-03-22T16:23:58.7245869Z ==============================================================================
2022-03-22T16:23:59.1330565Z Syncing repository: ptebic (Git)
2022-03-22T16:23:59.1546627Z ##[command]git version
2022-03-22T16:23:59.1935715Z git version 2.25.1
2022-03-22T16:23:59.2003451Z ##[command]git config --get remote.origin.url
2022-03-22T16:23:59.2140973Z ##[command]git clean -ffdx
2022-03-22T16:23:59.2225193Z Removing internal_packages/download_internal_packages/
2022-03-22T16:23:59.2226217Z Removing test/downloads/
2022-03-22T16:23:59.2226782Z Removing test/logs/
2022-03-22T16:23:59.2247685Z ##[command]git reset --hard HEAD
2022-03-22T16:23:59.2509432Z HEAD is now at 2625acd Comment torchvision installation
2022-03-22T16:23:59.2533010Z ##[command]git config gc.auto 0
2022-03-22T16:23:59.2590389Z ##[command]git config --get-all<EMAIL_ADDRESS>2022-03-22T16:23:59.2654379Z ##[command]git config --get-all http.proxy
2022-03-22T16:23:59.2702061Z ##[command]git config http.version HTTP/1.1
2022-03-22T16:23:59.2845544Z ##[command]git -c http.extraheader="AUTHORIZATION: bearer ***" fetch --force --tags --prune --progress --no-recurse-submodules origin
2022-03-22T16:23:59.6855506Z ##[command]git -c http.extraheader="AUTHORIZATION: bearer ***" fetch --force --tags --prune --progress --no-recurse-submodules origin +2625acdfffd030292b01aeb1904587eb7e303006
2022-03-22T16:23:59.8007933Z From https://dev.azure.com/aiinfra/PyTorch/_git/ptebic
2022-03-22T16:23:59.8009827Z  * branch            2625acdfffd030292b01aeb1904587eb7e303006 -> FETCH_HEAD
2022-03-22T16:23:59.8511992Z ##[command]git checkout --progress --force 2625acdfffd030292b01aeb1904587eb7e303006
2022-03-22T16:23:59.8516846Z HEAD is now at 2625acd Comment torchvision installation
2022-03-22T16:23:59.8556958Z ##[section]Finishing: Checkout ptebic@xiaoyu/shiproom to s

Do you have any idea why the permission is suddenly missing? How can I fix this?
Agent and Worker's Diagnostic Logs
Logs are located in the agent's _diag folder. The agent logs are prefixed with Agent_ and the worker logs are prefixed with Worker_. All sensitive information should already be masked out, but please double-check before pasting here.

Hi @xiaoyu-work, What version of the agent was ued in the successful pipeline and in the failed one?

Hi @max-zaytsev, they are both 2.192.0

I've reached out to the host owner. He helped me to fix it manually. I'll keep updating this issue if this error happens again in next 2 days. Thanks.

@max-zaytsev I got this permission error again.
The host owner said someone probably ran something as root. You can see a subset of files are all owned by root and hence ciagent (pipeline user) cannot remove those.
(base) REDMOND.gokuma@GCR-OPENPAI-35:/home/ciagent/gpu2/_work/5/s/test/downloads/test_babel_tnlr_training/babel/src/babel/azure/ml/babel/dependencies/transformers210$ ls -l
total 2784
-rw-r--r-- 1 root root 1537 Mar 22 10:11 activations.py
-rw-r--r-- 1 root root 471 Mar 22 10:11 another_try.py
-rw-r--r-- 1 root root 15359 Mar 22 10:11 benchmark_utils.py
drwxr-xr-x 2 root root 4096 Mar 22 10:11 commands
-rw-r--r-- 1 root root 7791 Mar 22 10:11 configuration_albert.py
-rw-r--r-- 1 root root 11841 Mar 22 10:11 configuration_auto.py
-rw-r--r-- 1 root root 5246 Mar 22 10:11 configuration_bart.py
-rw-r--r-- 1 root root 8702 Mar 22 10:11 configuration_bert.py
-rw-r--r-- 1 root root 1604 Mar 22 10:11 configuration_camembert.py
-rw-r--r-- 1 root root 5812 Mar 22 10:11 configuration_ctrl.py
-rw-r--r-- 1 root root 7065 Mar 22 10:11 configuration_distilbert.py
-rw-r--r-- 1 root root 6865 Mar 22 10:11 configuration_electra.py
-rw-r--r-- 1 root root 4363 Mar 22 10:11 configuration_encoder_decoder.py
-rw-r--r-- 1 root root 9955 Mar 22 10:11 configuration_flaubert.py
-rw-r--r-- 1 root root 8520 Mar 22 10:11 configuration_gpt2.py
-rw-r--r-- 1 root root 3075 Mar 22 10:11 configuration_longformer.py
-rw-r--r-- 1 root root 998 Mar 22 10:11 configuration_marian.py
-rw-r--r-- 1 root root 1533 Mar 22 10:11 configuration_mmbt.py
-rw-r--r-- 1 root root 8352 Mar 22 10:11 configuration_openai.py
-rw-r--r-- 1 root root 13612 Mar 22 10:11 configuration_reformer.py
Do you know if the pipeline user (ciagent) will somehow run our script in pipeline as root so the permissions of the files were changed?
This issue happens pretty randomly. It works a couple of times today, but suddenly it stopped working.

Hi @xiaoyu-work yes, seems like there is some lack of permissions for user which runs agent - to remove these files.
If ciagent is set up as agent user - it should not do anything as root... Do you know maybe if there are some other programs running as root - which could change permissions for these files?
Is it possible to share some minimal sample to reproduce this issue on our side - if you suspect that it could be related to some agent issue?

@xiaoyu-work btw do you use containers in your pipeline?

@anatolybolshakov

Yes seems those files were run by root user, which changes the file permission. But I've checked the host login history, there is no one log in to the host as root to run it. So it might come from agent user?
I don't think this can be reproduced. Even for our pipeline, we saw this error occasionally. Now it works, but we got this error twice before.
Yes we use containers in our pipeline.

If it is possible that somehow pipeline user ciagent can have root permission to run it?

@anatolybolshakov, I got this error again. Do you know how can I identify if the files were run by root user? Like, any log particular indicates this?

@xiaoyu-work it would probably make sense to check whole pipelines logs for modifications of this file - to understand if there is some change made by agent and not something else in system.

@xiaoyu-work is this issue still actual for you?

@xiaoyu-work I'm closing this for now. Please reopen if you still have any questions.


added save_without_historical_record to documents

Codecov Report

Merging #286 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #286   +/-   ##
=======================================
  Coverage   97.62%   97.62%
=======================================
  Files          15       15
  Lines         590      590
  Branches       72       72
=======================================
  Hits          576      576
  Misses          7        7
  Partials        7        7

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b62857...081c04a. Read the comment docs.

@lee-seul thanks for catching this missing documentation! 🙌


Uncaught TypeError: Cannot read property 'length' of undefined

[Enter steps to reproduce below:]

Set time to 30m
Set youtube url to https://www.youtube.com/watch?v=Zu5AU_wluLo

Atom Version: 1.0.0
System: Mac OS X 10.10.3
Thrown From: editor-background package, v1.0.1
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At /Users/maxlang/.atom/packages/editor-background/lib/youtube.coffee:167

TypeError: Cannot read property 'length' of undefined
  at Request._callback (/Users/maxlang/.atom/packages/editor-background/lib/youtube.coffee:167:35)
  at self.callback (/Users/maxlang/.atom/packages/editor-background/node_modules/request/request.js:368:22)
  at emitOne (events.js:77:13)
  at Request.emit (events.js:166:7)
  at Request.onRequestError (/Users/maxlang/.atom/packages/editor-background/node_modules/request/request.js:1025:8)
  at emitOne (events.js:77:13)
  at ClientRequest.emit (events.js:166:7)
  at TLSSocket.socketOnEnd (_http_client.js:283:9)
  at emitNone (events.js:72:20)
  at TLSSocket.emit (events.js:163:7)
  at _stream_readable.js:891:16
  at process._tickDomainCallback (node.js:384:13)

Commands
     -9:35 core:paste (atom-text-editor#editor-background.video.youTubeURL.editor.mini.is-focused)

Config
{
  "core": {},
  "editor-background": {
    "mouseFactor": 8,
    "box3d": {
      "mouseFactor": 8
    },
    "image": {
      "overlayColor": {
        "red": 0,
        "green": 0,
        "blue": 0,
        "alpha": 1
      },
      "opacity": 10
    },
    "other": {
      "treeViewOpacity": 85
    },
    "text": {},
    "video": {
      "endTime": "30m",
      "youTubeURL": "https://www.youtube.com/watch?v=niHGkBk_xuI"
    },
    "blurRadius": 0,
    "imageURL": "",
    "youTubeURL": "https://www.youtube.com/watch?v=Zu5AU_wluLo",
    "animationSpeed": 50,
    "opacity": 0
  }
}

Installed Packages
# User
atom-beautify, v0.28.7
atom-color-highlight, v3.0.9
atom-jshinter, v0.1.3
atom-typescript, v4.8.4
autocomplete-emojis, v2.2.2
editor-background, v1.0.1
esformatter, v1.7.1
file-icons, v1.5.8
javascript-snippets, v1.0.0
jsformat, v0.9.0
jshint, v1.3.8
language-javascript-semantic, v0.2.1
linter, v1.1.0
minimap, v4.10.1
minimap-color-highlight, v4.1.4
node-debugger, v1.1.0
tree-view-background, v0.3.1

# Dev
No dev packages

30min ?
no no no, 30sec will be better
choose some fragment from video, it will be soft wrapped (mixed, seamless)
plugin convert video to images because of performance
array of images is faster than decoding each frame
so if you choose 30 min video there are a lot of frames(images)....
i see that you choose video with music, sound is not processed by plugin
you can use your default media player to play your music
i think that is better that music is played outside editor and not sticked
but, of course error is error and i will check this bug when i will have time
thanks for report!

tree-view-background is in conflict with background-editor


A filter to add or subtract a value from a given date using the methods Add and Subtract

Pull request for issue #170.

Looks good, thanks!
Can you please:

Rename them to amAdd, amSubtract to keep it consistent with the other filters
Add tests for them

Thanks!

Thank you @urish ;)
All done.

Thanks!

p.s. I have improved the tests, they now explicitly state the expected result as you originally intended to do:
	it('should subtract 1 hour from date', function () {
		var date = new Date(2000, 1, 1, 0, 0, 0);
		expect(amSubtract(date, 1, 'hours').toString()).toMatch(/^Mon Jan 31 2000 23:00:00/);
	});

Released as 0.10.3


[Google Maps] Support marker clustering

Edited by amirh:
This is a request to  support marker clustering similarly to https://developers.google.com/maps/documentation/android-sdk/utility/marker-clustering

@m7amdElberawy
The issue at https://github.com/flutter/flutter/issues/26863 has been closed and moved here. Future collaboration on this issue will be done here.

any update?

@pro100svitlo no update yet; sorry for the delay.
Since this issue is not related to Firebase; I'll close this and reopen the original issue in the flutter/flutter repo. Please continue to track it there.


Bootstrapper fails if $env:path has empty strings

Due to faulty installers, many PATH environment variables end up with consecutive semicolons, which turns into a blank line with $env:path -split ';'. While Test-Path returns False when given non-empty strings, it throws an error when it encounters the empty string. This causes the build.ps1 script to fail on machines where $env:path has consecutive semicolons.
It happens in the section that checks for nuget.exe on the path.

Just to point out that this was fixed by cake-build/resources#5.

@luisgoncalves good catch!


feat(analytical): Enable cython app on local vertex map graph

Committed-by: siyuanzhang.zsy from Dev container
Fixes #2309

Codecov Report

Merging #3081 (9269e37) into main (f3a2eb0) will decrease coverage by 6.86%.
Report is 3 commits behind head on main.
The diff coverage is 0.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3081      +/-   ##
==========================================
- Coverage   42.65%   35.79%   -6.86%
==========================================
  Files         100      100
  Lines       10812    10825      +13
==========================================
- Hits         4612     3875     -737
- Misses       6200     6950     +750

Files Changed
Coverage Δ

python/graphscope/analytical/udf/compile.py
25.05% <0.00%> (ø)

python/graphscope/tests/unittest/test_udf_app.py
0.00% <0.00%> (ø)

... and 29 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26c2813...9269e37. Read the comment docs.


Update catalan messages.po

New lines translated
You can check it @jordimas @darccio @ferranrego @MiquelAdell @mvdan @johnnydement @Mejans @surfdude29

Just to check, did you see the comment from @jordimas on your own repo?
https://github.com/ivanbea/social-app/commit/860c2c3fdd72cadb19c9489a28adb34639189b0b#r141348104


Allow enabling extenstions using migrations

It would be nice to have Ecto.Migration.extension method which would allow us to run:
defmodule MyApp.Migrations.EnableExtensions do
  use Ecto.Migration

  def change do
    create_if_not_exist extension("uuid-ossp")
  end
end

Or similar code. Similar to #1805

Extensions are very postgresql specific. I would personally rather see them in a separate package, something like PostgresEcto that could also include other postgresql specific extensions - macros wrapping fragments for some common functions, etc.

I can try to produce something like that.

Unfortunately it is not possible to implement something like the above in a third party project. In any case, I don't mind having Postgres specific behaviour in Ecto most times, specially this one which is a very minor feature.

@josevalim I can accept that challenge.

Maybe we should look into ways of making it possible for extensions to provide such behaviour?

@michalmuskala I would love to and tried a couple times in the past. The best solution I had in mind is to define one protocol per adapter and I am not confident it is the best way to go.


Fix extra electrode groups ephys

Should fix #1163.
Also related to https://github.com/catalystneuro/neuroconv/issues/932 as that should be more or less the last place where we did not priority set.

Thanks for the review @pauladkisson


parsestr

		// TODO: make this plural when `parse` can handle multiple values.
		secret.ValueTransformed = parsedStr
		currentState.Increment(secret.Name)
		secrets.Store(secret.Name, secret)

You don’t have to make it plural. Just make sure ValueTransformed is a valid concataneted json string of all transformed values when len(secret.values) > 1

This could already have been fixed.
I need to double check.

Does not apply. Depending on the size of the array the value can be plural, singular, or in case of a template appliacation it could not be either.
Keeping the name ValueTransformed makese sense.
closing this as WONTFIX.


Start fleshing out tests.

This is mainly to test Heroku CI.

Additional configuration was missing in base branch (crystal-heroku).


Memory leak keeps growing when mounting / unmounting elements

Hi, I was running this repo and playing around with the widgets demo, and I noticed that when I was mounting / unmounting elements ( opening and closing elements like the calendar / colour picker), the amount of memory leak kept growing and growing ? this seems like a big issue ? is this a known issue and can it maybe get fixed by altering the reconciler file ?
Steps to reproduce.

run the widget demo.

Close the demo immediately

check the memory leak log

run the widget demo again

repeatedly open and close the colour picker

Close the demo.

See that the memory leak / in blocks has increased

I spent quite some time trying to trace down some of these leaks but I haven't been able to find them. I can take a look at the calendar implementation, but if you'd like to help identifying all the leaks, building a demo that helps to test and isolate leaks would be welcome


Allow @EqualsAndHashCode methods to be final

I'm using EqualsVerifier to cover autogenerated methods. For one class where I'm using Lombok's @EqualsAndHashCode, EqualsVerifier fails the verification with this error:
Subclass: equals is not final.
Make your class or your equals method final, or supply an instance of a redefined subclass using withRedefinedSubclass if equals cannot be final.

But I do not see a way to ask Lombok to mark its generated equals method final.
Can we add a new annotation parameter for this feature? I'd be happy to implement it, but I'd like an expert's blessing that this is the right way forward.

Disclaimer: I'm just a Lombok fan, not a team member.

I'm using EqualsVerifier to cover autogenerated methods.

IMHO you shouldn't. Correctness of the generated code is the responsibility of the generator rather than of its user. Sure, someone manually overriding the generated equals may blow it and this should be covered by a test.

Make your class or your equals method final

This could be useful and rather simple, but I'm unsure if it's useful enough. The generated method is based on the article linked from your link, so it uses canEqual and AFAIK equals itself needn't be ever overridden.

Can we add a new annotation parameter for this feature?

I guess, there should be a configuration key as well. AFAIK all my generated equals and hashCode methods can be final.
Anyway, awaiting a team member's blessing is the way to go.

It helps to do this when you are replacing hand-coded or old generated
methods by lombok annotations to have tests in place. It's not useless.
EqualsVerifier will complain about any class that is not @Value annotated
really, not just about the equals method. Just making that method final
won't help.
Honestly, annotating classes with @Value to make the whole thing final is
a good idea any time you can get away with it. Hibernate model classes
can't be done like that unfortunately, but for dto's it should be the norm.
In cases where you can't make everything final, use
EqualsVerifier..forClass(classToVerify).suppress(Warning.STRICT_INHERITANCE).verify().
That is slightly more typing but it works.
On Wed, Sep 26, 2018 at 12:00 AM Maaartinus<EMAIL_ADDRESS>wrote:

Disclaimer: I'm just a Lombok fan, not a team member.
I'm using EqualsVerifier to cover autogenerated methods.
IMHO you shouldn't. Correctness of the generated code is the
responsibility of the generator rather than of its user. Sure, someone
manually overriding the generated equals may blow it and this should be
covered by a test.
Make your class or your equals method final
This could be useful and rather simple, but I'm unsure if it's useful
enough. The generated method is based on the article linked from your link
https://www.artima.com/lejava/articles/equality.html, so it uses
canEqual and AFAIK equals itself needn't be ever overridden.
Can we add a new annotation parameter for this feature?
I guess, there should be a configuration key as well. AFAIK all my
generated equals and hashCode methods can be final.
Anyway, awaiting a team member's blessing is the way to go.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rzwitserloot/lombok/issues/1872#issuecomment-424516350,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKCRSXYBTP9xcVHzDsWNhc5VRJQyGmHks5ueqd9gaJpZM4W5Z-l
.

--
"Don't only practice your art, but force your way into it's secrets, for it
and knowledge can raise men to the divine."
-- Ludwig von Beethoven

Why we closed this issue


PyTorch 0.4.0 Compatibility Issue

The current codebase has issues with PyTorch 0.4 release. Any plan for updating the repo regarding the new API of PyTorch? @jwyang
Most issues can be solved by simply converting cuda tensor to numpy. However, I have some trouble correcting this line: https://github.com/jwyang/faster-rcnn.pytorch/blob/8e029c56f7227a30a5b8efa90a8b235f9ee14479/lib/model/rpn/proposal_target_layer_cascade.py#L133

have you solved this problem???
i also met this after i change the 1/num_example to 1/num_example.item() or .float()

okay, i have try to solve this problem.

and this is the reason~
x = torch.randn(5, 4)
x.index(1)
pytorch would print the same problem. the solution is we have to wrap the tensor format to a tuple format by add '[]' to the offset
another problem is is_device, u just comment the line, that would be okay...
another thing I do not understand is pytorch provide a better API to load data, why we use the blob to load the data. which makes things looks so complicated!
another solution is use index_select(0, offset.view(-1).long())
however, I am not sure if the results is right.
finally, pytorch version changes a lot which makes me feel frustrated to adapt the newest version!!!

Not yet. Consider PyTorch 1.0 is coming, I would stick with 0.3 for now.

I also encounter this problem about this code in pytorch 0.4.0,  try to change the code  index(offset.view(-1)) to the index((offset.view(-1),)),  and get right.
I think the index function is update in pytorch 0.4.0 but din't update in document.

now the master supports pytorch-0.4.0 and tensorboard. feel free to use them.

Thanks, much appreciated.


Move Test framework to a separate Library

Context
For splitting test targets and regular targets, 'MoonlightTests' should be moved to a separate library.
Checklist

[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation including README.md
[x] My changes generate no new warnings
[x] My changes generate no breaking changes

I would have still suggested you to separate other targets into their own libraries as well.
For example - using Moonlight in SwiftUI doesn't require us to use any UIKit stuff, hence including them when importing Moonkit doesn't make much sense to me. We should be able to import only what we need.

okay, in the case of SwiftUI this is a valid point, done.


6.7.0 is causing puppeteer + Chromium download

What should happen?
npm i foundation-sites should not install puppeteer as well
What happens instead?
puppeteer is required, triggering a Chromium download
Possible Solution
Move the puppeteer requirement to devDependencies.
Test Case and/or Steps to Reproduce (for bugs)
How to reproduce:

npm i foundation-sites

Context
Puppeteer triggers a 100MB+ Chromium download. On slow connections like mine, it takes a while.
Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] There are no other issues similar to this one.
[x] The issue title and template are correctly filled.

My entire team has just got our Angular project development come to a screeching halt over this.

Yes, this is only useful for development environment and shouldn't be required when installing foundation-sites.

Sorry. It got added as a dependency instead of a dev dependency. I completely overlooked that. I will get that fixed in 6.7.1. Thanks.


Remove unused NominalDecl args from ConformanceLookupTable

Based on recent swift-evolution discussion, I started looking into what would be necessary to make structural types conform to protocols by removing reliance on NominalDecl.
This is a very simple first cleanup PR that simply removes some instances of NominalDecl where they were completely unused in the existing code.

@swift-ci Please smoke test


CLI: Windows Compatibility

Known Issues Addressed:
#362
#202
#180 (a tiny bit)
Updates:

Added compatibility for Windows to "get_terminal_size" as to prevent cli.py from crashing when downloading a stream.
Added to the test_cli.py script for the newly added functionality (will most likely need revising).

For Linux and OSX users, the os.popen call for "stty size" has been replaced with a subprocess.check_output call. This tweak prevents stderr from displaying an invalid bash command error on Windows every time the progress bar is updated; because this puts the "display_progress_bar" function into a different process, this edit also has a side effect of reducing download time slightly (bonus!).
If either attempt at getting the terminal geometry fails, the default values are returned as follows in an attempt to prevent an error: rows: 80, columns: 24.

Coverage decreased (-1.1%) to 76.426% when pulling e29cc8c0366185b449af93d113d4d88a9fcb1fd4 on Medstar117:windows-compatibility into 7e0cd4e8639ab5754227eafa8a98613939c8b61a on nficano:master.

Issue seems to be fixed already -- please re-open if this isn't the case :)


Support for other KMS systems

AWS, Azure, more!

Help wanted: anyone who has credentials or billing accounts for these!

From https://github.com/smallstep/certificates/tree/master/kms

I might be interested in taking on either AWS KMS or Vault (have access to both)

vault would be awesome @RichiCoder1 !

Yes! Let me know if I can help. The code has actually been refactored over to here now: https://github.com/sigstore/sigstore/tree/main/pkg/kms

Took a swing at vault, luckily not too bad: sigtore/sigstore#44

Vault is done now. @RichiCoder1 are you still thinking of adding AWS support?

I am! Should be reasonably simple, just might take me a bit longer to get to it.

will implement for Azure -> https://github.com/sigstore/cosign/issues/399

I am almost finished implementing AWS KMS support -- should have a PR in sigstore by EOD.

Hey guys, just checking. Is AWS KMS support working at this stage (I am testing with 1.2.1). The documentation (KMS.md) seems to indicate it should, but I am not having much success so far. Just checking before opening some issues.

@joelapatatechaude AWS KMS support should be working as of v0.6.0. Please file a bug and tag me if it is broken.

Created #1194

Closing since we have a bunch now and separate bugs for individual ones remaining!


fix: MD046/code-block-style

Code block style set to fenced caught some accidental blocks from indenting.
It also found a broken table

We're in the process of pulling this commit to the private repository so we can verify the updates on staging. We'll post a new private repo PR # shortly and then close this PR.

Private repo PR 4961 replaces this pull request. We moved the commit to the private repo so we can validate formatting changes by using automated validation and staging. We'll close this PR now.
@nschonni - Thank you!


Support for Mac OS Monterey

On Mac OS Monterey the installations says
/usr/local/Homebrew/Library/Homebrew/os/mac/version.rb:33:in `block in from_symbol': unknown or unsupported macOS version: :dunno (MacOSVersionError)
can someone adapt it to run with Monterey

Is that on Intel or ARM?
I tried install with ARM and worked fine:
❯ brew install corelocationcli
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> Updated Formulae
Updated 6 formulae.

Warning: Cask 'corelocationcli' is already installed.

To re-install corelocationcli, run:
  brew reinstall corelocationcli

~
❯ brew reinstall corelocationcli
==> Downloading https://github.com/fulldecent/corelocationcli/releases/download/3.2.0/Core
==> Downloading from https://objects.githubusercontent.com/github-production-release-asset
######################################################################## 100.0%
==> Uninstalling Cask corelocationcli
==> Unlinking Binary '/opt/homebrew/bin/CoreLocationCLI'
==> Purging files for version 3.2.0 of Cask corelocationcli
==> Installing Cask corelocationcli
==> Linking Binary 'CoreLocationCLI' to '/opt/homebrew/bin/CoreLocationCLI'
🍺  corelocationcli was successfully installed

It is an ARM
I will try again.

I'm having the same issue with an Intel Mac running Monterey.

It was my mistake I had to reinstall the arm version of brew.

This should be fixed now
released to Brew at https://github.com/Homebrew/homebrew-cask/pull/122637#pullrequestreview-955865737


Missing License

Could you provide a license?
I assume that - as you uploaded these to GitHub - you intend this to be free software, so I would appreciate it if you would provide a license.
I'd like to adapt some parts of your preamble etc.

yes, this would work, but for software, the CC licenses are usually not recommended, read e.g.
https://creativecommons.org/faq/#can-i-apply-a-creative-commons-license-to-software
which recommends licenses from the following:
https://opensource.org/licenses
https://opensource.org/licenses
If you want to go for the most permissive (as is CC-BY-4.0 from the CC licenses), the MIT License should be fine (also because it is extremely short and easy to understand), for prohibiting commercial you can use the GPLv3.
Another option - especially regarding that this is LaTeX code - is to release this under the Latex Project Public License, which additionally obliges derived works to be named different than the original work, but probably this is not necessary as these aren't packages you intend do distribute on CTAN or something like that.
I personally use MIT, or (for package-like stuff) dual-license with LPPL and GPLv3, but I'm of course also not an expert and this might be personal flavor.
Thanks for the effort you are taking.

It seems to me that LaTeX projects and LaTeX code is somewhat curious when it comes to licenses. I’ve tried to find other projects that also should have this problem to see what they are doing:

I think this project could be classified as somewhat of an open textbook. These seem to be mostly licensed under GPL (for example the Stacks Project), Creative Commons (for example the HoTT book.) or something along the lines of “do what you want” like MIT, CC0 or Public Domain.

The open access policies of publishers like Springer, Elsevier or CUP seem to recommend their authors to use CC BY or stronger. But I don’t know if they have the publishing of LaTeX code in mind.

Who should have the publishing of LaTeX code in mind is the arXiv: they seem to mostly use Creative Commons.

Tom Leinster’s book “Basic Category Theory” was published in 2014 by CUP, and then became freely available in 2016 on arXiv. The preface of this free version reads as follows:

This book was first published by Cambridge University Press in 2014, and is
now being published on the arXiv by mutual agreement. […] This electronic version is not only free; it is also freely editable. […] In legal terms, this text is released under the Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International licence (CC BY-NC-SA 4.0). […] Broadly speaking, the licence allows you to edit and redistribute this text in any way you like, as long as you include an accurate statement about authorship and copyright, do not use it for commercial purposes, and distribute it under this same licence. In technical terms, all you need to do in order to edit this book is to download the source files from the arXiv and use LaTeX […] in the usual way. […]

It seems to me that at least in this case, the publisher (CUP) assumes that the choosen CC license also applies to the source code.

The posts on math.stackexchange.com, tex.stackexchange.com, stackoverflow.com, etc. are licensed under CC BY-SA 4.0.

I think the problem with LaTeX projects is that they often consists of roughly two parts:

The actual content of the project, i.e., the actual text inside the document environment. This is, for most intense and purposes, basically plain text, enhanced with some markup elements to specify the structure of the document (\section, etc.), as well as some syntax for mathematical notations (^ for exponents, _ for indices, etc.).

Some TeX code (or abstractions thereof) that is interpreted during runtime of the compiler.

After thinking about it for some time I came to the following conclusions (for now):

I personally think that the GPL is too strong for this kind of project, as it forces everyone who wants to use parts of the project to also use the GPL.

The MIT license seems to require attribution only in a rather weak sense: It is enough to simply retain a copy of the license file in the source code. For a LaTeX project, which will later be compiled into a PDF document, this basically removes the attribution. This is probably okay for projects that do not contain any actual content of their own (see next point), but I find it otherwise somewhat troublesome.

The LPPL license is probably the correct approach for projects that do not contain any actual content on their own, e.g., packages or templates.

A Creative Commons license can probably be used for projects that are about their actual content, and whose TeX components is (more or less) not so interesting.

In view of the above, I will probably put this project under CC BY-4.0. This should allow anyone to reuse its contents and its code, and the problems of using Creative Commons for code should hopefully not really matter for this specific project.

Alright, seems like you looked into that deeper than I did myself, as I did not really think about differentiating internal tex code from typesetted documents (be it the source code, wich is mostly just text as you mentioned, or the processed document).
So I agree with your viewpoint here (I will also have to rethink licensing on my projects now...)
Thanks for spending your time with this (and for the great document, of course)!

I already added the LICENSE file to the source repository last week, and have now also synced these changes to GitHub. (I originally wanted to fix the CI build before syncing the repositories, but it seems that this will take longer than expected.)


Sprint 2 Learning

yeah, this is a deliverable to our customer who is the professor, who expects us to learn something and we are being graded on this, thank you


grpc channel stub cannot end request when timeout exceed，eg：timeout=30ms，request ended after 200ms

What version of gRPC and what language are you using?
python
grpcio                             1.54.0
grpcio-tools                       1.54.0
What operating system (Linux, Windows,...) and version?
linux
What runtime / compiler are you using (e.g. python version or version of gcc)
Python 3.10.14 (main, May  6 2024, 19:42:50) [GCC 11.2.0] on linux
What did you do?
Just write a grpc client multit-thread demo, set the timeout 30ms for per request, and grpc server "<IP_ADDRESS>:1234" will always sleep for 60ms. So what to be expected should be that all requests end with timeout exceed in 30ms. But Actually, after lots of request time sended and grep elapsed time to stastics, many request cost much more than 30ms.
grpc_server/demo_client.py
#!/usr/bin/env python
# -*- coding: utf-8 -*-

from api import test_pb2, test_pb2_grpc
import os
import sys
import argparse
import logging
import time
import toml
from threading import Thread
from multiprocessing import Process

import grpc

grpc_channel_options = [
    ('grpc.max_send_message_length', -1),
    ('grpc.max_receive_message_length', -1),
    # ('grpc.client_idle_timeout_ms', 10 * 1000)
]
channel = grpc.insecure_channel("<IP_ADDRESS>:1234",
                                grpc_channel_options)

def start():

    request_cnt = 0
    while request_cnt < total:
        node = None
        try:
            # node, channel = client.get_channel()
            start_time = time.perf_counter()

            stub = test_pb2_grpc.TestStub(channel)
            req = test_pb2.Req(msg="hello")

            if args.api == "echo":
                rsp = stub.Echo(req, timeout=0.03)
            if args.api == "proxyecho":
                rsp = stub.ProxyEcho(req, timeout=1)
            elif args.api == "redisecho":
                rsp = stub.RedisEcho(req, timeout=1)

            if request_cnt % 1000 == 0:
                logging.info("request cnt %d, rsp %s", request_cnt, rsp.msg)
        except Exception as e:
            elapsed = (time.perf_counter() - start_time) * 1000
            # logging.error("exception, %s", e)
            print("exception %s, elapsed %f" % (e, elapsed))

        request_cnt += 1
        # time.sleep(0.002)

parser = argparse.ArgumentParser()
parser.add_argument("--api",
                    type=str,
                    default="echo",
                    help="echo/redisecho/proxyecho")

if __name__ == "__main__":
    args = parser.parse_args()
    # time.sleep(10000)

    total = 10000000
    worker_num = 32
    threads = []
    [threads.append(Thread(target=start)) for i in range(worker_num)]

    for th in threads:
        th.start()
    for th in threads:
        th.join()

Run the program nohup python grpc_server/demo_client.py for nearly 5 min, and print the time cost of top 10 longest request

What did you expect to see?
All request ended in 30ms
What did you see instead?
Some requests ended in 100ms+.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

Any help? @gnossen @XuanWang-Amos @hassox @beccasaurus

@johnsondeng  I wasn't able to repro the issue setting up a simple server against your client.
Please provide the server side script also to be able to repro this issue. If possible a docker image with your server/client setup.

@johnsondeng I wasn't able to repro the issue setting up a simple server against your client.
Please provide the server side script also to be able to repro this issue. If possible a docker image with your server/client setup.

issue.zip
The code and dockerfile is in zip attach files. I build a simple image for this issue,  and can still repro the issue after run the client for 5min.
dockerfile
FROM python:3.10

WORKDIR /workspace

RUN pip install grpcio==1.65.0 grpcio-tools==1.65.0

RUN mkdir /opt/protoc && cd /opt/protoc \
    && wget https://github.com/protocolbuffers/protobuf/releases/download/v27.2/protoc-27.2-linux-x86_64.zip \
    && unzip protoc-27.2-linux-x86_64.zip \
    && export PATH=$PATH:/opt/protoc/bin/ \
    && protoc --version

Can anyone help？ @gnossen @sourabhsinghs @XuanWang-Amos

@johnsondeng , i have tried again to repro this issue with suggested setup. these are my findings.

Elapsed Time Range
Number of Calls

(10, 20]
0

(20, 30]
0

(30, 40]
0

(40, 50]
0

(50, 60]
3200119

(60, 70]
64

(70, 80]
2

(80, 90]
0

(90, 100]
0

Do you have any suggestion on why we are unable to see similar delays ?
Did another test to see constant overhead for timeout delay . sharing graph for that also.

@johnsondeng , i have tried again to repro this issue with suggested setup. these are my findings.
Elapsed Time Range	Number of Calls
(10, 20]	0
(20, 30]	0
(30, 40]	0
(40, 50]	0
(50, 60]	3200119
(60, 70]	64
(70, 80]	2
(80, 90]	0
(90, 100]	0
Do you have any suggestion on why we are unable to see similar delays ?
Did another test to see constant overhead for timeout delay . sharing graph for that also.

I produce a histogram too,  it seems like that  requests which time cost > 60ms is more than yours.  I think maybe hardware diff? My container is running on Mac M1.
But my question is, why the request > 60ms cannot end requests  in 50ms(timeout params setting in rpc call)。 Your histogram above also shows 64 request in (60, 70]。The timeout is very inaccurate。
0.00 - 10.00: 0
10.00 - 20.00: 0
20.00 - 30.00: 0
30.00 - 40.00: 0
40.00 - 50.00: 0
50.00 - 60.00: 125234
60.00 - 70.00: 197
70.00 - 80.00: 33
80.00 - 90.00: 19
90.00 - 100.00: 8
100.00 - 500.00: 1

@johnsondeng In data you've shared looks like 99.794% of RPCs (125234/125492) ends in 50.00 - 60.00 ms. Latency in rest of the RPC can be attributed to Hardware / GIL / Network etc.


Support for only record Internal Audio

QuickTime player can support record internal audio, but just support Audio Recroding, not support Screen Recroding.

Kap is a screen recorder. We don't have any plans to only record audio.

Duplicate of https://github.com/wulkano/Kap/issues/735

Kap是一个屏幕录像机。我们没有计划只录制音频。

I expect to be able to record internal sounds while recording the screen, not external sounds.

Duplicate of #735

I expect to be able to record internal sounds while recording the screen, not external sounds.

To record system audio, you can install the Soundflower plugin for Kap via Preferences @jet10000!


chore: prepare v2.0.0-beta.17 release

chore: prepare v2.0.0-beta.17 release

😅 again

I see we'll fix this in another PR


how can i remove the slider ?

im trying to remove the colorpicker slider , can i do it ?

If you want to remove all sliders simply use data-wcp-sliders="w"  in your html.
refer to https://github.com/fujaru/jquery-wheelcolorpicker/wiki/Plugin%3AOptions

thanks


Should export named exports and default export when named exports are classes (declare)

We have a specific case where we need to export both named classes and a default function from a flow declaration file.  As soon as we add a default export, flow seems to not let us import the named class types.  We can import the default type and type types, but not class types.
flow versions: 0.20.0 and 0.21.0
Here's a simple example:
declare module Foo {
    declare class Foo { foo: number }
    declare function exports (n:number): Foo;
}

Importing Foo from another module fails:
import type { Foo } from 'Foo';

src/blah.js:7
7: import type { Foo } from 'Foo';
^^^ Named import from module Foo. This module has no named export called Foo.

If I switch Foo to a structural type, there is no error:
declare module Foo {
    declare type Foo = { foo: number };
    declare function exports (n:number): Foo;
}

Since we're extending / subclassing, this isn't an option for us.
I noticed that these tests don't assert that class types work correctly: https://github.com/facebook/flow/tree/master/tests/declare_type

Hey folks.  This issue has been open a while.  Just wondering if there is anything else I should do.  I created a very simple, reproducible case.  Is something in the works?  Did it get fixed in 0.22 and I didn't notice?  Just hoping to get some idea of when this might get fixed.  Thanks!

Sorry for the delay in response here!
So your best bet is probably to define your declare module as follows:
declare module "Foo" {
  declare class Foo { foo: number; }
  declare var exports: {
    (): (n: number) => void,
    Foo: Class<Foo>,
  };
}

The reason you're hitting issues here is because the use of an export named exports tells Flow to use that (exclusively) as the type of module.exports for the declared module. In other words: A declare for something called exports in a declare module always overrides all other declares inside the declare module
I have it on my queue to bring declare module syntax into the 21st century some point soon (I want to build proper ES export syntax for declare module to simplify this story) -- but for now the above definition should get you where you need to be.

(As usual: Please re-open if you still have issues)

Thanks for responding about the future plans.  We've got a different work-around that we may as well keep until proper support for ES export is implemented. :)  Looking forward to it. Thanks!


Invalid memory comparisons

3563e5c5d55cb3012336abbff73c0acac820f8f2 changes a number of strncmp calls to memcmp and this leads to out-of-bounds memory reads, in particular where peek_string is used. These are detected during fuzzing and, while probably benign, cause problems with coverage guidance as there are many ways to get essentially the same crash with slightly different inputs.

Do you have examples of these sorts of crashes?

Do you have examples of these sorts of crashes?

Aye, here's an example where memcmp will be used to compare 5 bytes out of bounds:
$ echo -n "=" |./build/fuzz.parse -
Reading 1 bytes from -
=================================================================
==159391==ERROR: AddressSanitizer: global-buffer-overflow on address 0x561545705c01 at pc 0x56154575e7c6 bp 0x7ffc19fced20 sp 0x7ffc19fce4d0
READ of size 5 at 0x561545705c01 thread T0
    #0 0x56154575e7c5 in MemcmpInterceptorCommon(void*, int (*)(void const*, void const*, unsigned long), void const*, void const*, unsigned long) crtstuff.c
    #1 0x56154575ecb9 in memcmp (/home/stevie/code/prism/build/fuzz.parse+0xb0cb9) (BuildId: 88df46cf0c232021)
    #2 0x56154580d540 in parser_lex /prism/src/prism.c:6639:62
    #3 0x561545808f27 in parse_program /prism/src/prism.c:14317:5
    #4 0x561545808da2 in pm_parse /prism/src/prism.c:14561:12
    #5 0x56154589ba02 in pm_parse_serialize /prism/src/prism.c:14584:23
    #6 0x5615458bcaf2 in harness /prism/fuzz/parse.c:7:5
    #7 0x5615458bcc2a in LLVMFuzzerTestOneInput /prism/fuzz/fuzz.c:8:5
    #8 0x56154580257d in ExecuteFilesOnyByOne /AFLplusplus/utils/aflpp_driver/aflpp_driver.c:255:7
    #9 0x561545802388 in LLVMFuzzerRunDriver /AFLplusplus/utils/aflpp_driver/aflpp_driver.c
    #10 0x561545801f48 in main /AFLplusplus/utils/aflpp_driver/aflpp_driver.c:300:10
    #11 0x7f639fe29d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #12 0x7f639fe29e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #13 0x561545744264 in _start (/home/stevie/code/prism/build/fuzz.parse+0x96264) (BuildId: 88df46cf0c232021)

0x561545705c01 is located 63 bytes to the left of global variable '<string literal>' defined in 'src/prism.c:14573:34' (0x561545705c40) of size 2
0x561545705c01 is located 31 bytes to the left of global variable '<string literal>' defined in 'src/prism.c:14566:34' (0x561545705c20) of size 6
  '<string literal>' is ascii string 'PRISM'
0x561545705c01 is located 0 bytes to the right of global variable '<string literal>' defined in 'src/prism.c:14418:31' (0x561545705c00) of size 1
  '<string literal>' is ascii string ''
SUMMARY: AddressSanitizer: global-buffer-overflow crtstuff.c in MemcmpInterceptorCommon(void*, int (*)(void const*, void const*, unsigned long), void const*, void const*, unsigned long)
Shadow bytes around the buggy address:
  0x0ac328ad8b30: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x0ac328ad8b40: 00 00 00 00 00 00 00 00 00 00 00 00 07 f9 f9 f9
  0x0ac328ad8b50: 00 00 07 f9 f9 f9 f9 f9 00 04 f9 f9 00 00 00 00
  0x0ac328ad8b60: 00 00 00 f9 f9 f9 f9 f9 00 04 f9 f9 00 07 f9 f9
  0x0ac328ad8b70: 00 00 00 00 00 00 00 00 00 02 f9 f9 f9 f9 f9 f9
=>0x0ac328ad8b80:[01]f9 f9 f9 06 f9 f9 f9 02 f9 f9 f9 00 04 f9 f9
  0x0ac328ad8b90: 00 00 00 00 00 00 00 f9 f9 f9 f9 f9 00 00 00 00
  0x0ac328ad8ba0: 03 f9 f9 f9 f9 f9 f9 f9 00 00 00 07 f9 f9 f9 f9
  0x0ac328ad8bb0: 06 f9 f9 f9 00 f9 f9 f9 00 00 00 00 00 05 f9 f9
  0x0ac328ad8bc0: f9 f9 f9 f9 00 00 00 00 00 07 f9 f9 f9 f9 f9 f9
  0x0ac328ad8bd0: 04 f9 f9 f9 00 01 f9 f9 06 f9 f9 f9 06 f9 f9 f9
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb
==159391==ABORTING

That looks more like a case where the return value from peek_string is not being correctly checked:
https://github.com/ruby/prism/blob/c8ea6f24e228391a2bf384d7fa57188456eef8aa/src/prism.c#L5020-L5029
whereas here we just use the return unconditionally:
https://github.com/ruby/prism/blob/c8ea6f24e228391a2bf384d7fa57188456eef8aa/src/prism.c#L6639
I don't think this is a memcmp vs. strcmp issue.

With this patch
diff --git a/src/prism.c b/src/prism.c
index f8c0beb3..0772fb23 100644
--- a/src/prism.c
+++ b/src/prism.c
@@ -6636,7 +6636,7 @@ parser_lex(pm_parser_t *parser) {

                 // = => =~ == === =begin
                 case '=':
-                    if (current_token_starts_line(parser) && memcmp(peek_string(parser, 5), "begin", 5) == 0 && pm_char_is_whitespace(peek_offset(parser, 5))) {
+                    if (current_token_starts_line(parser) && strncmp((const char *)peek_string(parser, 5), "begin", 5) == 0 && pm_char_is_whitespace(peek_offset(parser, 5))) {
                         pm_token_type_t type = lex_embdoc(parser);

                         if (type == PM_TOKEN_EOF) {

all the crashing test cases are "fixed". FWIW, I'm not a fan of using C strings anywhere; it'd be better to pass around a pointer and a length (like a golang slice or similar).


The app doesnt work anymore out of box

It seems h2 db is corrupted, hence it just doesnt work. Either a new h2 db has to be re-created or we should have a way to generate one on the fly, when user doesnt pass the h2 db file to its home dir.

This sample app is out-of-date and is no longer maintained. To get started with Aura, see the Aura Open Source Developer Guide.
I'm closing out this issue.


Clean up usability and documentation mess

Expected Behavior
OpenJsad should be useable and documented
Actual Behavior

the website is down
there are two different versions of openjscad - the old one has flaws and the new one is not ready
google search points to outdated documentation

Steps to Reproduce the Problem

try using openjscad with an existing openjscad project - the website is down - work around it
try fixing a problem with a parameter definition of type choice - a web search points you to https://en.wikibooks.org/wiki/OpenJSCAD_Quick_Reference which is outdated
get frustrated

To avoid this frustration i'd suggest to be more consistent in the plan for moving to version 2 including the necessary infrastructure and documentation. As I a start I'd suggest to open up github discussion to have a more consistent experience in the dicussion forum. Then the DNS entry for openjscad.org should be made more flexible so that a week long outtage as the one we are experiencing with #779 should not have such a bad effect.
It should be made more clear where the current documentation is and how a migration from V1 to V2 can be done. Also I'd love to get #448 going which is long overdue due to the complexity of the environment and project. I'd love to see more simplicity in the approaches.

I (and others) understand the frustration, but no one expects four(4) major data centers to go down at the same time. OpenJSCAD isn’t the only website down still.
In addition, those domain names are owned by individuals, who have drifted away from the project. Request for help was sent, but... yeah.

@z3dev  thank you for your swift answer which clarifies the DNS problem. When following a workaround such as https://creator.buildbee.com/ and then the documentation link you end up in https://openjscad.org/dokuwiki/doku.php which is also offline at this time - what's the workaround for that one?
The major question is to avoid such disruption and confusion in the future as well as make it easier to get to know openjscad and use components others have designed and more. This issue is calling for some strategic move that would position openjscad better. The current outage is IMHO just a clear warning what happens if the strategic part is not accounted for. I don't even know where the discussion/chatroom ended up so that's why i am asking to move it to github discussion got get more focussed.

I am personally a fan of using all the GitHub resources. GitHub pages for web hosting (both docs and online web app), and GitHub Discussions. Everything can be contained in this single repo.

@danmarshall please remember that the websites need that little Perl script. as far as I know, only static content can be served up by GIST. is there something that I missed?
For the API documentation, it's possible to create both HTML and MARKDOWN versions. The question is... where to put the MARKDOWN versions.
Discussions? Do you mean Team discussions? That's the only discussions that I know about at GitHub.

@z3dev  Discussions are for creating a forum/chat room and are effectively handled like issues so that e.g. issues changes and discussion entries can be linked. See https://github.com/somnathrakshit/geograpy3/discussions for an example.
The MARKDOWN can be put directly in the github pages which are effectively just a branch gh-pages of  the main gitrepository. Where is the content and the perl script right now?

Always something new at GitHub.... discussions are enabled.

See #785 for a first followup

I have updated the documentation to the wikibooks version but if there is a more recent one let me know!
https://creator.buildbee.com

https://en.wikibooks.org/w/index.php?title=OpenJSCAD_User_Guide&action=history shows the history of the page. I also had the assumption that this is the official documentation but it seems not to be any more and at least that should be pointed out

As discussed earlier, WIKIBOOKS is the V1 Documentation from a long while back. I’d be glad to use WIKIBOOKS again if they improved the review process. But waiting 6 months for some old fart to review two changes is ridiculous. I doubt that you can even make changes to those pages. I doubt that anyone can delete those pages.
😔

@z3dev is the newer v1 documentation checked in anywhere?

@z3dev is the newer v1 documentation checked in anywhere?

Sadly, no.
Both V1 and V2 WIKIs are still offline. I’ve been following the OHV recovery process but it’s going slow. I’ll start some discussions with OHV this week, and try recovering everything.
I had an old copy of the V2 wiki and put it online at openjscad.xyz/guide.html which forwards to the wiki. This is the new website.

Just added a hint to the wikibooks page about the deprecation:
https://en.wikibooks.org/w/index.php?title=OpenJSCAD_Quick_Reference&type=revision&diff=3826690&oldid=3583623

@WolfgangFahl thanks.
Do you know if books can be removed from WIKIBOOKS?

@z3dev  with admin right pages can be deleted in a wiki. There is a deletion log then but the page won't be visible any more and robots won't pick it up. It better practice to remove all content and place a redirect on the page to make sure outdated content is not referenced any more.

Closing this issue. Let's open new issues for anything left over.


Fix for Xaudio2 bugs

Fix issue with voices not being returned properly, causing them to grow out of control and start lagging.
Add sample rate to the voice key to prevent those from being re-used by another frequency. This should fix distortion with multiple audio sources.

This can be applied to master branch as well.

Excellent!


denippet#choosable(-1) cause error too many arguments

Hello!
according to the help file, we can pass an argument for direction.
However, when I excecute the command like this
vim.fn['denippet#choosable'](-1)

It causes error like this
E5108: Error executing lua: Vim:E118: Too many arguments for function: denippet#choosable
stack traceback:
	[C]: in function 'choosable'
	/Users/ryoppippi/.config/nvim/lua/plugin/nvim-cmp.lua:120: in function 'on_keymap'
	...oppippi/.local/share/nvim/lazy/nvim-cmp/lua/cmp/core.lua:145: in function 'callback'
	....local/share/nvim/lazy/nvim-cmp/lua/cmp/utils/keymap.lua:133: in function <....local/share/nvim/lazy/nvim-cmp/lua/cmp/utils/keymap.lua:127>

Looking at the actual implementation, it appears that this function does not accept any arguments

I don't see the documentation updated. The implementation is correct.
Fixed.

Okay thanks!!


Build fails due to OpenSSL and database configuration issues

While trying to set up the PostHog locally, I encountered a series of issues related to OpenSSL and the database configuration.
Here is a brief description of what happened:

After resolving a few package-related issues, I ran into an error related to OpenSSL. The error message was:

Error: Configuration property "enable.ssl.certificate.verification" not supported in this build: OpenSSL not available at build time
   at HighLevelProducer.Client (/path/to/plugin-server/node_modules/.pnpm/node-rdkafka-acosom@2.16.1_ts-node@10.9.1_typescript@4.9.5/node_modules/node-rdkafka-acosom/lib/client.js:54:18)
   at HighLevelProducer.Producer (/path/to/plugin-server/node_modules/.pnpm/node-rdkafka-acosom@2.16.1_ts-node@10.9.1_typescript@4.9.5/node_modules/node-rdkafka-acosom/lib/producer.js:75:10)
   at new HighLevelProducer (/path/to/plugin-server/node_modules/.pnpm/node-rdkafka-acosom@2.16.1_ts-node@10.9.1_typescript@4.9.5/node_modules/node-rdkafka-acosom/lib/producer/high-level-producer.js:93:12)
   at createKafkaProducer (/path/to/plugin-server/src/kafka/producer.ts:16:22)
   at createHub (/path/to/plugin-server/src/utils/db/hub.ts:130:47)
   at processTicksAndRejections (node:internal/process/task_queues:95:5)
   at startPluginsServer (/path/to/plugin-server/src/main/pluginsServer.ts:232:56)

I've tried re-installing rdkafka and OpenSSL - found this thread which helped me install OpenSSL without errors but I'm not sure if it completely fixed the root cause as I'm still facing the same issue.

Lastly, I encountered an error related to a missing database table graphile_worker.jobs. The error message was:
psycopg2.errors.UndefinedTable: relation "graphile_worker.jobs" does not exist

I tried to troubleshoot this, but I'm not certain whether it's due to missing database migrations, a misconfiguration, or some other reason.
Can someone please help? TIA.

Hey 👋
I ran into this exact same issue today. Eventually solved it (with the help of @benjackwhite)
Make sure you have these ENV variables set in your .zshrc (or equivalent) file:
export PATH="/opt/homebrew/opt/openssl@3/bin:$PATH"
export LDFLAGS="-L/opt/homebrew/opt/openssl@3/lib"
export CPPFLAGS="-I/opt/homebrew/opt/openssl@3/include"
export PKG_CONFIG_PATH="/opt/homebrew/opt/openssl@3/lib/pkgconfig"

Then purge the node_modules in your plugin-server and re-install them without the cache:
(env) ➜  posthog git:(master) cd plugin-server && rm -rf node_modules/ && cd ..
(env) ➜  posthog git:(master) pnpm i --force --dir plugin-server

You should see Kafka running an install script
node_modules/.pnpm/node-rdkafka-acosom@2.16.1_ts-node<EMAIL_ADDRESS>Running install script

I had to also rebuild rdkafka in the plugin-server directory.
cd plugin-server && pnpm rebuild node-rdkafka


Add Full Word search

Adds support for anchor based searching.
Closes #147.

Issues with inserting accent chars. My first guess would be the interpreter for unicode/ASCII in search causes accents to be placed differently. Backspace is not following the interpreter rule. So either use pure ASCII convention or pure Unicode convention for adding/removing accents.
Example: #bi<backspace>

Same rules as first letters is not being applied to full words regarding K vs ^ (khakka) or a vs E (oorha).
Example: Searching #Kwlsw is not showing results for #^wlsw.

First letters never had a rule about halant. It is not common knowledge which words end with a halant character. It should be added to the exceptions like above example of #Kwlsw
Example: Searching #jin ky B is not showing the line ਜਿਨੑ ਕੇ ਭਾਗ ਬਡੇ ਹੈ ਭਾਈ ਤਿਨੑ ਸਾਧੂ ਸੰਗਿ ਮੁਖ ਜੁਰੇ ॥

Hitting the <end> key inside the search box will move the cursor to the end as expected and also highlights the last search result. Problem is hitting space after this will activate the highlighted search result instead of adding a space to the search box. Only the <enter> key should be activating the highlighted search result.
Example: #jw kw<end><space>

Vishraam causes an issue for following search.
Example: #KUbu KUbu works. #KUbu KUbu<space> fails. Line is ਖੂਬੁ ਖੂਬੁ ਖੂਬੁ ਖੂਬੁ ਖੂਬੁ ਤੇਰੋ ਨਾਮੁ ॥

Incorrect highlighting. See first letter in results 3-5.


Search Bar enhancement

Description
Enhanced the search bar:

Added link to another search icon with proper size
changed the placeholder size
added outline to text box

Fixes #774
Replace issue_no with the issue number which is fixed in this PR
Screenshots
before:

after:

Checklist

[ x] Tests have been added or updated to cover the changes
[ x] Documentation has been updated to reflect the changes
[ x] Code follows the established coding style guidelines
[ x] All tests are passing

@thestarsahil pls add the gssoc labels


Update LTR 3.28.11

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Trying to update LTR without the backports (to see if it builds on windows)

@conda-forge-admin, please rerender

@gillins should we merge this even if there will be no update for windows users for now?

I'm happy for you to merge, no strong thoughts. I guess it shows how hard it is to keep updated once things aren't using the current pinning....

@gillins what do you think we need to move forward?

I don't think we can really update LTR safely without some sort of force push thing from main. Not really my area unfortunately. I think you are good to merge this for now and fix it up in the next LTR version.

Sorry, my bad for removing the LTR from migrations...
We still have a long way until the next major LTR (feb 2024) with monthly patch releases, so I will merge this for now, but try to fix it later to see if I can have build in windows for the following months.

I will pin the backports PR instead, has it fixes some stuff and close this one.


Pretrained model on cpu only

Hello,
I tried what @hangzhaomit suggested to be able to run on cpu, but it didn't work.
When I do not change anything in test.py I get this error:
AttributeError: module 'torch._C' has no attribute '_cuda_setDevice'
When I comment the lines
76. torch.cuda.set_device(args.gpu) (or give as argument -1)
108. segmentation_module.cuda()
I get the error:
AssertionError: Torch not compiled with CUDA enabled
I also did not find .to(device) anywhere in the code.
Could you please share how you fix these problems and run on a non CUDA device?
Thank you very much in advance.
Originally posted by @nefelipk in https://github.com/CSAILVision/semantic-segmentation-pytorch/issues/117#issuecomment-475057813

Hi, did you solve this problem?

Can you please help with this?

Can you please help with this?

Works! Thanks so much for your work.

Hi, how did you solve this problem? I am having the same issue

CPU-only solution: https://github.com/tejaswigowda/semseg-pytorch

@tejaswigowda  Model run with GPU took 3 sec, but CPU takes 30 sec around. Is that normal?


[WIP] Windows fpm

Will contain fixes for issue #20

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (91f35dd) 94.15% compared to head (50f5c01) 94.15%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   94.15%   94.15%
=======================================
  Files           4        4
  Lines        1249     1249
=======================================
  Hits         1176     1176
  Misses         73       73

Flag
Coverage Δ

ctests
94.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add ability to install packages + xgboost model deployer + example

I've added the ability to deploy containers and install additional packages using pip, as well as an integration test for the XGBoost model, and an example walkthrough (using an iPython Notebook) of building and deploying Clipper, and then using it with XGBoost.
closes #341

Can one of the admins verify this patch?

In regards to your issue, my output actually doesn't look like that - I thought I would try and find the lines where execution seemed to crash, but I didn't get the same lines logged. Perhaps this may be why as well? We might be running different things/different versions

LGTM. Nice work!

jenkins test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1169/
Test PASSed.


[GA/ios] add brew update

Closing, this is due to a deprecated syntax of the specific homebrew tap we try to install.


Update Argo Workflows app to the latest version

This issue is a:

[ ] Bug Report
[x] Enhancement suggestion
[ ] Feature request

If the issue concerns an existing Marketplace application, please tag the maintainer of the application (found in manifest.yaml) below:
Enhancement suggestions / feature requests
Currently, we have Argo Workflows version v3.0.3 installed by Civo marketplace as a newer version 3.4.8 is available, so it needs to be updated to the newer version. We kindly request an update to the latest version of Argo Workflows on the Civo marketplace to ensure that we have access to the most recent features and improvements.
@protosam  I request you to please take a look into this.

Sorry @aakashnagpal4567, but I'm going to have to decline to participate in maintaining this change.
As of the last time I checked, Civo is still lagging behind on the release cycle of k3s. I have moral objections to supporting the product.
cc @markboost - I'm sorry if I wasn't clear previously, you need to put priority in maintaining the software being deployed. This starts with k3s. This is literally why we can't see eye to eye.
Sincerely,
Sam


Option -p 2 breaks svg file

Try optimize file with -p 2 option and got next results:
Original: https://gist.github.com/shyr1punk/be4c9e65d0da86a8e1f8e80fb7a0083c
With -p 2 option: https://gist.github.com/shyr1punk/3d9c534c6de600ee8e6653fa3744a48f
No -p 2 option: https://gist.github.com/shyr1punk/564966b8b03729f3952d3481b8067733

Not sure if this is an issue with svgo since you are explicitly using a low precision?

Looks like this is fixed. I cannot reproduce the problem with -p 2


chore: release main

:robot: I have created a release *beep* *boop*
@opentelemetry/instrumentation-aws-sdk: 0.2.0
Features

AWS-SDK SNS Context propagation (#728) (78cd4e1)

@opentelemetry/instrumentation-graphql: 0.26.1
Bug Fixes

adding nested input variables in graphql plugin (#720) (7a7d3a3)

@opentelemetry/instrumentation-nestjs-core: 0.27.0
Features

report path and route with fastify backend in nestjs-core instrumentation (#714) (62a04c7)

@opentelemetry/auto-instrumentations-node: 0.26.1
Dependencies

The following workspace dependencies were updated

dependencies

@opentelemetry/instrumentation-graphql bumped from ^0.26.0 to ^0.26.1

This PR was generated with Release Please. See documentation.

@Rauno56 if you approve I can release this since it only affects one package

@NathanielRN @willarmiros can you confirm the version 0.2.0 is correct for aws?

Looks like the action is clearing old reviews. I'll see what i can do to stop that from happening.

@dyladan yep that's ok, instrumentation is still non-stable, the other AWS packages (id generator, propagator, resource detectors) are stable.

There is nothing preventing this PR from being merged, right?
I would really be glad to publish the aws-sdk instrumentation with the sns context propagation inject feature.

/easycla

Sorry this took so long to merge

:robot: Release for @opentelemetry/auto-instrumentations-node is at https://github.com/open-telemetry/opentelemetry-js-contrib/releases/tag/auto-instrumentations-node-v0.26.1 :sunflower:

:robot: Release for @opentelemetry/instrumentation-aws-sdk is at https://github.com/open-telemetry/opentelemetry-js-contrib/releases/tag/instrumentation-aws-sdk-v0.2.0 :sunflower:

:robot: Release for @opentelemetry/instrumentation-graphql is at https://github.com/open-telemetry/opentelemetry-js-contrib/releases/tag/instrumentation-graphql-v0.26.1 :sunflower:

:robot: Release for @opentelemetry/instrumentation-nestjs-core is at https://github.com/open-telemetry/opentelemetry-js-contrib/releases/tag/instrumentation-nestjs-core-v0.27.0 :sunflower:


saucelabs

See https://github.com/saucelabs-sample-scripts/Node-Selenium/blob/master/selenium-webdriver/selenium-webdriver-simple.js
@addyosmani @philipwalton
Selenium Assistant has been useful for local testing and running tests on Travis CI via download browsers. One of the biggest issues at the moment is lack of windows support and the download times (as well as automated Safari support).
I'm thinking it would be relatively easy to add saucelabs support, would just require changes with some of the method names, some additional methods.
I'm questioning if it's useful or not beyond making it easy to do local vs remote testing.
API Changes

[x] getBrowser(browserId, release) => getLocalBrowser(browserId, release)
[x] getAvailableBrowsers() => getLocalBrowsers()
[x] downloadBrowser() => downloadLocalBrowser()

Addition

[x] setSaucelabsCredentials()
[ ] getSaucelabsBrowser(browserId, release, options)

Minor Details

[ ] printAvailableBrowserInfo() - Flag whether saucelabs is supported or not
[ ] killWebDriver() Would need to kill connection with saucelabs etc

+1
In my experience the hardest part about webdriver testing is writing the tests and dealing with the flakiness/driver inconsistencies. But If you already have code that runs the tests locally, it's relatively simple to just set the selenium server URL to point to the Sauce Labs cloud.
The one other thing you'll want to remember to do is update the Sauce Labs tests with the results, since that doesn't happen automatically. Here's some code I wrote to do that prior to webdriver.io supporting this in their runner:


Tonel file format will introduce many changes in P12

With Pharo 12 changing the format for Tonel, any PR's will have many changes not related to the actual change.
I propose moving the current main branch to Pharo-11 and making the main branch a Pharo 12 as default.
The script in the Tonel documentation (https://github.com/pharo-open-documentation/pharo-wiki/blob/master/General/ExportFormats.md) can be used to convert the complete package at once.

Tonel is now pinned to ver 1.0


Support retrieving workspace of path dependencies in cargo

What are you trying to accomplish?

Given a repository like so:
.
├── cli_crate
│   ├── Cargo.toml
└── workspace
    ├── Cargo.toml
    └── nested
        └── Cargo.toml

Where workspace represents a workspace containing only nested, and cli_crate has a path dependency to nested, and nested inherits any property from the root (e.g. edition =  { workspace = true }).
If we point dependabot to only cli_crate, it will error out when running cargo update as though it can fetch the path dependency nested, it cannot resolve this as it also needs the workspace due to this inheritance. Fix Cargo file_fetcher to support this logic of locating the root of a workspace member who is a path dependency, and fetching it.
For negative cases, where the root cannot be located or is incorrect, emit an error message (instead of when running cargo update which can be confusing to understand).

Closes #9533
Anything you want to highlight for special attention from reviewers?

How will you know you've accomplished your goal?

Unit tests added (see reference repository: https://github.com/Jefffrey/dependabot-cargo-test/tree/master)
Checklist

[x] I have run the complete test suite to ensure all tests and linters pass.
[x] I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
[x] I have written clear and descriptive commit messages.
[x] I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
[x] I have ensured that the code is well-documented and easy to understand.

Approving based on @RobJellinghaus ' approval :)

Thanks Abdul! Sorry I don't have approver ability myself for some reason, let me know if you can diagnose that. Thanks @Jefffrey for this work! Do you have any other Rust Dependabot issues you are looking at?

Thanks so much @Jefffrey !

This led to a regression that causes Dependabot runs to fail in multi-package Cargo workspaces - see #10584. Any chance this could either get reverted or fixed forward?

This led to a regression that causes Dependabot runs to fail in multi-package Cargo workspaces - see #10584. Any chance this could either get reverted or fixed forward?

Sorry about this! I've raised a PR to revert these changes: #10599


Segmentation fault when ImageAcquisitionManager is created and quit() is called in interpreter

Describe the bug
When Harvester is initialized and ImageAcquisitionManager is created, exiting the interpreter causes a segmentation fault.
To Reproduce
Simply the following code in the interpreter will do:
>>> from harvesters.core import Harvester
>>> h = Harvester()
>>> h.add_cti_file("/path/to/cti_file")
>>> h.update_device_info_list()
>>> iam = h.create_image_acquisition_manager(0)
>>> quit()
Segmentation fault

Expected behavior
Python interpreter should be able exit without the segmentation fault.
Desktop (please complete the following information):

OS: Red Hat Enterprise Linux 7.4

Hi Severi,
Thank you for the report. Could you tell me which version of Harvester Core did you use?
PS. I'll privately reply tomorrow to the message that you recently sent to me! Thanks!

Hi @kazunarikudo!
I'm using version 0.post0.dev615. I also noticed that the interpreter sometimes exits with words
pure virtual method called
terminate called without an active exception
Aborted

instead of Segmentation fault.

Thank you for making a try! If you don't see the segfault it should be the change that I made for issue #40. And yes, I also saw pure virtual method called just a couple of hours ago when I closed the Harvester GUI. Of course, calling pure virutal method is absolutely wrong so there must be something wrong with the design and I would have to investigate it anyway. By the way, you told me "the interpreter sometimes exists with...". Do you mean it's literally sometimes but not always?

By the way, you told me "the interpreter sometimes exists with...". Do you mean it's literally sometimes but not always?

Yes, most of the time the interpreter exits with just Segmentation fault.

Okay, so could you please upgrade the Harvester Core and check the situation doesn't change. The latest version is 615. You should be able to upgrade it just typing pip install -U harvesters.

It's not necessary to try today but I would like to introduce you how to log the execution of Harvester. I'll attach a logging configuration file but please assume you have already got the file and its name is logging.report.ini. Just copying the file, you will tell Harvester the location where the configuration file is located using an environment variable HARVESTER_LOGGING_CONFIG. To setup the environment variable, just type export HARVESTER_LOGGING_CONFIG=path/to/logging.report.ini. Then you launch your interpreter. In the meantime, you'll see logging messages every code execution. I'll show you a block that I excepted from the example code you introduced.

In [1]: from harvesters.core import Harvester

In [2]: h = Harvester()

In [3]: h.add_cti_file('/Users/kznr/dev/genicam/bin/Maci64_x64/TLSimu.cti')
2018-10-05 23:58:36,559 :: harvesters :: INFO :: Added /Users/kznr/dev/genicam/bin/Maci64_x64/TLSimu.cti to the CTI file list.

In [4]: h.update_device_info_list()
2018-10-05 23:58:45,435 :: harvesters :: INFO :: Discarded the device information list.
2018-10-05 23:58:45,446 :: harvesters :: INFO :: Initialized GenTL Producer, /Users/kznr/dev/genicam/bin/Maci64_x64/TLSimu.cti.
2018-10-05 23:58:45,448 :: harvesters :: INFO :: Opened System module, TLSimu.cti.
2018-10-05 23:58:45,451 :: harvesters :: INFO :: Opened Interface module, XX::InterfaceA.
2018-10-05 23:58:45,454 :: harvesters :: INFO :: Opened Interface module, XX::InterfaceB.
2018-10-05 23:58:45,455 :: harvesters :: INFO :: Updated the device information list.

In [5]: iam = h.create_image_acquisition_manager(0)
2018-10-05 23:58:53,104 :: harvesters :: INFO :: Opened Device module, TLSimuMono.
2018-10-05 23:58:53,105 :: harvesters :: INFO :: URL: Local:SIDEV.xml;380;6214
2018-10-05 23:58:53,113 :: harvesters :: INFO :: URL: Local:SIDEVTL.xml;600;fcc
2018-10-05 23:58:53,115 :: harvesters :: INFO :: URL: Local:SITLI.xml;700;2fd0
2018-10-05 23:58:53,118 :: harvesters :: INFO :: URL: Local:SITL.xml;940;1310
2018-10-05 23:58:53,121 :: harvesters :: INFO :: Opened DataStream module 0 of TLSimuMono.
2018-10-05 23:58:53,121 :: harvesters :: INFO :: URL: Local:SIDS.xml;300;ce4
2018-10-05 23:58:53,122 :: harvesters :: INFO :: Instantiated an ImageAcquisitionManager object for TLSimuMono.

In [6]: quit()
2018-10-05 23:59:03,340 :: harvesters :: INFO :: Closed DataStream module 0 of TLSimuMono.
2018-10-05 23:59:03,341 :: harvesters :: INFO :: Disconnected the port from the NodeMap of TLSimuMono.
2018-10-05 23:59:03,342 :: harvesters :: INFO :: Closed Device module, TLSimuMono.
2018-10-05 23:59:03,342 :: harvesters :: INFO :: Destroyed the ImageAcquisitionManager object which TLSimuMono had belonged to.
2018-10-05 23:59:03,343 :: harvesters :: INFO :: Started resetting the Harvester object.
2018-10-05 23:59:03,343 :: harvesters :: INFO :: Removed the all CTI file from the list.
2018-10-05 23:59:03,343 :: harvesters :: INFO :: Discarded the device information list.
2018-10-05 23:59:03,344 :: harvesters :: INFO :: Closed Interface module, XX::InterfaceA.
2018-10-05 23:59:03,344 :: harvesters :: INFO :: Closed Interface module, XX::InterfaceB.
2018-10-05 23:59:03,345 :: harvesters :: INFO :: Closed System module, TLSimu.cti.
2018-10-05 23:59:03,345 :: harvesters :: INFO :: Closed /Users/kznr/dev/genicam/bin/Maci64_x64/TLSimu.cti.
2018-10-05 23:59:03,345 :: harvesters :: INFO :: Completed resetting the Harvester object.

Unfortunately, I have not seen any segmentation fault on Ubuntu or macOS but you would able to reproduce the phenomenon and we could find a clue in the logging message.

logging.report.ini.zip
Please note that you should remove the environment variable typing unset HARVESTER_LOGGING_CONFIG. Otherwise, Harvester will keep growing a log file (it's named harvester.log by default) and I know it's not good for you! ;-)

Please excuse me, I had inject a bug at 53b92260c4d2cc30d8563b0460b4279f6c246a2d which crashes Harvester while it's connecting a camera but it's been fixed at d05f7d9d2c6f039d87832f6890c54bd07dc5f57a.

I have just confirmed the phenomenon on Windows with a tangible device.

Hi Severi, I have committed a change at e326309dbb041d384146ab742f3413b5b1530507 and I expect the change fixes the issue you are facing. Please let me know if it didn't. In addition, I have changed an identifier from ImageAcquisitionManager to ImageAcquirer (see https://github.com/genicam/harvesters/issues/49 about the detail). I'm sorry having broken the compatibility. -Kazunari

I have committed a change at e326309 and I expect the change fixes the issue you are facing.

I can confirm that your commit indeed has fixed the issue. :+1: Thank you!

I have changed an identifier from ImageAcquisitionManager to ImageAcquirer ... I'm sorry having broken the compatibility.

No worries, it was just a small fix. :)


Boolean type maps to String?

Hey,
Just wondering what I'm missing, how come you're mapping Boolean types to String in DynamoDB? I thought DynamoDB supported a Boolean type?

Dynamoose predates support for Boolean types in DynamoDB.   I've been thinking about adding a option to toggle the behavior.  I do not want to change the current default behavior as it would break the existing deployments.


Add option to use permissions boundary on the default roles

what

Allow the use of a permissions boundary on the default role

why

In our organisation, permissions boundaries are a requirement for all engineers to provide to roles. Allowing them to be attached to the default roles allows us to create node groups without creating our own roles and policies.

The change is entirely optional and non-breaking.

references

The eks_cluster module you provide also allows this.

Thanks for the assist @nitrocode 👍  -- I need to remember that I can do that for simple changes like I mentioned.

Thanks all :)


terminal error

Type: Bug
When I try to execute my python programme using the terminal inside VScode, there's only a red cross appearing in front of the command without any other error reports.
But when I type the same command (python test.py) into the powershell terminal, my python program runs just like I wish.
I have restarted VScode, restarted my computer, and even uninstalled and reinstalled python for my computer. But VScode still doesn't work!
Until now, I still couldn't figure out what goes wrong......
VS Code version: Code 1.76.0 (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:22:44.506Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
AMD Ryzen 7 5800H with Radeon Graphics          (16 x 3194)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
13.86GB (4.14GB free)

Process Argv
--crash-reporter-id 61856dad-caf4-4ccb-bae0-dcd822929e23

Screen Reader
no

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

codespaces
Git
1.13.10

latex-workshop
Jam
9.7.0

vscode-language-pack-zh-hans
MS-
1.76.2023030109

csharp
ms-
1.25.4

remote-wsl
ms-
0.76.1

cpptools
ms-
1.14.4

cpptools-extension-pack
ms-
1.3.0

makefile-tools
ms-
0.6.0

vetur
oct
0.37.3

markdown-preview-enhanced
shd
0.6.8

open-in-browser
tec
2.0.0

errorlens
use
3.7.0

local-history
xyz
1.8.1

markdown-all-in-one
yzh
3.5.0

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
pynewvextcfv2:30669238
azure-dev_surveyone:30548225
pyindex848:30662994
nodejswelcome1:30587005
282f8724:30602487
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
showlangstatbar:30672706
vsctsb:30677850
pythonb192:30669360

It looks like your report is missing some important information, See this wiki page for more info on how to report actionable terminal issues.
/label info-needed terminal

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


A2C - Merging

Once this is merged rev1 and rev2 of the analog to CAN board will be finished completely
Rev1 layout was done with Ethan and Rev2 was already reviewed, so I just need permission to merge

Resistor was changed from 3.3k to 330 👍

Resistor was changed from 3.3k to 330 👍


Bulk API

Many scenarios require the insertion of a lot of data in bulk where using Prisma's normal create operations are too slow. For cases like these we should add a dedicated bulk API which probably also will become its own query capability.

@janpio is a bulk API on the roadmap? this would be an amazing addition.

+1

We have a createMany operation now: https://www.prisma.io/docs/concepts/components/prisma-client/crud#create-multiple-records
This issue can be closed now.

what about upsertMany? is it on prisma roadmap?

@xiaoyu-tamu Follow https://github.com/prisma/prisma/issues/4134 for upsertMany
Right now you can workaround that by filtering with findMany and then creating non existent records with createMany


use stable python 3.7 in travis CI

Travis so has some issues with running python 3.7 natively, and due to that initially the old, no longer updated, "-dev" version was used. This PR fixes that by using a stable release - much more likely used by the end users of the package.
Related travis issue: https://github.com/travis-ci/travis-ci/issues/9815

Pull Request Test Coverage Report for Build 104

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.7%) to 85.087%

Files with Coverage Reduction
New Missed Lines
%

monero/numbers.py
2
97.62%

monero/ed25519.py
5
69.9%

Totals

Change from base Build 36:
-0.7%

Covered Lines:
833

Relevant Lines:
979

💛  - Coveralls


ElectrumX Wallet - V1.2.4 - WINDOWS Version - How to SEND Assets ?

Hi,
I use the 1.2.4 wallet version along with a nano ledger.
I want to send an asset, but cannot see the option to do so.
Is it possible ? If not, how can I move my assets ?

Hi there,
We use GitHub issues as a place to track bugs and other development-related issues.
Please see the link below to our dedicated support line:
Help Center :  Click Here
Ticket ID: WB240
Note: Click on the live chat icon at the bottom corner of the page to start a conversation.

You cannot currently move assets of of ledger hardware without sideloading a custom app

Ok thanks. What side app should I use ?

https://github.com/kralverde/app-ravencoin

Hello @gaetan29  A support ticket has been opened for you. Kindly Talk to Support on the Live support page so that your issue can be looked into. CLICK HERE


xds: test clean up

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

/assign @adisuissa

Hi @adisuissa, I don't why this cannot be retested, could you help me trigger it? Thanks!


Difference to binaryan

What is the motivation for creating a 'second binaryan.js',  or what are the differences?
Isn't WAST already an AST?

OK, WAST is an AST in text format, got it ;)
still, why not use
var module = new Binaryen.Module();
var parser = new Binaryen.SExpressionParser(input);
parser.get_root().dump();
var s_module = parser.get_root().getChild(0);

@pannous what is first binaryan.js?

Oh I meant binaryen with e https://github.com/WebAssembly/binaryen / https://github.com/WebAssembly/binaryen/blob/master/bin/binaryen.js

True npm install --save -g binaryen doesnt work unlike yours