From 35d4c381034103e3ba0538e6f5c892e5b5ab25b4 Mon Sep 17 00:00:00 2001 From: TomNicholas Date: Mon, 18 May 2026 16:02:07 -0400 Subject: [PATCH] fix: also require kerchunk for test_read_netcdf3 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. --- docs/releases.md | 3 +++ virtualizarr/tests/test_parsers/test_netcdf3.py | 1 + 2 files changed, 4 insertions(+) diff --git a/docs/releases.md b/docs/releases.md index 7b12814f..fcc7aa3c 100644 --- a/docs/releases.md +++ b/docs/releases.md @@ -13,6 +13,9 @@ ### Internal changes +- 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). + By [Tom Nicholas](https://github.com/TomNicholas). + ## v2.6.1 (3rd May 2026) 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. diff --git a/virtualizarr/tests/test_parsers/test_netcdf3.py b/virtualizarr/tests/test_parsers/test_netcdf3.py index 87d24f60..da09a028 100644 --- a/virtualizarr/tests/test_parsers/test_netcdf3.py +++ b/virtualizarr/tests/test_parsers/test_netcdf3.py @@ -8,6 +8,7 @@ from virtualizarr.tests.utils import obstore_http +@requires_kerchunk @requires_scipy def test_read_netcdf3(netcdf3_file, array_v3_metadata, local_registry): filepath = str(netcdf3_file)