-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Check if zarr store supports consolidated metadata #10457
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
Merged
TomNicholas
merged 47 commits into
pydata:main
from
TomNicholas:check_supports_consolidated_metadata
Jul 2, 2025
Merged
Changes from 1 commit
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
01e7518
new blank whatsnew
TomNicholas 83e553b
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas e44326d
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 4e4eeb0
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas d858059
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas d377780
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 3132f6a
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 900eef5
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 4c4462f
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 5b9b749
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas fadb953
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 57d9d23
Merge branch 'main' of https://github.com/TomNicholas/xarray
TomNicholas 11170fc
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 0b8fa41
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas f769f85
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 4eef318
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 29242a4
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 5d15bbd
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas b4ed8ee
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 6a23bfb
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas d87b209
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 08cb041
add test for store which explicitly does not support consolidated met…
TomNicholas 9cc9105
check for consolidated metadata support
TomNicholas c817869
WrapperStore now only imported and used if zarr v3
TomNicholas 50cff3e
use dedicated decorator
TomNicholas a47b2e2
use getattr in case property not defined
TomNicholas 76f16fe
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 9b39529
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas e174d9c
implement with_read_only
TomNicholas 5b441b2
Merge branch 'check_supports_consolidated_metadata' of https://github…
TomNicholas e395c60
Merge branch 'main' into check_supports_consolidated_metadata
TomNicholas 1829807
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas de02ced
Merge branch 'main' of https://github.com/pydata/xarray
TomNicholas 8167b83
Merge branch 'main' into check_supports_consolidated_metadata
TomNicholas 1e30664
Merge branch 'check_supports_consolidated_metadata' of https://github…
TomNicholas 101b280
change test to not expect warning if consolidated metadata not supported
TomNicholas 26a402c
move store definition outside of test
TomNicholas 1ce9d87
try setting Store=None when zarr-python v3 not installed
TomNicholas ad4d79c
remove Store type hint entirely to avoid import issues
TomNicholas 09915dd
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 8252f9f
try WrapperStore = object
TomNicholas e550851
ensure WrapperStore is defined as something even if zarr isn't presen…
TomNicholas 470e701
Merge branch 'main' into check_supports_consolidated_metadata
TomNicholas 586c7a2
release note
TomNicholas e673608
Merge branch 'check_supports_consolidated_metadata' of https://github…
TomNicholas 758eea4
correct RST syntax
TomNicholas e3f0d7b
Merge branch 'main' into check_supports_consolidated_metadata
TomNicholas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
check for consolidated metadata support
- Loading branch information
commit 9cc91057d03f045a75d3785bd365fad93a23006f
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will be needed for Zarr >3.0.0,<3.1.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right - I had something like that then removed it 🤦
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added in a47b2e2