Skip to content

[do not merge] test new dtypes in zarr-python #10332

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

d-v-b
Copy link
Contributor

@d-v-b d-v-b commented May 19, 2025

This PR tests compatibility between xarray and a branch of zarr-python that supports more numpy data types. Do not merge this PR.

@d-v-b
Copy link
Contributor Author

d-v-b commented May 19, 2025

cc @ianhi

@github-actions github-actions bot added the topic-NamedArray Lightweight version of Variable label May 20, 2025
@d-v-b
Copy link
Contributor Author

d-v-b commented May 20, 2025

this PR is less useful than I thought it would be, because it seems that the test CI jobs don't use the dependencies declared in pyproject.toml. What's the reason for this limitation?

@github-actions github-actions bot added CI Continuous Integration tools dependencies Pull requests that update a dependency file labels May 20, 2025
@TomNicholas TomNicholas added the run-upstream Run upstream CI label May 20, 2025
@github-actions github-actions bot added topic-backends topic-zarr Related to zarr storage library io labels May 20, 2025
@@ -34,7 +34,7 @@ io = [
"h5netcdf",
"scipy",
'pydap; python_version<"3.10"',
"zarr",
"zarr @ git+https://github.com/d-v-b/zarr-python.git@feat/fixed-length-strings",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah just modify ci/requirements/environment.yml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration tools dependencies Pull requests that update a dependency file io run-upstream Run upstream CI topic-backends topic-documentation topic-NamedArray Lightweight version of Variable topic-zarr Related to zarr storage library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants