Revert "Vendor kerchunk netCDF3 reader (#397)" (#408)

zarr-developers · Jan 30, 2025 · 92f04de · 92f04de
1 parent 81a76f0
commit 92f04de
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 414 deletions.
diff --git a/docs/releases.rst b/docs/releases.rst
@@ -63,7 +63,7 @@ Documentation
 Internal Changes
 ~~~~~~~~~~~~~~~~
 
-- Vendor netCDF3 reader from kerchunk. (:pull:`397`) By `Tom Nicholas <https://github.com/TomNicholas>`_.
+- Add netCDF3 test. (:pull:`397`) By `Tom Nicholas <https://github.com/TomNicholas>`_.
 
 .. _v1.2.0:
 

diff --git a/virtualizarr/readers/netcdf3.py b/virtualizarr/readers/netcdf3.py
@@ -26,7 +26,7 @@ def open_virtual_dataset(
         virtual_backend_kwargs: Optional[dict] = None,
         reader_options: Optional[dict] = None,
     ) -> Dataset:
-        from virtualizarr.vendor.kerchunk.netCDF3 import NetCDF3ToZarr
+        from kerchunk.netCDF3 import NetCDF3ToZarr
 
         if virtual_backend_kwargs:
             raise NotImplementedError(
@@ -38,9 +38,6 @@ def open_virtual_dataset(
             loadable_variables,
         )
 
-        if reader_options is None:
-            reader_options = {}
-
         refs = NetCDF3ToZarr(filepath, inline_threshold=0, **reader_options).translate()
 
         # both group=None and group='' mean to read root group

diff --git a/virtualizarr/vendor/kerchunk/LICENSE.txt b/virtualizarr/vendor/kerchunk/LICENSE.txt
diff --git a/virtualizarr/vendor/kerchunk/__init__.py b/virtualizarr/vendor/kerchunk/__init__.py
diff --git a/virtualizarr/vendor/kerchunk/netCDF3.py b/virtualizarr/vendor/kerchunk/netCDF3.py