FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

MAINT: suppress google-crc32c pure-python fallback warning by basnijholt · Pull Request #942 · pipefunc/pipefunc · GitHub

MAINT: suppress google-crc32c pure-python fallback warning - #942

Merged
basnijholt merged 15 commits into
mainfrom
fix/run-status-cleanup
Mar 6, 2026
Merged

MAINT: suppress google-crc32c pure-python fallback warning#942
basnijholt merged 15 commits into
mainfrom
fix/run-status-cleanup

Conversation

Copy link
Copy Markdown
Collaborator

Summary

Suppress the google-crc32c pure-python fallback RuntimeWarning that fires on every pipefunc import when the C extension isn't compiled.

Import chain: pipefunc → map/__init__.py → _zarr.py → zarr → numcodecs → google_crc32c 💥

The fix adds a warnings.filterwarnings call in map/__init__.py right before the zarr import (which is already inside a suppress(ImportError) block).

Test plan

  • pipefunc-cli status no longer shows the warning
  • from pipefunc import Pipeline no longer shows the warning
  • All 39 tests in test_run_status_cli.py pass

codecov Bot commented Mar 6, 2026
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
pipefunc/map/__init__.py 100.00% <100.00%> (ø)
pipefunc/map/_prepare.py 100.00% <ø> (ø)
pipefunc/map/_storage_array/_zarr.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

codspeed-hq Bot commented Mar 6, 2026
edited
Loading

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 6 untouched benchmarks


Comparing fix/run-status-cleanup (f79b53a) with main (0c4637a)

basnijholt changed the title fix: suppress google-crc32c pure-python fallback warning MAINT: suppress google-crc32c pure-python fallback warning Mar 6, 2026
basnijholt merged commit 2ba54b6 into main Mar 6, 2026
22 checks passed

github-actions Bot commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

✅ PR Title Formatted Correctly

The title of this PR has been updated to match the correct format. Thank you!

basnijholt deleted the fix/run-status-cleanup branch March 6, 2026 20:05
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
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.

1 participant


Back | FazBrowse Home | New Git URL