made update_period optional on Updater

Author: evvaaaa

src/fastcs/attributes.py

@@ -26,7 +26,8 @@ async def put(self, controller: Any, attr: AttrW, value: Any) -> None:
 class Updater(Protocol):
     """Protocol for updating the cached readback value of an ``Attribute``."""
 
-    update_period: float
+    # If update period is None then the attribute will not be updated as a task.
+    update_period: float | None
 
     async def update(self, controller: Any, attr: AttrR) -> None:
         pass

src/fastcs/backend.py

@@ -124,6 +124,8 @@ def _add_attribute_updater_tasks(
     for attribute in single_mapping.attributes.values():
         match attribute:
             case AttrR(updater=Updater(update_period=update_period)) as attribute:
+                if update_period is None:
+                    continue
                 callback = _create_updater_callback(
                     attribute, single_mapping.controller
                 )

src/fastcs/backends/epics/gui.py

@@ -27,7 +27,10 @@
 from pydantic import ValidationError
 
 from fastcs.attributes import Attribute, AttrR, AttrRW, AttrW
-from fastcs.backends.epics.util import EpicsNameOptions, _convert_attribute_name_to_pv_name
+from fastcs.backends.epics.util import (
+    EpicsNameOptions,
+    _convert_attribute_name_to_pv_name,
+)
 from fastcs.cs_methods import Command
 from fastcs.datatypes import Bool, Float, Int, String
 from fastcs.exceptions import FastCSException
@@ -65,13 +68,17 @@ def _get_pv(self, attr_path: list[str], name: str):
             ]
             + [
                 _convert_attribute_name_to_pv_name(
-                    attr_name, self.epics_name_options.pv_naming_convention, is_attribute=False
+                    attr_name,
+                    self.epics_name_options.pv_naming_convention,
+                    is_attribute=False,
                 )
                 for attr_name in attr_path
             ]
             + [
                 _convert_attribute_name_to_pv_name(
-                    name, self.epics_name_options.pv_naming_convention, is_attribute=True
+                    name,
+                    self.epics_name_options.pv_naming_convention,
+                    is_attribute=True,
                 ),
             ],
         )

src/fastcs/backends/epics/ioc.py

@@ -30,7 +30,6 @@ class EpicsIOCOptions:
     name_options: EpicsNameOptions = EpicsNameOptions()
 
 
-
 class EpicsIOC:
     def __init__(
         self, pv_prefix: str, mapping: Mapping, options: EpicsIOCOptions | None = None
@@ -74,7 +73,7 @@ def _add_sub_controller_pvi_info(self, pv_prefix: str, parent: BaseController):
                     _convert_attribute_name_to_pv_name(
                         path,
                         self._name_options.pv_naming_convention,
-                        is_attribute=False
+                        is_attribute=False,
                     )
                     for path in child.path
                 ]
@@ -96,7 +95,9 @@ def _create_and_link_attribute_pvs(self, pv_prefix: str, mapping: Mapping) -> No
             ]
             for attr_name, attribute in single_mapping.attributes.items():
                 pv_name = _convert_attribute_name_to_pv_name(
-                    attr_name, self._name_options.pv_naming_convention, is_attribute=True
+                    attr_name,
+                    self._name_options.pv_naming_convention,
+                    is_attribute=True,
                 )
                 _pv_prefix = self._name_options.pv_separator.join(
                     [pv_prefix] + formatted_path
@@ -169,7 +170,9 @@ def _create_and_link_command_pvs(self, pv_prefix: str, mapping: Mapping) -> None
             ]
             for attr_name, method in single_mapping.command_methods.items():
                 pv_name = _convert_attribute_name_to_pv_name(
-                    attr_name, self._name_options.pv_naming_convention, is_attribute=True
+                    attr_name,
+                    self._name_options.pv_naming_convention,
+                    is_attribute=True,
                 )
                 _pv_prefix = self._name_options.pv_separator.join(
                     [pv_prefix] + formatted_path

src/fastcs/backends/epics/util.py

@@ -51,10 +51,16 @@ def _convert_attribute_name_to_pv_name(
         return attr_name.title().replace("_", "")
     elif naming_convention == PvNamingConvention.CAPITALIZED:
         return attr_name.upper().replace("_", "-")
-    elif naming_convention == PvNamingConvention.CAPITALIZED_CONTROLLER_PASCAL_ATTRIBUTE:
+    elif (
+        naming_convention == PvNamingConvention.CAPITALIZED_CONTROLLER_PASCAL_ATTRIBUTE
+    ):
         if is_attribute:
-            return _convert_attribute_name_to_pv_name(attr_name, PvNamingConvention.PASCAL, is_attribute)
-        return _convert_attribute_name_to_pv_name(attr_name, PvNamingConvention.CAPITALIZED)
+            return _convert_attribute_name_to_pv_name(
+                attr_name, PvNamingConvention.PASCAL, is_attribute
+            )
+        return _convert_attribute_name_to_pv_name(
+            attr_name, PvNamingConvention.CAPITALIZED
+        )
     return attr_name