Skip to content

Fix NumCodecs import with zfpy post-releases - #893

Open
justavibedev wants to merge 1 commit into
zarr-developers:mainfrom
justavibedev:fix/zfpy-post-release
Open

justavibedev wants to merge 1 commit into
zarr-developers:mainfrom
justavibedev:fix/zfpy-post-release

Conversation

@justavibedev

Copy link
Copy Markdown

Fixes #892.

With zfpy 1.0.1.post0 installed, importing NumCodecs raises a ValueError before the caller can use any codec. This showed up during test collection on #891 and also reproduces with the 0.17.1 release.

The ZFPY compatibility check now uses packaging.version.Version for both packages. The tests cover the released zfpy version, a NumPy local version, and an older zfpy post-release that still needs the compatibility warning.

With the actual 1.0.1.post0 wheel, the version, ZFPY and checksum modules had 155 tests pass and seven skip. I also checked importing NumCodecs in an environment without zfpy. The docs built locally with warnings and the new release-note link rendered. Codex assisted with implementation and verification.

@codecov

codecov Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (4afa628) to head (426cbd6).

Additional details and impacted files
@@             Coverage Diff              @@
##             main      #893       +/-   ##
============================================
+ Coverage   89.71%   100.00%   +10.28%     
============================================
  Files          27        27               
  Lines         914       916        +2     
============================================
+ Hits          820       916       +96     
+ Misses         94         0       -94     
Files with missing lines Coverage Δ
src/numcodecs/zfpy.py 100.00% <100.00%> (+78.72%) ⬆️

... and 2 files with indirect coverage changes

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Importing numcodecs fails with zfpy 1.0.1.post0 installed

1 participant