-
Notifications
You must be signed in to change notification settings - Fork 54
Manifest arrays use arrayv3metadata #429
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
abarciauskas-bgse
merged 81 commits into
zarr-python-3.0
from
manifest-arrays-use-arrayv3metadata
Feb 18, 2025
Merged
Changes from 80 commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
2a01bfa
Added zarray_to_v3metadata and test
abarciauskas-bgse 17fd547
Working on manifest array tests
abarciauskas-bgse e5666ab
Fix test_manifests/test_array#TestConcat tests
abarciauskas-bgse 5a8cc4c
Passing TestStack tests and add fixture
abarciauskas-bgse 4c0b616
All test_manifests/test_array tests passing
abarciauskas-bgse ac2f787
Compressors should be list
abarciauskas-bgse 5503c60
Passing dmrpp tests
abarciauskas-bgse 1272051
Merge branch 'main' into manifest-arrays-use-arrayv3metadata
abarciauskas-bgse 1f36755
Passing test_hdf.py tests
abarciauskas-bgse 7098803
Start to work on kerchunk tests
abarciauskas-bgse ce2284c
Add method to convert array v3 metadata to v2 metadata for kerchunk (…
abarciauskas-bgse c9853d5
Fix fixtures and mark xfail netcdf3
abarciauskas-bgse 209dae3
Test for convert_v3_to_v2_metadata
abarciauskas-bgse e7205ef
Deduplicate fixture for array v3 metadata
abarciauskas-bgse d65e457
Parse filters and compressors from v3 metdata for v2 metadata
abarciauskas-bgse 190c20f
Rewrite extract_codecs
abarciauskas-bgse 47f5ddd
Refactor convert_to_codec_pipeline
abarciauskas-bgse 5d15608
Fix hdf integration tests
abarciauskas-bgse 908bc52
Test for convert_to_codec_pipeline
abarciauskas-bgse 4a8bfdd
Refactor get_codecs and its tests
abarciauskas-bgse d05cec3
Fix most integration tests and writer tests
abarciauskas-bgse ff23eeb
Fix xarray tests
abarciauskas-bgse 8560f2d
Working on integration tests
abarciauskas-bgse 97d0a71
Add expected type
abarciauskas-bgse 669ce52
Mark datetime tests xfail
abarciauskas-bgse b794dab
Upgrade xarray for tests
abarciauskas-bgse 825142d
xfail some unsupported zarr-python 3 data types
abarciauskas-bgse 6684125
Require zarr
abarciauskas-bgse 5e82de4
Remove zarr dep
abarciauskas-bgse f57b48d
import zarr, explicit dependency
abarciauskas-bgse b811959
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 8c5139b
Add zarr as a dependency
abarciauskas-bgse eb2a86c
Merge branch 'manifest-arrays-use-arrayv3metadata' of github.com:zarr…
abarciauskas-bgse 15ac7a7
Merge branch 'main' into manifest-arrays-use-arrayv3metadata
abarciauskas-bgse 5359762
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] c808351
Min numcodecs version
abarciauskas-bgse bd50167
numcodecs>=0.15.1 in environment and upstream.yml conda env files
abarciauskas-bgse ed97704
Working on mypy errors
abarciauskas-bgse a3c190e
Fix mypy errors and tests
abarciauskas-bgse 95886b9
Remove ZArray class
abarciauskas-bgse a0f72b2
Just return metadata's shape
abarciauskas-bgse aad511f
Create update metadata function
abarciauskas-bgse b357b04
Fix typing for update_metadata
abarciauskas-bgse 08e877a
Check for regular chunk grid in manifest instantiation
abarciauskas-bgse f040459
Remove obsolete codecs code
abarciauskas-bgse 495d660
Fix chunks function and add docstring
abarciauskas-bgse a262f0b
Remove custom zattrs type
abarciauskas-bgse bcd68a0
Move some imports and make update_metadata a private method
abarciauskas-bgse f0ce778
Remove zarr.py
abarciauskas-bgse 0518488
Add zarr to other ci env files
abarciauskas-bgse 0712979
Fixture array_v3_metadata uses array_v3_metadata_dict
abarciauskas-bgse c40915d
No need for union type for CodecPipeline
abarciauskas-bgse cdaca53
Use type alias
abarciauskas-bgse 2415e07
Add comment
abarciauskas-bgse 9366d69
Update virtualizarr/manifests/array_api.py
abarciauskas-bgse d590cfc
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 6394207
Revised copy_and_replace_metadata to be in utils and called correctly
abarciauskas-bgse ea9fd56
Update virtualizarr/translators/kerchunk.py
abarciauskas-bgse 0ee2b48
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 86d1de5
Refactor create v3 array metadata
abarciauskas-bgse fe8305f
Rename to create_v3_array_metadata
abarciauskas-bgse 0f5b32d
Fix some codecs fixtures
abarciauskas-bgse 97bc7cd
Use global vars and simple fixture for creating codec pipelines
abarciauskas-bgse b5a1dc6
Remove redundant create_codec_pipeline fixture
abarciauskas-bgse 12c6260
Fix docstring
abarciauskas-bgse 4b555b6
Use create_v3_array_metadata in from_kerchunk_refs
abarciauskas-bgse c245b0a
Add links to zarr-python 3.0 issues for big endian, datetime and time…
abarciauskas-bgse 23ac776
Reorganize conftest
abarciauskas-bgse 402af7f
Remove obsolete comment
abarciauskas-bgse 3e2e8ec
Rename function numcodec_config_to_configurable
abarciauskas-bgse 92293a6
Fix parameters in docstring for convert_to_codec_pipeline
abarciauskas-bgse eccc215
Revert change to pytest mark skipif for astropy
abarciauskas-bgse 8fa72b2
Remove commented arguments
abarciauskas-bgse f747681
Add classes to test_codecs and make zarr_array a fixture
abarciauskas-bgse ba2e467
Add tests for extract_codecs
abarciauskas-bgse 68695c9
Add test for get_codec_config
abarciauskas-bgse f22bf96
Remove obsolete comment
abarciauskas-bgse 8af37a0
Add test for copy_and_replace_metadata
abarciauskas-bgse d4946ab
Add release notes
abarciauskas-bgse 2c43c61
Attempt to fix rst links
abarciauskas-bgse 05cd57a
Move convert_v3_to_v2_metadata to utils
abarciauskas-bgse 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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.