[Bug]:   Unable to install stretchly on an M1 due to "hdiutil: attach failed - no mountable file systems"

Version

[X] I'm using version 1.12.0

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

What operating system are you using?
macOS
Operating System Version
macOS Monterey v12.6
Reproduction steps

Followed instructions to install through brew
Executed brew update && brew install --cask --no-quarantine stretchly
The following error is returned

~ brew update && brew install --cask --no-quarantine stretchly
Already up-to-date.
==> Caveats
This application is not signed. See https://github.com/hovancik/stretchly#application-signing for details.

==> Downloading https://github.com/hovancik/stretchly/releases/download/v1.12.0/stretchly-1.12.0-arm64.dmg
Already downloaded: /Users/allen/Library/Caches/Homebrew/downloads/a54ed94c04df64177f3e7a6aa7def63dfc47aade78c2372569af4bba7232ce43--Stretchly-1.12.0-arm64.dmg
==> Installing Cask stretchly
Warning: macOS's Gatekeeper has been disabled for this Cask
hdiutil: attach failed - no mountable file systems
==> Purging files for version 1.12.0 of Cask stretchly
Error: Failure while executing; `hdiutil attach -plist -nobrowse -readonly -mountrandom /private/tmp/d20221027-35338-vwzikd /private/tmp/d20221027-35338-vwzikd/a54ed94c04df64177f3e7a6aa7def63dfc47aade78c2372569af4bba7232ce43--Stretchly-1.12.0-arm64.cdr` exited with 1. Here's the output:
hdiutil: attach failed - no mountable file systems

Tried to install from dmg as well but it does not open
...
Reproduces how often: 100%

Expected Behavior
Stretchly to be installed and usable
Actual Behavior
Stretchly fails to install due to dmg not opening and brew error above
Relevant log output
~ brew update && brew install --cask --no-quarantine stretchly
Already up-to-date.
==> Caveats
This application is not signed. See https://github.com/hovancik/stretchly#application-signing for details.

==> Downloading https://github.com/hovancik/stretchly/releases/download/v1.12.0/stretchly-1.12.0-arm64.dmg
Already downloaded: /Users/allen/Library/Caches/Homebrew/downloads/a54ed94c04df64177f3e7a6aa7def63dfc47aade78c2372569af4bba7232ce43--Stretchly-1.12.0-arm64.dmg
==> Installing Cask stretchly
Warning: macOS's Gatekeeper has been disabled for this Cask
hdiutil: attach failed - no mountable file systems
==> Purging files for version 1.12.0 of Cask stretchly
Error: Failure while executing; `hdiutil attach -plist -nobrowse -readonly -mountrandom /private/tmp/d20221027-35338-vwzikd /private/tmp/d20221027-35338-vwzikd/a54ed94c04df64177f3e7a6aa7def63dfc47aade78c2372569af4bba7232ce43--Stretchly-1.12.0-arm64.cdr` exited with 1. Here's the output:
hdiutil: attach failed - no mountable file systems

Preferences
N/A (not installed)

Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I don't have m1 to check unfortunately :(
Googling around found similar issues with electron apps but no answers there: https://github.com/balena-io/etcher/issues/3086 https://github.com/evolus/pencil/issues/622
So tried searching this error in electron or electron-builder, but no luck

It's super strange because stretchly installed on my other m1 just fine.

@hovancik found the root cause. I was trying to install stretchly on my work laptop so I don't stare at the screen for 8 hours straight. JAMF (apple device management software 🤮) has a setting that prevents software from being installed without direct approval from admin. Super annoying but completely unrelated to stretchly itself. Thank you for the quick turn around!


Attempting to load invalid JSON in body raises exception

This change catches the exception when invalid JSON is provided to json.loads the or "{}" that was in the code after json.loads didn't actually do anything, as the JSON libarary raises an expection vs return . I believe this change moves the functionality to expected behaviour i.e. if the JSON fails to load then an empty dictionary is the result.

@andyfase i think we need to be explicit when failing on loading incoming json data and failing on it, especially when the handler deliberatly has chosen to "load_json": https://github.com/sloev/python-lambdarest/blob/master/lambdarest/init.py#L365
i suggest we instead raise a custom malformed_json_error
that we then can catch here: https://github.com/sloev/python-lambdarest/blob/master/lambdarest/init.py#L325
and return a proper bad-request-400 response
cause if the handler expects json and is given a body that cant be loaded then client should know i think :-)

@andyfase i think we need to be explicit when failing on loading incoming json data and failing on it, especially when the handler deliberatly has chosen to "load_json": https://github.com/sloev/python-lambdarest/blob/master/lambdarest/init.py#L365
i suggest we instead raise a custom malformed_json_error
that we then can catch here: https://github.com/sloev/python-lambdarest/blob/master/lambdarest/init.py#L325
and return a proper bad-request-400 response
cause if the handler expects json and is given a body that cant be loaded then client should know i think :-)

Yes your right - I missed that you can pass in load_json=False in the decorator. I was just confused by the or {} statement. Ok let me rework this and update the PR.

Yes your right - I missed that you can pass in load_json=False in the decorator. I was just confused by the or {} statement. Ok let me rework this and update the PR.

@andyfase sweet, looking forward to it!!!
:-) thanks!

@andyfase sweet, looking forward to it!!!
:-) thanks!

@andyfase i'm gonna close down this pr if no new content last 20 days :-)


Update reserve() method and tests to stop parsing headers

Small change since https://github.com/CVEProject/cve-services/issues/427 was fixed in https://github.com/CVEProject/cve-services/pull/475.

Code looks good!
I changed the base branch to merge this into to cve-services-2.1.0 since the fix is not yet available in production (which is currently running 1.1.1). Once 2.1.0 is released, we can merge our branch to master and make a new release. We can also add all other 2.1.0 features onto this branch going forward.


Support for parsing with a variable number of separator

Is it possible to support parsing an arbitrary number of separators as if they were one? e.g.
let line = "ID: 5, favourite numbers: 1---6-2-6---3-3333---------4"
let res: (usize, vec<usize>) = prse::parse!(line,"ID: {}, favourite numbers: {:-:}");
println!("{:?}",res.1") // 1,6,2,6,3,3333,4

Same issue. Possible solution (not perfect, but works pretty well):
let (res0, it): (usize, _) = prse::parse!(line,"ID: {}, favourite numbers: {:-:0}");
let res1: Vec<usize> = it.filter(|s|s!=Ok("")) //filter out empty strings between separators
                                      .map(|x|x.unwrap().parse::<usize>().unwrap())
                                      .collect();

In fact, I'm using try_parse!, and ? operator makes it even shorter.
For single variables (not iterators) with unknown number of separators, scan into &str and use trim.

I've opened a pull request for this

Shouldn't you bump the version number? It's a major change, maybe 1.1.0?

Yep, just have few more changes I want to make before pushing a release out


Fix incorrect output dimensions in stacked bijectors

Fixes #2065 (still needs testing)

Ok, is the revision in Project.toml automatically updated?

Ok so turns out that wasn't the fix, I had to add dispatches for output_length and transform. Please take a look and then run the tests again when you're ready.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/variational/advi.jl
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/variational/advi.jl
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
1492

💛  - Coveralls

I don't think this test failure is related to this PR...? but I am not sure from first glance.

FWIW the tests passed locally for me.
Is anything else needed before merging?


Upgrade jackson-databind to 2.9.10

jackson-databind <IP_ADDRESS> is vulnerable to several deserialization vulnerabilities. The latest stable version of 2.9 (2.9.10) mitigates these.
Fixes #2031

Thanks @mhitchens for this and #2030

@loosebazooka No problem. Speaking of this and #2030, is there a jib release planned that would incorporate those?


Validate VAT Reg.No. format does not work correct

Description:
Validate VAT Reg.No. format does not work correct.
If you leave out the country code, VAT will still work in Germany and you can register.
This should not be the case.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37571.
[created from NEXT-37571, comment 486928]

[public]

FixVersion Change: <IP_ADDRESS>, <IP_ADDRESS>

This issue will be / was solved in Shopware version <IP_ADDRESS>, <IP_ADDRESS>.
[created from NEXT-37571, comment 510432]


Add change type alias

@nomisRev, I re-opened ChangeTypeAlias along with more tests to show that the recipe handles every case aside from Import aliases.
I've created a new issue to discover how we may add type attribution to imports, since the information does not exist on the FIR.
Note: That ChangeTypeAlias does a lot more than a search and replace, because it tracks name space conflicts, guarantees the name is unique, and will update all the references to the type alias while ignoring shadowed names.


ExpressionBuilder.ExpressionFunctions is in different namespace

There is a note explaining what the EF shorthand is for

EF is a shorthand “using” notation to define ExpressionBuilder.ExpressionFunctions.

It seems that at some point in time this changed to Microsoft.Toolkit.Uwp.UI.Animations.Expressions.ExpressionFunctions

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

ID: 859dcabb-2805-6fd7-2816-22eb0f8e8a48
Version Independent ID: 569050ed-7e91-680a-6992-0e4cf7145dd5
Content: Relation based animations - UWP applications
Content Source: windows-apps-src/composition/relation-animations.md
Product: uwp
Technology: platform
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

Am I supposed to close this ticket?


ParticleEditor - Ability to select emiter as particle

This would allow you to make beautiful particle explosions with sparkling particles flying around

We'd appreciate PRs, i don't believe anyone on the team will implement this.


[Bug] Segmentation fault

Your current environment information
ibibverbs not available, ibv_fork_init skipped
Collecting environment information...
PyTorch version: 2.1.1+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A
OneFlow version: path: ['/opt/conda/lib/python3.10/site-packages/oneflow'], version: 0.9.1.dev20241019+cu118, git_commit: d23c061, cmake_build_type: Release, rdma: True, mlir: True, enterprise: False
Nexfort version: none
OneDiff version: 1.2.1.dev15+g241fe57d
OneDiffX version: none
GCC version: (GCC) 8.5.0 20210514 (Red Hat 8.5.0-22)
Clang version: Could not collect
CMake version: version 3.30.4
Libc version: glibc-2.28
Python version: 3.10.14 (main, Mar 21 2024, 16:24:04) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-5.4.241-1-tlinux4-0017.7-x86_64-with-glibc2.28
Is CUDA available: True
CUDA runtime version: 12.1.105
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA L40
Nvidia driver version: 525.125.06
cuDNN version: Probably one of the following:
/usr/lib64/libcudnn.so.8.9.7
/usr/lib64/libcudnn_adv_infer.so.8.9.7
/usr/lib64/libcudnn_adv_train.so.8.9.7
/usr/lib64/libcudnn_cnn_infer.so.8.9.7
/usr/lib64/libcudnn_cnn_train.so.8.9.7
/usr/lib64/libcudnn_ops_infer.so.8.9.7
/usr/lib64/libcudnn_ops_train.so.8.9.7
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              192
On-line CPU(s) list: 0-191
Thread(s) per core:  2
Core(s) per socket:  96
Socket(s):           1
NUMA node(s):        1
Vendor ID:           AuthenticAMD
CPU family:          25
Model:               17
Model name:          AMD EPYC 9K84 96-Core Processor
Stepping:            0
CPU MHz:             2600.034
BogoMIPS:            5200.06
Hypervisor vendor:   KVM
Virtualization type: full
L1d cache:           32K
L1i cache:           32K
L2 cache:            1024K
L3 cache:            32768K
NUMA node0 CPU(s):   0-191
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm rep_good nopl cpuid extd_apicid amd_dcm tsc_known_freq pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw topoext perfctr_core invpcid_single ibpb vmmcall fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 avx512_bf16 clzero xsaveerptr wbnoinvd arat avx512vbmi umip avx512_vbmi2 vaes vpclmulqdq avx512_vnni avx512_bitalg avx512_vpopcntdq rdpid fsrm
Versions of relevant libraries:
[pip3] diffusers==0.30.3
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.26.4
[pip3] onnx==1.17.0
[pip3] onnxruntime==1.19.2
[pip3] onnxruntime-gpu==1.18.0
[pip3] open-clip-torch==2.20.0
[pip3] torch==2.1.1
[pip3] torchaudio==2.1.1
[pip3] torchsde==0.2.6
[pip3] torchvision==0.16.1
[pip3] transformers==4.44.2
[pip3] triton==2.1.0
[conda] blas                      1.0                         mkl
[conda] cudatoolkit               11.8.0               h6a678d5_0
[conda] mkl                       2023.1.0         h213fc3f_46344
[conda] mkl-service               2.4.0           py310h5eee18b_1
[conda] mkl_fft                   1.3.10          py310h5eee18b_0
[conda] mkl_random                1.2.7           py310h1128e8f_0
[conda] numpy                     1.26.4          py310h5f9d8c6_0
[conda] numpy-base                1.26.4          py310hb5e798b_0
[conda] open-clip-torch           2.20.0                   pypi_0    pypi
[conda] pytorch-cuda              12.1                 ha16c6d3_5    pytorch
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] torch                     2.1.1                    pypi_0    pypi
[conda] torchaudio                2.1.1               py310_cu121    pytorch
[conda] torchsde                  0.2.6                    pypi_0    pypi
[conda] torchvision               0.16.1                   pypi_0    pypi
[conda] triton                    2.1.0                    pypi_0    pypi
🐛 Describe the bug
I encountered a Segmentation fault issue while using OneDiff in ComfyUI, with no additional error information. I am seeking some assistance.
I encountered two types of Segmentation fault errors:
Stack trace (most recent call last) in thread 256218:
Segmentation fault (Signal sent by the kernel [(nil)])

Stack trace (most recent call last) in thread 417043:
Segmentation fault (Address not mapped to object [(nil)])

After testing, this error is unrelated to a single workflow and is also unrelated to previously executed workflows. I currently have no leads and have not been able to reproduce it again; the occurrence seems to be quite random.

This error is likely originating from OneFlow, and I have also seen similar issues in other issues(#393  #1080 ), but none of them have provided a solution.
@strint Could you please take a look at this, or do you have any suggestions for a solution?


Checkpoints for the fine-tuned models

Hi,
Thank you for the amazing repo and documentation. Would it be possible to provide the best checkpoints for the model fine-tuned on the MaRVL task?
Thanks

Hi,
I sadly do not have the time right now to prepare the checkpoints for the fine-tuned models. However, training on Marvl/NLVR2 took us less than 2 days with a single 3090 so you should be able to quickly train your own model.

No problem. Thank you!


Using bootstrap in README is causing symbol void error

Using the bootstrap code that is currently in the README (displayed on the package's homepage) results in an error. To be clear, the code is
(let ((bootstrap-file (concat user-emacs-directory "straight/bootstrap.el"))
      (bootstrap-version 2))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

which results in the following error:

Bootstrapping straight.el...
if: straight.el bootstrap failed: Symbol’s value as variable is void: straight-repository-branch

Using straight.el/master/install.el works, however:
(let ((bootstrap-file (concat user-emacs-directory "straight/bootstrap.el"))
      (bootstrap-version 2))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/master/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

t

This was very much a dumb mistake on my part, sorry. The problem should be fixed now with no changes necessary on your end; please let me know if it is not.

Lovely. Fixed on my end.

got hit by this bug in 2019!

@shackra Can't reproduce. Can you open a new issue with the full contents of your init-file, your operating system, and Emacs version?

It was fixed by changing the branch develop to master
no rush, dear Sir


make install failed

by compiling the current version I have an error like
cmake -DCMAKE_INSTALL_PREFIX=/usr .
make install
[100%] Built target caer
Install the project...
-- Install configuration: "Release"
-- Installing: /usr/lib/x86_64-linux-gnu/pkgconfig/libcaer.pc
CMake Error at cmake_install.cmake:36 (file):
file INSTALL cannot copy file "/home/binh/Neuromorphics/libcaer/libcaer.pc"
to "/usr/lib/x86_64-linux-gnu/pkgconfig/libcaer.pc".
Makefile:61: recipe for target 'install' failed
make: *** [install] Error 1
Could you help me?

Are you running as root? To copy files to system directories you'll need to run make install as root or via sudo.

you are right. I just follow your README without thinking.
Please change your README
make install
into
sudo make install
best thanks


GKE Failed: Create Address error`

Hey, I'm running on Google Container Engine. Initially on 1.5.6, but then I upgraded to 1.6.1 (which I afterwards learned that it would break all git push, but...gave me an excuse to setup docker build for all images, so something positive came out of it).
Latest version of Workflow.
First issue I saw was that suddenly some of my requests would hang forever. I manually added a healthcheck to the load balancer, and that worked around the issue:

Then I decided now to check the cloud console, and I saw a lot of errors related to Deis Router:

Any suggestions on how to fix this would be appreciated. I'm currently migrating everything from Deis 1.13 on AWS to Deis Workflow on GKE.

This looks like an internal GCP error having nothing to do with the router other than a coincidence that the IP it's unable to reserve happens to be for the router.

I "Fixed" it by reconfiguring deis-router as type: ClusterIP, and then as type: LoadBalancer (Which created a new load balancer).
Could be related to my experimentation. I'll close this issue, and hope that the issue doesn't surface again.


Add parameter for optional DB init scripts.

The PostgreSQL docker image allows mounting a volume containing SQL scripts that are executed when the docker image is started. This PR exposes this functionality in the Github Action.

hi @danielweller-swp, pwd is not working with this docker in docker case. The first part of -v must be the host path, which is /home/runner/work/{repo}/{repo} in this case.

hi @danielweller-swp, pwd is not working with this docker in docker case. The first part of -v must be the host path, which should be /home/runner/work/{repo}/{repo} in this case.
It costs me a whole damn day to figure it out :(

Thanks, you are completely correct - this is not working as intended. Sorry for the inconvenience it caused! I'll try to figure out whether this can be handled in a generic way and update this PR.

@hank-cp: Thanks again for your input. I fixed the action to correctly compute the path and added a test for the action (in form of a Github Action workflow).

@danielweller-swp  Cool and THX~
Not sure when this PR will be accepted, but I actually would like to add some more changes upon your PR to support inject postgres.conf.
According to postgres docker's doc
$ docker run -d --name some-postgres -v "$PWD/my-postgres.conf":/etc/postgresql/postgresql.conf postgres -c 'config_file=/etc/postgresql/postgresql.conf'

could do the job.

as repo maintainer is not here can we maintain another repo?

@shirshak55: I've created a fork on my account, I guess I can maintain this for a while.
@hank-cp: Feel free to submit your PR there.


Include warnings against using volatile

There seems to be a strong feeling within the C# language community (e.g. gitter discussions) that volatile should no be used, there are other ways of dealing with the problems it was intended to help solve.
The existing documentation does nothing to discourage use of volatile yet it seems most experts think it should be discouraged.
Please add information or guidance or at least some clarification about this and if volatile has been superseded these days by other methods (interlocked API etc) then please explain that.

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

ID: d226514d-c7cb-0d1b-b9dc-fb791399dc47
Version Independent ID: aa5d1078-7f89-98ce-7fdc-107c9ed3eebc
Content: volatile - C# Reference
Content Source: docs/csharp/language-reference/keywords/volatile.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

That's a great suggestion @Korporal
Can you add a permalink to the recent gitter discussions?
I've added this to our backlog to address as we update the language referenced.

@BillWagner - Is this helpful?
https://gitter.im/dotnet/csharplang?at=607b146006e2e024e85cad8c


build(deps): bump @sentry/react-native from 4.12.0 to 5.14.1

This PR resolves PHIRE-485
Description
Bumps sentry to latest

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Need help with something? Have a look at our docs, or get in touch with us.

sourcemaps work we can merge


It's causing Activity to leak

Hi,
It's a wonderful library which makes it quite easy to fetch device location. However, I am facing memory leak situation and I believe it's due to library.
I have an Activity A which opens Activity B. B uses this library to fetch the location.
If I press back button on B, it moves to A. LeakCanary then looks for retained instances and gives a total of 4 retained instances.
I checked the code of library as the developer has recently implemented Lifecycle awareness and it seems good. All listeners are getting removed in the code. But I think the context I am passing in LocationProvider.Builder() is causing the leak as suggested by LeakCanary.
I am using the addObserver after initialising LocationProvider and removeObserver in onDestroy() as I saw in sample code.
Kindly have a look :)

Searching through the web, I got this: https://github.com/googlesamples/android-play-location/issues/26
It seems the memory leak bug is in Google Play Services' LocationListener itself and not this library.
To solve the issue, I passed applicationContext to the library instead of activity context.


Merge different approaches to generate devicetree

It would be good for LiteX to generate suitable device tree for Linux / Zephyr (potentially micropython in future).
I started (but never finished) a tool to take a csr.cvs file and generate a Linux Device Tree file - see https://github.com/mithro/litex-devicetree
It was targeting producing suitable dts for or1k as part of the LCA2018 Miniconf tutorial (see https://github.com/timvideos/litex-buildenv/wiki/HowTo-LCA2018-FPGA-Miniconf). It seems to be used here;
https://github.com/timvideos/litex-buildenv/blob/005f7463bd5e022ee3e3d8cc90efde57e8465b86/scripts/build-linux.sh#L97-L131
A much better solution would be to allow modules to provide devicetree fragments in the same way that they can provide CSRs and stuff.

I agree, let's track this in https://github.com/litex-hub/litex-devicetree (i just transfered the issue), once we'll have something more generic than the actual json2dts.py of linux-on-litex-vexriscv, we could also use it there.


🐛 [firebase_storage] Error while uploading image to firebase storage

Bug report
Describe the bug
On the web platform, when uploading a file to the storage, the debugger pauses and some errors are printed.
However, the file is uploaded successfully.
Sample code
As it's on the web, the accurate code that causes error is not known.
But by looking at the error logs, it seems subscribing to snapshotEvents is where it happens.
final UploadTask uploadTask = userRef.putData(
  await file.readAsBytes(),
  SettableMetadata(contentType: file.mimeType),
);
uploadTask.snapshotEvents.listen((TaskSnapshot event) async {
  if (event.state == TaskState.running) {
    final double progress = event.bytesTransferred / event.totalBytes;
    EasyLoading.showProgress(
      progress,
      status:
          'Uploading Image...\n${(progress * 100).toStringAsFixed(0)}% Completed',
    );
  }
});

await uploadTask.whenComplete(() => null);

Expected behavior
File should upload without any errors.

Additional context
This is only happening on web after updating from ^10.3.2 to ^10.3.3

Storage Upload Error Logs
TypeError: Cannot read properties of undefined (reading 'STATE_CHANGED')
at startListen (http://localhost:60869/packages/firebase_storage_web/src/interop/storage.dart.lib.js:561:97)
    at Object._runGuarded (http://localhost:60869/dart_sdk.js:40843:7)
    at [_subscribe] (http://localhost:60869/dart_sdk.js:34405:17)
    at [_createSubscription] (http://localhost:60869/dart_sdk.js:33673:46)
    at _BroadcastStream.new.listen (http://localhost:60869/dart_sdk.js:33638:53)
    at new _ForwardingStreamSubscription.new (http://localhost:60869/dart_sdk.js:38261:55)
    at [_createSubscription] (http://localhost:60869/dart_sdk.js:38177:16)
    at _MapStream.new.listen (http://localhost:60869/dart_sdk.js:38174:41)
at [_listenToStream] (http://localhost:60869/packages/async/src/stream_group.dart.lib.js:225:35)
at [_onListen] (http://localhost:60869/packages/async/src/stream_group.dart.lib.js:168:70)
    at Object._runGuarded (http://localhost:60869/dart_sdk.js:40843:7)
    at [_subscribe] (http://localhost:60869/dart_sdk.js:34405:17)
    at [_createSubscription] (http://localhost:60869/dart_sdk.js:33673:46)
    at _BroadcastStream.new.listen (http://localhost:60869/dart_sdk.js:33638:53)
    at new _ForwardingStreamSubscription.new (http://localhost:60869/dart_sdk.js:38261:55)
    at [_createSubscription] (http://localhost:60869/dart_sdk.js:38177:16)
    at _HandleErrorStream.new.listen (http://localhost:60869/dart_sdk.js:38174:41)
    at new _ForwardingStreamSubscription.new (http://localhost:60869/dart_sdk.js:38261:55)
    at [_createSubscription] (http://localhost:60869/dart_sdk.js:38177:16)
    at _MapStream.new.listen (http://localhost:60869/dart_sdk.js:38174:41)
at uploadImageToFirebase (http://localhost:60869/packages/whats_that_anime/pages/HomePage/functions/upload_image_to_firebase.dart.lib.js:46:35)
    at uploadImageToFirebase.next (<anonymous>)
    at http://localhost:60869/dart_sdk.js:40641:33
    at _RootZone.runUnary (http://localhost:60869/dart_sdk.js:40511:59)
    at _FutureListener.thenAwait.handleValue (http://localhost:60869/dart_sdk.js:35438:29)
    at handleValueCallback (http://localhost:60869/dart_sdk.js:35999:49)
    at _Future._propagateToListeners (http://localhost:60869/dart_sdk.js:36037:17)
    at [_complete] (http://localhost:60869/dart_sdk.js:35864:25)
    at Object._cancelAndValue (http://localhost:60869/dart_sdk.js:40905:24)
    at http://localhost:60869/dart_sdk.js:21397:17
    at Object._checkAndCall (http://localhost:60869/dart_sdk.js:5279:16)
    at Object.dcall (http://localhost:60869/dart_sdk.js:5284:17)
    at FileReader.<anonymous> (http://localhost:60869/dart_sdk.js:108960:106)

But surprisingly, the debugger pauses for exception in

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.22622.436], locale en-IN)
Checking Android licenses is taking an unexpectedly long time...[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.3)
[√] Android Studio (version 2021.2)
[√] VS Code (version 1.69.2)
[√] Connected device (3 available)
[√] HTTP Host Availability

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.17.6
Flutter SDK 3.0.5
whats_that_anime 2.1.0+4

dependencies:
- cached_network_image 3.2.1 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- chewie 1.3.4 [cupertino_icons flutter provider video_player wakelock]
- cloud_firestore 3.4.0 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- cupertino_icons 1.0.5
- desktop_drop 0.3.3 [flutter flutter_web_plugins cross_file]
- dotted_border 2.0.0+2 [flutter path_drawing]
- firebase_auth 3.5.0 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.20.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_storage 10.3.3 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]
- flutter_easyloading 3.0.5 [flutter flutter_spinkit]
- flutter_native_splash 2.2.5 [args flutter flutter_web_plugins image js lint meta path universal_io xml yaml]
- get_storage 2.0.3 [flutter get path_provider]
- http 0.13.4 [async http_parser meta path]
- image_picker 0.8.5+3 [flutter image_picker_android image_picker_for_web image_picker_ios image_picker_platform_interface]
- internet_connection_checker_plus 1.0.1 [flutter http]
- motion_toast 2.3.0 [flutter]
- package_info_plus 1.4.3 [flutter package_info_plus_platform_interface package_info_plus_linux package_info_plus_macos package_info_plus_windows package_info_plus_web]
- paginate_firestore 1.0.3 [flutter bloc flutter_bloc cloud_firestore provider]
- pasteboard 0.2.0 [flutter flutter_web_plugins js]
- url_launcher 6.1.5 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]
- video_player 2.4.5 [flutter html video_player_android video_player_avfoundation video_player_platform_interface video_player_web]

dev dependencies:
- change_app_package_name 1.1.0
- flutter_lints 2.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph]
- icons_launcher 1.2.1 [args image path universal_io yaml]

transitive dependencies:
- archive 3.3.1 [crypto path]
- args 2.3.1
- async 2.8.2 [collection meta]
- bloc 7.2.1 [meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.1 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.2.0
- charcode 1.3.1
- clock 1.1.0
- cloud_firestore_platform_interface 5.7.0 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.8.0 [cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- collection 1.16.0
- cross_file 0.3.3+1 [js meta]
- crypto 3.0.2 [typed_data]
- csslib 0.17.2 [source_span]
- fake_async 1.3.0 [clock collection]
- ffi 2.0.1
- file 6.1.2 [meta path]
- firebase_auth_platform_interface 6.4.0 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 4.1.0 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.5.0 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 1.7.1 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_storage_platform_interface 4.1.12 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 3.3.1 [async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]
- flutter_bloc 7.3.3 [flutter bloc provider]
- flutter_blurhash 0.7.0 [flutter]
- flutter_cache_manager 3.3.0 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_plugin_android_lifecycle 2.0.7 [flutter]
- flutter_spinkit 5.1.0 [flutter]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- get 4.6.5 [flutter]
- html 0.15.0 [csslib source_span]
- http_parser 4.0.1 [collection source_span string_scanner typed_data]
- image 3.2.0 [archive meta xml]
- image_picker_android 0.8.5+1 [flutter flutter_plugin_android_lifecycle image_picker_platform_interface]
- image_picker_for_web 2.1.8 [flutter flutter_web_plugins image_picker_platform_interface]
- image_picker_ios 0.8.5+6 [flutter image_picker_platform_interface]
- image_picker_platform_interface 2.5.0 [cross_file flutter http plugin_platform_interface]
- intl 0.17.0 [clock path]
- js 0.6.4
- lint 1.8.2
- lints 2.0.0
- matcher 0.12.11 [stack_trace]
- material_color_utilities 0.1.4
- meta 1.7.0
- nested 1.0.0 [flutter]
- octo_image 1.0.2 [flutter flutter_blurhash]
- package_info_plus_linux 1.0.5 [package_info_plus_platform_interface flutter path]
- package_info_plus_macos 1.3.0 [flutter]
- package_info_plus_platform_interface 1.0.2 [flutter meta plugin_platform_interface]
- package_info_plus_web 1.0.5 [flutter flutter_web_plugins http meta package_info_plus_platform_interface]
- package_info_plus_windows 2.0.0 [package_info_plus_platform_interface ffi flutter win32]
- path 1.8.1
- path_drawing 1.0.0 [vector_math meta path_parsing flutter]
- path_parsing 1.0.0 [vector_math meta]
- path_provider 2.0.11 [flutter path_provider_android path_provider_ios path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- path_provider_android 2.0.16 [flutter path_provider_platform_interface]
- path_provider_ios 2.0.10 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.7 [ffi flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.6 [flutter path_provider_platform_interface]
- path_provider_platform_interface 2.0.4 [flutter platform plugin_platform_interface]
- path_provider_windows 2.1.0 [ffi flutter path path_provider_platform_interface win32]
- pedantic 1.11.1
- petitparser 5.0.0 [meta]
- platform 3.1.0
- plugin_platform_interface 2.1.2 [meta]
- process 4.2.4 [file path platform]
- provider 6.0.3 [collection flutter nested]
- rxdart 0.27.5
- sky_engine 0.0.99
- source_span 1.8.2 [collection path term_glyph]
- sqflite 2.0.3 [flutter sqflite_common path]
- sqflite_common 2.2.1+1 [synchronized path meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.0 [charcode source_span]
- synchronized 3.0.0+2
- term_glyph 1.2.0
- test_api 0.4.9 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.1 [collection]
- universal_io 2.0.4 [collection crypto meta typed_data]
- url_launcher_android 6.0.17 [flutter url_launcher_platform_interface]
- url_launcher_ios 6.0.17 [flutter url_launcher_platform_interface]
- url_launcher_linux 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_macos 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_platform_interface 2.1.0 [flutter plugin_platform_interface]
- url_launcher_web 2.0.12 [flutter flutter_web_plugins url_launcher_platform_interface]
- url_launcher_windows 3.0.1 [flutter url_launcher_platform_interface]
- uuid 3.0.6 [crypto]
- vector_math 2.1.2
- video_player_android 2.3.8 [flutter video_player_platform_interface]
- video_player_avfoundation 2.3.5 [flutter video_player_platform_interface]
- video_player_platform_interface 5.1.3 [flutter plugin_platform_interface]
- video_player_web 2.0.12 [flutter flutter_web_plugins video_player_platform_interface]
- wakelock 0.6.2 [flutter meta wakelock_macos wakelock_platform_interface wakelock_web wakelock_windows]
- wakelock_macos 0.4.0 [flutter flutter_web_plugins wakelock_platform_interface]
- wakelock_platform_interface 0.3.0 [flutter meta]
- wakelock_web 0.4.0 [flutter flutter_web_plugins js wakelock_platform_interface]
- wakelock_windows 0.2.0 [flutter wakelock_platform_interface win32]
- win32 2.7.0 [ffi]
- xdg_directories 0.2.0+1 [meta path process]
- xml 6.1.0 [collection meta petitparser]
- yaml 3.1.1 [collection source_span string_scanner]

Hello,
Thanks for the report, I can reproduce on the demo app


go does not pull in packages

Does not work!
dep ensure -add github.com/gizak/termui

The above fails:
could not find project Gopkg.toml, use dep init to initiate a manifest

without doing the above, go does not fetch packages using /v3

Apart from that
Apart from that the thing looks really great, except I cannot make it work!
Go get <---------->
go get fails as it says there is no toml file
Please let me know how to use this
thanks

Go modules???
in the .md file it says:

It is not necessary to go get termui, since Go will automatically manage any imported >dependencies for you. Do note that you have to include /v3 in the import statements as shown in >the 'Hello World' example below.

However, this does not seem to work at all

@zorgrian have you tried doing
$ go mod init github.com/zorgrian/yourpkgname

then try running the code?


add correct check for validation ret value

Codecov Report

Merging #309 (c734c84) into main (6e1e212) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #309   +/-   ##
=======================================
  Coverage   81.78%   81.78%
=======================================
  Files          18       18
  Lines        2437     2437
  Branches      425      425
=======================================
  Hits         1993     1993
  Misses        298      298
  Partials      146      146

Files Changed
Coverage Δ

src/ansys/pyensight/core/utils/adr.py
36.84% <0.00%> (ø)


Add Receiver::new_sender to channels

This change gives some channel::Receivers the ability to spawn new senders at will.
Currently, Receivers are disconnected as soon as the Sender count reaches zero, as new Senders can only be created by cloning an existing one. Disconnected receivers do not block on recv and such, instead they just fail immediately.
This didn't quite fit my use case, where I need to have the ability to create new Senders when I want to, and also get the non-blocking behavior if there is no Sender alive. With the current API, I have to keep a prototype Sender which is always alive and which I can clone whenever I want. The problem is that since the prototype is always alive, the channel never gets disconnected. That means calls to recv stay blocking, even when we know that no message will ever be sent through the Sender prototype, because its only purpose is to produce new Senders.
This change creates a new Receiver type that supports creating new Sender instances. That means, I don't need to keep a Sender prototype anymore, so my Receiver is non-blocking when I know there is no real Sender alive. But even after the channel was disconnected, you can "reconnect" it by spawning new senders.
Obviously this method cannot appear in the API of channel flavours like tick or never, since the point of those is to expose only a receiver. This means, we need two different types for Receiver. I put that new API in a separate submodule so that it's easy to import and document. That submodule only supports creating unbounded channels for now, as that was my use case. If you're interested we may be able to add support for bounded and zero channels, but there is AFAIK no risk in keeping an incomplete API here.
Please let me know if there is anything you need from me to get this merged faster, be it documentation, tests, etc.
Cheers :)

hey, this api is interesting to me as well.

Obviously this method cannot appear in the API of channel flavours like tick or never, since the point of those is to expose only a receiver. This means, we need two different types for Receiver. I put that new API in a separate submodule so that it's easy to import and document. That submodule only supports creating unbounded channels for now, as that was my use case. If you're interested we may be able to add support for bounded and zero channels, but there is AFAIK no risk in keeping an incomplete API here.

However, I didn't particularly like this submodule arrangement... How about making new_sender returning some dummy sender for tick and never? maybe this will be called full()?

ping @oowekyala @taiki-e regarding my api idea: https://github.com/crossbeam-rs/crossbeam/pull/750#issuecomment-1876415286


The plugin is very slow

After the change where it pops up in new window. The plugin has become very slow. Even scroll up and down of response takes for ever.
Expected outcome
What should happen?
Actual outcome
What happened?
Versions
App: <IP_ADDRESS>-stable
Chrome: 49.0.2623.112
Platform: Windows (7)
Steps to reproduce

Thank you for issue report.
I agree that in some cases (like really big request) it can take some time to scroll down/up. But it's a natural behavior and it works as intended.


Add a <EntityPicker> field to the scaffolder

This PR introduce an additional EntityPicker to the already existing OwnerPicker.  While this is already possible with the OwnerPicker it feels wrong to choose that one as it's designed for choosing an owner and I also wanted other defaults for the defaultKind. So I made the defaultKind configurable.
I also decided to change the implementation of the OwnerPicker to be based on the EntityPicker, just with a different default configuration. But based on this comment one could argue that their implementations might differ in the future. However, for now I'm not sure if that is actually planned. I put it into a separate commit if you don't want to include these changes.
Based on that you could also argue whether a generic EntityPicker is a good idea or not. Some use cases may ask for a more specific picker, but I think these are tied closely to the organization. But if you are just starting with the scaffolder I think it's easier to have a more generic field to get started and be able to migrate to a more specific one later. All of this should be better than bloating up the specific OwnerPicker to support more generic use cases 😉
PS: This PR also contains a small fix for a missing key property on a react element.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Think we probably want to merge #5677 first, and then use that method to expose this EntityPicker too?

Think we probably want to merge #5677 first, and then use that method to expose this EntityPicker too?

@benjdlambert  Sure, I can adjust the PR once it is merged

@Rugvip @benjdlambert  Now it is merged, I gave this one an update

As a separate issue I'm thinking that we need to create a docs route similar to /scaffolder/actions that can list the custom fields and their corresponding parameters yaml.

Is defaultKind: System required for EntityPicker in order to strip out the system: prefix here? @Fox32

I am probably just misinterpreting the intent/usage of the EntityPicker, but I got to this same thread for the same reason that @regicsolutions and @paul-cummings did, which was to figure out a way to end up with values selected by the EntityPicker/OwnerPicker that did not include the "group:default" prefix. For example, instead of a value of "group:default/platform team", just return "platform-team".
I am using v1.12.0 and the NextScaffolderPage, so using all the latest stuff. Below is the parameter from my template.yaml where I am using the EntityPicker and trying to just get back the group value selected without the prefixed stuff.
domain:
  title: Domain
  type: string
  description: What domain does this app belong to?
  ui:field: EntityPicker
  ui:options:
    allowedKinds:
      - Group
    defaultKind: Group

However, ${{ parameters.domain }} returns "group:default/platform-team". Is what I am trying to do possible with the EntityPicker out of the box, or do I need to create my own picker implementation based on the EntityPicker to do this?

Anyone have an answer to @my3sons question? Is there a way just to reference the name not the prefix? The prefix is causing issues when passed in the steps of the scaffolder.


[Bug]: distutils.version deprecated

What happened?
I got a warning that distutils.version is deprecated, and the version comparison does not work locally.
Steps to Reproduce
While writing a conversion for https://datadryad.org/stash/dataset/doi:10.5061%2Fdryad.d2547d840 I used the following code:
from datetime import datetime
from neuroconv.datainterfaces import SpikeGLXRecordingInterface

file_path = "/Users/bendichter/Downloads/doi_10.5061_dryad.d2547d840__v9/Pt01_g0/Pt01_g0_imec0/Pt01.imec0.ap.bin"

interface = SpikeGLXRecordingInterface(file_path)

metadata = interface.get_metadata()
metadata["NWBFile"].update(session_start_time=datetime(1900,1,1))
interface.run_conversion("Pt01_out.nwb", metadata=metadata)

This returns a warning and an error.
/Users/bendichter/dev/neuroconv/src/neuroconv/tools/spikeinterface/spikeinterface.py:1003: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  distutils.version.LooseVersion(pynwb.__version__) >= "1.3.3"
/Users/bendichter/opt/miniconda3/lib/python3.9/site-packages/setuptools/_distutils/version.py:346: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)

I am also getting a traceback:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In [4], line 1
----> 1 interface.run_conversion("Pt01_out.nwb", metadata=metadata)

File ~/dev/neuroconv/src/neuroconv/datainterfaces/ecephys/baserecordingextractorinterface.py:173, in BaseRecordingExtractorInterface.run_conversion(self, nwbfile_path, nwbfile, metadata, overwrite, stub_test, starting_time, use_times, write_as, write_electrical_series, es_key, compression, compression_opts, iterator_type, iterator_opts)
    170 else:
    171     recording = self.recording_extractor
--> 173 write_recording(
    174     recording=recording,
    175     nwbfile_path=nwbfile_path,
    176     nwbfile=nwbfile,
    177     metadata=metadata,
    178     overwrite=overwrite,
    179     verbose=self.verbose,
    180     starting_time=starting_time,
    181     use_times=use_times,
    182     write_as=write_as,
    183     write_electrical_series=write_electrical_series,
    184     es_key=es_key or self.es_key,
    185     compression=compression,
    186     compression_opts=compression_opts,
    187     iterator_type=iterator_type,
    188     iterator_opts=iterator_opts,
    189 )

File ~/dev/neuroconv/src/neuroconv/tools/spikeinterface/spikeinterface.py:1003, in write_recording(recording, nwbfile_path, nwbfile, metadata, overwrite, verbose, starting_time, use_times, write_as, es_key, write_electrical_series, write_scaled, compression, compression_opts, iterator_type, iterator_opts)
   1000 if nwbfile is not None:
   1001     assert isinstance(nwbfile, pynwb.NWBFile), "'nwbfile' should be of type pynwb.NWBFile"
   1002 assert (
-> 1003     distutils.version.LooseVersion(pynwb.__version__) >= "1.3.3"
   1004 ), "'write_recording' not supported for version < 1.3.3. Run pip install --upgrade pynwb"
   1005 write_as = "raw" if write_as is None else write_as
   1006 compression = "gzip" if compression is None else compression

File ~/opt/miniconda3/lib/python3.9/site-packages/setuptools/_distutils/version.py:90, in Version.__ge__(self, other)
     89 def __ge__(self, other):
---> 90     c = self._cmp(other)
     91     if c is NotImplemented:
     92         return c

File ~/opt/miniconda3/lib/python3.9/site-packages/setuptools/_distutils/version.py:352, in LooseVersion._cmp(self, other)
    350 if self.version == other.version:
    351     return 0
--> 352 if self.version < other.version:
    353     return -1
    354 if self.version > other.version:

TypeError: '<' not supported between instances of 'str' and 'int'

It also looks like the __version__ method of getting the python version isn't working properly.
import pynwb

print(pynwb.__version__)

'unknown'

Using this method, which only works in 3.8+, works fine:
from importlib.metadata import version
print(version("pynwb"))

2.2.0+24.g4a9ee922.dirty

Operating System
macOS
Python Executable
Python
Python Version
3.9
Package Versions
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] Have you ensured this bug was not already reported?

Can you please provide a full traceback?

@CodyCBakerPhD do you think we could build a utility that uses the latest method for getting the package version if the version of Python supports it? We should also move from distutil.version.LooseVersion to packaging.version.parse

Already available: https://github.com/NeurodataWithoutBorders/nwbinspector/blob/dev/src/nwbinspector/utils.py#L133-L155
which we get for free from our DANDI dependency.
Without a traceback, I can't tell where you're getting it from, though.

I'm curious, if you pip show pynwb in that environment, what does it say? This may affect the Inspector's helper function too

(base) MacBook-Pro-8:sandbox bendichter$ pip show pynwb
Name: pynwb
Version: 2.2.0+24.g4a9ee922.dirty
Summary: Package for working with Neurodata stored in the NWB format
Home-page: https://github.com/NeurodataWithoutBorders/pynwb
Author: Andrew Tritt
Author-email<EMAIL_ADDRESS>License: BSD
Location: /Users/bendichter/dev/pynwb/src
Requires: h5py, hdmf, numpy, pandas, python-dateutil, setuptools
Required-by: nwbinspector, dandi, neuroconv

Hmm, I've seen that dirty tag before occasionally when working on dandi stuff, and it can cause all sorts of issues on top of version calculations.
Does
python -c "
from nwbinspector.utils import get_package_version

print(get_package_version("pynwb"))
"

return what we expect?

2.2.0+24.g4a9ee922.dirty

OK well, nonetheless packaging.version.Version seems to handle it fine.
from packaging.version import Version

Version("2.2.0+24.g4a9ee922.dirty") <= Version("2.1.0")
>>> False

do you want to me to make the fix or do you have it covered?

I can do itx


Culling test

Currently iterate Y from top of the world to the bottom and checks every x,z (16,16) in the chunk.
Can be optimized.
I assume this function lags the game while checking chunks

PR duplicates https://github.com/zardoy/minecraft-web-client/pull/152/files and does not make any sense to me

/deploy


tests: Use Result::expect() throughout

Result::expect() was added in Rust 1.4. Using it tidies up the code,
and also helps by printing error details, eg, printing syntax error
details if a regex fails to compile. It adds a colon followed by the
Debug output from any error, making the periods in messages
unnecessary.

Thanks!


Add support for Fuse subgraphs

We already opened some allocations to support Fuse subgraphs but we are getting an error when we want to close these allocations. Is it possible to add support to close them both without getting error and without getting rewards (If I'm not wrong this network is not already supported for rewards right?)
"failureReason": "Failed to prepare tx call data: Failed to resolve CAIP2 ID from the provided network alias: fuse"
Maybe this PR is related:
https://github.com/graphprotocol/indexer/pull/939/commits/25854480617e26cbc42ecaf19dfddb671f6ef8f5
Thank you

Hey @bleiva thanks for reporting. This should be resolved with https://github.com/graphprotocol/indexer/pull/949.


Search query error when using withCount

@yajra
I just faced a weird problem while using search for a counted query.
        $query = ProgramSession::select([
            'program_sessions.*',
        ])->withCount([
            'attendances',
        ]);

If I were to add that withCount function the aggregate count query fails. Since the wheres are not being added to the query inside, instead they are being added the the main query.

What error are you getting please provide more details

I have the same issue. If we use withCount the aggregate Count is always 0.
For some reasons, the query use bindings from withCount

You can also put the counts in as attributes instead, then you can skip the "with" entirely.

Can you please check again using v8.5.1, this be fixed by #1737 and #1730. Thanks!


Restore JSDoc configuration in eslint configuration file(s) and emit errors in CI

In https://github.com/microsoft/botbuilder-js/commit/7b7484b1a85e6196203b43fcb40bc3c271b258a2 via #2814 the JSDoc eslint ovrerides were removed.
We may restore parts of the overrides, but the JSDoc lint violations will be treated as errors in CI.
Overrides snippet:
"FunctionDeclaration": true,
"MethodDefinition": true,
"ClassDeclaration": true,
"ArrowFunctionExpression": false,
"FunctionExpression": false

Moving out to R12 with the expectation that this will come online in phases (potentially a package-by-package basis)


Infer is not finding frameworks

infer -- clang -c ViewController.m
ViewController.m:8:9: fatal error: 'UIKit/UIKit.h' file not found
#import <UIKit/UIKit.h>
^
1 error generated.

The error you get is from clang, not Infer, as you can test by trying clang -c ViewController.m on its own. You need to use a compilation command that works for your project. See http://fbinfer.com/docs/analyzing-apps-or-projects.html for examples.


Bump to node-forge 0.10.0

I rule and have completed some work on Case Manager that's ready for review!
This pull request makes the following changes:

Bumps node-forge

For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

@xmunoz try a fresh yarn install, that usually clears it up!


Adding changelog for release-0.12

Description of changes:
Adding change log for the 0.12 release.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.12


Automatic data curation and feature extraction of museum images of critically endangered mollusks

I'm bringing an image dataset (about 8,000 images) that contain two different angle views of about a half a million specimens of North American freshwater bivalve shells.
Freshwater bivalves are the most endangered animals on the planet, and many of these species have suffered serious population level declines and range contractions over the course of the last century. The OSU Museum of Biological Diversity Mollusk Division houses the largest freshwater bivalve collection in the world, and furthermore contains about a quarter of all known museum specimens of endangered, threatened, and extinct species. We have specimens not only from the majority of watersheds in North America, but in many cases from the same sites collected at multiple different time periods. This makes the OSUM Mollusk Division's collection a very powerful resource to ask questions about continental-scale changes in phenotype correlated with anthropogenic disturbance (dams, pollution) and climate change.
The dataset consists of images of whole drawers of specimens from two angles -- top down, and 45º. The drawers contain individual boxes of specimens called "lots". 1 lot is the set of all the specimens of a species collected at a single place and time. All lots in the collection have a unique numeric catalogue number which is printed on the top right corner of a cardstock label in the box.
All images were taken using the same lighting setup and contain an Calibrite ColorChecker Nano and a QP Card QP101 Calibration Card with mm scale bar.
(A sample from the dataset can be downloaded here)
My goal is to get help to use CV / ML methods to:

segment both images of each drawer of specimens into lots.
Use OCR to capture the catalogue number of each lot from its label and add the number to the image metadata
assign GUIDs to the images and make the dataset available online for use for morphological analysis.

I would definitely be interested in testing some hypotheses about the distribution of different morphological traits and color patterns using this dataset. It would be the largest dataset of its kind in existence for mollusks.
Please reach out if you're interested in collaborating on some or all of this!
-Nate
Originally posted by @nfshoobs in https://github.com/Imageomics/Image-Datapalooza-2023/issues/3#issuecomment-1676450117

Additional info to address questions from pitch:

All images are from the same 2 cameras, the 2 view images are taken simultaneously for each drawer, and contain the same color checker and scale bar.
A catalogue number is a UID, and is typewritten or printed on each label.
The labels that bear the catalogue number  are generally in the same position in each box (upper left) and the catalogue number is in the same part of each label (upper left).
The catalogue numbers all have accompanying metadata in the collection database (i.e. taxonomic identification with GUID to global names architecture, collecting date, geolocation, ecological description, condition of each shell (weathered, freshly dead, etc)
There are not multiple box lots that have the same catalogue number in the dataset (i.e. if two boxes have the same catalogue number, it would probably be due to a clerical error in the collection, or an error in the OCR).
The number of boxed samples per drawer vary. There can be as few as 1 (at the extreme side, we have a single sample of many very large specimens taking up a whole drawer), or as many as ~80 (in the case of a drawer containing many very small boxed samples)
The number of individuals per boxed sample vary. Could be as few as 1, or as many as 1240 (though the majority of samples have fewer than 30 specimens). We also have a count of individuals for each catalogue number in the database.
-There are fewer than 10 box sizes (there is a set of standard sizes we use), and these boxes have edges that are generally contrasty. See example files. This should help segmentation.

One major potential of this research is that the ending dataset would be a large annotated set of species images that are expertly identified, which can be then used to train a model that identifies new images automatically. This would be incredibly useful for conservation and management, as "ability to identify mussel species" is a rare skill and there is high demand for ID expertise from state and federal wildlife agencies.


Option to escape Excel-unfriendly strings

Is your feature request related to a problem?
When a csv file contains a string with a leading -, +, or =, Excel will treat it as a formula field and throw an error.
Describe the solution you'd like
Love an option to auto-detect these leading characters and escape them properly with a single leading single-quote '.

Hi @MrTemple,
Thank you for the suggestion. That is quite a neat feature. I have added tho the backlog. Currently, I am a bit busy with regular work, so I would encourage you to give it a try and implement it yourself, since I don't know when I can get to it, sorry.
In any case, thank you for the feedback.

Thanks @dehesa.
Where do you suggest I start? in _lowlevelWrite(field: String), somewhere either in 3.A or between 3.A and 3.B? (This is used for the Encoder too, right?)
And do you think I'd need to add another char to the result.reserveCapacity?
Also, I haven't contributed to an open source project yet. Is this the general process to follow? https://github.com/firstcontributions/first-contributions

@MrTemple did you ever implement this? thank you

@aehlke No, I didn't have time to dig under the hood and make the change the way I'd have liked. For my purposes it was faster to just extend String with an excelSafe property.
This has all the escaping that seems necessary for excel. I'm pulling many tens of thousands of very long, complexly-formatted, code-containing issue descriptions from various systems (Jira, Github, etc). These seem to have pretty much the ugliest content you're likely to run into, and this escaping logic seems to wrangle them properly without spilling out of the cell.
Note about the character length, I settled on 1000 kind of randomly because it worked on my version of excel, and I didn't happen to need the extra info in the fields I was pulling from. You may want to experiment with a higher limit.
Also the excelNumSafeHashPrefixed is just an uber hack to prevent excel monkeying with long numbers. When you open a csv there's no way to prevent excel from turning a serial number or barcode value like<PHONE_NUMBER>4634 into 6.43e13. If you remember, you can reformat the cells (custom format of 0 is the only way), but if you accidentally save the csv, excel squashes the number forever. 🤦♂ I do a lot of opening and saving and reimporting of the csv that I export. I use this to force it to write out the field as #64342363424634 and then I have to remember to remove it with excelNumSafeHashPrefixRemoved. Ugly, but we're dealing with software written by boomers, for boomers here. 🤣

extension String {

        /// Trims whitespaces and newlines, and wraps any unsafe strings for CSV import to excel using ="stringContents", while escaping any double-quotes within the field.
    public var excelSafe: String {
            // Trim whitespaces and newlines.
        let string = self.trimmingCharacters(in: .whitespacesAndNewlines)

            // Any double-quotes or newlines must be escaped.
        let forbiddenCharacters = CharacterSet(charactersIn: "\"\r\n")

            // If the first character is an equal, minus, or plus sign, excel will treat it like a formula and display "#NAME". If the string contains double-quotes or newlines it will spill out of the cell.
        guard
            !string.hasPrefix("="),
            !string.hasPrefix("-"),
            !string.hasPrefix("+"),
            string.rangeOfCharacter(from: forbiddenCharacters) == nil
        else {
                // Note: Different versions of Excel dislike long formulae (255, 8192 characters?), but will still import and display them (with the ="..." visible in the field.
            let escapedString = string.replacingOccurrences(of: "\"", with: "\"\"")
            let truncatedString = String(escapedString.prefix(1000))
            return "=\"\(truncatedString)\""
        }

        return string
    }

    public var excelSafeOneLine: String {
        return self.replacingOccurrences(of: "\n", with: " ")
            .replacingOccurrences(of: "\r", with: " ")
            .replacingOccurrences(of: "\t", with: " ")
            .excelSafe
    }

    public var excelNumSafeHashPrefixed: String {
        return "#\(self)"
    }

    public var excelNumSafeHashPrefixRemoved: String {
        var string = self
        if string.hasPrefix("#") {
            string.removeFirst()
        }
        return string
    }
}

Thanks for the detailed info! I ended up using a different library but it also has this issue so I appreciate this


On the problem of dragging elements across levels in tree recursive components

原来的move方法中是可以返回relatedContext参数的，其中能解构出 index，element，list，component，但是我在递归组件中跨层级移动元素relatedContext中只能解构出list，componet两个参数而且list值也不对，请您帮助我下，谢谢
The original move method can return relatedcontext parameters, in which ‘index’, ‘element’, ‘list’ and ‘component’ can be deconstructed, but I can only deconstruct ‘list’ and ‘component’ parameters in relatedcontext when I move elements across levels in recursive components, and the list value is also wrong. Please help me. Thank you

@lybyq please share a sample repo where i can see the issue.

Hello, this is the tree recursive component I wrote
<draggable         v-show="isOpen"         v-if="isFolder"         v-model="tree.children"         tag="ul"         class="wrapper"         :class="{'space': tree.level === 2, 'space2': tree.level === 3}"         v-bind="options"         :move="onMove"         @start="onStart"         @end="onEnd"         @change="updateDrag"         @unchoose="unchoose"       >         <transition-group>           <StudyTree             v-for="(item, idx) in tree.children"             :key="item.id || idx + '_s'"             :tree="item"             :index="idx"             :courseId="courseId"             v-bind="$attrs"           />         </transition-group>       </draggable>

If I drag each other at the same level of the tree, the move method returns the correct parameters

But if I drag across levels, the move method returns only two parameters

Please help or provide solutions. Thank you Anish
拜托您给与下帮助，或者提供下解决思路，谢谢您

@lybyq please provide a reproduction repo and re-open the ticket


Designs: backcountry search

Description:
As a backcountry user, I want to be able to

search and find all options on a list
search by camping location, name, subarea or even a landmark (such as a lake)
define my experiences in any order
see different options for location/time frames
be able to easily find the details of the amenities that the backcountry area offers (bear caches, pit toilets, tent pads)
be able to do this all easily from my phone, tablet or desktop

This ticket is to create designs for how the public will search for backcountry experiences.
Acceptance Criteria:

[ ] Designs are in figma and linked here

Development Checklist:

[ ] ...
[ ] ...
[ ] ...

Dependencies

Blocked by
Blocking
Reference: #14

Relevant documentation as reference
Definition of Ready

[ ] Acceptance criteria are included
[ ] Wireframes are included (if applicable)
[ ] Design / Solution is accepted by Product Owner (if applicable)
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done

In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

Code review:

[ ] Code is peer reviewed and has passed CI/CD tests

QA:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Code is potentially shippable to the production environment
[ ] Functional features have been tested and passed by QA
[ ] UI components tested by designer
[ ] Code is deployed to PROD when moved to 'done' column (unless requested otherwise by PO)

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes:

Refining this further may need to wait until Pavel returns. @LindsayMacfarlane @pavelcreates

Hi @pavelcreates - can you please take a look at this ticket and refine if needed before Thursday's sprint planning. Thanks!

I think this looks good for now - I feel like this will need further refinement once we've spoken to external users. There are still a lot of unknowns before we go into designs. However, what we have here is a good high-level.


Typo on step 43 of the Beta  Learn the CSS Box Model by Building a Rothko Painting

Affected page

To reproduce

Recommended fix, suggestions

Screenshots

Hi @katkelsch1 !
Thank you for finding this typo.
I have added the first timers only label since this is an easy fix.
For those interested in fixing this issue, here is the file you need to change.
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/14-responsive-web-design-22/learn-the-css-box-model-by-building-a-rothko-painting/step-043.md
Proposed fix:
The border-radius property accepts up to four values to round the top-left, top-right, bottom-right, and bottom-left corners.
Please also read through our Contributing docs.
Happy contributing!

I'm going to start looking into this.

Thank you for fixing the issue. It was the first time for me on Github. I'm
a total newbie in the webdev coding bootcamp.😀
On Sat, Jan 22, 2022, 11:03 AM Krzysztof G. @.***>
wrote:

Closed #44871 https://github.com/freeCodeCamp/freeCodeCamp/issues/44871
via #44877 https://github.com/freeCodeCamp/freeCodeCamp/pull/44877.
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/44871#event-5937229516,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXNEDULZMYEXVYWWEPEFGC3UXLWP3ANCNFSM5MQSVHWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Switch network doesn't work with non default chains

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.2.7
Current Behavior
I added some new chains when setting up the providers.
When i use SwitchNetwork from the useNetwork  hook with chains i added e.g BSC, BSC Testnet etc, the selected network and list of networks do not show any longer.
There is no error and if i refresh the page, it now shows that i am connected to the right network
Expected Behavior
SwitchNetwork should work well with default and non default chains
Steps To Reproduce
Add new chains to the provider on setup and then use the code below to select and switch between chains
import { useNetwork } from 'wagmi'

const App = () => {
  const [{ data, error, loading }, switchNetwork] = useNetwork()

  return (
    <>

        {data.chain?.name ?? networkData.chain?.id}{' '}
        {data.chain?.unsupported && '(unsupported)'}

      {switchNetwork &&
        data.chains.map((x) =>
          x.id === data.chain?.id ? null : (
            <button key={x.id} onClick={() => switchNetwork(x.id)}>
              Switch to {x.name}
            </button>
          ),
        )}

      {error &&
{error?.message}
}
    </>
  )
}

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

I'm using a workaround for this using utilizing window.ethereum. This solution doesn't rely on any library as long as you're targeting browser MetaMask users. You can modify my solution to support your use case:
// We'll use this in a few places
const ETHEREUM_CHAIN_ID = '0x1';

First, I have an addEthereumChain helper method that I use in the (unlikely) case that user does not have Ethereum chain added to their wallet, it will trigger user to add ethereum chain. We'll use this helper in the next part:
const addEthereumChain = async () => {
  if (typeof window.ethereum === 'undefined') return;
  return await window.ethereum.request({
    method: 'wallet_addEthereumChain',
    params: [
      {
        chainId: ETHEREUM_CHAIN_ID,
        chainName: 'Ethereum',
        rpcUrls: [`https://mainnet.infura.io/v3/${INFURA_ID}`],
      },
    ],
  });
};

Next I have the switchNetwork function that allows me to trigger MetaMask to ask user to switch to Ethereum network:
const switchNetwork = async () => {
  if (typeof window.ethereum === 'undefined') return;
  try {
    await window?.ethereum?.request({
      method: 'wallet_switchEthereumChain',
      params: [{ chainId: ETHEREUM_CHAIN_ID }],
    });
    return true;
  } catch (error) {
    if ((error as any).code !== 4902) return;
    await addEthereumChain();
  }
  return false;
};

Then I have a switchOnChainChange to switch to Eth network when chain changes while user is on:
const switchOnChainChange = async () => {
  if (typeof window.ethereum?.on === 'undefined') return;

  return window.ethereum.on('chainChanged', (chainId: string) => {
    console.log(`chain changed to ${chainId}`);

    if (chainId !== ETHEREUM_CHAIN_ID) {
      switchNetwork();
      // HERE: do whatever else you want, I'm triggering a toast
    } else {
      // HERE: do whatever else you want, I'm removing the toast on valid chain switch
    }
  });
};

Now that we have all of this, here's the React custom hook that will do the rest of the work:
export const useSwitchToEthereum = () => {
  return React.useEffect(() => {
    if (typeof window === 'undefined' || typeof window.ethereum === 'undefined') return;

    // prompts user to switch to Eth network when chain changes while user is on
    switchOnChainChange();

    // This is for when user first lands on the page and is not on the ethereum chain
    // @ts-ignore
    if (window.ethereum.chainId !== ETHEREUM_CHAIN_ID) {
      switchNetwork();
      // HERE: do whatever else you want, I'm triggering a toast that loads on page load
    } else {
      // HERE: do whatever else you want, I'm removing the toast on valid chain switch
    }
  }, []);
};

Finally, import this custom hook and add it in your top component (I'm using next.js so I have it in my _app.tsx file):
const App = ({ Component, pageProps }: AppProps) => {

  useSwitchToEthereum();

  return <Component {...pageProps} />;
};

export default App;

I have yet to thoroughly test this so I don't recommend adding this to production code until you've done your testing. So far, it's been working great for me.
If you have any issues feel free to reach out on twitter @undeterrable

+1 to this, was about to raise the same issue haha!   Thanks so much for sharing your work around

Looks like there's an issue with MetaMask, where is emits a disconnect event before switching chain.
https://github.com/MetaMask/metamask-extension/issues/13375#issuecomment-1046125113
Until this gets fixed, I'm adding a shim to help prevent this from happening. https://github.com/tmm/wagmi/pull/190

OK thanksThough, the bug doesn't happen with default chains in wagmi-------- Original message --------From: awkweb @.>Date: Sun, 20 Feb 2022, 01:00To: tmm/wagmi @.>Cc: layinka @.>, Author @.>Subject: Re: [tmm/wagmi] Switch network doesn't work with non default chains (Issue #180)
Looks like there's an issue with MetaMask, where is emits a disconnect event before switching chain.
MetaMask/metamask-extension#13375 (comment)
Until this gets fixed, I'm adding a shim to help prevent this from happening. #190
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Yup, that's correct. It looks like it only occurs when you use chains not built-in to MetaMask.

Looks like this was an issue on MM's side and works again when testing (using this CodeSandbox).
@layinka can you try out and re-open this issue if it doesn't work?

@tmm I still have this issue when switching between BSC and BSC Testnet. Not sure how related this is to "non default chains".
Seems like I get as you already mentioned disconnected during the switching process.
This Error bubbels up, not sure where it is coming from though.
Error: underlying network changed (event="changed", network={"name":"bnb","chainId":56,"ensAddress":null,"_defaultProvider":null}, detectedNetwork={"name":"bnbt","chainId":97,"ensAddress":null,"_defaultProvider":null}, code=NETWORK_ERROR, version=providers/5.6.1)
    at Logger.makeError (index.js?dd68:219:1)
    at Web3Provider.eval (base-provider.js?4ba1:971:1)
    at Generator.next (<anonymous>)
    at fulfilled (base-provider.js?4ba1:5:43)

While adding connectors, there was an option named shimChainChangedDisconnect that had solved this issue. However, it was removed in 0.3.x and onwards.
This issue has resurfaced in the above sandbox as well.

Looks like this was an issue on MM's side and works again when testing (using this CodeSandbox).
@layinka can you try out and re-open this issue if it doesn't work?

Yeah, the shimChainChangedDisconnect option was a hack. Better that this is fixed by MM. Will think about it some more.

I'm facing the same issue in my project, any workaround meanwhile MM fix it?
Thanks in advance!

I'm facing the same issue in my project, any workaround meanwhile MM fix it? Thanks in advance!

A workaround I used was to use a boolean state variable, with initial value as false.
Its value changes from false to true when the connect function is called.  (connect from useConnect hook)
And from true to false when the disconnect function is called. (disconnect from useDisconnect hook)
The useConnect hook also returns a boolean value isDisconnected.
So, a network change makes the value of isDisconnected false, however the state variable still has a value true.

Hi there
I am having this issue on a webapp that has wagmi. Any options on how to fix this on the app - like a workaround?
I use this list of networks below, it  at first lets me connect to any of them. Then when I start like a transaction, it will randomnly change to whatever is the FIRST in the list, in this case bsc. Then I need to disconnect and reconnect to get it to work.
Any tips would be appreciated, thank you
const { chains, provider } = configureChains(
[
bsc, goerli, polygon, mainnet
],
[
publicProvider(),
]
);
const { connectors } = getDefaultWallets({
appName: 'MyApp',
chains
});
const wagmiClient = createClient({
autoConnect: true,
connectors,
provider
});

any fixes yet ?

Yup, was fixed in 0.12.6 by https://github.com/wagmi-dev/references/pull/177


remote calls of docker image result in images returned with version and also with

Output of docker version:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 22:11:10 2016
 OS/Arch:      linux/amd64

Output of docker info:
Client:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 22:11:10 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 22:11:10 2016
 OS/Arch:      linux/amd64
i809781@mo-d6c40141a:~$ docker info
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 215
Server Version: 1.12.0
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 134
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-21-generic
Operating System: Ubuntu 16.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.851 GiB
Name: mo-d6c40141a
ID: 7SCG:FJK3:BLDB:4EEO:3UIQ:MBOM:LF7K:SZ4H:HWIV:5HHP:EZOB:DG5Z
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Http Proxy: http://proxy.wdf.sap.corp:8080
Https Proxy: http://proxy.wdf.sap.corp:8080
No Proxy: localhost,sap.corp
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
AWS
Steps to reproduce the issue:

on machine 1 configure docker listening TCP port
on machine 2 set DOCKER_HOST to point to machine 1
on machine 2 call docker images

Describe the results you received:
A list where each container has 2 entries one with a version number and one with version <none>
Describe the results you expected:
Only the ones with version numbers should be returned
Additional information you deem important (e.g. issue happens only occasionally):

This is not a bug; images without a tag are shown if an image is pulled by digest If you use docker images --digests, you'll see that the DIGEST column is not empty for those images;
docker images --digests
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
nginx               <none>              sha256:30e3a72672e4c4f6a5909a69f76f3d8361bd6c00c2605d4bf5fa5298cc6467c2   f1fae62370b4        7 days ago          54.3 MB

Given that an image with a specific digest can be tagged under multiple names/tags, those images can be shown separately.
Docker 1.13 improves this though; if only a single image is present for a digest, the images are combined on a single line;
docker images --digests
REPOSITORY          TAG                 DIGEST                                                                    IMAGE ID            CREATED             SIZE
nginx               alpine              sha256:30e3a72672e4c4f6a5909a69f76f3d8361bd6c00c2605d4bf5fa5298cc6467c2   f1fae62370b4        7 days ago          54.3 MB

Related issue; https://github.com/docker/docker/issues/15508, and pull request https://github.com/docker/docker/pull/23996
I'll close this issue, because this will be improved in 1.13, but feel free to comment after I closed


New type of requirements.txt file and add requirements-ci.txt

Context
The idea is to:

Pin strictly in the requirements.txt file;
Have no version pins in the requirements-ci.txt file (exceptions apply);
Pin strictly in the doc/requirements.txt file.

This way:

Users will have a combination of dependency versions for which our package should work (requirements.txt file);
On our CI, we see if a new version of a dependency breaks something (requirements-ci.txt file), but this doesn't affect users.

The same applies to the documentation requirements where the documentation build should continue to work even with new dependency versions being released.
Changes
Created an updated requirements.txt file by:

Creating a clean conda environment;
Installing the plugin with pip install -e .;

a) Saving the dependencies to a temporary req.txt file via pip freeze >> req.txt
b) Removing the PennyLane-Honeywell entry from req.txt;
4. Installing pytest and related packages and verifying that the test suite runs;
5. Making req.txt the new requirements.txt file.

requirements-ci.txt: created by copying requirements.txt and removing the pinnings (except for semantic versioning which is pinned in PennyLane's setup.py file),
doc/requirements.txt: created in a similar fashion to how requirements.txt created, just that make docs was being used for "installation".

[sc-17333]


Notify Integration

Trello backlink https://trello.com/c/xewAy5s7/123-spike-allow-user-to-create-a-form-with-a-title-that-sends-email-to-govuk-notify
End User can press a button that sends an email using Notify, including the title, text input, and submission time

To update where I got with this today - the docs for Notify are indeed really good. I set up a basic integration to send an email via notify, but nothing was coming through when I tested - I have a feeling this is mostly because the file is not being run and I'm new to ruby so don't fully know what I'm doing, but I didn't have time to investigate further.
I've opened a draft PR link above and pushed all my changes up so far.

I've also just seen that the PR is failing on tests so I have indeed done something wrong with the ruby. Will investigate when I'm back!


Add libraries

https://developer.twitter.com/en/docs/developer-utilities/twitter-libraries
https://pypi.org/project/python-twitter/
See also Learn to Code and link swap there.

https://github.com/ckoepp/TwitterSearch
And topic on github. Twitter api and tweepy

Wrapper for premium searches https://twitterdev.github.io/search-tweets-python/#
Snippets https://twitterdev.github.io/


Parameters for activemq_addresses as a dictionary

Refactor activemq_addresses to behave in regards to parameters as activemq_address_settings, ie. each address iten in the list may have a parameters dict containing python-cased queue paramters.
    activemq_addresses:
      - name: DLQ
        anycast:
          - name: DLQ
            parameters:
              durable: True
      - name: client123.pubsub
        multicast:
          - name: client123.pubsub.foo
            parameters:
              max_consumers: 1
              delay_before_dispatch: 3

Fix #169

Thank you!


[ISSUE #8909] Move nullcheck ahead

Which Issue(s) This PR Fixes

Fixes #8909
Brief Description

Move nullcheck ahead
How Did You Test This Change

No test.

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

Project coverage is 47.52%. Comparing base (321670a) to head (9d3ed5d).
Report is 1 commits behind head on develop.

Files with missing lines
Patch %
Lines

...rocketmq/common/config/AbstractRocksDBStorage.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8910      +/-   ##
=============================================
- Coverage      47.54%   47.52%   -0.03%
- Complexity     11729    11735       +6
=============================================
  Files           1304     1304
  Lines          91058    91058
  Branches       11678    11677       -1
=============================================
- Hits           43295    43274      -21
- Misses         42415    42435      +20
- Partials        5348     5349       +1

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


[GHSA-56gj-927p-mfph] Jenkins Aqua Security Serverless Scanner Plugin showed plain text password in job configuration form fields

Updates

Affected products

Comments
It was brought to our attention via https://github.com/google/osv.dev/issues/2075 that the separator character should a slash and not a colon

I think what we have on file is the correct package name, but the incorrect ecosystem. Packagist doesn't have a package with the name org.jenkins-ci.plugins/aqua-serverless.
https://packagist.org/?query=org.jenkins-ci.plugins%2Faqua-serverless
where as the maven entry is there
https://mvnrepository.com/artifact/org.jenkins-ci.plugins/aqua-serverless
Many thanks for the correction and I'll correct the correction on the backend 👍


Support for data-focus-within in RAC

Provide a general summary of the feature here
Originally opened in the discussion: https://github.com/adobe/react-spectrum/discussions/4849#discussion-5463826
For all grid components that support interactive children, I'd like to have possibility to style the <Item/> component based on whether the focus is contained in the descendant of this <Item/>.
For example:
My solution requires to style a <Row/> component when it's being interacted with. Currently, this can be done with using data-hovered or data-focus-visible selectors. However, when my focus moves to one of the interactive children, there's no attribute on the <Row/> component I could use, that I could use to apply the same styles as e.g data-hovered (that'd signify that this row is being interacted with)
🤔 Expected Behavior?
Additional prop/data-attribute that would be enabled once the focus is on the element's descendants. Best candidate seems to be using focus-within selector, but I'm open to hear about drawbacks of this or some other ideas.
😯 Current Behavior
No way of knowing that focus is contained in the element's descendants.
💁 Possible Solution
Possibly adding the useFocusWithin hook to the TableRow component.
🔦 Context
I'd like to style a grid role="row" element based on whether any of the descendants is currently focused.
💻 Examples
No response
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Just to add to @reidbarber comment, we'd also want to add focus-within tracking to Cell as well

Hello guys, do we want to add a separate data-focus-within and leave the existing data-focused intact, or is it okay to just enhance data-focused to remain activated even when the focus shifts to one of its children?
I might be missing some details here, but if the latter is okay then I think this can be solved as simple as changing useFocusRing() in TableRow to useFocusRing({ within: true }), and in the case of GridList, it seems this is already handled by useGridListItem() and useSelectableItem() within it:
https://github.com/adobe/react-spectrum/blob/d2049a5831e57fcad7a2d489fa0eacbe155a6db9/packages/%40react-aria/gridlist/src/useGridListItem.ts#L62-L71
If what we want is the former.. then a little more guidance would be of great help 🙏


PISysAudit configuration security

Add rules to see if the PISysAudit tool itself is setup securely. For example, only run sign scripts (Check remote computers' Execution Policy), remote servers have a least privilege configuration appropriate for running PISysAudit.

Created #345 to address this issue without scoping specifically to the PI Security Audit Tools themselves.


Create getting_started.md

top links for learning and getting started with java

Hey, move Java section to README.md > Technologies and Languages > Java

sure

moved it, now you can merge.


Jekyll

By making a PR you are agreeing to that contributing a hello world project allows the repo/project owner to use the hello world app in examples for their own use.

[ ] only the most simple basic hello world app
[ ] remove all tests and commands for tests and dependencies for tests
[ ] must say "Hello World" on the main component
[ ] only include the minimal components (1 ideally)
[ ] no routing, not extras whatsoever
[ ] no styling
[ ] nothing else !
[ ] must work by running npm install and npm start (or equivalent)
[ ] must be in the folder named <tech-name>
[ ] add build steps and which folder to serve in the README.md
[ ] verify that the app once built can be served from whatever folder the built assets are placed in
[ ] verify that a Pull Request doesnt already exist. If it does, feel free to contribue to it.

Jekyll requires a Ruby toolchain. Are we sure we wanna add this to hello world?

 The caveat is we need to have docker installed.
Reference: Bookworm Jekyll template/theme > preview.sh L31-L35


Update gocql to pull in https://github.com/gocql/gocql/pull/1321

Updates gocql to pull in https://github.com/gocql/gocql/pull/1321

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

Looks good to me. Could you also update the failed test? The failed test is common/persistence/persistence-tests/historyV2PersistenceTest.go
Update the line 164 from "if resp.NextPageToken == nil {" to "if len(resp.NextPageToken) == 0 {"
should fix the test

Coverage increased (+0.04%) to 67.886% when pulling 9056a9cd3aa654b3d74321b5f061913f2acd6515 on AngerM:master into 2348e969a91ce16e280670df06ce048a30e49878 on uber:master.


Duplicate headers

I'm getting the following with makedocs:
LoadError: duplicate header id 'deprecations'

https://github.com/JuliaStats/DataFrames.jl/blob/master/NEWS.md
For some files, it's common to have multiple headings with the same name.

Apologies for the slow response, was out. I've got some refactoring that needs to be pushed once I've updated docs and checked for regressions, but I believe that this is fixed with that. Probably should be sometime tomorrow or Monday I think.

The refactor was 36d09ffb376c5510357c31d24a59191f0c9c9f39, which includes support for duplicate headers. It's undocumented currently, but this is basically how it works:
A document can have any number of headers with the same name and no warnings will be raised so long as none of them are referenced in a {ref}. If you do want to refer to a particular one of the duplicate headers then use the following {ref#...} syntax:

# Header

[The custom header name]({ref#custom-name})...

## Header

# [Header]({ref#custom-name})

The link that wraps the third header gets removed from the final output.
This approach seems to work alright, let me know if you run into trouble.

Thanks!


feat: :sparkles: support for polish language

I add polish support and some test with correct fixtures examples. I don't know if I did something wrong, polish support works almost fine besides a few cases that I mentioned in Issue.

There is still a problem with hundreds and a few other things, maybe it's a configuration issue in the strategies / figures_converter/languages/pl.rb file. I don't know how to solve it, I added some tests with correct examples. Most of the things are practically similar to what in Czech I tried to copy the content from upload to pl.rb but it was throwing me errors.
NoMethodError:          undefined method result' for nil:NilClass
        options.gender.result

`
Maybe with your help I can figure out this problem.

@sectasy0 if you just copy over all of translations/cs.rb onto translations/pl.rb you'll get quite a bit further. At least it will get rid of mechanical errors. Unfortunately I think one needs to speak Polish to properly construct the translation rules around gender/ordinals/etc.

@dblock I'm native polish speaker. All translations are properly constructed in .yml files as well as correct examples.
Example with hundreds:
 expected: "dziewięćset dziewiętnaście"
             got: "dziewięć _ sto dwieście trzysta czterysta pięćset sześćset siedemset osiemset dziewięćset dziewiętnaście"

@sectasy0 want to finish this?

@sectasy0 want to finish this?

Yeah, but I don't quite understand why it returns all the things, I just copied everything from czech language which is basically the same as in Polish in terms of numbers, wanna help me a little?

I checked out your code and rn all specs pass. Add ones that fail and I can try to take a look?


Feature/password hasher

Django has interesting behavior when it comes to password hashers.
I guess it walks through the list of hashers until it finds one that works on the specified password, then it goes and re-encodes the password using the first hasher specified in the settings file.
Which is cool because it's efficient (I guess?) but kinda a pain because I wrote the new password hasher that will apparently only be used twice per old password; Once to verify the old password and re-encode it for the data migration, and once the first time the user logs in to their account. After that the password will be re-encoded using the first specified hasher.
Man I'm bringing this up in the meeting as a "what you learned this week" type thing.
I'm also probably gonna have to redo a bit of it since it's only used twice per qualified user. Oh well. NBD.

Aww dangit I just had a thought. If Django is gonna be all stubborn and do it how it wants anyway, I can just skip the whole dang password hasher and encode it all in Django's format in the update script.

whomp whomp

You might want to look at 608a47a0dd1c5ee58382d476974d263cf6739ea8 to see how the hasher can be simplified. It was hard for me to explain yesterday, but the only method you actually needed to implement was verify, since we never need to encode a password after the initial import (and the encoding is always the same "RubyPasswordHasher$1$$").


mp3 and advert folder after cleanup wrong

I used the Tool. After "SD Karte säubern", the files in the mp3 and advert folder are wrong. Instead of 0001 there is 001, which is no more the right file to find for the DF-Player. In the mp3 and advert folder, 4 digit file names are necessary (up to 1000 files). Correction is necessary.

this issue should be fixed in version 1.1.9

I have retested, it's working fine now, thank you


[WIP] Show appropriate spaceship error message when having insufficient permissions

I'm adding some more things here now

@mfurtak @KrauseFx
caller_locations(1,1).first.label

@lacostej Yeah, updated and unified the code. Agreed re GitHub issue. If you notice something, please PR on it 👍

This is so beautiful 👍

I want to add some tests for this before merging

that is so beautiful 😎


Enable intendation folding in Code Blocks [markdown]

The same as with the main content -- reading is easier than writing . So please add ability to fold the code in { ``` <<_>> ``` } .


The chart "Properties of combined heat and power (CHP) plants" does not load (both on beta and on live)

When I open the chart Properties of combined heat and power (CHP) plants on the Beta or on the Live server the chart freezes at "loading".

Nice catch, it's still referring to the micro chp/fuel cell queries which have been removed during the recent heat project