Skip to content

add zarr tests#4228

Draft
MMesch wants to merge 1 commit into
emscripten-forge:emscripten-3xfrom
MMesch:mmesch/zarr-tests
Draft

add zarr tests#4228
MMesch wants to merge 1 commit into
emscripten-forge:emscripten-3xfrom
MMesch:mmesch/zarr-tests

Conversation

@MMesch

@MMesch MMesch commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

Basic zarr tests as in this PR are currrently failing. I'm opening to iterate on it and get it running. Currently this is based on the latests zarr.

Would be itnerest to hear feedback from @DerThorsten and @davidbrochart .

@davidbrochart

Copy link
Copy Markdown
Collaborator

Zarr uses threads to do non-blocking disk I/O, so that can't work in WASM. I don't know if there is a way to configure it so that it doesn't use threads.
I ported watchfiles to WASM in anychange, it would be a similar change.

@MMesch

MMesch commented Jan 12, 2026

Copy link
Copy Markdown
Contributor Author

What does work is:

dependencies:
  - zarr < 3
  - numcodecs < 0.16

Can we expect any problems using zarr < 3?

@davidbrochart

Copy link
Copy Markdown
Collaborator

It looks like in v3 they use a thread pool executor internally, which was not the case in v2.
I don't know if we can still use v2. At least v3 mentions pyodide compatibility: https://github.com/zarr-developers/zarr-python/releases/tag/v3.0.0.

@MMesch

MMesch commented Jan 12, 2026

Copy link
Copy Markdown
Contributor Author

zarr > 3 issue on pyodide pyodide/pyodide-recipes#227

@MMesch

MMesch commented Jan 12, 2026

Copy link
Copy Markdown
Contributor Author

recipe is pinned to <v3 https://github.com/pyodide/pyodide-recipes/blob/main/packages/zarr/meta.yaml

@DerThorsten we should maybe do the same?

@agriyakhetarpal

Copy link
Copy Markdown

Hi! You may refer to zarr-developers/zarr-python#1903 and pyodide/pyodide-recipes#584 for Zarr v3 support.

@MMesch

MMesch commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

nice, will have a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants