vertical time uom

Author: valentin-gauthier-geosiris

energyml-utils/src/energyml/utils/data/crs.py

@@ -106,6 +106,10 @@ class CrsInfo:
 
     vertical_unknown: Optional[str] = None
     """Free-text vertical CRS descriptor."""
+    
+    time_uom: Optional[str] = None
+    """Unit of measure for time coordinates (e.g. ``"s"``, ``"min"``, ``"h"``)."""
+
 
     # ------------------------------------------------------------------
     # Rotation / azimuth
@@ -443,9 +447,10 @@ def _from_abstract_local3dcrs(
 
     # --- Vertical UOM (length or time) ------------------------------------
     vertical_uom: Optional[str] = _uom_to_str(get_object_attribute_no_verif(crs_obj, "vertical_uom"))
+    time_uom = _uom_to_str(getattr(crs_obj, "time_uom", None))
     if vertical_uom is None:
         # time_uom only present on LocalTime3dCrs
-        vertical_uom = _uom_to_str(getattr(crs_obj, "time_uom", None))
+        vertical_uom = time_uom
 
     # --- Axis order --------------------------------------------------------
     axis_order_raw = get_object_attribute_no_verif(crs_obj, "projected_axis_order")
@@ -494,6 +499,7 @@ def _from_abstract_local3dcrs(
         projected_unknown=projected_details.get("unknown"),
         vertical_epsg_code=vertical_details.get("epsg_code"),
         vertical_uom=vertical_uom,
+        time_uom=time_uom,
         z_increasing_downward=z_increasing_downward,
         vertical_wkt=vertical_details.get("wkt"),
         vertical_unknown=vertical_details.get("unknown"),
@@ -621,6 +627,11 @@ def _from_local_engineering_compound_crs(
     vert_axis_uom_raw = get_object_attribute(crs_obj, "vertical_axis.uom")
     if vert_axis_uom_raw is not None:
         vert_axis_uom = _uom_to_str(vert_axis_uom_raw)
+        
+    is_time = get_object_attribute(crs_obj, "vertical_axis.is_time")
+    time_uom = None
+    if is_time is not None and str(is_time).lower() in ("true", "1", "yes"):
+        time_uom = vert_axis_uom
     vert_axis_dir_raw = get_object_attribute(crs_obj, "vertical_axis.direction")
     if vert_axis_dir_raw is not None:
         d = str(vert_axis_dir_raw)
@@ -693,6 +704,7 @@ def _from_local_engineering_compound_crs(
         # when the resolved CRS carries an explicit direction (not the None sentinel).
         vertical_epsg_code=vert_info.vertical_epsg_code if vert_info else None,
         vertical_uom=(vert_info.vertical_uom if vert_info else None) or vert_axis_uom,
+        time_uom=time_uom,
         z_increasing_downward=(
             vert_info.z_increasing_downward
             if vert_info and vert_details_raw is not None and vert_details_raw.get("z_increasing_downward") is not None

energyml-utils/src/energyml/utils/data/representation_context.py

@@ -12,6 +12,7 @@
 from energyml.utils.epc_utils import extract_uuid_and_version_from_obj_path, get_property_kind_by_title, get_property_kind_uuid_from_property_object
 from energyml.utils.introspection import get_obj_uri, get_obj_uuid, get_object_attribute, is_enum, search_attribute_matching_name
 from energyml.utils.data.helper import RgbaColor, ScalarRenderingInfo, read_graphical_rendering_info
+from energyml.utils.data.crs import extract_crs_info
 
 NO_KIND = "NO_KIND"
 
@@ -149,7 +150,7 @@ def _collect_crs(self):
                         crs_uuid = getattr(crs, "uuid", None)
                         if crs_uuid is not None and crs_uuid not in crs_uuids:
                             crs_uuids.add(crs_uuid)
-                            self.crs_infos.append(CrsInfo.from_crs_object(crs, self.workspace))
+                            self.crs_infos.append(extract_crs_info(crs, self.workspace))
                     else:
                         logging.warning(f"CRS {get_obj_uri(crs_ref)} not found in workspace")
 
@@ -230,6 +231,11 @@ def projected_uom(self) -> Optional[str]:
     @property
     def vertical_uom(self) -> Optional[str]:
         """Return the vertical unit of measure (e.g. "m") if available from CRS info, or None."""
+        if self.vertical_is_time:
+            # For time-based representations, return the time unit of measure if available
+            for ci in self.crs_infos:
+                if ci.time_uom is not None:
+                    return ci.time_uom
         for ci in self.crs_infos:
             if ci.vertical_uom is not None:
                 return ci.vertical_uom