Skip to content

Update to zarr-v3-compatible released version of Kerchunk (0.2.8) - #412

Merged
TomNicholas merged 5 commits into
mainfrom
kerchunk_0.2.8
Feb 3, 2025
Merged

Update to zarr-v3-compatible released version of Kerchunk (0.2.8)#412
TomNicholas merged 5 commits into
mainfrom
kerchunk_0.2.8

Conversation

@TomNicholas

@TomNicholas TomNicholas commented Feb 1, 2025

Copy link
Copy Markdown
Member

Long-awaited part of #392, as it allows us to pin zarr-python>=3!

cc @mpiannucci the warning in the icechunk docs can be updated now (though virtualizarr still needs to be fixed to work with the latest version of icechunk).

  • Closes part of Planning explicit dependency on Zarr v3 #392
  • Tests added
  • Tests passing
  • Full type hint coverage
  • Changes are documented in docs/releases.rst
  • New functions/methods are listed in api.rst
  • New functionality has documentation

@TomNicholas TomNicholas added Kerchunk Relating to the kerchunk library / specification itself dependencies Updates a dependency labels Feb 1, 2025
@TomNicholas

TomNicholas commented Feb 1, 2025

Copy link
Copy Markdown
Member Author

We need kerchunk to be on conda for the CI to work https://github.com/conda-forge/kerchunk-feedstock @martindurant

@martindurant

Copy link
Copy Markdown
Member

The bot should pick it up later today.

@martindurant

Copy link
Copy Markdown
Member

conda forge PR is merged, artefact should be available soon.

@codecov

codecov Bot commented Feb 2, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.17%. Comparing base (83a6f10) to head (eb494cf).
Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   77.81%   84.17%   +6.36%     
==========================================
  Files          31       31              
  Lines        1839     1839              
==========================================
+ Hits         1431     1548     +117     
+ Misses        408      291     -117     

see 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TomNicholas

Copy link
Copy Markdown
Member Author

Only failure is an icechunk one which also occurs on main (and I've mentioned it upstream earth-mover/icechunk#665 (comment)) so I will merge this now.

Thank so much @martindurant !

@TomNicholas
TomNicholas merged commit 14f58ba into main Feb 3, 2025
@TomNicholas
TomNicholas deleted the kerchunk_0.2.8 branch February 3, 2025 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Updates a dependency Kerchunk Relating to the kerchunk library / specification itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants