Skip to content

Commit 74b4fa1

Browse files
authored
fix: also require kerchunk for test_read_netcdf3 (#998)
NetCDF3Parser lazily imports kerchunk.netCDF3 inside __call__, but the test only had @requires_scipy. In environments that ship scipy without kerchunk (e.g. pixi min-deps once a transitive dep pulled in scipy) the test raised ModuleNotFoundError instead of being skipped.
1 parent 9f90d2e commit 74b4fa1

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

docs/releases.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,9 @@
1515

1616
### Internal changes
1717

18+
- Mark `test_read_netcdf3` as also requiring `kerchunk`. It already had `@requires_scipy`, but `NetCDF3Parser` lazily imports `kerchunk.netCDF3`, so the test raised `ModuleNotFoundError` in any environment with scipy but not kerchunk (e.g. pixi `min-deps` once a transitive dep started pulling in scipy).
19+
By [Tom Nicholas](https://github.com/TomNicholas).
20+
1821
## v2.6.1 (3rd May 2026)
1922

2023
Adds end-to-end support for inlined chunk references in `ChunkManifest` (read via Kerchunk parsers, write via Kerchunk and Icechunk writers), plus Zarr-Python 3.2.0 compatibility and several bug fixes.

virtualizarr/tests/test_parsers/test_netcdf3.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
from virtualizarr.tests.utils import obstore_http
99

1010

11+
@requires_kerchunk
1112
@requires_scipy
1213
def test_read_netcdf3(netcdf3_file, array_v3_metadata, local_registry):
1314
filepath = str(netcdf3_file())

0 commit comments

Comments
 (0)