Merge pull request #22 from fractal-analytics-platform/bug_fixes

Bug fixes

src/ngio/ngff_meta/fractal_image_meta.py

@@ -823,9 +823,7 @@ def space_axes_unit(self) -> SpaceUnits:
             raise ValueError("Spatial axes must have a unit.")
         if return_type not in SpaceUnits.allowed_names():
             raise ValueError(f"Invalid space unit {return_type}.")
-        if isinstance(return_type, str):
-            return_type = SpaceUnits(return_type)
-        return return_type
+        return SpaceUnits(return_type)
 
     @property
     def pixel_size(self) -> PixelSize:
@@ -850,8 +848,7 @@ def time_axis_unit(self) -> TimeUnits | None:
         if len(types) == 0:
             return None
         elif len(types) == 1:
-            assert isinstance(types[0], TimeUnits)
-            return types[0]
+            return TimeUnits(types[0])
         else:
             raise ValueError("Multiple time axes found. Only one time axis is allowed.")
 

src/ngio/tables/tables_group.py

@@ -156,14 +156,17 @@ def list(
 
         list_of_tables = self._get_list_of_tables()
         self._validate_list_of_tables(list_of_tables=list_of_tables)
+
+        all_table_types = ["roi_table", "feature_table", "masking_roi_table"]
+
         if table_type is None:
             return list_of_tables
 
         else:
-            if table_type not in ["roi_table", "feature_table", "masking_roi_table"]:
+            if table_type not in all_table_types:
                 raise ValueError(
                     f"Table type {table_type} not recognized. "
-                    " Allowed values are: 'roi', 'feature', 'masking_roi'."
+                    f" Allowed values are: {all_table_types}"
                 )
             list_of_typed_tables = []
             for table_name in list_of_tables: