Fix joining of paths in hdf file (was OS-dependent)

h5grove/content.py

@@ -1,7 +1,6 @@
 from typing import Dict, Generic, Optional, Sequence, TypeVar, Union
 import h5py
 import numpy as np
-import os
 
 from .models import Selection
 
@@ -13,6 +12,7 @@
     attr_metadata,
     get_array_stats,
     get_entity_from_file,
+    hdf_path_join,
     parse_slice,
     sorted_dict,
 )
@@ -189,9 +189,9 @@ def __init__(self, path: str, h5py_entity: h5py.Group, h5file: h5py.File):
 
     def _get_child_metadata_content(self, depth=0):
         return [
-            create_content(self._h5file, os.path.join(self._path, child_path)).metadata(
-                depth
-            )
+            create_content(
+                self._h5file, hdf_path_join(self._path, child_path)
+            ).metadata(depth)
             for child_path in self._h5py_entity.keys()
         ]
 

h5grove/utils.py

@@ -158,3 +158,10 @@ def get_array_stats(data: np.ndarray) -> Dict[str, Union[float, int, None]]:
         "mean": cast(np.mean(data)),
         "std": cast(np.std(data)),
     }
+
+
+def hdf_path_join(prefix, suffix):
+    if prefix == "/":
+        return f"/{suffix}"
+
+    return f'{prefix.rstrip("/")}/{suffix}'

test/test_utils.py

@@ -0,0 +1,13 @@
+from h5grove.utils import hdf_path_join
+
+
+def test_root_path_join():
+    assert hdf_path_join("/", "child") == "/child"
+
+
+def test_group_path_join():
+    assert hdf_path_join("/group1/group2", "data") == "/group1/group2/data"
+
+
+def test_group_path_join_trailing():
+    assert hdf_path_join("/group1/group2/", "data") == "/group1/group2/data"