h5netcdf 1.7.1__py3-none-any.whl → 1.7.2__py3-none-any.whl

h5netcdf/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '1.7.1'
-__version_tuple__ = version_tuple = (1, 7, 1)
+__version__ = version = '1.7.2'
+__version_tuple__ = version_tuple = (1, 7, 2)
 
 __commit_id__ = commit_id = None

h5netcdf/core.py

@@ -12,7 +12,7 @@ from packaging import version
 
 from . import __version__
 from .attrs import Attributes
-from .dimensions import Dimension, Dimensions
+from .dimensions import Dimension, Dimensions, _check_classic_unlimited
 from .utils import (
     CompatibilityError,
     Frozen,
@@ -1030,12 +1030,8 @@ class Group(Mapping):
 
     @dimensions.setter
     def dimensions(self, value):
-        if self._format == "NETCDF4_CLASSIC":
-            unlimited_dims = list(filter(lambda s: s in [None, 0], value.values()))
-            if len(unlimited_dims) > 1:
-                raise CompatibilityError(
-                    "NETCDF4_CLASSIC format only allows one unlimited dimension."
-                )
+        if self._root._format == "NETCDF4_CLASSIC":
+            _check_classic_unlimited(value)
 
         for k, v in self._all_dimensions.maps[0].items():
             if k in value:

h5netcdf/dimensions.py

@@ -1,12 +1,24 @@
 import weakref
 from collections import OrderedDict
-from collections.abc import MutableMapping
+from collections.abc import Mapping, MutableMapping
 
 import numpy as np
 
 from .utils import CompatibilityError
 
 
+def _check_classic_unlimited(value, unlimited=None):
+    if isinstance(value, Mapping):
+        multiple_unlimited_dimensions = sum(v in (None, 0) for v in value.values()) > 1
+    else:
+        multiple_unlimited_dimensions = unlimited and value in (None, 0)
+
+    if multiple_unlimited_dimensions:
+        raise CompatibilityError(
+            "Only one unlimited dimension allowed in the NETCDF4_CLASSIC format."
+        )
+
+
 class Dimensions(MutableMapping):
     def __init__(self, group):
         self._group_ref = weakref.ref(group)
@@ -25,14 +37,8 @@ class Dimensions(MutableMapping):
             raise RuntimeError("H5NetCDF: Write to read only")
         if name in self._objects:
             raise ValueError(f"dimension {name!r} already exists")
-        if (
-            size in [0, None]
-            and self._unlimited()
-            and self._group._format == "NETCDF4_CLASSIC"
-        ):
-            raise CompatibilityError(
-                "Only one unlimited dimension allowed in the NETCDF4_CLASSIC format."
-            )
+        if self._group._root._format == "NETCDF4_CLASSIC":
+            _check_classic_unlimited(size, self._unlimited())
 
         self._objects[name] = Dimension(self._group, name, size, create_h5ds=True)
 

h5netcdf/tests/test_h5netcdf.py

@@ -211,7 +211,7 @@ def write_h5netcdf(tmp_netcdf, compression="gzip", format="NETCDF4"):
     if ds.data_model == "NETCDF4_CLASSIC":
         with raises(
             CompatibilityError,
-            match="NETCDF4_CLASSIC format only allows one unlimited dimension.",
+            match="Only one unlimited dimension allowed",
         ):
             ds.dimensions = {"x": 4, "y": 5, "z": 6, "unlimited": None, "empty": 0}
 
@@ -2903,7 +2903,7 @@ def test_raise_on_closed_file(tmp_local_netcdf):
     v = f.create_variable("hello", ("x",), float)
     v[:] = np.ones(5)
     f.close()
-    with pytest.raises(
+    with raises(
         ValueError,
         match=f"I/O operation on <Closed h5netcdf.File>: '{tmp_local_netcdf}'",
     ):
@@ -3013,3 +3013,30 @@ def test_attributes_list(tmp_local_netcdf, attr):
         assert hf.attrs["foo"][0] == attr[0]
         assert hf.attrs["foo"][1] == attr[1]
         assert isinstance(hf.attrs["foo"], list)
+
+
+def test_group_dimensions(tmp_local_netcdf):
+    # regression test for https://github.com/h5netcdf/h5netcdf/issues/293
+    with h5netcdf.File(tmp_local_netcdf, mode="w") as f:
+        group = f.create_group("data")
+        dims = {"y": 3, "x": 3, "z": None, "z1": None}
+        group.dimensions = dims
+        assert list(group.dimensions) == ["y", "x", "z", "z1"]
+        group.dimensions["z2"] = None
+        assert list(group.dimensions) == ["y", "x", "z", "z1", "z2"]
+
+
+def test_group_dimensions_classic(tmp_local_netcdf):
+    # regression test for https://github.com/h5netcdf/h5netcdf/issues/293
+    with h5netcdf.File(tmp_local_netcdf, mode="w", format="NETCDF4_CLASSIC") as f:
+        group = f.create_group("data")
+        dims = {"y": 3, "x": 3, "z": None, "z1": None}
+        with raises(CompatibilityError, match=r"Only one unlimited dimension allowed"):
+            group.dimensions = dims
+        assert list(group.dimensions) == []
+        dims = {"y": 3, "x": 3, "z": None}
+        group.dimensions = dims
+        assert list(group.dimensions) == ["y", "x", "z"]
+        with raises(CompatibilityError, match=r"Only one unlimited dimension allowed"):
+            group.dimensions["z1"] = None
+        assert list(group.dimensions) == ["y", "x", "z"]

{h5netcdf-1.7.1.dist-info → h5netcdf-1.7.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: h5netcdf
-Version: 1.7.1
+Version: 1.7.2
 Summary: netCDF4 via h5py
 Author-email: Stephan Hoyer <shoyer@gmail.com>, Kai Mühlbauer <kmuehlbauer@wradlib.org>
 Maintainer-email: h5netcdf developers <devteam@h5netcdf.org>

h5netcdf-1.7.2.dist-info/RECORD

@@ -0,0 +1,16 @@
+h5netcdf/__init__.py,sha256=Y0EBCcmlJctwl1kCmj7yLijTVy9AioBTr2091vInAtw,456
+h5netcdf/_version.py,sha256=veZJrH40zJtbmH4sd91I-OwvwjV0ha3xLhPSXrkCusQ,704
+h5netcdf/attrs.py,sha256=GSq2lzv9q0tEkJBqr2Hyrwo0YJnWZHxtLRhhG9NVf4w,4575
+h5netcdf/core.py,sha256=Dd0q1XNwrrqzCWSPauwGDFwtrVu55APIrhwOs0Gd7nQ,69508
+h5netcdf/dimensions.py,sha256=BO126-7r9B0sF_bbcCSzTMnQY5wH7e_DSkbUO4F9q_o,8609
+h5netcdf/legacyapi.py,sha256=MIZlht5Ad4hDFF1Slz2vXmKkgbv7Fhhf2YwNIe16Lfk,7682
+h5netcdf/utils.py,sha256=btxKI-VZP-Wn0Rk_wmnhnUls-mrxO42w0s_uX_su4FI,6528
+h5netcdf/tests/conftest.py,sha256=qS7XTZxos0NIRFtMCJwVEyx0paZw8Le1loPK1MtoQ_0,2350
+h5netcdf/tests/pytest.ini,sha256=ruJxrLdCIA4bCPVuPQjxsLSlvVxuIsIakK6iQOmz-ak,107
+h5netcdf/tests/test_h5netcdf.py,sha256=nQoKgCym6xxigicOEF-2nSBm5ba0k568GatfZnSo8BI,118407
+h5netcdf-1.7.2.dist-info/licenses/AUTHORS.txt,sha256=LTKzUh9o4Wc_oT3aFC48cyDCCP6tdm6VEV_6RrNy4uo,272
+h5netcdf-1.7.2.dist-info/licenses/LICENSE,sha256=Xer1Jg8iL_n9Da0xt0S99blk6tsg9tee_JdgH1rWTjs,1505
+h5netcdf-1.7.2.dist-info/METADATA,sha256=_Gh6qQTBa-Oy7Q4wm8vMAoa7EJN1sdSHjjf2IuVMDb8,13396
+h5netcdf-1.7.2.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+h5netcdf-1.7.2.dist-info/top_level.txt,sha256=Fb_KIpOE6MBqjSvxV1Ay7oYce1mdmQ1pO9JQJPDeGqg,9
+h5netcdf-1.7.2.dist-info/RECORD,,

h5netcdf-1.7.1.dist-info/RECORD

@@ -1,16 +0,0 @@
-h5netcdf/__init__.py,sha256=Y0EBCcmlJctwl1kCmj7yLijTVy9AioBTr2091vInAtw,456
-h5netcdf/_version.py,sha256=wcauG7D_isNqKUiP6L0WuH_5Dwe0oyUJa9LG3cC4WHk,704
-h5netcdf/attrs.py,sha256=GSq2lzv9q0tEkJBqr2Hyrwo0YJnWZHxtLRhhG9NVf4w,4575
-h5netcdf/core.py,sha256=Fg_68fF9vcyozlk-2vO1Qa0U_oRbvWfE976ax--CvxM,69698
-h5netcdf/dimensions.py,sha256=pxtt3ID55bTMp6djIpAka3RgXzioIUs9qBBgMqd9HhU,8336
-h5netcdf/legacyapi.py,sha256=MIZlht5Ad4hDFF1Slz2vXmKkgbv7Fhhf2YwNIe16Lfk,7682
-h5netcdf/utils.py,sha256=btxKI-VZP-Wn0Rk_wmnhnUls-mrxO42w0s_uX_su4FI,6528
-h5netcdf/tests/conftest.py,sha256=qS7XTZxos0NIRFtMCJwVEyx0paZw8Le1loPK1MtoQ_0,2350
-h5netcdf/tests/pytest.ini,sha256=ruJxrLdCIA4bCPVuPQjxsLSlvVxuIsIakK6iQOmz-ak,107
-h5netcdf/tests/test_h5netcdf.py,sha256=FWH2xJfIrBkUg7QEqOTvhrJYVvMU6flkDdkX-NWjI7s,117170
-h5netcdf-1.7.1.dist-info/licenses/AUTHORS.txt,sha256=LTKzUh9o4Wc_oT3aFC48cyDCCP6tdm6VEV_6RrNy4uo,272
-h5netcdf-1.7.1.dist-info/licenses/LICENSE,sha256=Xer1Jg8iL_n9Da0xt0S99blk6tsg9tee_JdgH1rWTjs,1505
-h5netcdf-1.7.1.dist-info/METADATA,sha256=tqEPGLxlIdLwfJjNy2NV60F_5jz95xr5YQ13KzfZa6I,13396
-h5netcdf-1.7.1.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-h5netcdf-1.7.1.dist-info/top_level.txt,sha256=Fb_KIpOE6MBqjSvxV1Ay7oYce1mdmQ1pO9JQJPDeGqg,9
-h5netcdf-1.7.1.dist-info/RECORD,,