h5netcdf 1.7.0__tar.gz → 1.7.1__tar.gz

{h5netcdf-1.7.0 → h5netcdf-1.7.1}/CHANGELOG.rst

@@ -1,6 +1,11 @@
 Change Log
 ----------
 
+Version 1.7.1 (October 16th, 2025):
+
+- Fix regression where attributes with list of strings were written with h5py low-level API instead of high-level API (:issue:`291`, :pull:`292`).
+  By `Kai Mühlbauer <https://github.com/kmuehlbauer>`_
+
 Version 1.7.0 (October 15th, 2025):
 
 - Fix unintentional changes in test suite (:pull:`277`).

{h5netcdf-1.7.0 → h5netcdf-1.7.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: h5netcdf
-Version: 1.7.0
+Version: 1.7.1
 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.0 → h5netcdf-1.7.1}/h5netcdf/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '1.7.0'
-__version_tuple__ = version_tuple = (1, 7, 0)
+__version__ = version = '1.7.1'
+__version_tuple__ = version_tuple = (1, 7, 1)
 
-__commit_id__ = commit_id = 'g9cb422fb6'
+__commit_id__ = commit_id = 'g251854355'

{h5netcdf-1.7.0 → h5netcdf-1.7.1}/h5netcdf/attrs.py

@@ -87,10 +87,11 @@ class Attributes(MutableMapping):
 
         self._check_dtype(dtype)
 
+        is_plain_string = dtype.kind in {"S", "U"} and dtype.metadata is None
         if (
-            dtype.kind in {"S", "U"}  # for strings
-            and dtype.metadata is None  # but not special h5py strings
-            and not isinstance(value, (list, self._h5py.Empty))
+            is_plain_string  # for simple strings
+            and (not isinstance(value, list) and self._format == "NETCDF4_CLASSIC")
+            and not isinstance(value, self._h5py.Empty)
             and self._h5py.__name__ == "h5py"
         ):
             # create with low level API to get fixed length strings

{h5netcdf-1.7.0 → h5netcdf-1.7.1}/h5netcdf/tests/test_h5netcdf.py

@@ -2999,3 +2999,17 @@ def test_is_classic(tmp_local_netcdf):
 
     out = subprocess.run(["ncdump", "-k", tmp_local_netcdf], capture_output=True)
     assert out.stdout.decode().strip() == "netCDF-4 classic model"
+
+
+@pytest.mark.parametrize(
+    "attr", [("un", "deux"), ["un", "deux"], ("one", "two"), ["one", "two"]]
+)
+def test_attributes_list(tmp_local_netcdf, attr):
+    # regression test for https://github.com/h5netcdf/h5netcdf/issues/291
+    with h5netcdf.File(tmp_local_netcdf, mode="w") as hf:
+        hf.attrs["foo"] = attr
+
+    with h5netcdf.File(tmp_local_netcdf, mode="r") as hf:
+        assert hf.attrs["foo"][0] == attr[0]
+        assert hf.attrs["foo"][1] == attr[1]
+        assert isinstance(hf.attrs["foo"], list)

{h5netcdf-1.7.0 → h5netcdf-1.7.1}/h5netcdf.egg-info/PKG-INFO

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