Fix private member access for pyright SLF001

Author: GDYendell

src/fastcs/attributes.py

@@ -131,15 +131,17 @@ def allowed_values(self) -> list[str] | None:
         return self._allowed_values
 
     async def process(self, value: T) -> None:
-        if self._write_display_callback is not None:
-            await self._write_display_callback(self._datatype.dtype(value))
-
         await self.process_without_display_update(value)
+        await self.update_display_without_process(value)
 
     async def process_without_display_update(self, value: T) -> None:
         if self._process_callback is not None:
             await self._process_callback(self._datatype.dtype(value))
 
+    async def update_display_without_process(self, value: T) -> None:
+        if self._write_display_callback is not None:
+            await self._write_display_callback(self._datatype.dtype(value))
+
     def set_process_callback(self, callback: AttrCallback[T] | None) -> None:
         self._process_callback = callback
 

src/fastcs/backends/epics/ioc.py

@@ -33,7 +33,7 @@ def _get_input_record(pv_name: str, datatype: DataType) -> RecordWrapper:
 
 
 def _create_and_link_read_pv(pv_name: str, attribute: AttrR) -> None:
-    record = _get_input_record(pv_name, attribute._datatype)
+    record = _get_input_record(pv_name, attribute.datatype)
 
     async def async_wrapper(v):
         record.set(v)

src/fastcs/backends/tango/dsr.py

@@ -211,7 +211,7 @@ def register_dev(dev_name: str, dev_class: str, dsr_instance: str) -> None:
     dsr_name = f"{dev_class}/{dsr_instance}"
     dev_info = DbDevInfo()
     dev_info.name = dev_name
-    dev_info._class = dev_class
+    dev_info._class = dev_class  # noqa
     dev_info.server = dsr_name
 
     db = Database()

src/fastcs/controller.py

@@ -17,6 +17,12 @@ def path(self) -> list[str]:
         """Path prefix of attributes, recursively including parent ``Controller``s."""
         return self._path
 
+    def set_path(self, path: list[str]):
+        if self._path:
+            raise ValueError(f"SubController is already registered under {self.path}")
+
+        self._path = path
+
     def _bind_attrs(self) -> None:
         for attr_name in dir(self):
             attr = getattr(self, attr_name)
@@ -29,13 +35,9 @@ def register_sub_controller(self, name: str, sub_controller: SubController):
             raise ValueError(
                 f"Controller {self} already has a SubController registered as {name}"
             )
-        if sub_controller.path:
-            raise ValueError(
-                f"SubController is already registered under {sub_controller.path}"
-            )
 
         self.__sub_controller_tree[name] = sub_controller
-        sub_controller._path = self.path + [name]
+        sub_controller.set_path(self.path + [name])
 
     def get_sub_controllers(self) -> dict[str, BaseController]:
         return self.__sub_controller_tree