[BUG] Deepspeed repeatedly requests to lock files and gets stuck

Describe the bug
When training with deepspeed and taking the ZeRO2 configuration, the program got stuck and did not exit after training. I had to use ctrl+c to end the process.
It showed the process stuck at File "/data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/deepspeed/ops/transformer/inference/triton/matmul_ext.py", line 66, in put       with FileLock(self.lock_path): .
I printed the log of the filelock, and found that in repeated attempts to get the lock of the file.
2024-02-27 15:56:22,424 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:254] - DEBUG: Attempting to acquire lock<PHONE_NUMBER>25296 on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock
2024-02-27 15:56:22,425 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:254] - DEBUG: Attempting to acquire lock<PHONE_NUMBER>15056 on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock
2024-02-27 15:56:22,427 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:254] - DEBUG: Attempting to acquire lock<PHONE_NUMBER>60912 on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock
2024-02-27 15:56:22,429 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:254] - DEBUG: Attempting to acquire lock<PHONE_NUMBER>34224 on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock
2024-02-27 15:56:22,431 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:266] - DEBUG: Lock<PHONE_NUMBER>25296 not acquired on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock, waiting 0.05 seconds ...
2024-02-27 15:56:22,433 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:266] - DEBUG: Lock<PHONE_NUMBER>15056 not acquired on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock, waiting 0.05 seconds ...
2024-02-27 15:56:22,434 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:266] - DEBUG: Lock<PHONE_NUMBER>60912 not acquired on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock, waiting 0.05 seconds ...
2024-02-27 15:56:22,436 - /data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/filelock/_api.py[line:266] - DEBUG: Lock<PHONE_NUMBER>34224 not acquired on /data2/pfluo/.triton/autotune/Fp16Matmul_2d_kernel.pickle.lock, waiting 0.05 seconds .

I modified the file deepspeed/ops/transformer/inference/triton/matmul_ext.py by commenting out lines 66-69, and the program terminated properly.
https://github.com/microsoft/DeepSpeed/blob/aed599b4422b1cdf7397abb05a58c3726523a333/deepspeed/ops/transformer/inference/triton/matmul_ext.py#L66-L69
And I found that the process also got stuck while executing ds_report. I'm not sure that such a solution has any impact on the training process. Also, I'm using an NFS filesystem, and I'm not sure if that has an effect on filelock, which in turn triggers this error.
To Reproduce
Steps to reproduce the behavior:

Clone the project https://github.com/BAAI-DCAI/Bunny
Finetune the model with zero2 configration

{
    "fp16": {
        "enabled": "auto",
        "loss_scale": 0,
        "loss_scale_window": 1000,
        "initial_scale_power": 16,
        "hysteresis": 2,
        "min_loss_scale": 1
    },
    "bf16": {
        "enabled": "auto"
    },
    "train_micro_batch_size_per_gpu": "auto",
    "train_batch_size": "auto",
    "gradient_accumulation_steps": "auto",
    "zero_optimization": {
        "stage": 2,
        "overlap_comm": true,
        "contiguous_gradients": true,
        "sub_group_size": 1e9,
        "reduce_bucket_size": "auto"
    }
}

After the training is finished, the processes get stuck.

Expected behavior
The training procedure should terminate normally.
ds_report output
[2024-02-27 16:50:14,887] [INFO] [real_accelerator.py:191:get_accelerator] Setting ds_accelerator to cuda (auto detect)
--------------------------------------------------
DeepSpeed C++/CUDA extension op report
--------------------------------------------------
NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
async_io ............... [NO] ....... [OKAY]
fused_adam ............. [NO] ....... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_lion ............... [NO] ....... [OKAY]
 [WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
evoformer_attn ......... [NO] ....... [NO]
fused_lamb ............. [NO] ....... [OKAY]
fused_lion ............. [NO] ....... [OKAY]
inference_core_ops ..... [NO] ....... [OKAY]
cutlass_ops ............ [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
ragged_device_ops ...... [NO] ....... [OKAY]
ragged_ops ............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
 [WARNING]  sparse_attn requires a torch version >= 1.5 and < 2.0 but detected 2.2
 [WARNING]  using untested triton version (2.2.0), only 1.0.0 is known to be compatible
sparse_attn ............ [NO] ....... [NO]
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
--------------------------------------------------
DeepSpeed general environment info:
torch install path ............... ['/data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/torch']
torch version .................... 2.2.0+cu118
deepspeed install path ........... ['/data2/pfluo/micromamba/envs/torch220/lib/python3.10/site-packages/deepspeed']
deepspeed info ................... 0.13.2, unknown, unknown
torch cuda version ............... 11.8
torch hip version ................ None
nvcc version ..................... 11.8
deepspeed wheel compiled w. ...... torch 2.2, cuda 11.8
shared memory (/dev/shm) size .... 503.68 GB

Screenshots
If applicable, add screenshots to help explain your problem.

System info (please complete the following information):

OS: Ubuntu 20.04.6 LTS
GPU count and types: One device equipped with 8 x A6000
Interconnects (if applicable) N/A
Python version 3.10.13
Any other relevant info about your setup

Launcher context
I tried both deepspeed and torchrun.

I have the same issue. Running rm -rf ~/.triton resolves it for me.

Hello @pengfei-luo @fffffarmer, this is because deepspeed will save triton autotune cache when it exits, and given your home dir is on a NFS, such saving could be slow. The workaround would be to set the TRITON_CACHE_DIR environment variable to point to your local hard disk.
If the autotune cache is disabled, like the way you comment it out (or delete the cache), it will not impact the quality of the training but is likely going to impose a 1-3 minutes delay when you start/resume the training as triton will perform autotune from scratch.
We plan to add a warning to explain this to the user when we detect the current cache dir is on a different file system.
Ref: https://github.com/microsoft/DeepSpeed/blame/4520edd61ce1235f87d062aa075cff412ae11a73/deepspeed/ops/transformer/inference/triton/matmul_ext.py#L53C42-L53C58


Directory Rearrangement

Hi
This PR to fix this : 👇

Into this: 👇

Thanks

Looks good to me..Thanks for that

Thanks


look into the other ways of windows sandboxing in chrome code

mentioned in https://media.ccc.de/v/37c3-11811-writing_secure_software questions at 43:00.
Getting into the mentioned methods would be the next thing.

moved to https://github.com/matu3ba/sandboxamples/issues/1.


HTTP Header parameters should be Stringified?

This issue pertains to the following package(s):

[X] GraphQL Playground - Electron App
[X] GraphQL Playground HTML
[ ] GraphQL Playground
[ ] GraphQL Playground Express Middleware
[ ] GraphQL Playground Hapi Middleware
[ ] GraphQL Playground Koa Middleware
[ ] GraphQL Playground Lambda Middleware

What OS and OS version are you experiencing the issue(s) on?
OSX 10.13.6
What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
<IP_ADDRESS>19
What is the expected behavior?
Custom Http Header should be stringified
What is the actual behavior?
If an object is send as HTTP Header,  the actual data send is the string representation of the object [Object Object]

What steps may we take to reproduce the behavior?
Try to send a query with object as http Header
{
"authorization": {"token":"sdsd"}
}

Please provide a gif or image of the issue for a quicker response/fix.

Thanks, a man for this answer. This should get included in docs.

Thanks, a man for this answer. This should get included in docs.


Update legoMain.c

Changed some variables from lunch, will continue tomorrow

Just submitted since it was working, could undo if need be, but would rather commit to main branch than keep it on the side.


Use a centralized store/state manager to cleanly handle all toasts

So, the code as of right now relies on the DOM to handle the state of the toasts. Because of this, there are problems with how we keep track of what's new and not. The temporary solution was to add attributes to new and old values. However, with this method, we are not able to cleanly separate logic for each part of this library. We need to have functions responsible for the UI (document.createElement) and logic for state handling, calculations (i,e. timers before automatically closing), With this, I might make breaking changes, so take note if there's anyone using this right now.

Added main state manager on commit #9643ec1831b39b5f72e254e2cac110e511443c2b


Improve visibility of important information

We recently worked through a ticket with azure support because we did not realize that reservation permissions are separate from subscription permissions. The azure support engineer ended up reaching out to their peers because they too did not realize that subtle but very important distinction between the two and how to manage them.
This is a very important paragraph:

You can manage access to reservations orders and reservations independently from the subscriptions that get the reservation discount. When you give someone permissions to manage a reservation order or the reservation, it doesn't give them permission to manage the subscription. Similarly, if you give someone permissions to manage a subscription in the reservation's scope, it doesn't give them rights to manage the reservation order or the reservation.

Secondly, we granted permission to someone and then they couldn't manage the reservation because it was the 'order' that we needed to give them access to.  I didn't know they were two different things when first visiting the article and immediately went to "add or change users who can a mange a reservation." This section talks about orders but doesn't help you manage them, that information is up in the "Reservation Order and Reservation" section. At the very least, add a statement to the "Add or change who can manage a reservation" section that references the "Reservation Order and Reservation".
e.g.

Similarly, if you give someone permissions to manage a subscription in the reservation's scope, it doesn't give them rights to manage the reservation order or the reservation. For more information on managing Reservation Orders, see //Reservation Order and Reservation//

I made the mistake of scrolling down and finding the powershell command for splitting a reservation which references get-reservationorder, then saying "there must be a way to do this in the UI" and finding it there by chance. Only after coming back to this page to submit feedback did I find the section at the top that talks about how to access the reservation order...

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

ID: 9be8cdc6-1a0e-25d6-8131-a56c5c0029a5
Version Independent ID: d27499e5-c8d4-2de0-273d-09f64aec3121
Content: Manage Azure Reservations
Content Source: articles/cost-management-billing/reservations/manage-reserved-vm-instance.md
Service: cost-management-billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@jonwbstr Thanks for bringing this to our attention. This issue is being assigned to the content author to evaluate and update as appropriate.
@bandersmsft FYI

@jonwbstr Added emphasis to the article, and clarified in the introduction with https://github.com/MicrosoftDocs/azure-docs-pr/pull/117755. Thanks.
#please-close


⚠ Bofrost.de has degraded performance

In 180f94f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1150 ms

Resolved: Bofrost.de performance has improved in 1d1130b after 35 minutes.


Where can customers track the progress of phase 2 for Azure Redis TLS 1.0/1.1 phase-out?

The documentation mentioned that "Phase 2 is postponed because of COVID-19. We strongly recommend that you begin planning for this change now and proactively update clients to support TLS 1.2 or later." Is there anywhere customers can track the progress of this change?

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

ID: 3d5e31f8-a816-2b22-96c0-f763fa67f9e6
Version Independent ID: d1ff1bc0-c88e-f51f-c6d8-fb9f9b212b94
Content: Remove TLS 1.0 and 1.1 from use with Azure Cache for Redis
Content Source: articles/azure-cache-for-redis/cache-remove-tls-10-11.md
Service: cache
GitHub Login: @flang-msft
Microsoft Alias: franlanglois

@yichungyu Thank you for the feedback. This is being assigned to the content author for updates.

Hi @yichungyu . I forwarded your inquiry to our PM team who will reach out to you with some information. I will close this issue here. If you feel the need this is unsatisfactory, please do reopen the issue.
#please-close


Smartcard forwarding: wrong string passed to server

Situation: Trying to forward a smartcard plugged into a REINER SCT cyberJack e-com USB smartcard reader to a Windows 8.1 machine.
xfreerdp from Debian Jessie - which is Version 1.1.0.-beta1 - works, if I specify /smartcard:cyberJack as parameter.  If I try the same with the latest xfreerdp from the nightly builds (1.2.1-dev), strange things happen.
With the Version from Debian Jessie, if I start the "cyberJack device manager" on the windows machine, the reader is displayed as "REINER SCT cyberJack pp_a2<PHONE_NUMBER>) 00 00" and everything works fine. The same with the nightly build returns a reader name of "cyberJack pp_a2 0" and it doesn't pass all the tests in the cyberJack device manager (CTAPI fails, then the device becomes completely inaccessible).
Did you add some kind of input validation for the name string, and made it a little too restrictive, so it hiccups on the "()"'s, maybe? Or what causes the truncated name?
Kind Regards,
Stefan
(sendspamhere ät stefanbaur döt de)

#4312 fixed a bunch of smartcard related issues bringing it to a usable state again.
Reopen, if your specific issue was not fixed.


CategoryColumn typo

[Enter feedback here]
First one is "SampleBarChartData.ColumnCategory," should be "SampleBarChartData.CategoryColumn,"
To get the demo to work took a long time, simply following the steps was not enough. Perhaps the whole page needs a review as karma updates have changed some things.

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

ID: 9e31ebbc-e248-fc7f-24cf-6e5e9ae04eef
Version Independent ID: 85ba273c-34bf-bb77-ff62-a9bfc454c63f
Content: Introduction to unit tests for Power BI visual projects - Power BI
Content Source: powerbi-docs/developer/visuals/unit-tests-introduction.md
Service: powerbi
Sub-service: powerbi-custom-visuals
GitHub Login: @mberdugo
Microsoft Alias: monaberdugo

Hi @maddogmikeb, thanks for pointing out the typo and for your other feedback about the page. I am forwarding your comments on to the article owner for consideration.

Thank you, @maddogmikeb, for your feedback. I fixed the typo, and will work on updating the document.
@uve, who can help me update this article?

This tutorial has been updated and will be published in the next few days.
https://docs.microsoft.com/power-bi/developer/visuals/unit-tests-introduction


Ifc export

Hi, can you when load to three.js make export to ifc file, add simple function export2ifc ()

I never worked with ifc and there are other solutions ready.
Like the https://3dviewer.net/ and the https://github.com/IFCjs

There is new library https://github.com/ThatOpen/engine_clay can you try to make export to ifc


Add basic blocking mechanisms solution for multiple CRs

Description:

when an upgrade becomes enable, check if there are other upgrade CRs currently running (with status upgradeNotCompleted) and if there are, don't start the current upgrade until they are done
add new member in ClusterGroupUpgradeSpec: BlockingCRs to hold a list of both the blocking CRs and their namespaces
the current basic functionality assumes that if UOCR A contains B as blocking CR, then B should also contain A as blocking
add KUTTL tests for this scenario

/lgtm


VERIFY SIFT-CLI

Hello,
I'm having difficulties verifying the following. Please help how to resolve this issue.
ubuntu@ubuntu:~$ gpg --keyserver pgp.mit.edu --recv-keys 22598A94
gpg: keyring /home/ubuntu/.gnupg/secring.gpg' created gpg: keyring /home/ubuntu/.gnupg/pubring.gpg' created
gpg: requesting key 22598A94 from hkp server pgp.mit.edu
gpg: /home/ubuntu/.gnupg/trustdb.gpg: trustdb created
gpg: key 22598A94: public key "SANS Investigative Forensic Toolkit<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1
ubuntu@ubuntu:~$ gpg --verify sift-cli-linux.sha256.asc
gpg: can't open `sift-cli-linux.sha256.asc'
gpg: verify signatures failed: file open error

Sounds like you didn’t download all the files. Make sure they are in the directory you are running the commands from.
Sent from my iPhone

On Sep 27, 2017, at 21:33, PAULABAILEN<EMAIL_ADDRESS>wrote:
Hello,
I'm having difficulties verifying the following. Please help how to resolve this issue.
ubuntu@ubuntu:$ gpg --keyserver pgp.mit.edu --recv-keys 22598A94
gpg: keyring /home/ubuntu/.gnupg/secring.gpg' created gpg: keyring/home/ubuntu/.gnupg/pubring.gpg' created
gpg: requesting key 22598A94 from hkp server pgp.mit.edu
gpg: /home/ubuntu/.gnupg/trustdb.gpg: trustdb created
gpg: key 22598A94: public key "SANS Investigative Forensic Toolkit<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg: imported: 1
ubuntu@ubuntu:$ gpg --verify sift-cli-linux.sha256.asc
gpg: can't open `sift-cli-linux.sha256.asc'
gpg: verify signatures failed: file open error
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

hello, i am having difficulty verifying the follwoing.  Please help.
cosign verify-blob --key sift-cli.pub --signature sift-cli-linux.sig sift-cli-linux
Error: verifying blob [sift-cli-linux]: signature not found in transparency log
main.go:74: error during command execution: verifying blob [sift-cli-linux]: signature not found in transparency log

I'll take a look, that's if you just want to verify the installer.
You can simply download cast and install sift that way now. It handles everything.


Learn More Page

Learn more landing page

[ ] Move about page
[ ] Add time line and story
[ ] Link to this http://www.austinchronicle.com/news/2009-06-05/789960/
[ ] Get involved / Get Hacking anchor button
[ ] Coreteam
[ ] Values & Code of Conduct

Get Involved / Get Hacking links
Come hang
meetup
Talk with us
slack
twitter
Connect and learn
survey
Work
policy page
github
project page

We talked at the 1/31 meetup about ditching this page - need to make sure this content is redistributed to existing page (example, code of conduct to resource page) etc.


Clarify package manager instructions

Summary

Verified instructions install the latest 3.1 release. No updates need to be made regarding this.
Added clause about SDK vs runtime.
Added paragraph about checking installed versions.

Fixes #16569
Fixes #17443
Fixes #16911
FIxes #17408
Note to reviewer: These changes were written once and ported to each document. So you only need to really review a single file.

@gewarren I guess that's what I get for assuming my paste system worked correctly 😁
Swapped to an include file...


[Android] ScrollView always is vertical

Trying to create an horizontal ScrollView seems to not be working.
new ScrollView(Orientation.Horizontal) {

}

The sample: https://github.com/jsuarezruiz/comet-travel-app-challenge/blob/main/src/CometTravelApp/CometTravelApp/Views/HomeView.cs#L129

Fixed 45c268a1c80d0c98789005b2accf38ecff7eaa82


Bluetooth: host: Configurable time to run slave conn param update

Enable possibility to configure time a slave will wait until
start of connection parameters update procedure after BT connection
is established.
Jira: DESK-606
Signed-off-by: Piotr Pryga<EMAIL_ADDRESS>

@ppryga could you please remove internal Jira references from the commit message? This is upstream and so it doesn't belong here.

@ppryga could you please remove internal Jira references from the commit message? This is upstream and so it doesn't belong here.

corrected


OCM. search for federated users doesn't contain ocis instance info

related:
https://github.com/owncloud/ocis/pull/9788
https://github.com/owncloud/ocis/issues/9702
https://github.com/owncloud/ocis/issues/9735
Steps:
Given using server "LOCAL"
And "Alice" has created the federation share invitation
And using server "REMOTE"
And "Brian" has accepted invitation
When user "Brian" searches for federated user "ali" using Graph API

Actual: response doesn't contain info about ocis instance (no identities section).

Expected: response contains identities section to understand which ocis instance this user is from.
"identities": [
                {
                    "issuer": "federation-ocis-server",
                    "issuerAssignedId": "b49ed57c-c562-4740-a5be-4ebc8b947100"
                }
            ]

This can be useful if we will be sharing more than 2 ocis instances to separate users (this can be in case of same names)
CC @rhafer @tbsbdr

Yes, there will need to be some way to present a UI so that a user can distinguish which federated instance each user comes from. The API response probably needs to have a federated domain name (or something similar), and then someone needs to think about how best to present something in client UIs that will be user-friendly.


nixos/tests/docker-tools-overlay: fix test

change virtualisation.docker.storageDriver from deprecated overlay to overlay2. Using overlay dockerd fails to start with error: ERROR: the overlay storage-driver has been deprecated and removed
https://docs.docker.com/engine/deprecated/#legacy-overlay-storage-driver

Fixes build of nixosTests.docker-tools-overlay (fails since 2023-11-14):
https://hydra.nixos.org/job/nixos/trunk-combined/nixos.tests.docker-tools-overlay.x86_64-linux
https://hydra.nixos.org/build/274675056
Log:
docker # [   27.585476] dockerd[970]: failed to start daemon: error initializing graphdriver: [graphdriver] ERROR: the overlay storage-driver has been deprecated and removed; visit https://docs.docker.com/go/storage-driver/ for more information: overlay
docker # [   27.598244] systemd[1]: docker.service: Main process exited, code=exited, status=1/FAILURE
docker # [   27.600730] systemd[1]: docker.service: Failed with result 'exit-code'.
docker # [   27.604406] systemd[1]: Failed to start Docker Application Container Engine.
docker # [   29.779517] systemd[1]: docker.service: Scheduled restart job, restart counter is at 3.
docker # [   29.783500] systemd[1]: docker.service: Start request repeated too quickly.
docker # [   29.787914] systemd[1]: docker.service: Failed with result 'exit-code'.
docker # [   29.793353] systemd[1]: Failed to start Docker Application Container Engine.
docker # [   29.797414] systemd[1]: docker.socket: Failed with result 'service-start-limit-hit'.
docker # Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
docker: output:
cleanup
kill machine (pid 8)

First failed after https://github.com/NixOS/nixpkgs/pull/264461 (docker: move default from 20.10 to 24) was merged
According to https://docs.docker.com/engine/deprecated/#legacy-overlay-storage-driver that was the version in which overlay was removed:

Legacy overlay storage driver
Deprecated in Release: v18.09 Removed in Release: v24.0

Listed test maintainers:
@lnl7
@roberth

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg test docker-tools-overlay


Add the spotless plugin to apply java style to our project

https://github.com/google/google-java-format is now the way to reliably enforce the Google Java style guide.  This PR adds a plugin that is capable of reformating the project code with:gradlew spotlessApply.
I'll update our CI to use gradlew spotlessCheck to verify that the code is the right style which is why I've removed java as a filetype for checkstyle plugin.
I've also updated our contributing instructions to instruct users to add the intellij plugin so that IntelliJ formats their code using google-java-format.
related to #1140

I guess were still stuck manually updating google_checks.xml to keep it in sync whenever the google format changes

Nah I've removed .java from the file extensions that checkstyle is using.
I've just kept it around for the other styles.
On Tue, Dec 6, 2016 at 4:02 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

I guess were still stuck manually updating google_checks.xml to keep it in
sync whenever the google format changes
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/1144#issuecomment-265271200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HTu_mx1bTO8G9v2ke9GY10rHXVv6ks5rFc1vgaJpZM4LF3xX
.

oh cool. didn't catch that


iOS talk to Android and vice versa?

Not an issue, just a question, will this library allow for iOS devices to talk peer-to-peer with Android devices and vice versa?  Is there an example that accounts for this?
Thanks, great work guys!

@VNAPNIC Thank you for the reply! Is there any plan to do so in the future, or is this a technical limitation that can't be overcome?
Thanks again

@VNAPNIC Thank you for the reply! Is there any plan to do so in the future, or is this a technical limitation that can't be overcome?
Thanks again

I think to connect both platforms, should using wifi p2p

I think to connect both platforms, should using wifi p2p

That's wonderful!  I'll start making my app now, I hope you can get the cross-platform improvement up soon, thanks for all your hard work!

That's wonderful!  I'll start making my app now, I hope you can get the cross-platform improvement up soon, thanks for all your hard work!

@Sford4 were you successful connecting iOS and Android together using wifi p2p?

@Sford4 were you successful connecting iOS and Android together using wifi p2p?

I interpreted @VNAPNIC 's last comment to mean that the package currently cannot accomplish this, but he plans to update the package in the near future to allow connecting iOS and Android together using wifi p2p.
Is that correct, @VNAPNIC ?

I think to connect both platforms, should using wifi p2p

I am also excitedly waiting.

Well, I tried it and it was not working for me. I tinkered around it for 1 hour and I did not get it working.

I think to connect both platforms, should using wifi p2p

Hi @VNAPNIC, has there been any progress in this area?
We appreciate all your hard work, thank you!

Hi,
I'm facing the same "issue" but in my case I don't need to establish a connection between the two devices, I'm just using this lib to advertise users using the same app nearby by passing a custom ID to the deviceName parameter.
While it's working perfectly on Android / Android or iOS / iOS, I'd like Android devices to be able to see iOS ones and vice versa, but I don't need to establish a connection between Android & iOS. Does anyone have an idea? I've already tried to switch to Stategy.Wi_Fi_P2P, doesn't seems to do the trick.
Thanks

Hi ! Any news on Android <-> iOS communication @VNAPNIC ?

Hi! Waiting here for Android <-> iOS communication. Any news?

@VNAPNIC  Also anxiously waiting for android to iOS communication...pretty please

@VNAPNIC I'd like to implement a cross platform app and that would be great if Android and iOS devices could talk to each other. Maybe is there any plan to implement this feature in the near future?


Exposing types.Container directly on template options

Right now the api type types.Container is exposed directly on the docker ps formatter.
I think this is a mistake as these fields were not available before, and API types are subject to change over time (and that's ok because of API versioning), meanwhile the CLI interfaces must remain constant (or relatively constant).
Having types.Container directly available on the formatter users can assume this is a stable interface.
Namespacing under another field (".ContainerDetails", or some other much cleverer name) it is probably ok, but still likely way overkill for the CLI list format.
There's just too many ways things can conflict and cause problems down the road.

/cc @vdemeester

It seems that the only purpose of preProcessor in docker ps is to capture if .Size is used in the template (so that we could set opts.size = true). I think we could ignore any field encountered other than .Size at the pre-processing stage. If there are unknown fields, it will be captured later anyway. Maybe we could use a map[string]bool instead?

@cpuguy83 Add a PR #28347 to remove types.Container from preProcessor and redefined preProcessor as map[string]bool. Please take a look.

git@github.com:docker/docker.git
@

[ ]

Ah, I see it's not as bad as I thought since we're only embedding in the preprocessor.


Support default prefix if translation key not found

Supporting default or global keys is essential for large projects when using the concept of sub-keys. It allows to define a fallback to a general key prefix if the sub-key is not defined.
E.g. in case of the missing key "home.subpage1.error" the last sub-key ("error"), shall be prefixed with the userdefined default prefix (e.g. "global") and returns the translation of the key ("global.error"). If this key is also missing, the initial key ("home.subpage1.error") shall be returned.

Well, you have defined this would be essential. :) But yes: a process of falling back to a standard translation is a common use case.
At the moment, we advice the strategy using one (or multiple) fallback languages defining the key in question. That's also because we treat the complete translation key as one unit, but not split its parts for individual resolving algorithms.


[Feature]: Changing pie chart color

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
When the data is connected from api, you aren't allowed to alter the color for the pie graph.
Why should this be worked on?
It is important to change the color for the graph since color is a critical factor for user experience

ticket created in Jira-https://zemoso.atlassian.net/browse/TNOSB-530

Hi @hubearmy , i was trying to alter the chart color i am getting the option to alter chart color except pie chart.
Currently i am loading data from statically, Can you please indicate which segment of chart should have the custom color for the below graph?

The color of the pie (the four colors inside) should be able to alter. I
can't do that when the data is connected from api (js mode)/
Raushan Kumar Gupta @.***> 於 2024年8月20日 週二 下午5:57寫道：

Hi @hubearmy https://github.com/hubearmy , i was trying to alter the
chart color i am getting the option to alter chart color except pie chart.
Currently i am loading data from statically, Can you please indicate which
segment of chart should have the custom color for the below graph?
image.png (view on web)
https://github.com/user-attachments/assets/825a0876-2d93-4f94-a094-cbe27805b55d
—
Reply to this email directly, view it on GitHub
https://github.com/appsmithorg/appsmith/issues/35737#issuecomment-2298460658,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ6EWQAILP523YPFZ6DWLQ3ZSMHI5AVCNFSM6AAAAABMW4A2G6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJYGQ3DANRVHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @hubearmy , If we want to alter the color of each section of pie chart we need to take n input as each section will contain unique color.  Please give your suggestion on it.
In case of other chart, we are applying one color only:

What I mean is there’s no place or way to alter the color of the pie chart
when you series data is connected from database/api.
[image: 螢幕擷取畫面_21-8-2024_104754_app.appsmith.com.jpeg]
Raushan Kumar Gupta @.***> 於 2024年8月21日 週三 上午10:36寫道：

Hi @hubearmy https://github.com/hubearmy , If we want to alter the
color of each section of pie chart we need to take n input as each
section will contain unique color. Please give your suggestion on it.
In case of other chart, we are applying one color only:
image.png (view on web)
https://github.com/user-attachments/assets/3ebf49b5-f161-4011-86db-6aa4bf960586
—
Reply to this email directly, view it on GitHub
https://github.com/appsmithorg/appsmith/issues/35737#issuecomment-2300347833,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ6EWQCRUJSJIKOTYTE4K7TZSP4JJAVCNFSM6AAAAABMW4A2G6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBQGM2DOOBTGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @hubearmy , Attached pic is not visible for me. Can you please provide the pic again. Like how you are connecting to api.
Points to consider:
If we want to add the option to alter the color then, we have to take n input for the color for each section of pie chart so that it will contain unique color to represent various segment of the data.
Any thoughts on it.

hey @raushan3737 , can you help me with setting up the data sources for local development, I couldn't find any means to contact you so I am texting you here maybe we can connect over discord or mail?

Hi @hubearmy, can you please provide how you are connecting to API?
If we add data statically unique colors are coming on the pie chart automatically. For the other graphs we have an option to change the colors manually.


RPCs + Spawn calls not in order

Describe the bug
When spawning an object and then trying to reference that object via an RPC it is possible that the RPC arrives before the object gets spawned.  This bug happens because Spawn calls and RPCs use different ReliableSequenced channels internally and thus lose the guarantee of order.
To Reproduce
Steps to reproduce the behavior:

Spawn an object using NetworkObject::Spawn
Invoke a ClientRpc containing the id of the NetworkObject as a parameter immediately after spawning the object on another NetworkObject.
Have the ClientRpc search for the object with NetworkSpawnManager.SpawnedObjects[objNetID]
Error appears in some cases. This is a race condition.

Actual outcome
KeyNotFound Exception is thrown.
Expected outcome
Because RPCs and Spawn calls are in order I'm able to reference my NetworkObject
Environment (please complete the following information):

OS: any
Unity Version: any
MLAPI Version: 0.1.0

Yes, this issue is driving me crazy. I will probably make a solution I already saw somewhere to Invoke() the method after a given time. It would be nice to make some asynchronous solutions.

Fixed by #948


added StrictHostKeyChecking switch

I added this to allow automated logins while using maestro-ng. The tunnel fails if the client has never logged in unless this flag is set.

Thanks for the PR @ejether! One thing before I merge this: Please use booleans instead of the string 'yes' or 'no' in the method arguments.

Hey, Thanks @jmagnusson. I updated the PR, I hope its what you had in mind.

Thank you for the improvement @ejether. I made the argument optional, so it won't be set to either yes or no unless specifically passed in. Also added a unit test for it.

Thanks for accepting it, Nice improvements!


⚠ Bofrost.lu has degraded performance

In 35504fb, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 999 ms

Resolved: Bofrost.lu performance has improved in bab8d57 after 6 minutes.


Sarcasm undetected

Model does not function as expected in misogyny detection when there are indirect sentences or sarcasm involved.
Possible fixes:

balance dataset to include labelled sarcastic comments
fine-tuning specifically for sarcasm detection
transfer learning : fine-tune pre-trained sarcasm detection models for misogyny detection

can i get assigned?

I want to be assigned to this

please assign me

For participants requesting to be assigned, check the assignee list to make sure it has not been assigned to someone already. It will be assigned to you later on if the participant who's been assigned to it does not deliver. Do ensure you have a commit within every 30 minutes, failing which the issue will be assigned to the next requesting participant.

@RishavSinha20 assigned to the issue

@neethanm is assigned to the issue

assignn

Can I be assigned this issue?

can i get assigned if this is still open?

@Python-is-better-than-Java  is assigned to the issue

@ssether04 is assigned to the issue

@poorvisuparn can I get this issue?

Issue assigned to @srijan-nodes


latest release of sphinx_rtd_theme 3.0..0 doesn't work with sphinx 8.0.2

We get this error :
Command sphinx-build -j 4 docs/source docs/build/html failed with exit code 2
Sphinx build command is expected to run in proper way

Python Version: 3.11.9
Sphinx Version: 8.0.2
RTD Theme Version: 3.0.0

Hi. Can you provide more detailed information? What's the exact error? How can we reproduce the issue?

I did a quick test locally and I'm able to build the docs using 3.0.0 and Sphinx 8.0.2. Feel free to re-open with more information.

okay it works now, it was some other errors that has to be adapted with latest sphinx version


Decimal numbers dissapear

json = JSONBuilder::Generator.new
json.float = 1.3
json.compile! # "{\"float=\": }"

The issue here is you are using json.float = 1.3 as a setter, if you remove the = you would get expected results. The new API in 3.0 should help as well.


Error pushing to registry while following the documentation guide

Hi,
I am following this guide to setup a local registry.
On the step to push the image I am getting the following error:
$ docker push localhost:5000/hello-mine:latest
a8219747be10: Image push failed
FATA[0000] Error pushing to registry: mux: variable "hello-mine" doesn't match, expected "^(?:[a-z0-9]+(?:[._-][a-z0-9]+)*/){1,4}[a-z0-9]+(?:[._-][a-z0-9]+)*$"

I think that I have followed the previous steps properly:
$ docker ps|grep regis
92ccd2969bc2        registry:2.0        "registry cmd/regist   16 minutes ago      Up 16 minutes       <IP_ADDRESS>:5000->5000/tcp     jolly_heisenberg

&
$ docker images|grep hello
hello-world                         latest               91c95931e552        12 days ago         910 B
localhost:5000/hello-mine           latest               91c95931e552        12 days ago         910 B

Hi!
If you are reporting a new issue (that does not have any duplicates already open) we would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

Following the bot advices.
I've search for the issue but I couldn't find it.

How reproducible: Always
Steps to Reproduce:
Follow this guide: https://docs.docker.com/registry/deploying/ until the docker push localhost:5000/hello-mine:latest command
Actual Results:
a8219747be10: Image push failed
FATA[0000] Error pushing to registry: mux: variable "hello-mine" doesn't match, expected "^(?:[a-z0-9]+(?:[._-][a-z0-9]+)*/){1,4}[a-z0-9]+(?:[._-][a-z0-9]+)*$"

Expected Results:
Get the image uploaded to the new local registry.
Additional info:
Nothing to add.

I'm pretty sure the mux errors is related to the docker registry v2 (docker/distribution) when pushing an image to a v2 registry in graph/push.go. I'm trying to reproduce btw

Registry 2.0 only supports Docker 1.6 and newer clients. If you need to support older Docker clients, you'll have to run both a Registry 1.0 and 2.0, as is described in this section of the same document: Configure Nginx with a v1 and v2 registry
Could you test your setup, using a Docker 1.6 client?

Hi @thaJeztah,
After creating this issue I figured out that the version of the client could be a problem so I updated it, but even with the new version I get the same problem:
$ docker version
Client version: 1.6.0
Client API version: 1.18
Go version (client): go1.4.2
Git commit (client): 4749651
OS/Arch (client): darwin/amd64
Server version: 1.6.0
Server API version: 1.18
Go version (server): go1.4.2
Git commit (server): 4749651
OS/Arch (server): linux/amd64

Hi @thaJeztah it looks like that's the issue. I've updated the client after the creation of this issue and now I am getting a certificate error non related with this one.
As far as I can tell I can close this ticket if you are happy with it.
Thanks for the help!

Thanks for confirming it's resolved. I'll close the issue :)


Add support for glam feature in i_float

As discussed in iShape-Rust/iFloat#3.

I will open a PR for i_triangle once this is merged.


Getting more done in GitHub with ZenHub

Hola! @sergeyklay has created a ZenHub account for the kr organization. ZenHub is the leading team collaboration and project management solution built for GitHub.

How do I use ZenHub?
To get set up with ZenHub, all you have to do is download the browser extension and log in with your GitHub account. Once you do, you’ll get access to ZenHub’s complete feature-set immediately.
What can ZenHub do?
ZenHub adds a series of enhancements directly inside the GitHub UI:

Real-time, customizable task boards for GitHub issues;
Burndown charts, estimates, and velocity tracking based on GitHub Milestones;
Personal to-do lists and task prioritization;
“+1” button for GitHub issues and comments;
Drag-and-drop file sharing;
Time-saving shortcuts like a quick repo switcher.

Add ZenHub to GitHub
Still curious? See more ZenHub features or read user reviews. This issue was written by your friendly ZenHub bot, posted by request from @sergeyklay.

@sergeyklay Why would we need this?

:neutral_face: hmmm. even don't understand how it happened and why. sorry guys


add plugin maintainer user

Description
Hi,
Request release permissions for Aqua plugins https://github.com/jenkinsci/aqua-security-scanner-plugin

plugin-aqua-microscanner
plugin-aqua-security-scanner
plugin-aqua-serverless

Looking forward for my first release to artifactory.

Committers/maintainers
@oranmoshai
@xB-2048
Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

please fill in the template

please fill in the template

Updated the template. Please let me know if I miss something.

@ mention an existing maintainer

Given AFAIU you're adding a new person, all checkboxes should be checked. (I mean, and check it is actually true, not just checking them on the screen :wink:)

Permissions to @sudhirbatchu9383 are approved

we need @oranmoshai sign-off, or a jira ticket raised by aqua_maintainers, in https://issues.jenkins-ci.org/ (INFRA project)

Permissions to @sudhirbatchu9383  are approved


Math magicians: full website

What does this PR do?
This PR is raised to implement the following features: 👍 :

[x] Add react-router
[x] Add Home, Calculator, and Quote pages
[x] Add style using CSS modules

Description of the tasks completed 👇

[x] Add page navigation.
[x] Add React-router.
[x] Add bable to transpile modern JavaScript syntax into code that can run in all environments when writing React components
[x] Use the linter tool that analyzes my source code to flag programming errors, bugs, stylistic errors, and suspicious constructs. 💯
[x] Check the accessibility of this project which means that websites, tools, and technologies are designed and developed so that people with disabilities ♿ can use them. For more information about Web accessibility Click here☺

How should it be tested
Clone the repository
git clone https://github.com/UbdaNam/math-magicians.git

Navigate to the folder
cd math-magicians

Checkout the branch
git checkout Add-react-router

Run app
npm start

Thanks for reviewing my PR 😊

Thank you @HAMMAS-SALEEM for reviewing my PR 😀 . I will take care of the changes you requested

Thank you @elisha2kyakpo1 for approving my PR


Fix for audio playback on Windows clients going silent when audio device is invalidated

Turns out that some AI voice gen app that Richard was using was inadvertently causing the Parsec client audio playback device to be invalidated. I was unable to repro this myself using the same software however it is 100% reproducible on Richard's setup.
My solution: simply reinitialize the IAudioClient device, just like we do when detecting the default device has changed (in the case where playback is happening on the default device). The solution is generic and solves the problem in the future for any other scenario where the device is invalidated for w/e reason.

Whoops, please ignore.


Fix minOccurrences and maxOccurrences Handling in X12 to Ballerina Schema Conversion

Purpose
Fixes: https://github.com/ballerina-platform/ballerina-library/issues/6658 , https://github.com/ballerina-platform/ballerina-library/issues/6610
This PR addresses two issues in the EDI tool:

Incorrect Handling of minOccurrences and maxOccurrences in Schema.json from X12 XSD Files:
This fix resolves an issue where maxOccurrences/minOccurrences values from X12 XSD files were not accurately converted into schema.json, leading all values to default to 1 irrespective of the defined occurrences in the XSD.

This is a part of the sample X12 file;

Before Fix:  Values of maxOccurrences and minOccurrences are not properly converted into Ballerina EDI schema.

After Fix: The same segment now correctly converts to schema.json with maxOccurrences and minOccurrences.

2. Enabling Documentation Comments in Generated Code:
Add documentation comments to the automatically generated public functions, enhancing the developer experience of the connector user.
Before Fix:

After Fix:

Tested for X12 810, 850 examples, and works as expected.


Requesting range off end of file does not return 416 status code

Describe the bug
If we request a byte range that is beyond the end of the file the server returns the full content of the file and ignores the byte range.
To Reproduce
Steps to reproduce the behavior:
Run up a server with a small file in it:
wsgidav --host=<IP_ADDRESS> --port=65160 --auth=anonymous --root=`pwd`/tmp

(I have tried cheroot and uvicorn servers)
Use curl to request content:
This works:
$ curl -H 'Range: bytes=10-20' -i http://<IP_ADDRESS>:65160/pyproject.toml
HTTP/1.1 206 Partial Content
date: Thu, 06 Apr 2023 18:08:34 GMT
server: uvicorn
Content-Length: 11
Last-Modified: Thu, 06 Apr 2023 17:16:56 GMT
Content-Type: application/octet-stream
Date: Thu, 06 Apr 2023 18:08:34 GMT
ETag: "120114958-1680801416-3302"
Accept-Ranges: bytes
Content-Range: bytes 10-20/3302

Ask for a byte range that crosses the threshold and this works:
$ curl -H 'Range: bytes=3200-3400' -i http://<IP_ADDRESS>:65160/pyproject.toml
HTTP/1.1 206 Partial Content
date: Thu, 06 Apr 2023 18:10:21 GMT
server: uvicorn
Content-Length: 102
Last-Modified: Thu, 06 Apr 2023 17:16:56 GMT
Content-Type: application/octet-stream
Date: Thu, 06 Apr 2023 18:10:21 GMT
ETag: "120114958-1680801416-3302"
Accept-Ranges: bytes
Content-Range: bytes 3200-3301/3302

Now ask for a byte range that starts off the end of the file and this fails:
HTTP/1.1 206 Partial Content
date: Thu, 06 Apr 2023 18:09:09 GMT
server: uvicorn
Content-Length: 3302
Last-Modified: Thu, 06 Apr 2023 17:16:56 GMT
Content-Type: application/octet-stream
Date: Thu, 06 Apr 2023 18:09:10 GMT
ETag: "120114958-1680801416-3302"
Accept-Ranges: bytes
Content-Range: bytes 0-3301/3302

[build-system]
requires = ["setuptools", "lsst-versions >= 1.3.0"]
build-backend = "setuptools.build_meta"

...

The entire file is returned as shown by the Content-Range header.
The server is receiving the request properly:
11:09:10.785 - INFO    : <IP_ADDRESS> - (anonymous) - [2023-04-06 18:09:10] "GET /pyproject.toml" depth=0, range=bytes=10000-10020, elap=0.000sec -> 206 Partial Content
INFO:     <IP_ADDRESS>:53418 - "GET /pyproject.toml HTTP/1.1" 206 Partial Content

Expected behavior
The expectation is that this should return status code 416. It should definitely not return the full content of the file.
For example requesting a file from GitHub:
$ curl -H 'Range: bytes=100000-100003' -i https://raw.githubusercontent.com/lsst/resources/main/.github/workflows/build.yaml
HTTP/2 416
cache-control: max-age=300
content-security-policy: default-src 'none'; style-src 'unsafe-inline'; sandbox
content-type: text/plain; charset=utf-8
etag: "07230c3e6c40a24f5ab21c24344d1937f97cbd5c9bea12318aabbc05a3b3caa7"
strict-transport-security: max-age=31536000
x-content-type-options: nosniff
x-frame-options: deny
x-xss-protection: 1; mode=block
x-github-request-id: 187C:5B36:267AC3:2DD550:642F0968
accept-ranges: bytes
content-range: bytes */3045

Screenshots, Log-Files, Stacktrace
Insert stacktrace if applicable.

If applicable, add screenshots to help explain your problem.
If applicable, add a log file (consider --verbose).
Environment:
WsgiDAV/4.2.0 Python/3.10.10(64 bit) macOS-13.3-arm64-arm-64bit
Python from: .../miniconda/envs/lsst-scipipe-5.1.0/bin/python3.10

Which WSGI server was used (cheroot, ext-wsgiutils, gevent, gunicorn, paste, uvicorn, wsgiref, ...)?
cheroot and uvicorn.
Which WebDAV client was used (MS File Explorer, MS Office, macOS Finder, WinSCP, Windows, file mapping, ...)?
Simple curl.

I did have a quick look at the code and the error occurs in util.obtain_content_ranges.  The first match in the loop does correctly determine the first and last byte ranges but does not check to see if firstpos > filesize. Instead it drops out of that block and then tries the second match with reSuffixByteRangeSpecifier.search. This time it ends up doing firstpos filesize - larger_number and gets a negative answer which it then turns into firstpos = 0 and you end up with the whole file.
I did get the simple case to work by adding a check for firstpos > filesize in the first check block and then immediately returning [], 0 -- what I'm not sure about is what should happen if you ask for multiple ranges and only one of them is unsatisfiable.

Yes, I also looked into it a bit yesterday. As I understand it, invalid Range headers may result in a

Response 200, returning the complete file,
or
Response 416 with Content-Range: bytes */FILESIZE

I will try to get the latter working. (Multi-Part responses are currently not implemented btw.)


For Release 2.0 - Updated the priority of the APIs in producer_api_plugin and net_api_plugin to MEDIUM_HIGH

Updated the priority of the APIs in producer_api_plugin and net_api_plugin to MEDIUM_HIGH
Change Description
These apis should respond fast even if host is busy. Hence, the priority for it within net_api_plugin and producer_net_plugin needs to be updated.
Consensus Changes
NA
API Changes
Updates to net_api_plugin
Documentation Additions
Updated the priority of the APIs in producer_api_plugin and net_api_plugin to MEDIUM_HIGH as these apis should  respond fast even if host is busy.

Wrong title.  Not based on develop.


Pin the charmed-mysql snap version

Issue

We are curriently installing the 8.0/edge channel of the charmed-mysql snap. However, this can cause the charm to break if a new version of the snap is release that is buggy.

Solution

Pin the revision of the charmed-mysql snap instead of downloading the from the 8.0/edge channel (revision 44 which is the latest `8.0/edge revision)

(for the future, not necessarily this PR) thinking about how to keep this up to date—will the revision change infrequently enough that it makes sense to update manually or should we look into developing a renovate config to automatically update this?

A renovate config to auto update the revision may be a good idea


Incognito mode blocks sessionStorage and throws error

Steps to reproduce

Run yarn dev
Open web app in Incognito

The iframe content does not load at all.

According to https://stackoverflow.com/questions/24456891/iframe-in-chrome-error-uncaught-securityerror-failed-to-read-the-sessionstora this is caused by script blockers or blocking third party cookies/site-data.

Is it still relevant?
If not, please close the issue.
If yes, please transfer to the repo serlo/serlo-editor-as-lti-tool

I've tested in the serlo/serlo-editor-as-lti-tool and it seems to work. So, closing this issue


Fix: ValueError when the device_model parameter does not contain spaces

Fixed this issue.
Log with error:
File "/home/ha/.homeassistant/custom_components/xiaomi_gateway3/core/zigbee.py", line 654, in get_buttons
zigbee_model, _ = device_model.split(' ', 1)
ValueError: not enough values to unpack (expected 2, got 1)

Thanks! But I have fix with another way


id function is not compliant to the spec

As per the spec, id function selects elements by their unique ID and attribute for unique ID is declared using DTD.
Q. If no DTD is defined then is id attribute implicitly considered as unique ID of the element node?
Following test fails, because we are not handling DTD at this moment, right?
it.only('adhoc tests', () => {
	const testDocument = xml`<?xml version="1.0" encoding="UTF-8"?>
			<!DOCTYPE root [
					<!ELEMENT root (foo+)>
					<!ELEMENT foo (bar)>
					<!ELEMENT bar (#PCDATA)>
					<!ATTLIST foo name ID #REQUIRED>
			]>
			<root>
					<foo name="first">
							<bar>Some content</bar>
					</foo>
					<foo name="second">
							<bar>Another content</bar>
					</foo>
			</root>
`;
	const evaluator = new Evaluator({rootNode: testDocument});

	const actual = evaluator.evaluate(`count(id('first second'))`);

	expect(actual.numberValue).toBe(2);
})

This issue probably just needs documentation.

I've inferred that "the spec" above refers to XPath 1.0 (id, 5.2.1 Unique IDs).

Pertinent spec language

The id function selects elements by their unique ID (see [5.2.1 Unique IDs]). [...]
[...]
5.2.1 Unique IDs
An element node may have a unique identifier (ID). This is the value of the attribute that is declared in the DTD as type ID. No two elements in a document may have the same unique ID. If an XML processor reports two elements in a document as having the same unique ID (which is possible only if the document is invalid) then the second element in document order must be treated as not having a unique ID.

NOTE: If a document does not have a DTD, then no element in the document will have a unique ID.

A few notes:

I can confirm that we're out of [XPath 1.0] spec as reported

We're also out of [XPath 1.0] spec according to the last NOTE in the quoted language, by treating the id attribute as a unique identifier even in the absence of a DTD

This function is not included in the ODK XForms Specification (and according to my reading, this is explicit); but since we provide the functionality, I do consider it a bug

Both aspects of out-of-XPath-1.0-spec behavior are consistent with the native evaluators of each major browser (I checked in Chrome, Firefox, Safari); I would like to find a compliant implementation before we decide to address this

Q. If no DTD is defined then is id attribute implicitly considered as unique ID of the element node?

This is the current behavior, and as mentioned it is consistent with all of the major browsers. I believe the behavior is expected for HTML documents, and any non-HTML consideration is just unaddressed (I'd expect it never will be).

we are not handling DTD at this moment, right?

Correct.
I was curious if there is any precedent for DTD support or interest in it. Not that it's an exhaustive search, but this issue is the first to mention DTD across all of the getodk org.


FIX: respect the align and width parameter of table directives

Fix #524
simply add back some rules on both css and python side.

waiting for 0.11 to be released before merging


Rails routes

In this milestone, I achieved the following...
For given URLs, I created
A route.
An action in the correct controller.
A view file.

Status: APPROVED 🟢
Hi  @Cabraham1  👋 👋
I'm @TuyishimireEric, Your code reviewer,
Good job on implementing the project requirements! 🥇

Your project is complete!  There is nothing else to say other than... it's time to merge it 🦾
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, you can request a second opinion using this form.


Topics: add new topics for addrv2, output linking, package relay

This is 3 out of the 16 new topics I was scheduled to have submitted by now.  Sorry for the delay.  :man_facepalming:

@bitschmidty ﬁxed, thanks!

Edits made for @jnewbery feedback (thanks!).  This includes a new commit that automatically creates Markdown reference-style links for the topics (with the ability to define a short name for topics whose long names I don't want to frequently type) and the removal of the unannounced channels topic (which I'll put in a separate commit).

ACK ed307eb8f5b2a06d31dec30fbf0b813461bbcb7e
Looks great. New automatic topics links are :fire:


Add sample using custom serializer

Add a sample project using a custom serializer for new users.

#551 already covers it and its an adoption blocker needs attention.
Post #551 closure this samples has very little/none left.


General improvements

Hi there 🙌
I had some issues using this component.
This PR fix:

Height 0 not working on Android
Render custom active divider
Custom sortRowStyle props
Support scrollView with margin (navbar/tabbar/etc.)
Use onContentSizeChanged instead on onScroll
Fix scrollTo after release
Allow to disable scroll

Let's discuss about all these improvements

Hey @kemcake, looks great. I'll test it out to make sure it's all working correctly, and will get back to / merge if all is good.

Merged, your changes are in the 0.0.8 release on npm.

👍

🎉

This broke my page:

@antoinerousseau don't really see what's wrong, you're talking about the scroll after a drop ?

@kemcake  the card above the one moved moves below before starting any movement


ATA Support

[Enter feedback here]
You should update the end of support date for the ATA version:
https://techcommunity.microsoft.com/t5/security-compliance-identity/end-of-mainstream-support-for-advanced-threat-analytics-january/ba-p/1539181

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

ID: d587c5ad-4a52-c244-5142-f780cc3a2c96
Version Independent ID: e34d972f-ee87-9845-1165-76fdaf956c2c
Content: Advanced Threat Analytics versions
Content Source: ATADocs/ATA-versions.md
Product: advanced-threat-analytics
Technology: ****
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks for your suggestion. We'll investigate and get back to you.

Thanks for your comment. We've updated the table.


Support Kubernetes 1.24

Support Kubernetes 1.24. And update dependencies.

Update go modules
Build with go 1.19
Update dependencies

Stop using nilerr

Signed-off-by: kouki<EMAIL_ADDRESS>

https://github.com/cybozu-go/cattage/pull/19/commits/f792499e4e13d1ffab08c507e67c02903e7df400
I mistakenly set the following library to use v0.25.0, so I changed it back to use v0.24.7.

k8s.io/api
k8s.io/apimachinery
k8s.io/client-go

@zoetrope @zeroalphat
I'm sorry. Please review again.


Pull Request view still shows PR after I checkout out a different branch in terminal

Checkout some PR so that it shows in the GH PR view
Open terminal. In the terminal switch back to the main main branch
GH PR view still shows the PR 🐛

Are we listening on git events that change branches? Should we automatically get out of PR review mode here? Or am I missing some git events on my machine again?

Are we listening on git events that change branches? Should we automatically get out of PR review mode here? Or am I missing some git events on my machine again?

We are listening to these events, and we should automatically get out of PR review mode here. I do suspect you're missing git events again. Next time you see this can you go to the SCM view and refresh?

A different PR got shown in my GITHUB PULL REQUEST view. Going to the SCM view and hitting refresh did not change anything.

I have started seeing this too. I'm following up with @mjbvz to see if I'm doing something wrong with disposing of a webview view and reregistering it.

Hello,
Same issue here:
I see Some (not all of them) PRs even though they are all merged.

How can I hide them because there are no pending actions

@alan345 do you still have those branches checked out? If so, this is the intended behavior. If you don't want to show merged PRs you can set:
"githubPullRequests.useReviewMode": {
  "merged": false
}

@austinbutler it sounds like you're talking about the PR description/overview which shows in the editor. That is not related to the current branch and instead shows the PR description/overview of the PR that you opened it from.

Getting the same issue when I checkout master on my repo - the plugin thinks I'm still checked out on an old branch, showing me diff and comments related to that PR.
Works fine if I checkout to a separate branch, but my master is now always tainted with the old branch which will never go away.
Have tried toggling enable/disable on plugin, as well as reinstalling it - no change.

@robhung this sounds like https://github.com/microsoft/vscode-pull-request-github/issues/4642. I have some questions about your setup:

Do you have a clone of a repo with an upstream remote another remote that's a fork that you own of the upstream?
If so, is the remote of your master branch the upstream repo, or your fork?

Until I do figure it out you can "unpollute" your master branch by opening the .git/config file, finding the section that starts with [branch "master"], and deleting the github-pr-owner-number line from there.

This should be fixed now.

@robhung this sounds like #4642. I have some questions about your setup:

Do you have a clone of a repo with an upstream remote another remote that's a fork that you own of the upstream?
If so, is the remote of your master branch the upstream repo, or your fork?
I have the same repo checked out in two computers, but no clones of the repo on the same computer. I do have a clone of the VS code workspace for the same repo, does that affect the plugin?

Until I do figure it out you can "unpollute" your master branch by opening the .git/config file, finding the section that starts with [branch "master"], and deleting the github-pr-owner-number line from there.
This worked perfectly, thank you!


Access QuartzNet configurations

I want to use persistent store but I cannot set the configurations for QuartzNet like UsePostgres, "quartz.jobStore.type","quartz.jobStore.tablePrefix."
Can silkierQuartz do this?

yes , you can!


[feat] Add d3 legend formatting for Arc, Polygon and Scatter deck.gl maps

CATEGORY
Choose one

[ ] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Adds d3 legend formatting to Arc, Polygon, and Scatter deck.gl maps
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

BEFORE

AFTER

TEST PLAN

Visual inspection
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Did you include an npm update as part of this pr? I think it would be better to split such a large change in deps to a separate PR if that was intended

Codecov Report

Merging #7951 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #7951      +/-   ##
==========================================
- Coverage   65.58%   65.54%   -0.05%
==========================================
  Files         469      469
  Lines       22449    22474      +25
  Branches     2431     2441      +10
==========================================
+ Hits        14723    14730       +7
- Misses       7606     7624      +18
  Partials      120      120

Impacted Files
Coverage Δ

superset/assets/src/explore/controls.jsx
42.85% <ø> (ø)
:arrow_up:

...sualizations/deckgl/CategoricalDeckGLContainer.jsx
0% <ø> (ø)
:arrow_up:

...et/assets/src/explore/controlPanels/DeckScatter.js
0% <ø> (ø)
:arrow_up:

...c/visualizations/deckgl/layers/Polygon/Polygon.jsx
0% <ø> (ø)
:arrow_up:

superset/assets/src/visualizations/Legend.jsx
0% <0%> (ø)
:arrow_up:

...src/components/FilterableTable/FilterableTable.jsx
78.61% <0%> (-4.62%)
:arrow_down:

superset/db_engine_specs/postgres.py
95.45% <0%> (+22.72%)
: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 9233a63...e370a3c. Read the comment docs.

Did you include an npm update as part of this pr? I think it would be better to split such a large change in deps to a separate PR if that was intended

I did not but switched the dependency from d3-format to superset-ui as suggested by @villebro. Everything should be good to go now.

Are the package-lock.json changes still necessary?

This LGTM but I don't feel comfortable merging frontend PRs; @etr2460 or @mistercrunch can you take a look?


[rlgl] rlGetShaderBufferSize seems to always return 0

Description
The rlGetShaderBufferSize function seems to always return 0, even after uploading, processing and downloading data with an SSBO (all successfully). Looking at the code, the function is implemented using glGetInteger64v:
// Get SSBO buffer size
unsigned int rlGetShaderBufferSize(unsigned int id) {
    long long size = 0;
    glBindBuffer(GL_SHADER_STORAGE_BUFFER, id);
    glGetInteger64v(GL_SHADER_STORAGE_BUFFER_SIZE, &size);
    return (size > 0)? (unsigned int)size : 0;
}

After looking around a bit, it looks like glGetBufferParameteri64v is probably the correct function to be calling in order to get the size of the SSBO, see below:
// Get SSBO buffer size
unsigned int rlGetShaderBufferSize(unsigned int id) {
    long long size = 0;
    glBindBuffer(GL_SHADER_STORAGE_BUFFER, id);
    glGetBufferParameteri64v(GL_SHADER_STORAGE_BUFFER, GL_BUFFER_SIZE, &size);
    return (size > 0)? (unsigned int)size : 0;
}

With this implementation, the actual size of the buffer is correctly returned. From what I can tell, glGetInteger64v is used to query the maximum value of some property (I'm not exactly sure). Is the current implementation the intended behaviour, or should it actually return the current size of the buffer?
Environment
Windows 11, desktop:
INFO: GL: OpenGL device information:
INFO:     > Vendor:   NVIDIA Corporation
INFO:     > Renderer: NVIDIA GeForce RTX 3060 Laptop GPU/PCIe/SSE2
INFO:     > Version:  4.3.0 NVIDIA 546.30
INFO:     > GLSL:     4.30 NVIDIA via Cg compiler

Code Example
#include <stdio.h>
#include <stdlib.h>

#include "raylib.h"
#include "rlgl.h"

int main(void) {
    InitWindow(800, 600, "pathtracer");

    unsigned int target_size = 100;
    unsigned int buffer = rlLoadShaderBuffer(target_size, NULL, RL_DYNAMIC_COPY);

    unsigned int actual_size = rlGetShaderBufferSize(buffer);
    printf("target size: %u, actual size: %u\n", target_size, actual_size);

    rlUnloadShaderBuffer(buffer);
    CloseWindow();
}

With the glGetInteger64v implementation, the code always outputs target size: 100, actual size: 0, but with the glGetBufferParameteri64v implementation, it outputs target size: 100, actual size: 100.

@kal39 Good catch! Thanks for the improvement! Actually, it seems previous implementation required glGetInteger64i_v() instead of glGetInteger64v() as per the specs.


[Core] Update typing annotations for CaseInsensitive Dictionary

Changes for typing after API Review

in setitem value should be Any, while key should str since we are lowering the case for it

data can either be a Mapping or iterable of KV mappings

iter should be Iterator[str]

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-core

/check-enforcer override


RFC - Fully custom breakpoints object

With the current setup, we are only exposing/allowing certain breakpoints to hit and report back to the user and they are all max-width based breakpoints that range from xs/s/m/l/xl
I think a better approach would to allow the user to supply (almost) the entire object and that would have the potential to look exactly how the user wants it to. Take the following as an example, the user could use their own descriptive keys that fit their use case (only using these to show the customization potential):
{
   iphoneXS: '(max-width: 300px)',
   iphoneX: '(max-width: 400px)',
   ipad: '(max-width: 1000px)',
   iphoneXMaxHeight: '(max-height: 800px)',
}

With this approach, breakpoints would be 100% customizable, which would be ideal in many cases.
Then we would create the necessary window.matchMedia queries to fire when the window changes height/width and report back using the keys from their breakpoints object.

This is great, and along the lines of what I would have suggested instead of PR #15


Decl'NetCDF test 06 fails

See https://gitlab.maisondelasimulation.fr/pdidev/pdi/-/jobs/24108
[PDI][000002][22:09:16] *** error: Error while triggering event `write': Decl_netcdf plugin: Cannot change the access of a variable to parallel : (-49) NetCDF: Variable not found
[PDI][000003][22:09:16] *** error: Error while triggering event `write': Decl_netcdf plugin: Cannot change the access of a variable to parallel : (-49) NetCDF: Variable not found
[PDI][000001][22:09:16] *** error: Error while triggering event `write': Decl_netcdf plugin: Cannot change the access of a variable to parallel : (-49) NetCDF: Variable not found
[PDI][000000][22:09:16] *** error: Error while triggering event `write': Decl_netcdf plugin: Cannot change the access of a variable to parallel : (-49) NetCDF: Variable not found

mentioned in commit 6ceb649751db8aba6bc9ed003fe17d88b72160e6

In GitLab by @ksiero on Jan 18, 2021, 16:51
mentioned in commit bc72d7f99c137339bb78fc51d2ce762f02eccdee

mentioned in commit 584c7ac310846ac15b86cb29d3a0b1eb220e60e9

mentioned in commit 0cfc0168280ef81d62a36b86e65ed3d819ffcb7c

mentioned in commit 5742b3edbfd9f72aa4d4b4e9854a2e48822e8b4b

In GitLab by @ksiero on Jan 18, 2021, 15:15
mentioned in commit 4e2783cd0562e52e59add6d3cb2b56701b21c3be


jadx: 1.3.4 -> 1.3.5

Description of changes
https://github.com/skylot/jadx/releases/tag/v1.3.5
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

cc @veehaitch

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/845

Result of nixpkgs-review pr 170054 run on x86_64-linux 1

2 packages built:

apkleaks
jadx

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/502


Major Update to UI

Please run the app to view the changes

Quilly is all about staying super simple, and that's why I use lots of white space to keep things clean. Your changes look nice but they seem to move away from simplicity. However, the tweaks you made in index.html for changing placeholder texts and headings are totally cool and can be added.


[WIP] Add LogMessageContent option

TS Client has a logMessageContent option, we should add it to Java and .NET.
I want to solicit feedback on this especially for the .NET one, I only did it partially because it looks a little ugly.

SignalR work has moved to aspnetcore repo, and this repo will be archived. If you're still interested in pursuing this PR, please move it over to aspnetcore. Thanks.


Index Replicas

Is your feature request related to a problem? Please describe.
We need Index Replicas, like on Algolia we need to create replicas to add different sort rules.
Describe the solution you'd like
So we don't need to create different indexes to add different rules and always sync the differences in all of them.
The replicas would always be synced with the index data, just the setting would change.
Describe alternatives you've considered
Duplicating indexes.
Additional context
Example:
Product (index)

Product_Price_ASC (replicas)
Product_Price_DESC (replicas)
etc....

this is related to #730

Hello, Thanks for this feature proposal!
I will close this issue in favor of our public roadmap.
I invite everyone interested in this feature to upvote it on the roadmap.


Nudge cannot open Self Service?

The Nudge dialog box opens and eventually the only option is to "Update Device" but when we click "Update Device" the screen flickers and we are presented with an error "Self Service quit unexpectedly" with a long string of codes.
Nudge is configured with an actionButtonPath of "jamfselfservice://content?entity=policy&id=2909&action=execute" which points to the policy for our Erase-Install script.
If we instead launch the Self Service application BEFORE clicking "Update Device" on the nudge dialog box the nudge dialog box is able to launch the Erase-Install policy and upgrade the computer.
How can I configure Nudge so that the nudge dialog can execute the Jamf policy to Erase-Install the computer?  Can nudge launch self service app as well as the upgrade dialog box?
I have been working with the 'latest' version <IP_ADDRESS>564 and then attempted the latest pre-release, <IP_ADDRESS>657 with the same result.
(I need to run this on Big Sur, Monterey and Ventura to get these upgraded to Sonoma.)

Is your Jamf server on 11.6.1? There is a PI for this.
PI119354 - Self Service macOS Crashes when using URL schemes to open content.

Jamf doesn't always add PIs to the Known Issues list, so people report them in the jamf-product-issues channel in Macadmins Slack.

Closing this as it is a JAMF issue. Good luck patching your server.


Add SonarCloud Integration Workflow for Pixeebot

User description

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

Description

Introduced a new workflow to fix SonarCloud issues with Pixeebot.
The workflow uploads Sonar results upon completion of the SonarCloud code analysis.

Changes walkthrough 📝
Relevant filesEnhancement

sonarcloud-pixeebot.ymlAdd SonarCloud Integration Workflow for Pixeebot

.github/workflows/sonarcloud-pixeebot.yml
Created a new GitHub Actions workflow for SonarCloud integration.  Configured the workflow to trigger on completed check runs.  Added job to upload Sonar results to Pixeebot using a specific action.

+18/-0

💡 Penify usage:
Comment /help on the PR to get a list of all available Penify tools and their descriptions

Summary by CodeRabbit

New Features

Introduced a new GitHub Actions workflow to enhance SonarCloud integration for the Pixeebot project.
Added a job to upload Sonar analysis results, improving code quality monitoring.

:white_check_mark: Build VTEX-SDK-dotnet 2.3.1064 completed (commit https://github.com/guibranco/VTEX-SDK-dotnet/commit/529b83111e by @guibranco)


Improper Periodicity

Does the current version support a periodicity2, phase2 and k2 for improper terms? If not can this be integrated into the next release? @j-wags
We are trying to capture this energetic behavior with the improper term.

Hi, @jmaat. Yes, it should. Both the stable version in master and the development version in the topology branch. Let us know if it doesn't as it is likely a bug.

I'm not sure I understand how SMIRNOFF trefoil impropers can possibly go past +- 90 degrees, since one of the atoms is already an infinite distance away from the others at that point. How can your function above even approach those limits?

Reflecting what Andrea said, we should be able to handle that. I'll be up
in Irvine today and we can try it out.
I'm also thinking about what John said. I think I see what he means, but
I'd like to go over the math for calculating that angle, and figure out
what its range can be if you have time (I'm also happy to do impropers 101
as self study if you're busy today)
On Fri, Feb 8, 2019, 6:19 AM John Chodera<EMAIL_ADDRESS>wrote:

I'm not sure I understand how SMIRNOFF trefoil impropers can possibly go
past +- 90 degrees, since one of the atoms is already an infinite distance
away from the others at that point. How can your function above even
approach those limits?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openforcefield/openforcefield/issues/176#issuecomment-461817060,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APkp187n7nphPBupTnAI0EXb7TrEw3Ggks5vLYdfgaJpZM4aq9Hn
.

@jchodera I think this was intended as an illustration of the type of double-well behavior we might want to support via a function like that, but it hasn't been scaled to vary over the appropriate range of angles.

@jmaat I think this is resolved now? If so, can you close the issue?


[WIP] Bosonic backend support for threshold and Fock measurement

Context: Currently, the Bosonic backend does not support the MeasureFock and MeasureThreshold measurement operations. This pull request aims to add support for it. Ref : issue #571
Description of the Change: Adding support for measure_threshold and measure_fock
Benefits:
Possible Drawbacks: N/A
Related GitHub Issues:

Thanks @JeyKelly for this PR! Feel free to ask any questions here, and you can comment directly on the code itself 🙂

@josh146 I've checked the result of the test and it seems like the test was designed to raise a non-implemented error when calling the MeasureThreshold operation (~strawberryfields\tests\backend\test_bosonic_backend.py line 664). Assuming the Fock state case works and post-measurement is updated properly, should I update the pull request to include removing the line of code and adding two extra modes to test_measurement verifying the proper sample shapes/states after applying MeasureFock and MeasureThreshold?

@nquesada I have revisited the measure_threshold method in the way you suggested. I have ran into a few issues when testing, and this is why I didn't update the test file yet. If I ran the circuit with a Fock() state parameter of 7, the circuit returns nan values for the weights (hence can't even run without a measurement), and if I put more, the fidelity of vacuum is bigger than one. This doesn't seem to be related to these modifications, but I thought I would mention it before creating an automatic test for the measurement.
That being said, the measure_fock method still remains and I'm not sure on how to approach updating the state (and even if the measurement is done properly). Do you have a similar piece theory that would help achieve this one?

Hi @JeyKelly --- The PR is looking really nice. For tests I'd suggest a few:

Prepare the vacuum and check that when you use threshold measurements you never get a click.
Prepare the odd cat states (with p=1) and also verify that you never get click.
This somewhat trivial, but check that the shapes of the covariances and vectors of means are correct when you get a click/no click.

Hi @JeyKelly --- The PR is looking really nice. For tests I'd suggest a few:
* Prepare the vacuum and check that when you use threshold measurements you never get a click.

* Prepare the odd cat states (with `p=1`) and also verify that you never get click.

* This somewhat trivial, but check that the shapes of the covariances and vectors of means are correct when you get a click/no click.

@nquesada Hi, thanks for the response. Just to be sure, do you mean for automatic tests?

Hi @JeyKelly --- Yes, I mean automatic tests. You can start by using the backend independent tests here:
https://github.com/XanaduAI/strawberryfields/blob/master/tests/backend/test_threshold_measurement.py
To test the bosonic backend it should be as easy as adding "bosonic:" in this line: https://github.com/XanaduAI/strawberryfields/blob/afde2997ce04d44612232be2a354946f608c0a7f/tests/backend/test_threshold_measurement.py#L24

Then you could add tests specific to the bosonic backend in here https://github.com/XanaduAI/strawberryfields/blob/master/tests/bosonic_files/test_backend_bosoniccircuit.py
This could be the ones I described before involving cat states and the ones checking the shapes of the updated coefficients, vectors of means and covariance matrices are correct.

To test the bosonic backend it should be as easy as adding "bosonic:" in this line:
https://github.com/XanaduAI/strawberryfields/blob/afde2997ce04d44612232be2a354946f608c0a7f/tests/backend/test_threshold_measurement.py#L24

Hi Again @nquesada, by using the test test_two_mode_squeezed_measurements in this file, it made me realize that running the circuit with two identical squeezed state (on two uninteracting modes) and measuring them gives me a different result. It seems that the self.fidelity_vacuum() is the culprit. Multiple calls to the function in one program increments its value when the state gets updated post-measurement (namely the self.weights) on uninteracting modes, when there is "no click" (measurements decrease the weights so it doesn't seem to run into problems). I'm not sure if the problem lies in the use of it in the code I submitted here or if the function itself has a flaw, but this prevents running the test because the vacuum fidelity goes above one in this case. Like I've mentioned previously, the fidelity is also problematic (above one) when calculated on a Fock state of more than 7 photons, even on a clean version of strawberryfields.

Hi @JeyKelly --- That is because two-mode squeezing is different from applying single-mode squeezing in two modes. As for your second question, you are correct that Fock states have a number of numerical issues. For the moment I suggest you set them aside (both in terms of preparing them or measuring them) and focus in threshold detection. Let's worry about Fock measurements in a future PR.

Hi @JeyKelly --- That is because two-mode squeezing is different from applying single-mode squeezing in two modes.

Yes, but I ran into fidelity problems by running the single-mode squeezing in two modes while trying to see where the problem was for two-mode squeezing (sorry, it might have been superfluous to mention it). Vacuum fidelity of the remaining mode increases (above 1) when there is a "no click" event in the first single-mode squeezed state.

Can you share a minimal non-working example of the issue you are finding with the two single-mode squeezed states @JeyKelly ?

Can you share a minimal non-working example of the issue you are finding with the two single-mode squeezed states @JeyKelly ?

import strawberryfields as sf

hbar = 2
nmodes = 2

prog_cat_fock = sf.Program(nmodes)
with prog_cat_fock.context as q:
    sf.ops.Catstate(alpha=0.1)   | q[0]
    sf.ops.Catstate(alpha=0.1)   | q[1]
    sf.ops.MeasureThreshold()    | q[0]
    sf.ops.MeasureThreshold()    | q[1]

eng = sf.Engine("bosonic", backend_options={"hbar": hbar})
result = eng.run(prog_cat_fock)

This will do the trick, or any combination of state close to unity vacuum fidelity (for example, by replacing both states with sf.ops.Squeezed(r=0.1)). When the weights are adjusted here post "no-click" with the c' = c/p0 line, vacuum_fidelity exceeds 1 when calculated for the q[1] mode.

@elib20 : I take the point that I should be equivalent to heterodyne post selected on the \alpha = 0 outcome. But this implementation is still correct. I guess it is hard to see how one is equivalent to the other since heterodyne calls generaldyne. Is your point that more tests should be added? Or that I should use reuse the functionality from heterodyne for the no click case here?


delete from cache by URL

This code add to use FastImage.deleteImage(url) method on ios and android.
It useful, when you want update image with same url or for more cache control.
Example:
FastImage.deleteImage(url).then(deletedImagePath => {
                    console.log(deletedImagePath);
};

Its async method.

is there an update on this? this is something that will be very useful for me. I need a way to delete/refresh the cache for a single image url instead of being forced to delete everything via FastImage.clearDiskCache and FastImage.clearMemoryCache


Add repeats to entity declarations in ioc.yaml

This is to allow repeating elements in ioc.yaml.
(details to be reviewd)
Add a new field repeat of type list to entity that allows you to iterate over a list and instantiate a number of the same entity. The context for each step will include the variables val and idx which can be used to differentiate the individual entities.
examples:
  - type: ADCore.NDPvaPlugin
   repeat: {{ range(5) | list }}
    NDARRAY_PORT: DET.CAM
    P: BL47P-EA-DET-01
    PORT: DET.PVA{{ val }}
    R: ":TX{{ val }}:"
    PVNAME: BL47P-EA-DET-01:TX{ val }}:PVA

  - type: epics.dbpf
    repeat: [ hello, world ]
    pv: BL47P-EA-DET-01:WORD{{ num }}
    value: "{{ val }}"

And combining this with #257 we could do
ioc.yaml
  - type: epics.dbpf
    repeat: {{ words | list }}
    pv: {{ prefix }}:WORD{{ num }}
    value: "{{ val }}"

env.yaml
words: [ now, is, the, time, for, all, good ]
prefix: BL47P-EA-DET-01

use of val and num are a little arbitrary - we could have overrides for these if that seems necessary

I like the concept, but rather than a reserved argument I wonder if it would be clearer to make a nested object:
  - type: ibek.repeat
    variable: index
    values: {{ range(5) | list }}
    entity:
      type: ADCore.NDPvaPlugin
      NDARRAY_PORT: DET.CAM
      P: BL47P-EA-DET-01
      PORT: DET.PVA{{ index }}
      R: ":TX{{ index }}:"
      PVNAME: BL47P-EA-DET-01:TX{{ index }}:PVA

As agreed in G13 - we will go with this.


Fix minor spelling issues.

Can you make this pull request to https://github.com/go-gitea/gitea?

Thanks!


DockPane Simple example in VB.NET

I'm trying to convert the DockpaneSimple sample to VB.NET.  Not able to compile due to 2 issues in BookmarkDockpaneViewModel.cs:

The line 82
_retrieveMapsCommand = new RelayCommand(() => RetrieveMaps(), () => true);
converted to
_retrieveMapsCommand = New RelayCommand(Function() RetrieveMaps(), Function() True)
The compiler is complaining that the expression RetrieveMaps() does not produce a value. RetrieveMaps() is a sub, so it doesn't return a value.

The line 96
ProjectItemsChangedEvent.Subscribe(OnProjectCollectionChanged, false);
converted to
ProjectItemsChangedEvent.Subscribe(OnProjectCollectionChanged, False)
The compiler is complaining that an argument was not specified for parameter 'args' of Private Sub OnProjectCollectionChanged.

To check this out I created a sample Dockpane app and added a button and the subscription.  I used Visual Basic intellisense to give me the syntax.
In regards to your item 1) this code snippet worked for me:
  Private _cmdExit As ICommand

  Protected Sub New()
    _cmdExit = new RelayCommand(AddressOf Execute, AddressOf CanExecute)
  End Sub

  Private Function CanExecute() As Boolean
    Return True
  End Function

  Private Sub Execute(obj As Object)
    MessageBox.Show("Button works!")
  End Sub

  Public ReadOnly Property CmdExit As ICommand
    Get
      Return _cmdExit
    End Get
  End Property

Regarding item 2) this worked for me:

  Protected Sub New()
    ProjectItemsChangedEvent.Subscribe(AddressOf Action)
  End Sub

  Private Sub Action(eventArgs As ProjectItemsChangedEventArgs)
    MessageBox.Show(eventArgs.ProjectItem.Type)
  End Sub