Remove logic to shorten compressed_size path attribute names

DiamondLightSource · Sep 17, 2024 · a9cb407 · a9cb407
1 parent fdbc041
commit a9cb407
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/odin_fastcs/odin_data.py b/src/odin_fastcs/odin_data.py
@@ -218,10 +218,7 @@ def _process_parameters(self):
 class FrameProcessorDatasetController(OdinAdapterController):
     def _process_parameters(self):
         for parameter in self._parameters:
-            short_path = parameter.uri[3:]
-            if short_path[0] == "compressed_size":
-                short_path[0] = "compressed"  # slightly shorter PV names
-            parameter.set_path(short_path)
+            parameter.set_path(parameter.uri[3:])
 
 
 def get_all_sub_controllers(

diff --git a/tests/test_controllers.py b/tests/test_controllers.py
@@ -81,7 +81,7 @@ async def test_fp_create_plugin_sub_controllers():
         ),
         OdinParameter(
             uri=["status", "hdf", "dataset", "compressed_size", "compression"],
-            _path=["hdf", "dataset", "whatever"],
+            _path=["hdf", "dataset", "compressed_size", "compression"],
             metadata={"type": "str"},
         ),
     ]
@@ -115,7 +115,7 @@ async def test_fp_create_plugin_sub_controllers():
             assert sub_controllers["DS"]._parameters == [
                 OdinParameter(
                     uri=["status", "hdf", "dataset", "compressed_size", "compression"],
-                    _path=["compressed", "compression"],
+                    _path=["compressed_size", "compression"],
                     metadata={"type": "str"},
                 )
             ]