feat: Add Velas Blockchain Currency & NPM Package

📝 Description

Added VLX Cryptocurrency
Added NPM Package for Velas App Integration

❓ Context

Impacted projects: hw-app-velas
Linked resource(s): https://github.com/LedgerHQ/ledgerjs/pull/827

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

@Z4karia our firmware apps team has informed us that this change is not ready to be merged at the moment, your PR will be considered as soon as the Nano app has been fully reviewed.
cc @Jdumas-Ledger @tjulien-ledger


Add test case : test_setting_concurrent_rebuild_limit

Signed-off-by: Chris Chien<EMAIL_ADDRESS>https://github.com/longhorn/longhorn/issues/3872
Add test case test_setting_concurrent_rebuild_limit

@cchien816 Please check @shuo-wu comments and fix the conflicts.

LGTM except for some minor comments.

@cchien816 Good job. Let's backport to 1.3.x and 1.2.x.


Added async keyword to function

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description

was missing async keyword in function, added it

Great eye, thanks for spotting this @MoofyWoofy ... And, thanks for your first contribution to the Docs repo!  Welcome to the Face Pile! 🥳


Doc fix for SELECT ... FINAL

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Short description (up to few sentences):
Fix description for FINAL statement - in can be used not only for CollapsingMergeTree, but also for ReplacingMergeTree tables.

It can be used for every kind of MergeTree (Aggregating, Graphite, ...) except plain MergeTree.


Processing images

Hi,
I am really interested in your project. I was wondering if there is a way to test the model on single images or even the webcam.
The first thing I need is to load the model from disk. I downloaded the .pth files you provided, but I am not sure how to load them and reconstruct the network.
My folder structure is:
models:
-file.pth
-fold_0
--OF_Horizontal
---AU01.pth
---AU02.pth
---...
-fold_1
--OF_Horizontal
---AU01.pth
After that it should be quite straight forward to send images into the network and visualize the output.
Thanks a lot,
Julian

Hello Julian, thank you for your interest in our work.
I just pushed new changes so it is possible to run a DEMO. See new function at solver.py
The demo is quite straighforward, you can read the updated README in order to run it..
To run it with the OF!=None, you have to calculate the OF in advance for your demo images. Our binary script to run the OF is temporaly out of service, but we are working on it (cc @fuankarion).
I think it should be ok like that, otherwise let me know it please.

Hello,
I would also be willing to test your network if you can provide a link to the weights.
Thanks,
Marc


CO2-em-removal files contain NaN sector dimensions

          Hi, Some feedback on the CO2-em-removal files:

In the CO2-em-removal files, the sector ids are NaN for the OAE sector:
double sector(sector) ; 		sector:_FillValue = NaN ; 		sector:long_name = "sector" ; 		sector:id = "1.0: CDR Industry; nan: CDR OAE; 0.0: CDR DACCS" ;
Also sector data type is double in these files compared to int64 in the corresponding CO2-em-anthro files.
Originally posted by @ShraddhaGupta28 in https://github.com/IAMconsortium/concordia/issues/32#issuecomment-1969073719

Thanks,  @ShraddhaGupta28 for reporting that. Indeed, we noticed in the checking with daria and etienne on #32 , that the sector coordinate breaks whenever data is missing, unfortunately, instead of supplying them as 0 valued entries. This should already be fixed by the code we prepared in #33 , but i'll make sure to check that the removals are correctly represented as well.


Tracking Vision Development.

There are some things missing in vision that should be added to complete the service.

[x] Add support for image = client.image(filename='./my.jpg'). (#2778, #2792)
[x] Make public method for multiple annotation detection(#2770).
[x] Move emotion attributes directly to face. face.joy instead of face.emotions.joy. (#2782)
[x] Add system tests #2526. (#2788, #2827, #2830, #2834, #2835, #2836, #2839)
[x] Add GAPIC support. (#2914, #2918, #2922, #2937, #2938)
[x] Add batch processing, annotation multiple images with one request.(#2897, #2967, #2978)
[ ] Rename Face.Landmark to something like Face.FacialFeature(https://github.com/GoogleCloudPlatform/google-cloud-python/pull/2833#discussion_r91315815).
[ ] Move snippets to docs/vision_snippets.py.
[ ] Add more examples and update documentation(#2754).
[ ] Refactor unit tests.

Long since launched.


How to ask for more quota?

Is your feature request related to a problem? Please describe.
New SEPAL users are accredited with a 1$ quota by default, which sometimes could be enough for users that only use the predefined recipes or some modules. However, for others, this quota could run out so fast, and usually, they get frustrated because they have zero clues on how to ask for more budget, even more, in some cases, they don't know that can ask for more resources. For others, it could be even worst, they are in a middle of a process and they are in a different time zone from Rome, they will have to wait hours until some administrator can see their quota request.
Describe the solution you'd like
We could create a button inside the resource's dashboard to automatically give the user a predefined amount of quota (i.e. 5% of their current budget) without waiting for approval, so they can finish their process. Afterward, we could create a form in which they can ask formally for more monthly quota (without exiting from SEPAL, a form to communicate with admins from inside the platform).

I have the feeling that @lpaolini and @cdanielw have coded something for us :

@dfguerrerom I'll create the new translation keys you'll just have to correct my typos


HTTP protocol doesn't follow redirect

I cannot open
uproot4.open("https://github.com/CoffeaTeam/coffea/raw/master/tests/samples/nano_dy.root")

yet I can open the place it 302 redirects to:
uproot4.open("https://raw.githubusercontent.com/CoffeaTeam/coffea/master/tests/samples/nano_dy.root")

Adding my comment from Slack, for the record:
This is a missed case:
https://github.com/scikit-hep/uproot4/blob/d8a4b7203952bb7bd85cbf6d2ed481d610c83e67/uproot4/source/http.py#L163-L174
doesn't check for response.status == 302 (redirect) and use the connection to try again at response.headers["location"].
Same for:
https://github.com/scikit-hep/uproot4/blob/d8a4b7203952bb7bd85cbf6d2ed481d610c83e67/uproot4/source/http.py#L244
which only returns True or False based on whether the status is 206 or not. This should be an issue asking to support redirects.
On the second issue of not being able to access the redirected file: I wasn't able to the first time, then I was every subsequent time. I verify that I can read the data in it. I don't know if this was a one-time GitHub thing or if it always needs to "warm up," or what.

When you are at it, please don't forget the other redirect status codes:
https://en.wikipedia.org/wiki/List_of_HTTP_status_codes#3xx_redirection


Where to Find Web Files?

Does anyone know where I can find the web files for postal? This way I can make changes and edits to its web interface. It would be best if you could just specify the directory it is located in.

You can find them in the app/views directory in HAML format.


openssh: 7.7p1 -> 7.9p1

added openssh_gssapi to make it easier to test the patched version
the HPN edition isn't available on top of 7.9p1 yet
fix-host-key-algorithms-plus.patch didn't apply anymore, assuming it's
fixed.
release notes: https://www.openssh.com/txt/release-7.9
/cc @aneeshusa
Motivation for this change
Package upgrade. Looks like we missed the 7.8p1 one, maybe because of all of the patches that make it annoying to upgrade?
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Failure on x86_64-darwin (full log)
Attempted: openssh
Partial log (click to expand)
building '/nix/store/0782dybi3wk39b41j447mpc2cdfxh9ni-curl-7.61.1.drv'...
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: retry: Resource temporarily unavailable
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: retry: Resource temporarily unavailable
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: retry: Resource temporarily unavailable
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: retry: Resource temporarily unavailable
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: Resource temporarily unavailable
builder for '/nix/store/0782dybi3wk39b41j447mpc2cdfxh9ni-curl-7.61.1.drv' failed with exit code 254
cannot build derivation '/nix/store/5yhqvcs98jw8v1gi18w1wankybcvisn4-openssh-7.9p1.tar.gz.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/rcinfnz94wwy0a52mq8z23xkjvdqywc0-openssh-7.9p1.drv': 1 dependencies couldn't be built
error: build of '/nix/store/rcinfnz94wwy0a52mq8z23xkjvdqywc0-openssh-7.9p1.drv' failed

Success on aarch64-linux (full log)
Attempted: openssh
Partial log (click to expand)
shrinking /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/bin/ssh-add
shrinking /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/bin/scp
shrinking /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/bin/ssh
gzipping man pages under /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/libexec  /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/bin
patching script interpreter paths in /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1
/nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1/bin/ssh-copy-id: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1...
/nix/store/yva1x98c92gacij62050xwwri3hs7s12-openssh-7.9p1

@GrahamcOfBorg build openssh_hpn openssh_gssapi

Failure on x86_64-darwin (full log)
Attempted: openssh_hpn, openssh_gssapi
Partial log (click to expand)
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: retry: Resource temporarily unavailable
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: retry: Resource temporarily unavailable
/nix/store/ic9jk332yd71lwvgdwsi8jh4jjmw5j63-cctools-binutils-darwin-wrapper/nix-support/setup-hook: fork: Resource temporarily unavailable
builder for '/nix/store/0782dybi3wk39b41j447mpc2cdfxh9ni-curl-7.61.1.drv' failed with exit code 254
cannot build derivation '/nix/store/mbr1lzg0m0vfrvmmg4bcaml5gzgqgixi-hpn-KitchenSink-7_8_P1.tar.gz.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/5yhqvcs98jw8v1gi18w1wankybcvisn4-openssh-7.9p1.tar.gz.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/jni36d02bp2hk2g0jniwqkchz2kfrj0d-openssh-gssapi.patch.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/iw8w1115x8szm8cvp2qsm7w3jbs6986b-openssh-7.8p1.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/x41rlcvgh4j9gxkg5c50am4m5a4znsxh-openssh-7.9p1.drv': 2 dependencies couldn't be built
error: build of '/nix/store/iw8w1115x8szm8cvp2qsm7w3jbs6986b-openssh-7.8p1.drv', '/nix/store/x41rlcvgh4j9gxkg5c50am4m5a4znsxh-openssh-7.9p1.drv' failed

Success on aarch64-linux (full log)
Attempted: openssh_hpn, openssh_gssapi
Partial log (click to expand)
shrinking /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1/bin/scp
shrinking /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1/bin/ssh
gzipping man pages under /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1/libexec  /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1/bin
patching script interpreter paths in /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1
/nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1/bin/ssh-copy-id: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1...
/nix/store/98334qwvhachv21kf89gw777ad076x69-openssh-7.8p1
/nix/store/ccyys0mvdj9fqfv2p56ik4bvnhc4ydx6-openssh-7.9p1

Success on x86_64-linux (full log)
Attempted: openssh_hpn, openssh_gssapi
Partial log (click to expand)
shrinking /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1/bin/sftp
shrinking /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1/bin/ssh-keygen
gzipping man pages under /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1/libexec  /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1/bin
patching script interpreter paths in /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1
/nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1/bin/ssh-copy-id: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1...
/nix/store/0573949flahx2r2p60r8r58k721gdp01-openssh-7.8p1
/nix/store/pmligmf1sj3gadagh7m2srj80irxhdac-openssh-7.9p1

5k rebuilds - this should probably go to staging.

oops, saw your comment too late


Shavee

Project description
Shavee is a Program to automatically decrypt and mount ZFS datasets using Yubikey HMAC as 2FA or any File on USB/SFTP/HTTPS drive with support for PAM to auto mount home directories.
Metadata

homepage URL:https://ashu.io/projects/shavee/
source URL:https://github.com/ashuio/shavee
license: mit
platforms: unix, linux, darwin, ...

I'm working on this. I contacted @ashuio to tag the latest version as the current is alpha and is missing a Cargo.lock file.

Hi, this was merged. And you can begin using it. I see it's only on unstable at the moment though.


scripts: add more options in qemu script

scripts: add more options in qemu script
I frequently need to change the number of processors, memory and VM name. This change allows the user to specify these options when booting CoreOS from the qemu script.

Trailing arguments are supported and added to the qemu invocation.
Using something like:
$ ./coreos_production_qemu.sh -name asdf -smp cpus=10

will change the name, number of processors, etc.
I'm not sure we need this mechanism as well since it doesn't hide a significant amount of abstraction (as, for example, the ignition flag does).

I see your point. There is, however, a minor downside to the solution you proposed, in that the command-line of qemu (e.g., ps -AF) looks confusing.
Would you be more open to an -image option to allow pointing to a COW-image of CoreOS instead of the one shipped by default?


Fix CUDA OOM Issues on Tesla T4

Closes #84

I'll eventually manage to forget this very useful and insightful commit history


⚠ ClamAV has degraded performance

In 74fb194, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 291 ms

Resolved: ClamAV performance has improved in c1f7202.


Fix CI. Bugfixes for splice and replace

CI: apply fixes from StyleCI,
Bugfix: splice on edge cases,
Bugfix: replace on a Chain,
CI: update scrutinizer,
CI: fix appveyor build

You can see the expected behaviours for array_splice on edge cases here:
https://repl.it/repls/WeakSteelblueTrials

Awesome, thank you for the PR


Cannot upgrade to 0.43

Describe the bug
Cannot upgrade to 0.43
Version
v0.42.3
To Reproduce
Execute pip3 install -U changedetection.io to no avail. Requirements are already satisfied. No upgrade performed.
Expected behavior
Finish upgrade to 0.43.
Desktop (please complete the following information):

OS: Win10
Browser: Firefox 114.0.2 (64-bit)

Hey! we just published 0.43.1 should work now

Just upgraded. TY :)


[WIP] Support for np.interp

Support for np.interp (first 3 arguments)
Initial commit for CI.

Codecov Report

Merging #3604 into master will decrease coverage by <.01%.
The diff coverage is 79.38%.

@@            Coverage Diff             @@
##           master    #3604      +/-   ##
==========================================
- Coverage   80.68%   80.68%   -0.01%
==========================================
  Files         393      393
  Lines       80485    80616     +131
  Branches     9164     9178      +14
==========================================
+ Hits        64942    65046     +104
- Misses      14127    14153      +26
- Partials     1416     1417       +1

I suspect the CI failure might be unrelated.
Leaving this in WIP status for now as I have ambitions to wire up the remaining arguments when I get a mo.

CI is failing due to the time limit on TravisCI.  Apparently, code coverage checks just push it over the time limit.  We have been seeing that on all recent PRs.

@sklam thanks for confirming

OK, so I am less keen on the idea of implementing left, right and period args.
I started with left, but the complexity is pretty insane (usual suspects: can be None; type consistency; array-like but not really as it won't take a tuple etc etc).
Therefore, I am un-WIP-ing the PR and would ask for it to be reviewed in its current state when you have a mo.

I found an inconsistency versus numpy - working on it...


RUM-7216: Add Slider semantics node mapper

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
Attention: Patch coverage is 25.49020% with 38 lines in your changes missing coverage. Please review.

Project coverage is 70.30%. Comparing base (41f157b) to head (bbc6703).
Report is 10 commits behind head on develop.

Files with missing lines
Patch %
Lines

...nal/mappers/semantics/SliderSemanticsNodeMapper.kt
15.91%
37 Missing :warning:

...ionreplay/compose/internal/utils/SemanticsUtils.kt
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2459      +/-   ##
===========================================
- Coverage    70.37%   70.30%   -0.07%
===========================================
  Files          780      781       +1
  Lines        28850    28901      +51
  Branches      4835     4839       +4
===========================================
+ Hits         20301    20317      +16
- Misses        7205     7233      +28
- Partials      1344     1351       +7

Files with missing lines
Coverage Δ

...ernal/mappers/semantics/RootSemanticsNodeMapper.kt
76.74% <100.00%> (+1.74%)
:arrow_up:

...ionreplay/compose/internal/utils/SemanticsUtils.kt
75.20% <0.00%> (-0.61%)
:arrow_down:

...nal/mappers/semantics/SliderSemanticsNodeMapper.kt
15.91% <15.91%> (ø)

... and 25 files with indirect coverage changes


Questions on switching other models with your sampling method

Dear Authors,
Thank you so much for your great work, as far as I am concerned, you only tried two types of pre-trained diffusion models in your work. Could you please shed some lights on ways to use other pretrained diffusion models (SD-v2.1 etc.) with your sampling method?
Thank you again!

Hi @TianwenZhou ,
Thank you for your interest. Actually, most of the previous work including DiffPIR, DPS, and DDRM, rely on diffusion UNets in image space (as image denoisers).
To release the generative power of large diffusion models such as SD, I recommend you have a look at https://arxiv.org/abs/2310.01110 & https://arxiv.org/abs/2311.15658 from @HJ-harry

Thank you for your letter, I will reply ASAP.BWTianwen Zhou

Dear @yuanzhi-zhu ,
Thank you so much for your prompt reply, it is really helpful for me! As I was browsing through the pretrained diffusion models, I only found unconditional model trained on 256x256 ImageNet, while models with other resolutions(e.g. 512x512) are conditional. Given the fact that training such a huge model from scratch is unrealistic, I wonder if you could kindly offer the method to load pretrained conditional 512x512 diffusion model to your sampling process.

unfortunately, to the best of my knowledge, there is no 512*512 image space diffusion model open-sourced.

So are there any ways that I can use a class-conditional 512x512 image space diffusion model with your sampling strategy?

sure, assume that you know the class of the corrupted image, you can use the guided score as $p(x)$.

Thank you, I'm also curious about ways to test directly on low resolution images, rather than degrading the HR image following the given degradation process

Thank you so much for your kind help and guidance! I'll definitely test these out

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here. Hurry, applications close on March 1, 2024.
Job Description:

Position: Senior Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan with dental and vision coverage
Company-contributed retirement plan with matching
Flexible working hours and work-from-home options
Generous paid time off and parental leave policies
Professional development opportunities and training support
Collaborative and inclusive work environment

Requirements:

Minimum of 5 years of experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Extensive experience with frameworks such as React, Angular, or Vue.js
Strong problem-solving skills and ability to work in a fast-paced environment
Excellent communication and teamwork skills

Note: This position is open to candidates from anywhere in the world. Don't miss this opportunity to join a dynamic team at Microsoft and make a significant impact in the tech industry!
RustMan88, @guiluchesi, @lisovskyvlad, @coyg, @yqrm, @zhamppx97, @matthieucan, @shahnawaz6alam, @joseph-hurtado, @nestorPons, @ferasassaf, @kushagra765, @Revlin, @guoxiaoqi, @ProgramForFun, @cjmdaixi, @uldissturms, @alexander480, @atmshop, @wingyiu

@TianwenZhou  Hi, tianwenZhou, have you solved this problem? I also want to try to use SD1.5 for inpainting task

@GONGJIA0208 Hi, are you willing to try conditional diffusion models with other resolutions?


fix column prune should retain at least one column for union all  (#41613)

cherry-pick #41613 to branch-3.0

run p0

run performance


storage: Add new implementation of lock waiting queue

What is changed and how it works?

Issue Number: Ref #13298
What's Changed:

Implements a new version of lock waiting queue, which is important for supporting resumable acquire_pessimistic_lock requests. (ref #13298)

* Make `storage::lock_manager` an directory
* Add new implementation of the lock waiting queue and other related stuff.

* Please refer to the Lock Waiting Queue section of the RFC for why this change is needed.
Some unresolved problems:

Stale entries in the queue: It's possible that the request is cancelled outside the queue (may be cancelled due to timeout in WaiterManager, which will be implemented in https://github.com/tikv/tikv/pull/13447 ). Possible solutions:

Use an atomic counter to count valid entries in the queue of each key. When it becomes 0 but the queue is not empty, remove it from the map.
Periodically iterate the map and remove the stale entries.
Introduce a priority queue that supports efficiently removing (by either introducing third-party library or implementing by ourselves), and remove the entry when it's cancelled outside.

Potential inefficient deadlock detection in some rare cases:

An example: let transaction 10 hold the lock, and the waiting queue be:  [20, 30, 50]
10 Release the lock, 20 is woken up, 30 & 50 is going to be notified after a delay. The queue: [30*, 50*]
Transaction 15 acquired the lock, then a request of transaction 40 arrives and waits for 15, then the queue is: [30*, 40, 50*]
Delayed wake up happened, pops 30 and stops at 40 since it arrives later than the previous waking up, then the queue becomes: [40, 50*]
Here 50 waits for 15, but no additional deadlock detection is performed. If 15 is also waiting for 50, the deadlock won't be found until the 50's lock-waiting request exceeds its timeout.
Possible solutions:

Ignore this case since it won't be frequent, just like we didn't handle the case that waiter won't be woken up if the leader transfers during waiting.
Iterate and rebuild the whole priority queue to get all entries to be woken up.
Use double queues to manage entries that have / haven't been scheduled for delayed wake up respectively.

Lacking of metrics

Related changes
Check List
Tests

Unit test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Release note
None

I always feel that there are several comments in the test case that do not correspond to the code, but it doesn't matter, it will be decided by the actual execution in the end, and there are no other problems.
LGTM.

@MyonKeminta
Are there any benchbot regression test results or local test results to verify the performance impact?

@MyonKeminta Are there any benchbot test results or local test results to verify the performance impact like improvements or regressions for this pr?

I think this PR itself should have no performance impact because the added functions are not used yet. I see https://github.com/tikv/tikv/pull/13447#issuecomment-1250682427 has some benchmark result after using these new functions.

/merge

/merge


Separated builtin signature generation into its own function, and added function for getting VName for builtins. [phab:D1486]

Created by craigbarber at 2017-04-04 20:07:01:

schroederc accepted this revision at 2017-04-04 20:08:15

craigbarber wrote at 2017-04-04 20:08:27:

cleaned up function signature

craigbarber wrote at 2017-04-04 20:25:41:

update merge from master

craigbarber closed via commit 5306a2c85957dadb49abef0a95434a4abdbb32b9 at 2017-04-04 20:27:31


No gradients provided for any variable

tensorflow : 2.0.0-beta1
python : 3.5.2
raise error
ValueError: No gradients provided for any variable: ['dense/kernel:0',

my code blew
from tensorflow import keras
import tensorflow as tf
import os

class Generator(keras.Model):

    def __init__(self, n_f=512, n_k=4):
        super(Generator, self).__init__()
        self.model = keras.models.Sequential([
            keras.layers.Input((100)),
            keras.layers.Dense(3 * 3 * n_f, activation=tf.nn.leaky_relu),
            keras.layers.Reshape((3, 3, n_f)),
            keras.layers.Conv2DTranspose(filters=n_f // 2, kernel_size=3, strides=2, padding='valid'),
            keras.layers.BatchNormalization(),
            keras.layers.LeakyReLU(),
            keras.layers.Conv2DTranspose(filters=n_f // 4, kernel_size=n_k, strides=2, padding='same'),
            keras.layers.BatchNormalization(),
            keras.layers.LeakyReLU(),
            keras.layers.Conv2DTranspose(filters=1, kernel_size=n_k, strides=2, padding='same'),

        ])

    def call(self, inputs, training=None, mask=None):
        return self.model(inputs, training=training, mask=mask)

class Discriminator(keras.Model):

    def __init__(self, n_f=512, n_k=4):
        super(Discriminator, self).__init__()

        self.model = keras.models.Sequential([
            keras.layers.Input((28, 28, 1)),
            keras.layers.Conv2D(filters=n_f, kernel_size=n_k, strides=2, padding='same', activation=tf.nn.leaky_relu),
            keras.layers.Conv2D(filters=n_f * 2, kernel_size=n_k, strides=2, padding='same'),
            keras.layers.BatchNormalization(),
            keras.layers.Conv2D(filters=n_f * 4, kernel_size=n_k, strides=2, padding='same'),
            keras.layers.BatchNormalization(),
            keras.layers.Flatten(),
            keras.layers.Dense(1),
        ])

    def call(self, inputs, training=None, mask=None):
        return self.model(inputs, training=training, mask=mask)

class GAN():

    def __init__(self, generator, discriminator, lr=1e-3):
        self.generator = generator
        self.discriminator = discriminator

        self.d_optimizer = keras.optimizers.Adam(learning_rate=lr, beta_1=0.5)
        self.g_optimizer = keras.optimizers.Adam(learning_rate=lr, beta_1=0.5)

    def _get_loss(self, logits, label):
        return tf.losses.mean_squared_error(logits, label)

    def _gan_loss_fn(self, generator, discriminator, input_noise, real_image):
        fake_image = generator(input_noise)
        real_logits = discriminator(fake_image)
        fake_logits = discriminator(real_image)

        g_loss = self._get_loss(fake_logits, tf.ones_like(fake_logits, dtype=tf.int32))
        d_loss = self._get_loss(real_logits, tf.ones_like(real_logits, dtype=tf.int32)) + \
                 self._get_loss(fake_logits, tf.zeros_like(fake_logits, dtype=tf.int32))

        return g_loss, d_loss

    def train(self, real_image, input_noise, verbose=False):
        g_loss, d_loss = self._gan_loss_fn(self.generator, self.discriminator, input_noise, real_image)
        with tf.GradientTape() as tape:
            grads = tape.gradient(g_loss, self.generator.trainable_variables)
            self.g_optimizer.apply_gradients(zip(grads, self.generator.trainable_variables))

        with tf.GradientTape() as tape:
            grads = tape.gradient(d_loss, self.discriminator.trainable_variables)
            self.d_optimizer.apply_gradients(zip(grads, self.discriminator.trainable_variables))

        if verbose:
            print("d_loss : %.5f , g_loss : %.5f" % (d_loss, g_loss))

    def save_weights(self, path):
        os.makedirs(path, exist_ok=True)
        self.generator.save_weights(os.path.join(path, 'generator'))
        self.discriminator.save_weights(os.path.join(path, 'discriminator'))

    def restore_weights(self, path):
        self.generator.load_weights(os.path.join(path, 'generator'))
        self.discriminator.load_weights(os.path.join(path, 'discriminator'))

if __name__ == '__main__':
    import numpy as np
    import sys
    print(sys.version)
    print(tf.__version__)

    gan = GAN(Generator(), Discriminator())
    batch_size = 64
    gan.train(np.random.rand(batch_size, 28, 28, 1).astype(np.float32),
              np.random.rand(batch_size, 100).astype(np.float32))

Hi,
Fixing the GradientTape
The main issue is that you are using the GradientTape wrong. The schematic correct way is the following:
# Set up the tape.
with tf.GradientTape() as tape:
    # Transform the data and compute the loss with the tape recording gradients.
    prediction = model(data)
    loss = loss(prediction)
# Now, gather the gradients and apply them.
grads = tape.gradient(loss, model.trainable_variables)
optimizer.apply_gradients(zip(grads, model.trainable_variables))

Additionally, in your case, you compute two intertwined losses and thus need to compute two sets of gradients (as you correctly sketched it). By default, GradientTape is non-persistent, i.e. it can only compute gradients once, after which it releases its resource variables. Thus, you need to explicitly make the tape consistent (tf.GradientTape(persistent=True)), and will probably want to explicitly del tape once gradients are computed to ensure the now-persistent variables are released upon the tape's garbage collection (avoiding a potential memory leak). See the docstring of tf.GradientTape for details (s/o to the developers; tensorflow is not always well-documented, but this class's docstring is really great!).
So, in your case:
with tf.GradientTape(persistent=True) as tape:
    # Compute the losses (note: I changed the method's signature; you
    # can use self.generator and self.discriminator in _gan_loss_fn)
    g_loss, d_loss = self._gan_loss_fn(input_noise, real_image)
# Compute and apply the first set of gradients.
g_grads = tape.gradient(g_loss, self.generator.trainable_variables)
self.g_optimizer.apply_gradients(zip(g_grads, self.generator.trainable_variables))
# Compute and apply the second set of gradients.
d_grads = tape.gradient(d_loss, self.discriminator.trainable_variables)
self.d_optimizer.apply_gradients(zip(d_grads, self.discriminator.trainable_variables))
# Delete the tape to ensure proper garbage collection and variables release.
del tape

Additional Issues
Now, if you do this, it still will not work, because there is another issue in your code: you compute losses with tf.int32 reference labels, which is incorrect if you really want to use an MSE loss. Change the dtype to tf.float32 and things will run. That being said, I am pretty sure this is not the correct loss function, and you actually want to use a binary cross-entropy loss (in which case the true labels should be of dtype tf.int32). I also think you interchanged the true_logits and fake_logits tensors...
I leave it to you to see which loss you need to set up, etc. At any rate, the losses tensors should be of dtype float for the tape to compute gradients.
Finally, this is a detail, but your losses print-out will not work either, as the losses are arrays of values; but you can simply tf.reduce_mean them upon printing (and preferably use tf.print rather than print) and everything should be fine :)

I'm very much obliged to you. @pandrey-fr ，but still the same problem.
from tensorflow import keras
import tensorflow as tf
import os

class Generator(keras.Model):

    def __init__(self, n_f=512, n_k=4):
        super(Generator, self).__init__()
        self.model = keras.models.Sequential([
            keras.layers.Input((100)),
            keras.layers.Dense(3 * 3 * n_f, activation=tf.nn.leaky_relu),
            keras.layers.Reshape((3, 3, n_f)),
            keras.layers.Conv2DTranspose(filters=n_f // 2, kernel_size=3, strides=2, padding='valid'),
            keras.layers.BatchNormalization(),
            keras.layers.LeakyReLU(),
            keras.layers.Conv2DTranspose(filters=n_f // 4, kernel_size=n_k, strides=2, padding='same'),
            keras.layers.BatchNormalization(),
            keras.layers.LeakyReLU(),
            keras.layers.Conv2DTranspose(filters=1, kernel_size=n_k, strides=2, padding='same'),

        ])

    def call(self, inputs, training=None, mask=None):
        return self.model(inputs, training=training, mask=mask)

class Discriminator(keras.Model):

    def __init__(self, n_f=512, n_k=4):
        super(Discriminator, self).__init__()

        self.model = keras.models.Sequential([
            keras.layers.Input((28, 28, 1)),
            keras.layers.Conv2D(filters=n_f, kernel_size=n_k, strides=2, padding='same', activation=tf.nn.leaky_relu),
            keras.layers.Conv2D(filters=n_f * 2, kernel_size=n_k, strides=2, padding='same'),
            keras.layers.BatchNormalization(),
            keras.layers.Conv2D(filters=n_f * 4, kernel_size=n_k, strides=2, padding='same'),
            keras.layers.BatchNormalization(),
            keras.layers.Flatten(),
            keras.layers.Dense(1),
        ])

    def call(self, inputs, training=None, mask=None):
        return self.model(inputs, training=training, mask=mask)

class GAN():

    def __init__(self, generator, discriminator, lr=1e-3):
        self.generator = generator
        self.discriminator = discriminator

        self.d_optimizer = keras.optimizers.Adam(learning_rate=lr, beta_1=0.5)
        self.g_optimizer = keras.optimizers.Adam(learning_rate=lr, beta_1=0.5)

    def _get_loss(self, logits, label):
        return tf.losses.mean_squared_error(logits, label)

    def _gan_loss_fn(self, generator, discriminator, input_noise, real_image):
        fake_image = generator(input_noise)
        real_logits = discriminator(fake_image)
        fake_logits = discriminator(real_image)

        g_loss = self._get_loss(fake_logits, tf.ones_like(fake_logits, dtype=tf.float32))
        d_loss = self._get_loss(real_logits, tf.ones_like(real_logits, dtype=tf.float32)) + \
                 self._get_loss(fake_logits, tf.zeros_like(fake_logits, dtype=tf.float32))

        return g_loss, d_loss

    def train(self, real_image, input_noise, verbose=False):
        with tf.GradientTape(persistent=True) as tape:
            g_loss, d_loss = self._gan_loss_fn(self.generator, self.discriminator, input_noise, real_image)

        grads = tape.gradient(g_loss, self.generator.trainable_variables)
        self.g_optimizer.apply_gradients(zip(grads, self.generator.trainable_variables))

        grads = tape.gradient(d_loss, self.discriminator.trainable_variables)
        self.d_optimizer.apply_gradients(zip(grads, self.discriminator.trainable_variables))

        if verbose:
            tf.print("d_loss : %.5f , g_loss : %.5f" % (d_loss, g_loss))

    def save_weights(self, path):
        os.makedirs(path, exist_ok=True)
        self.generator.save_weights(os.path.join(path, 'generator'))
        self.discriminator.save_weights(os.path.join(path, 'discriminator'))

    def restore_weights(self, path):
        self.generator.load_weights(os.path.join(path, 'generator'))
        self.discriminator.load_weights(os.path.join(path, 'discriminator'))

if __name__ == '__main__':
    import numpy as np
    import sys

    print(sys.version)
    print(tf.__version__)

    gan = GAN(Generator(), Discriminator())
    batch_size = 64
    gan.train(np.random.rand(batch_size, 28, 28, 1).astype(np.float32),
              np.random.rand(batch_size, 100).astype(np.float32))

That is due to the aforementioned inversion between the fake_logits and real_logits.
It should be:
fake_logits = discriminator(fake_image)
real_logits = discriminator(real_image)

Otherwise the g_loss is computed on the real image's processing by the discriminator ; since the generator has, in this erroneous case, nothing to do with the loss, it is just logical that no gradients could be computed.

@LieJiang Please make changes as suggested by  @pandrey-fr as I did the same and it works. Closing this issue as it has been resolved.


Custom image interpolation

Is it possible to use a custom image (not generated) as a starting and ending points to interpolate between them?

Nope they have to be generated. Check out the blog post at aiart.dev to understand more about what's going on here, which should clarify why it's not possible to use your own non generated images

There are models capable of this kind of thing, I think, but this is definitely not one of them!


Need a way to generate diagrams from Akka.NET

I think I have some idea for a global DotNet Tool and Nuget package that does that.
I've created a GitHub repository (https://github.com/Stelzi79/Akka-Diagrams) because this tool is a bit outside of the scope of a Chatbot system and would warrant it's own project.
Everybody is invited to help if they so desire!

Oh yes, if a tool needed to be built it would be bigger than the bot project.
My assumption was that there was a tool already available that provided that functionality.

Well my Google Foo didn't yielded any usable project so I assumed that there needs one to be created.


Add ability to delete storage pools

Separate from deleting volumes. Mentioned in #2893

+1

Came here to search for this before I opened it myself +1

+1

I'll +1 this as well.  We really need to be able to clean up unused pools.

+1

Note to self, when this is implemented, make sure to let @vincent99 know.

Note to @vincent99, @cjellick will tell you when you need to implement this :)

Is there some sort of guidance on a brute force method that's safe (i.e. emptying the storage_pool and storage_pool_host_map tables in the db or something?)
This is apparently preventing me from implementing convoy on my production network (and I really need it for several services :)

@guruvan personally i have tried to manually purge all the entities in both tables (if they were not already purged), and it worked. However as I was purging ALL, it was a straightforward process. YMMV
purged existing entities in storage_pool

and purged storage_pool_host_map directly

Here is some guidance on how to "manually" delete storage pools. I've tested this locally but not exhaustively. So, this isn't officially supported.
Storage pools can be deleted via the API. If you go to the storage pool API for your project (ie http://localhost:8080/v1/projects/1a5/storagepools/), you should be able to find your convoy-nfs or convoy-gluste storage pool. Note that in the above URL, the 1a5 is my environment id, it wont be the project id for everyone. Go to the API tab to get the proper root to your environment's API endpoint.
Once you've found it, you should be able to deactivate, then remove, then purge (reloading after each action to get the new possible actions).
That will delete the storage pool in the rancher API. But that doesn't clean up everything. You would also need to go to the Applications > Systems tab and delete the convoy-gluster (or whatever you called it) service.
That delete's the convoy containers but that still doesn't clean up everything. You need to go to the hosts and remove /etc/docker/plugins/convoy-gluster.spec file (name may be different) and then restart the docker daemon.
If you do all that, you'll have cleaned up everything you can. However, if you still have containers that are marked as using a convoy-gluster container, the docker daemon will slow down when it tries to manipulate those containers because it will retry for up to 30ish seconds to talk to the now non-existent driver.
Again, this is not a bulletproof officially supported clean up and you might still run into weird edge case issues, but hopefully it helps you out.

@cjellick thanks for this, it's very helpful for this and other issues I'm seeing as well! :) makes the whole process much more clear to me. I presume one should also remove the /var/lib/rancher/convoy/convoy-gluster-UUID as well. I presume I'll run into a few edge cases after having messed around manually in the db to get things running. Thanks to @picoCreator 's suggestion I've manually done some similar in the db, and had success. Using the API seems a more straightforward & safer process.
The last point is enlightening - I'll keep my eye out for this.

@ibuildthecloud this should be getting addressed as part of your volume driver/storage pool refactor?

Closing after talking to @cjellick
when you delete a storage infrastructure stack, the storage driver is deleted and when the driver is deleted, the storage pool IS deleted too.

Sometimes being subscribed to an issue for a year can pay off

Had the same issue after upgrade to Rancher v1.2.2. Convoy storage pool was removed, and replaced with rancher-nfs instead. But it still showed in UI.
Here is how to remove storage pool in v2-beta API:

Go to http://rancher-server:8080/v2-beta/projects/1a5/storagepools/
Search (Ctrl-F) for storage pool to remove (convoy-nfs)
Click on self link (e.g.: http://rancher-server:8080/v2-beta/projects/1a5/storagepools/1sp3)
Click deactivate, then remove, then purge (reloading after each action to get the new possible actions).

See @cjellick 's note for info about  environment IDs.

@beatcracker Thanks! Worked like a charm!

@beatcracker @fabiorauber you may also have to remove the storage plugin from the hosts, restarting Docker Engine around that. At least that is the case in Rancher v1.1. Something like:
systemctl stop docker.service
rm /etc/docker/plugins/<storage-pool-name>.spec
systemctl start docker.service

@zambon Thanks, docker was indeed complaining about a missing convoy plugin.


bug: openresty >= <IP_ADDRESS> is needed by apisix-1.2-0.noarch but openresty is <IP_ADDRESS>

Issue description
openresty >= <IP_ADDRESS> is needed by apisix-1.2-0.noarch but openresty is <IP_ADDRESS>

[root@VM_1_88_centos ~]# rpm -ivh --force apisix-1.2-0.el7.noarch.rpm
error: Failed dependencies:
        openresty >= <IP_ADDRESS> is needed by apisix-1.2-0.noarch
[root@VM_1_88_centos ~]# openresty -v
nginx version: openresty/<IP_ADDRESS>

Environment

apisix version (cmd: apisix version):
OS:

Minimal test code / Steps to reproduce the issue

What's the actual result? (including assertion message & call stack if applicable)
What's the expected result?

Remove dependency check when installing rpm
rpm -ivh apisix-1.2-0.el7.noarch.rpm --nodeps


Text appears under the toast box in modal pages on iOS 13

Hi,
Long time no see !!! after 4 years I'm using this perfect library & now I have this issue.
Now on iOS 13 when we toast, Text appears under the toast box in modal pages...

Here is a temporary solution for it.
on TTToast.swift
replace this line:
 msgLabel.center = parent.convert(center, to: self)
with this:
   if #available(iOS 13.0, *) {             let labelCenter = parent.convert(center, to: self)             msgLabel.center.x = labelCenter.x             msgLabel.center.y = bounds.midY         } else {             msgLabel.center = parent.convert(center, to: self)         }

Thank you for using this library! 😄
I didn't know the problem, I'll check later. And thank you for your solution. 👍


QueryBatcher hangs in awaitCompletion()

startJob() can be called multiple times and it is seen that awaitCompletion() hangs when startJob() is called after the job is stopped. Stack trace is attached
stack_trace.txt
	@Test
	public void jobReportStopJob() throws Exception{

		QueryBatcher batcher = dmManager.newQueryBatcher(new StructuredQueryBuilder().collection("XmlTransform")).withBatchSize(20).withThreadCount(20);

		AtomicInteger batchCount = new AtomicInteger(0);
		AtomicInteger successCount = new AtomicInteger(0);

		Set uris = Collections.synchronizedSet(new HashSet());

		batcher.onUrisReady(batch->{
			uris.addAll(Arrays.asList(batch.getItems()));
			batchCount.incrementAndGet();
			if(dmManager.getJobReport(queryTicket).getSuccessEventsCount() > 40){
				dmManager.stopJob(queryTicket);
			}

		});
		batcher.onQueryFailure(throwable -> throwable.printStackTrace());

		queryTicket = dmManager.startJob( batcher );
		batcher.awaitCompletion(Long.MAX_VALUE, TimeUnit.DAYS);

		queryTicket = dmManager.startJob( batcher );
		batcher.awaitCompletion(Long.MAX_VALUE, TimeUnit.DAYS);

		System.out.println("Success event: "+dmManager.getJobReport(queryTicket).getSuccessEventsCount());
		System.out.println("Success batch: "+dmManager.getJobReport(queryTicket).getSuccessBatchesCount());
		System.out.println("Failure event: "+dmManager.getJobReport(queryTicket).getFailureEventsCount());
		System.out.println("Failure batch: "+dmManager.getJobReport(queryTicket).getFailureBatchesCount());

		Assert.assertTrue(successCount.get() < 200);
		Assert.assertTrue(batchCount.get() == dmManager.getJobReport(queryTicket).getSuccessBatchesCount());

	}

If startJob is called twice, a warning is logged


Higher level DSL

I was thinking something like Logstash might be kind of cool, pipe first-party and third-party functions together, throw Kinesis inbetween, etc.

Oh... I like the concept: making the event bus (like Kinesis) agnostic to the function.


Transport maybe is not a EnhancedInstance.

Fix <Transport maybe is not a EnhancedInstance. For example, a TFrameTransport encapsulating TSocket.>

[ ] Add a unit test to verify that the fix works.
[x] Explain briefly why the bug exists and how to fix it.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

You should use instanceof rather than a catch.

Fix <Transport maybe is not a EnhancedInstance. For example, a TFrameTransport encapsulating TSocket.>

So, in TFrameTransport case, UNKNOWN is good enough?

Are you going to explain what you are proposing to change?

This PR has no update in half month. I have to close this.
If you want to continue on this, let's know.

Fix <Transport maybe is not a EnhancedInstance. For example, a TFrameTransport encapsulating TSocket.>

So, in TFrameTransport case, UNKNOWN is good enough?

The best way is to enumerate all the situations and take out the correct peers for these situations. However, I'm not sure of any other situations. TFrameTransport is just one of the ones I encounter in my own use.

If you still want to finish this PR, please follow the comments of the review.
About the UNKNOWN, I just want to make sure which field is going to be set as UNKNOW and how it looks like on the UI?

If you still want to finish this PR, please follow the comments of the review.

1、Which field is going to be set as UNKNOW？
A：Peer of exit span, likes below.

2、How it looks like on the UI?
A：I think it should be UNKNOWN in the peer, but I haven't tested this.

Peer as UNKNOWN? Then the node of topology would be unknown, this seems not right.
As on the client side, you should be able to get remote service logic name, or IP/DNS address, right?


Create samples (also to be used as E2E tests)

Plan:
Create a new top level dir samples and each sub-dir is an example integrating one or more API clients.
All of the samples will have a test which runs full integration (end to end) tests, and which can be automated en-mass. They will require ENV variables for private credentials and settings.
There are already elixir samples externally:
https://github.com/GoogleCloudPlatform/elixir-samples
We will use those to prioritize and seed this directory
Early Candidate:

Storage #117 - see pr #1233
Firestore #111 #1231
StackDriver #949
Spanner https://github.com/GoogleCloudPlatform/elixir-samples/issues/21
Vision ML

@dazuma Is this still the plan for this repo or should we close this issue?


Swallow underlying exception for config parsing

This will address the issue posed in aws/awscli#6696. Rather than modifying the existing exception flow, we'll use the from None syntax to swallow the underlying stacktrace for config parsing exceptions. This will prevent unnecessary information from being logged by default. Users who want this for debugging purposes will find it's still available under .kwargs['error'].

Codecov Report

Merging #2607 (8fa683b) into develop (6c6ca96) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #2607      +/-   ##
===========================================
- Coverage    95.41%   95.36%   -0.06%
===========================================
  Files           59       59
  Lines        11908    11907       -1
===========================================
- Hits         11362    11355       -7
- Misses         546      552       +6

Impacted Files
Coverage Δ

botocore/configloader.py
100.00% <100.00%> (ø)

botocore/compat.py
72.13% <0.00%> (-1.10%)
:arrow_down:

botocore/httpsession.py
90.94% <0.00%> (-0.42%)
:arrow_down:

botocore/hooks.py
95.27% <0.00%> (-0.40%)
:arrow_down:

botocore/utils.py
95.50% <0.00%> (-0.16%)
:arrow_down:

botocore/parsers.py
99.45% <0.00%> (-0.01%)
:arrow_down:

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 6c6ca96...8fa683b. Read the comment docs.


Use 'local' backend for Terraform modules by default

Benefit/Result/Outcome
A user could deploy Terraform from the repo without setting up dependencies.
In a future where MLZ doesn't provide BASH scripting to stand-up a Storage Account and Container for storing Terraform state, a local backend would instead allow the Terraform modules to be deployed independently.
Description
https://www.terraform.io/docs/language/settings/backends/local.html
Acceptance Criteria

Set the Terraform modules to local backend (and allowing the default path to be used)
Document (point to documentation) on how to use different backends
Document (point to documentation) on how to use the azurerm backend

Related to #354


allow use of ssl

No doubt it's a good PR. Thanks @croemmich :)

Release v0.1.4 including this patch. Thanks!

Thanks!


Print parser structure, part 1

ParserOps and ParserSyntax is already known concept from very first version of the lib. I reintroduced them into the parsing environment to allow parser combinators to be defined in the abstract context of P[_] (the Grammar class in the example).
The documentation generating code does not handle recursive parser definitions yet. So I kept most of the new code in test for now. It will be moved once I solve the recursion.

Codec changes are just indentation.
The new example in tests has a lot of copied code just to keep different examples independent.

Codecov Report

Merging #40 into master will increase coverage by 0.03%.
The diff coverage is 96.42%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage    99.3%   99.33%   +0.03%
==========================================
  Files           5        5
  Lines         144      151       +7
  Branches        2        2
==========================================
+ Hits          143      150       +7
  Misses          1        1

Impacted Files
Coverage Δ

src/main/scala/scalaz/parsers/Parsing.scala
99.01% <96.42%> (+0.07%)
:arrow_up:

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 7190870...8e2bf8a. Read the comment docs.


Inference Servers started with GPU on Windows cannot restart

Bug description
When we start an inference server with GPU (nvidia) on windows, we cannot restart the inference server, it crashes instantly.
To be able to access the GPU on windows without having the Nvidia Container Device Interface we are creating symlinks in the container when it start
https://github.com/containers/podman-desktop-extension-ai-lab/blob/ea02986c994f9eb60e587bcf7a35a309cde6c315/packages/backend/src/workers/provider/LlamaCppPython.ts#L124-L127
This cause a problem on restart as the symlinks already exists
Operating system
Windows 11
Installation Method
Other
Version
next (development version)
Steps to reproduce

asset experimental GPU enabled
start an inference server
assert nvidia-smi output the gpu when running inside the container
stop the container
assert stopped
start container
assert error logs

Relevant log output
INFO:     <IP_ADDRESS>:42542 - "GET /docs HTTP/1.1" 200 OK
ln: failed to create symbolic link '/usr/lib64/libcudadebugger.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libcuda.so': File exists
ln: failed to create symbolic link '/usr/lib64/libcuda.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libcuda.so.1.1': File exists
ln: failed to create symbolic link '/usr/lib64/libd3d12core.so': File exists
ln: failed to create symbolic link '/usr/lib64/libd3d12.so': File exists
ln: failed to create symbolic link '/usr/lib64/libdxcore.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvcuvid.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvcuvid.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvdxdlkernels.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-encode.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-encode.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-ml.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-opticalflow.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-opticalflow.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvoptix_loader.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvoptix.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvwgf2umx.so': File exists
ln: failed to create symbolic link '/usr/lib64/nvidia-smi': File exists
ln: failed to create symbolic link '/usr/lib64/libcudadebugger.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libcuda.so': File exists
ln: failed to create symbolic link '/usr/lib64/libcuda.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libcuda.so.1.1': File exists
ln: failed to create symbolic link '/usr/lib64/libd3d12core.so': File exists
ln: failed to create symbolic link '/usr/lib64/libd3d12.so': File exists
ln: failed to create symbolic link '/usr/lib64/libdxcore.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvcuvid.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvcuvid.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvdxdlkernels.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-encode.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-encode.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-ml.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-opticalflow.so': File exists
ln: failed to create symbolic link '/usr/lib64/libnvidia-opticalflow.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvoptix_loader.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvoptix.so.1': File exists
ln: failed to create symbolic link '/usr/lib64/libnvwgf2umx.so': File exists
ln: failed to create symbolic link '/usr/lib64/nvidia-smi': File exists

Additional context
No response

An easy fix would be to replace ln -s with ln -sfn.
What do you think @benoitf ?
Should I add the milestone 1.3 ?
Should I include this in this sprint @jeffmaury ?

An easy fix would be to replace ln -s with ln -sfn.
What do you think @benoitf ?
Should I add the milestone 1.3 ? Should I include this in this sprint @jeffmaury ?

Next sprint


Check the deploying results when deploying integration tests

Is your feature request related to a problem? Please describe.
As job<PHONE_NUMBER> indicates, we did not check the deployment result but only check whether the deploying command succeeded which gives us a false positive result
Describe the solution you'd like
Check whether the results of deploying command returns error
Describe alternatives you've considered

Additional context

Already done. Closing this


Fix issue with missing container name and namespace

Change Overview
When creating pods with sidecars, we have an issue with writing files or streaming logs from these pods, because such operations requires container name.
This PR fixes this issue.
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[x] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues

fixes #issue-number

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

Hi @chaitalisg,
Like mentioned in the PR description in case of multiple containers in a pod when we try to read logs there was a problem. But how do we know that it's the first container that we want to read the logs from?
cc @pavannd1

Good question. It is a calculated risk. When pods are created with sidecars, sidecar containers are placed after main container. So, I am using the first container. Let me know if you know a way to make this better!

When pods are created with sidecars, sidecar containers are placed after main container.

I don't think this is the case always.
I think we should pass the container name explicitly instead. For example if we have a utility to exec into a pod it should expect a container name, similarly in this case since we are trying to get logs of a container we should explicitly expect the container name.
This can default to the container name that we set in podOptions while creating the pod because the is the that we would be interested in.

I have raised this PR that fixes above problem
https://github.com/kanisterio/kanister/pull/2310#event-10306105151


Courseinformation

Branch that implements a way to enable the user to click on course information and be redirected to the corrospondig course-information git page.
Added a new coloumn in the repository table for html address, since url addresses will most likely not change that often, and will reduce the api calls to the git-providers.
Also improved the testing coverage for the gormDB implementation.

Coverage increased (+0.5%) to 38.575% when pulling b5837499909218226a00394d6e3235a04a9a7c8d on courseinformation into 4b9ccc3d009fc2540ee153d9f70aefb5f3483e1b on master.


d2i_CV_RESPONSE fails for CMS encapsulated SCVP-CV responses

NOTE: This issue was detected in OpenSSL 3.0.5 and still exists in 3.0.10.
The following code fails at d2i_CV_RESPONSE due to openssl raising error for implicit tagging.
const unsigned char *pData = NULL;
ASN1_OCTET_STRING *pOctetStr = NULL;
pData = (const unsigned char) buffer;
CMS_ContentInfo * cmsResponseContentInfo = d2i_CMS_ContentInfo(NULL, &pData, length);
if (!cmsResponseContentInfo) return retval;
if (CMS_get0_eContentType(cmsResponseContentInfo) != OBJ_nid2obj(NID_id_ct_scvp_certValResponse)) {
CMS_ContentInfo_free(pCmsResponse->cmsResponseContentInfo);
return retval;
}
pOctetStr = CMS_get0_content(cmsResponseContentInfo);
if (pOctetStr == NULL || *pOctetStr ==NULL) return retval;
pData = (*pOctetStr)->data;
//The following line of code fails at the d2i and returns NULL.
CV_RESPONSE * cvResponse = d2i_CV_RESPONSE(NULL, &pData, (*pOctetStr)->length);
The cause of this failure was identified to be the following lines of code in crypto/asn1_tasn_dec.c
In asn1_item_embed_d2i function, for
case ASN1_ITYPE_CHOICE:
/*
* It never makes sense for CHOICE types to have implicit tagging, so
* if tag != -1, then this looks like an error in the template.
*/
if (tag != -1) {
ERR_raise(ERR_LIB_ASN1, ASN1_R_BAD_TEMPLATE);
goto err;
}
This worked in OpenSSL 1.0.2 and the check for tag != -1 wasn't present there.
In our case, we have many structures within the SCVP CV RESPONSE which have implicit tagging. Example:
ASN1_CHOICE(CERT_REFERENCE) = {
ASN1_IMP(CERT_REFERENCE, d.pkc, PKC_REFERENCE, 0),
ASN1_IMP(CERT_REFERENCE, d.ac, AC_REFERENCE, 1)
} ASN1_CHOICE_END(CERT_REFERENCE)
ASN1_CHOICE(PKC_REFERENCE) = {
ASN1_IMP(PKC_REFERENCE, d.cert, X509, 0),
ASN1_IMP(PKC_REFERENCE, d.pkcRef, SCVP_CERT_ID, 1)
} ASN1_CHOICE_END(PKC_REFERENCE)
ASN1_CHOICE(AC_REFERENCE) = {
ASN1_IMP(AC_REFERENCE, d.attrCert, ATTR_CERT, 2),
ASN1_IMP(AC_REFERENCE, d.acRef, SCVP_CERT_ID, 3)
} ASN1_CHOICE_END(AC_REFERENCE)
The converse is also true in the i2d_CV_RESPONSE failing due to the tag check crypto/asn1/tasn_enc.c
In ASN1_item_ex_i2d function for the case ASN1_TYPE_CHOICE

Which one of the choices is chosen if the tagging is implicit? How should the decoder know?

The CERT_REFERENCE structure itself is defined as follows:
typedef struct CERT_REFERENCE_st {
int type;
union {
PKC_REFERENCE pkc;    / tag == 0 */
AC_REFERENCE ac;    / tag == 1 */
} d;
} CERT_REFERENCE;
So, the type holds which type the CERT_REFERENCE contains and we use it accordingly.

Does anyone know a workaround or is there a workaround for this?

I got my code to work (archiving the same successful behavior back in the days) by commenting out all the if (tag != -1) from both the encoder and decoder of the tasn.

Essentially, this revert back to the old behavior around version 1.1.0f  (more or less around the time that the tag != -1 was added?)
What was the reason the check was introduced and what benefit did it bring? Am I actually  doing the right thing?

I do not think simply removing the checks would be the right patch. These checks were added for purpose and are related to CVE-2020-1971.
See the commit messages:
https://github.com/openssl/openssl/commit/41d62636fd996c031c0c7cef746476278583dc9e
https://github.com/openssl/openssl/commit/1ecc76f6746cefd502c7e9000bdfa4e5d7911386
https://github.com/openssl/openssl/commit/f960d81215ebf3f65e03d4d5d857fb9b666d6920
https://github.com/openssl/openssl/commit/aa0ad2011d3e7ad8a611da274ef7d9c7706e289b

Hmm... looking at RFC 5055, should the definition of CertReference be instead:
ASN1_CHOICE(CERT_REFERENCE) = {
ASN1_SIMPLE(CERT_REFERENCE, d.pkc, PKC_REFERENCE),
ASN1_SIMPLE(CERT_REFERENCE, d.ac, AC_REFERENCE)
} ASN1_CHOICE_END(CERT_REFERENCE)

I do not think simply removing the checks would be the right patch. These checks were added for purpose and are related to CVE-2020-1971.
See the commit messages:
41d6263 1ecc76f f960d81 aa0ad20

Well, removing those checks (shown in my reply) allowed me to parse an SCVP response generated back in 2016 (which used either the 1.0.2 or 1.1.0f that was the on. Otherwise the tag != -1 (which would always be true because it is being passed by the upstream) will always prevent the flow from continuing like the old way.

That unfortunately does not mean that removing those checks is the correct fix. I do not think we are going to revert these commits.
It also can mean that the generated SCVP responses are not matching the standard.

Hum.. quick question @t8m, what is the correct (or in general, just how) to register an explicit TAG with the ASN1? If I am trying to decode and parse the ASN1 structure, is there a way to either:

Tell openssl to bypass the tag check because we instructed it to do so or
Manipulate the CMS structure before passing it over to openssl such that the tag would be a different value other than -1?

I do not think there is a simple way to do either. I also believe the encoding you've produced with the above definition with old openssl releases is incorrect.

I do not think there is a simple way to do either. I also believe the encoding you've produced with the above definition with old openssl releases is incorrect.

Yeah, it was incorrect, but we still need to parse the old response; while the old version of openssl before those commits (tag != -1) went in, we were still able to parse it just fine but now lost the ability to do so without any means to configure/accept it.


feat(core): submit UX improvements

RepoStatus now stores lock information
Removed verify_locks calls from Friendshipper in favor of RepoStatus locks
RepoStatus Files now have the lock owner, flags for submit status, and git state (added/ modified/etc)
Friendshipper submit UX has been improved to display if files can be submitted or not, colorizing them and displaying a warning icon if there will be a problem with submission.

Also closes #78


[next beta] Bump dcos-metrics version

High Level Description
This bump allows metrics to attempt to fall back to http after an https call to Mesos state fails (which can happen during disabled->permissive upgrades), and adds task information to the container metrics endpoint in dcos-metrics.
Related Issues

DCOS_OSS-1542 Fall back to HTTP when polling Mesos state in dcos-metrics
DCOS-17813 Include task name as a dimension via dcos-metrics API

Checklist for all PR's

[x] Included a test which will fail if code is reverted but test is not. (DCOS-17813 is fully covered; DCOS_OSS-1542 should have another test added for completeness, but I didn't want to hold this PR up further while we moved things round to accomodate that. There is a ticket to track this here)
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): changelog
[x] Test Results: link to CI job test results for component
[x] Code Coverage (if available): link to code coverage report

@mesosphere-mergebot label ready for review

@amitaekbote and @collinvandyck, would you kindly review this when you get a moment?

@mesosphere-mergebot bump-ee

Sad. This could have been on train 205 but I didn't know that this is ready, was also not pointed to it.

This is currently the only shippable PR against the master branch. Merging this through for beta2 w/o a train.

@mesosphere-mergebot bump-ee


pre-build orion pre-processor

is there a way to pre-build the pre-processor?
when compiling a tweak via github-actions this step seems to take the longest and I already wrote a docker image that comes with theos and swift, I'd like to ideally also have it have orion precompiled

Cache it

I'm aware of that, that does not really answer the question if it can be pre-built via cli invocation

Not super familiar with our Swift tooling but it looks like you should be able to pre-build the preprocessor via the swift-bootstraper.pl script.
Example invocation: https://github.com/theos/theos/blob/16f19fb1da4322513a9ad0283c12f418121773d1/makefiles/instance/rules.mk#L246

Thank you

FWIW invoking swiift-bootstrapper yourself may work for now but it’s an implementation detail so I wouldn’t recommend relying on it. Consider instead creating a new (temporary) Orion tweak, running make, and then deleting the tweak dir. That should be sufficient to cache the preprocessor in a stable manner.

Cache it

I already have this and it does nothing to cache swift preprocessor

Yes, it does, I don’t suggest broken code. Your build obviously has to succeed first in order to cache something

Yes, it does, I don’t suggest broken code. Your build obviously has to succeed first in order to cache something

I have a cache for the entire theos directory, infact my cache is pretty much the same as yours, so I don't know why yours works


Cannot read properties of undefined (reading 'fsPath')

Environment (please complete the following information):

macOS 14.6.1 (23G93)
Extension version v2.1.5

Describe the bug:
I just installed this extension for the first time and tried to run it. This is the error I encountered.
Repro steps:
Command Palette → "Microsoft Edge Tools: Open Browser"

Command 'Microsoft Edge Tools: Open Browser with DevTools' resulted in an error
Cannot read properties of undefined (reading 'fsPath')

Expected behavior:
Runs browser
Additional context:

AB#53838982

I too have this error

Same issue here. M1 Mac running Sequoia. All apps installed with Homebrew.
VS Code
Version: 1.93.1
Commit: 38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40
Date: 2024-09-11T17:20:05.685Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 24.0.0
Microsoft Edge Tools for VS Code
v2.1.6
Microsoft Edge
Version 129.0.2792.52 (Official build) (arm64)
*Extension Settings
"vscode-edge-devtools.browserArgs": []
"vscode-edge-devtools.browserFlavor": "Default",
"vscode-edge-devtools.defaultEntrypoint": "index.html",
"vscode-edge-devtools.defaultUrl": "about:blank"
"vscode-edge-devtools.headless": true
"vscode-edge-devtools.hostname": "localhost"
"vscode-edge-devtools.pathMapping": {
"/": "${workspaceFolder}"
},
"vscode-edge-devtools.port": 9222,
"vscode-edge-devtools.showWorkers": false,
"vscode-edge-devtools.sourceMapPathOverrides": {
"webpack:///./": "${webRoot}/",
"webpack:///src/": "${webRoot}/",
"webpack:///": "",
"webpack:///./~/": "${webRoot}/node_modules/",
"webpack://": "${webRoot}/",
"meteor://💻app/": "${webRoot}/"
},
"vscode-edge-devtools.sourceMaps": true,
"vscode-edge-devtools.timeout": 10000,
"vscode-edge-devtools.useHttps": false,
"vscode-edge-devtools.userDataDir": true,
"vscode-edge-devtools.webhint": true,
"vscode-edge-devtools.webhintInstallNotification": true,
"vscode-edge-devtools.webRoot": "${workspaceFolder}"

Here is the error output. I'm also on Mac M1 and Sequoia.
2024-09-29 21:22:48.183 [error] TypeError: Cannot read properties of undefined (reading 'fsPath')
at Qh (/Users/johannajohansson/.vscode/extensions/ms-edgedevtools.vscode-edge-devtools-2.1.6/out/extension.js:2:1509346)
at /Users/johannajohansson/.vscode/extensions/ms-edgedevtools.vscode-edge-devtools-2.1.6/out/extension.js:2:1507207
at h (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:161:210554)
at h.$executeContributedCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:161:211414)
at y.S (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:158:5968)
at y.Q (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:158:5734)
at y.M (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:158:4767)
at y.L (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:158:3602)
at i.value (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:158:2297)
at r.B (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:90:732)
at r.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:90:949)
at a.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:115:14463)
at i.value (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:184:8635)
at r.B (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:90:732)
at r.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:90:949)
at a.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:115:14463)
at MessagePortMain. (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:184:6761)
at MessagePortMain.emit (node:events:519:28)
at MessagePortMain.emit (node:domain:488:12)
at Object.MessagePortMain._internalPort.emit (node:electron/js2c/utility_init:2:2619) vscode-edge-devtools-view.launchHtml {"value":"ms-edgedevtools.vscode-edge-devtools","_lower":"ms-edgedevtools.vscode-edge-devtools"}

same here please

same here

any updates on this bug?


cabal2nix: split into a lightweight version and a wrapper

Current, the cabal2nix derivation contains both the executable, and a wrapper
that adds nix and nix-prefetch-scripts, which are required for some
features.
However, when calling callCabal2nix to create a derivation from a cabal file
at evaluation time,
these features are not actually used, but the huge closure of
nix-prefetch-scripts (which includes multiple vcs, as well as python and perl)
still needs to be fetched.
This commit splits cabal2nix into a lightweight version that is a standalone
static binary (cabal2nix-unwrapped), and a wrapper that includes the proper
dependencies in the path for full usage of the command line
utility (cabal2nix).
This commit also switches to the default ghc, to reduce the likelyhood of
building a different ghc when calling callCabal2nix.

Motivation for this change
Things done

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

[ ] NixOS
[X] macOS
[ ] other Linux distributions

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

@peti Do you know why cabal2nix was using ghc 8.8.1 in the first place?

Note for reviewers: I'm not sure what are the best practices for writing such a wrapper, so suggestions are definitely welcome there.

@cdepillabout Thanks for the review! Regarding your points:

I agree that it was a bit rash to change the ghc along with this change. But I also agree that aligning it to the default version of haskell-updates is probably for the best.
I feel too that it's weird to have cabal2nix-unwrapped living at the top-level, but I dont really know where else to put it (apart from inlining it both in cabal2nix and in haskellSrc2nix, which doesn't sound great). Maybe it can be available as cabal2nix.unwrapped, but that might make overriding much harder. Do you have a suggestion?
For symlinkJoin, I followed https://nixos.wiki/wiki/Nix_Cookbook#Wrapping_packages, but it shouldn't be hard to rewrite it as a plain mkDerivation. I'll do that now.

I feel too that it's weird to have cabal2nix-unwrapped living at the top-level, but I dont really know where else to put it (apart from inlining it both in cabal2nix and in haskellSrc2nix, which doesn't sound great)

Hmm, in make-package-set.nix, can you just get the normal unwrapped version from buildHaskellPackages instead of buildPackages?

Not really: we want make-package-set.nix to use the justStaticExecutables version to avoid the gigantic closure of haskell libraries. From there, I thought that is would be nice to make sure the actual cabal2nix binaries are shared between make-package-set.nix and the top-level cabal2nixm which implies using both justStaticExecutables and generateOptparseApplicativeCompletion, and maybe having a way to share these.
On the other hand, we can decide we don't care about the (binary) duplication, and just use justStaticExecutables buildHaskellPackages.cabal2nix in make-package-set.nix. However, this would mean using a different cabal2nix for each version of ghc, which may not be what we want.
In the end, if the duplication of binary is acceptable, then changing make-package-set.nix to use justStaticExecutables buildPackages.haskellPackages.cabal2nix and leaving the top-level unchanged might be a good solution. What do you think?

@madjar Sorry, I ninja-edited my last comment while you were typing, I think.
I think this PR looks fine as you have it.  Give me a day or so to actually play around with this and make sure it is working.

No worries! I think I'll make one more change to the way cabal2nix is
defined, to use runCommand instead of raw mkDerivation (to get the local
build and substituting settings right).
Le ven. 28 févr. 2020 à 17:09, Dennis Gosnell<EMAIL_ADDRESS>a
écrit :

@madjar https://github.com/madjar Sorry, I ninja-edited my last comment
while you were typing, I think.
I think this PR looks fine as you have it. Give me a day or so to actually
play around with this and make sure it is working.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/81125?email_source=notifications&email_token=AAA2UVJEZOIYOQ72H2BYPWTRE7QUDA5CNFSM4K4H4L2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENJBI5A#issuecomment-592581748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA2UVNCRMWDGAEWV46D24LRE7QUDANCNFSM4K4H4L2A
.

On second thought, I'm happy with this version that uses mkDerivation.
If you're satisfied with this, give me a sign. I'll make sure to rebase on top of a working commit, and merge.

@madjar This looks good to me.
I'd like to get @peti to have a quick look at this just in case.
When @peti signs off on it, I'm happy merging this in.

For @peti: the commit in question is a26545b.

Rebased on top of haskell-updates, and applied @Infinisil's recommendation.

I hope you don't mind, I just pushed a change so it doesn't need a whitelist of paths to be copied from the unwrapped derivation. Makes it a bit more succinct. The symlinkJoin approach is described here: https://nixos.wiki/wiki/Nix_Cookbook#Wrapping_packages

@Infinisil No worries! I think I used the whitelist to avoid copying position, but now that I look again, it seems to be overridden anyway.

@GrahamcOfBorg build cabal2nix
@GrahamcOfBorg build cabal2nix-unwrapped

@madjar The haskell-updates branch recently updated, so I rebased this PR again.
I've also tested this with the command:
$ nix-build -E 'with import ./. {}; haskellPackages.callHackageDirect { pkg = "pretty-simple"; ver = "<IP_ADDRESS>"; sha256 = "197fmg7j97x6fibzshmqc5h6mbrk7fl3ss5kqy8x8qyd23ai3rpc"; } {}'

As far as I could tell, it looks like it didn't have to pull in a bunch of language runtimes, so it worked as expected.

@madjar Thanks for putting this together!

My pleasure!


Integration 17.5.0

This integrates the following pull requests:

#3751 (issue: #3742)
#3752 (issue: #3729)
#3753 (issue: #3467)
#3755 (issue: #3754)
#3756 (issue: #3672)
#3760 (issue: #3679)
#3761 (issue: #3680)
#3763 (issue: #3745)
#3764 (issue: #3746)
#3765 (issue: #3725)
#3766 (issue: #3728)
#3767 (issue: #3743)
#3768 (issue: #3531)
#3769 (issue: #3759)
#3774 (issue: #3762)

Total: 15 pull request.
Merging this should also mark all other pull requests as merged. This integrated all changes into a single pull request for ease of overall testing and iteration through the pulls.

It seems the test for issue #1012 is taking long enough for its 200ms bias tolerance to be exceeded in the CI test terminal for Windows 8.1/chrome set up.


Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs.

Issue #, if available:
https://github.com/opensearch-project/alerting/issues/242
Description of changes:
Updated DEVELOPER_GUIDE.md to reference changes to the supported JDKs.
CheckList:
[x] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #338 (2ecefe2) into main (13d8161) will decrease coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Current head 2ecefe2 differs from pull request most recent head 21b932d. Consider uploading reports for the commit 21b932d to get more accurate results

@@             Coverage Diff              @@
##               main     #338      +/-   ##
============================================
- Coverage     78.50%   78.46%   -0.05%
+ Complexity      215      214       -1
============================================
  Files           173      173
  Lines          6975     6975
  Branches        916      916
============================================
- Hits           5476     5473       -3
- Misses         1009     1011       +2
- Partials        490      491       +1

Impacted Files
Coverage Δ

...ing/destination/client/DestinationEmailClient.java
72.50% <0.00%> (-5.00%)
:arrow_down:

...ing/model/destination/DestinationContextFactory.kt
66.66% <0.00%> (-3.71%)
:arrow_down:

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 13d8161...21b932d. Read the comment docs.

Successfully ran ./gradlew build locally as well.


Update Tracing instructions to highlight AppInsights

Tracking issue for completing the testing of AppInsights with Track2 SDKs (Blob Storage, Event Hubs, Cosmos) and then updating the repo's tracing instructions / samples to make AppInsights more prominent in the documentation and making it clear that the beta Monitor OpenElemetry Exporter is a secondary option.

CC: @ramya-rao-a
We might need to pick a good spot for this. Today we embed all instructions into each readme but if we think things are going to be fluid maybe this'd be a better fit for a link in our github wiki.

Adding @maorleger as he's also tracking tracing work in the repo.

For more context about what we have today:

OpenTelemetry Exporter for Azure Monitor - takes care of actually pushing those traces into Azure Monitor


[DebugInfo] Simplify test for DynamicSelfLocation

We only want to check that there is no "!dbg" until the next line, we can accomplish this by checking-same on the end-of-line regex marker.

@swift-ci please test


Youtube video replay problem

Current Behavior:
I use Vidstack together with youtube provider.
I have 2 problems when the video ends.

currentTime and duration are not equal so there is a gap at the end of the time slider.
On the replay button the label remains Pause and when clicked the video does not play again.

Expected Behavior:
The video needs to be restarted when the replay button is clicked.
Rebuild Steps:

Add Youtube video link as source.
Wait for the video to finish.
Click the replay button and see that the button does not work.

Environment:

Framework: React
Meta Framework: Next.js
Node: v20.11.0
Device: Apple M1
OS: MacOS 13.4
Browser<EMAIL_ADDRESS>
Anything else?

This has been fixed for the next release, happy to re-open if it persists after testing it again :)


Missing definitons in definitions.ts for setCamera

[ ] Ensure the setCamera allows individual properties to be set, and properties whose values were not passed to the parameters are set to the existing value.
[ ] Create moveCamera, move out the animate bits from setCamera to moveCamera

We've just released a (pretty stable) beta for version 2 of this plugin under the name @capacitor-community/google-maps.
It features a more complete and consistent API. Also the map can finally be rendered behind the webview in this new version.
Extensive documentation for v2 can be found here: https://capacitor-community.github.io/google-maps/
Since version 1 lacked a lot of (basic) features, did not have good coding practices, had a poor API and it is not recommended to use it anymore, we've dropped support for it. Therefore I will close this issue. We believe version 2 offers a better Developer Experience and is a much better fit for anyones use case.


[FEATURE] MySQL ADD PARTITION and DROP PARTITION Clauses in ALTER TABLE Statements Not Parsed Correctly by JSQLParser

Grammar or Syntax Description
The ADD PARTITION and DROP PARTITION clauses in ALTER TABLE statements are not correctly parsed by JSQLParser. The parser fails to recognize these operations and does not parse them correctly.
SQL Example
ALTER TABLE t1 ADD PARTITION (PARTITION p3 VALUES LESS THAN (2002));
ALTER TABLE t1 DROP PARTITION p3;

Additional context
Applicable RDBMS: MySQL 8.0
JSQLParser: 5.0
The parsing fails with errors, and it is necessary to add appropriate support for ADD PARTITION and DROP PARTITION clauses in ALTER TABLE statements.

Solved via #2108


[Question] How to render changeable picture efficiently enough?

I want to show square grid with about 100x100 colored cells. The state of the grid (i. e. the colors of the cells) shouldn't change frequently. However, the simplest solution, which I've tried: to create a widget and draw the grid in render function works very poorly, since rerendering is called too often. Is there any way to make it better?

makeGrid :: ()
  => (Int, Int)
  -> Widget s e
makeGrid (w, h)
  = createSingle () def {
      singleGetSizeReq = getSizeReq
    , singleRender = render
    }
  where
    cellSize = 10
    coord n = fromIntegral n * cellSize

    getSizeReq _wenv _node = (fixedSize $ coord w, fixedSize $ coord h)

    render _wenv node renderer = do
      let vp = node ^. L.info . L.viewport
          origin = Point (vp ^. L.x) (vp ^. L.y)
      drawInTranslation renderer origin do
        for_ [0 .. w - 1] \i -> do
          for_ [0 .. h - 1] \j -> do
            let rect = Rect (coord i) (coord j) (coord 1) (coord 1)
                s = Just $ BorderSide 0.1 gray
                bd = Border s s s s
            drawRectBorder renderer rect bd Nothing

@Lev135 Hi! In color picker widget there is a pattern with alternating colors to indicate transparency which is implemented using imageMem widget:
https://github.com/fjvallarino/monomer/blob/67f747a61f996389828a5845da923bdcb9cb3739/src/Monomer/Widgets/Singles/ColorPicker.hs#L265-L276
Perhaps you can use it to render your square grid.

Hi @Lev135!
The main problem is you're drawing quite a few items using a helper function that is not particularly efficient. This function is used to render widget borders in several scenarios, and because of that it's too general; I added a note for myself to review it for the most basic case.
Based on what I see from the code, is the objective only drawing a grid, or are you planning on drawing squares in the viewport?
If you are only going to draw the borders, the simplest/most efficient solution is just drawing ten lines horizontally and ten lines vertically.
If you need to draw individual blocks and want them to have their own borders (be it because there are spaces between blocks or the border colors are different), you could try using Renderer's renderRect, which is a lower level version of drawRect and drawBorderRect. In this case you'll have to call the begin/end functions manually:
beginPath renderer
setFillColor renderer rectColor
renderRect renderer rectBounds
fill renderer

Alternatively, if you only want to draw borders, you can call:
setStrokeColor renderer color
setStrokeWidth renderer width

In my case I have colored cells, so solution with drawing lines doesn't work. ReplacingdrawRect with renderRect doesn't seems to have a notable effect.  For @Deltaspace0 suggestion to use imageMem: I can't see any drawing primitives for such images... Maybe I've missed something. To be honest, I haven't quite figured out how it works and how it should solve my problem: render function will use a ready picture? For ColorPicker there should be no efficiency difficulies at all (untill we haven't several thousands of them).


menuconfig target aborts when Kconfig warnings are present

Describe the bug
west build -t menuconfig aborts when Kconfig warnings are present. This prevents the user from resolving the Kconfig warnings.
To Reproduce
For this purpose I'll use an exising Kconfig conflict. Any existing Kconfig symbol conflict will work to reproduce.
Steps to reproduce the behavior:

Enable CONFIG_POSIX_API=y
Enable CONFIG_NET_SOCKETS_POSIX_NAMES=y (this symbol conflicts with the previous)
Save configuration
Attempt to build, see build fails due to Kconfig warning
Issue west build -t menuconfig to resolve the Kconfig warning
See that the Kconfig warning prevents menuconfig from running

Expected behavior
Kconfig warnings should not prevent menuconfig from executing. A user would expect to use menuconfig as the primary strategy for resolving Kconfig warnings.
Impact
I must delete .config and start over
Environment (please complete the following information):

OS: Linux
Toolchain gnuarmemb
v2.3.0

This is another variant of this existing bug: https://github.com/zephyrproject-rtos/zephyr/issues/9573

Closing as duplicate of #9573


Creation of Diskshare in SharedConnection

Hi,
in our environment we are connecting to multiple diskshares and we dont know upfront how many of them will be. Also creation of DiskShare in constructor is an expensive operation and might fail due current connection issue. DiskShare should be created on demand  once its needed by SharedFile or SharedDirectory. Also the connection should be able to reconnect or created on demand so the connection can be reused by parallel requests

Delaying the disk share creation, e.g until the first getDiskShare() access happens, shouldn't be a problem.
But do you know a good way to detect disk share timeouts/session shutdowns? It would be good to implement this logic in the SharedConnection.getDiskShare() method and not "bloat" the SharedFile/Directory. Maybe you can share some stack traces/exceptions from your environment?

We are using it like this
public Session getSession() {         if (session == null || !session.getConnection().isConnected())             session = getConnection().authenticate(authenticationContext);         return session;     }
   public Connection getConnection() {         try {             if (connection == null) {                 connection = smbClient.connect(serverName);             }             //try to reconnect in case it got disconnected in mean time             if (!connection.isConnected()) {                 connection = smbClient.connect(serverName);             }         } catch (IOException ex) {             throw new SharedDriveConnectException("Unable to connect to server " + serverName, ex);         }         return connection;     }

@JozefDropco I had some time to work on the reconnect feature (Different branch): https://github.com/swaechter/smbjwrapper/commit/019bed6efa37405e1530cb79fc00201c6901c51f
What I can't surpress: The error message of the timed out socked. I guess a background thread


feat: better default stream support in client library

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕

PTAL below error:
Error:  /home/runner/work/java-bigquerystorage/java-bigquerystorage/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1beta2/BQV2ToBQStorageConverter.java:1: Line does not match expected header line of '^/\*$'. [RegexpHeader]

One unit test failed:
Error:  Failures:
Error:    JsonStreamWriterTest.testTwoParamNewBuilder_nullSchema:220 expected:<Stream[OrTable]Name is null.> but was:<Stream[]Name is null.>

Getting NPE in ITBigQueryWriteManualClientTest.testJsonStreamWriterBatchWriteWithDefaultStream
java.lang.NullPointerException
	at com.google.cloud.bigquery.storage.v1beta2.TableFieldSchema$Builder.setMode(TableFieldSchema.java:1647)
	at com.google.cloud.bigquery.storage.v1beta2.BQV2ToBQStorageConverter.ConvertFieldSchema(BQV2ToBQStorageConverter.java:69)
	at com.google.cloud.bigquery.storage.v1beta2.BQV2ToBQStorageConverter.ConvertTableSchema(BQV2ToBQStorageConverter.java:56)
	at com.google.cloud.bigquery.storage.v1beta2.JsonStreamWriter.newBuilder(JsonStreamWriter.java:253)
	at com.google.cloud.bigquery.storage.v1beta2.it.ITBigQueryWriteManualClientTest.testJsonStreamWriterBatchWriteWithDefaultStream(ITBigQueryWriteManualClientTest.java:320)
...

full trace


mediastreamer: use ffmpeg instead of ffmpeg_3

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/120705
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

including this in https://github.com/NixOS/nixpkgs/pull/123366


Deployment Report: Recursion error when generating report

A recursion error is possible under certain conditions when running the deployment report.

Addressed in release 1.4.0

There is a longer-term fix coming for this issue, reopening.

Long-term fix addressed in release 1.5.0


$MOR Pricing Model Inclusion

Tagging an issue here that popped up during conversation regarding issuance incentives and potential pricing models.
Might be helpful to create a model that shows an effective price being paid for the token based on an assumed amount of Ethereum staked. Pool assumptions etc.
A model that looks at it from the perspective of various capital profiles.
Existing DAO governances looking for additional beta on their treasury yields
Current OG ETH stakers (@tetranode types)
VC varieties (both small 7 mil with higher risk profiles and larger more established firms)
Degen (Scott from PR etc)
Crypto Native, OG etc. would help each pool visualize how $Mor fits in their respective desired outcomes
cc @DavidAJohnston

Seems that a community member has created a website to make many of these estimates.
https://morstats.info/


Ensure that we can consider #727

Consider the following for merge into develop if value is added:

[ ] #727
[x] #770

See #862. #727 would need to be a separate skeleton since not ever dever uses bower.