Skip to content

Commit acb0bb6

Browse files
neilSchroederneilSchroederpre-commit-ci[bot]
authored
Parse zarr v2 (#822)
* Implement Zarr V2 to V3 metadata conversion with fill value handling * Enhance parse_manifest_index to support V2 and V3 chunk key parsing * Refactor parse_manifest_index to improve regex pattern matching and error handling * Enhance get_chunk_mapping_prefix to support V2 and V3 chunk path parsing * Enhance build_chunk_manifest to calculate chunk grid shape for inline V2 data * Enhance test_virtual_dataset_zarr to handle dimension name checks for V2 and V3 formats * Remove redundant check for V2 format in get_metadata function * cleaning up * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * revert store * linting * fixing mypy typing * removing redundant code, linting * refactor zarr parsing to use strategy pattern for extensibility and maintainability, linted * refactor test, add tests to improve coverage of zarr parsing (97%), linted * adding v2 parsing as new feature * updating ZarrParser documentation * converting protocol to ABC * adding tests for sparse files being filled with default fill values * fix zeros list * adding comment about chunk key discovery * cleaning up a bit based on comments * fixing issue with conflicting test assertions around v2 metadata attributes * refactoring common bits of code * raise error on shard detection for v3 * test that sharded v3 array raises error * fixing mypy errors --------- Co-authored-by: neilSchroeder <neil.raymond.schroeder@gmail.clom> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent 7a13261 commit acb0bb6

File tree

3 files changed

+693
-108
lines changed

3 files changed

+693
-108
lines changed

docs/releases.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@
44

55
### New Features
66

7+
- `ZarrParser` now handles Zarr V2 and V3 array parsing.
8+
([#565](https://github.com/zarr-developers/VirtualiZarr/pull/822)).
9+
By [Neil Schroeder](https://github.com/neilSchroeder)
10+
711
### Breaking changes
812

913
### Bug fixes

0 commit comments

Comments
 (0)