76 pass backend options on init instead of run and provide pv name formatting options

src/fastcs/backends/epics/backend.py

@@ -7,17 +7,26 @@
 
 
 class EpicsBackend(Backend):
-    def __init__(self, controller: Controller, pv_prefix: str = "MY-DEVICE-PREFIX"):
+    def __init__(
+        self,
+        controller: Controller,
+        pv_prefix: str = "MY-DEVICE-PREFIX",
+        ioc_options: EpicsIOCOptions | None = None,
+    ):
         super().__init__(controller)
 
         self._pv_prefix = pv_prefix
-        self._ioc = EpicsIOC(pv_prefix, self._mapping)
+        self.ioc_options = ioc_options or EpicsIOCOptions()
+        self._ioc = EpicsIOC(pv_prefix, self._mapping, options=ioc_options)
 
-    def create_docs(self, options: EpicsDocsOptions | None = None) -> None:
-        EpicsDocs(self._mapping).create_docs(options)
+    def create_docs(self, docs_options: EpicsDocsOptions | None = None) -> None:
+        EpicsDocs(self._mapping).create_docs(docs_options)
 
-    def create_gui(self, options: EpicsGUIOptions | None = None) -> None:
-        EpicsGUI(self._mapping, self._pv_prefix).create_gui(options)
+    def create_gui(self, gui_options: EpicsGUIOptions | None = None) -> None:
+        assert self.ioc_options.name_options is not None
+        EpicsGUI(
+            self._mapping, self._pv_prefix, self.ioc_options.name_options
+        ).create_gui(gui_options)
 
-    def _run(self, options: EpicsIOCOptions | None = None):
-        self._ioc.run(self._dispatcher, self._context, options)
+    def _run(self):
+        self._ioc.run(self._dispatcher, self._context)

src/fastcs/backends/epics/gui.py

@@ -27,6 +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.cs_methods import Command
 from fastcs.datatypes import Bool, Float, Int, String
 from fastcs.exceptions import FastCSException
@@ -39,21 +43,45 @@ class EpicsGUIFormat(Enum):
     edl = ".edl"
 
 
-@dataclass
+@dataclass(frozen=True)
 class EpicsGUIOptions:
     output_path: Path = Path.cwd() / "output.bob"
     file_format: EpicsGUIFormat = EpicsGUIFormat.bob
     title: str = "Simple Device"
 
 
 class EpicsGUI:
-    def __init__(self, mapping: Mapping, pv_prefix: str) -> None:
+    def __init__(
+        self,
+        mapping: Mapping,
+        pv_prefix: str,
+        epics_name_options: EpicsNameOptions | None = None,
+    ) -> None:
         self._mapping = mapping
         self._pv_prefix = pv_prefix
+        self.epics_name_options = epics_name_options or EpicsNameOptions()
 
     def _get_pv(self, attr_path: list[str], name: str):
-        attr_prefix = ":".join([self._pv_prefix] + attr_path)
-        return f"{attr_prefix}:{name.title().replace('_', '')}"
+        return self.epics_name_options.pv_separator.join(
+            [
+                self._pv_prefix,
+            ]
+            + [
+                _convert_attribute_name_to_pv_name(
+                    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,
+                ),
+            ],
+        )
 
     @staticmethod
     def _get_read_widget(attribute: AttrR) -> ReadWidgetUnion: