Skip to content

Commit b157c86

Browse files
Changed get_signals to create_signals
1 parent 16ae772 commit b157c86

File tree

3 files changed

+16
-22
lines changed

3 files changed

+16
-22
lines changed

src/dodal/devices/electron_analyser/abstract_analyser.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,15 +70,15 @@ def __init__(self, prefix: str, name: str = "") -> None:
7070
self.acquisition_mode = epics_signal_rw(str, prefix + "ACQ_MODE")
7171

7272
# Read once per scan after data acquired
73-
self.energy_axis = self._get_energy_axis_signal(prefix)
73+
self.energy_axis = self._create_energy_axis_signal(prefix)
7474
self.binding_energy_axis = derived_signal_r(
7575
self._calculate_binding_energy_axis,
7676
"eV",
7777
energy_axis=self.energy_axis,
7878
excitation_energy=self.excitation_energy,
7979
energy_mode=self.energy_mode,
8080
)
81-
self.angle_axis = self._get_angle_axis_signal(prefix)
81+
self.angle_axis = self._create_angle_axis_signal(prefix)
8282
self.step_time = epics_signal_r(float, prefix + "AcquireTime")
8383
self.total_steps = epics_signal_r(int, prefix + "TOTAL_POINTS_RBV")
8484
self.total_time = derived_signal_r(
@@ -92,13 +92,17 @@ def __init__(self, prefix: str, name: str = "") -> None:
9292
super().__init__(prefix=prefix, name=name)
9393

9494
@abstractmethod
95-
def _get_angle_axis_signal(self, prefix: str = "") -> SignalR[Array1D[np.float64]]:
95+
def _create_angle_axis_signal(
96+
self, prefix: str = ""
97+
) -> SignalR[Array1D[np.float64]]:
9698
"""
9799
The signal that defines the angle axis. Depends on analyser model.
98100
"""
99101

100102
@abstractmethod
101-
def _get_energy_axis_signal(self, prefix: str = "") -> SignalR[Array1D[np.float64]]:
103+
def _create_energy_axis_signal(
104+
self, prefix: str = ""
105+
) -> SignalR[Array1D[np.float64]]:
102106
"""
103107
The signal that defines the energy axis. Depends on analyser model.
104108
"""

src/dodal/devices/electron_analyser/specs_analyser.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,7 @@ def __init__(self, prefix: str, name: str = "") -> None:
2525

2626
super().__init__(prefix, name)
2727

28-
def _get_angle_axis_signal(self, prefix: str = "") -> SignalR:
29-
if hasattr(self, "angle_axis"):
30-
return self.angle_axis
28+
def _create_angle_axis_signal(self, prefix: str = "") -> SignalR:
3129
angle_axis = derived_signal_r(
3230
self._calculate_angle_axis,
3331
min_angle=self.min_angle_axis,
@@ -46,9 +44,7 @@ def _calculate_angle_axis(
4644
axis = np.array([min_angle + offset + i * width for i in range(slices)])
4745
return axis
4846

49-
def _get_energy_axis_signal(self, prefix: str = "") -> SignalR:
50-
if hasattr(self, "energy_axis"):
51-
return self.energy_axis
47+
def _create_energy_axis_signal(self, prefix: str = "") -> SignalR:
5248
energy_axis = derived_signal_r(
5349
self._calculate_energy_axis,
5450
"eV",

src/dodal/devices/electron_analyser/vgscienta_analyser.py

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -32,20 +32,14 @@ def __init__(self, prefix: str, name: str = "") -> None:
3232

3333
super().__init__(prefix, name)
3434

35-
def _get_energy_axis_signal(self, prefix: str = "") -> SignalR[Array1D[np.float64]]:
36-
"""
37-
Override abstract and return epics signal
38-
"""
39-
if hasattr(self, "energy_axis"):
40-
return self.energy_axis
35+
def _create_energy_axis_signal(
36+
self, prefix: str = ""
37+
) -> SignalR[Array1D[np.float64]]:
4138
return epics_signal_r(Array1D[np.float64], prefix + "X_SCALE_RBV")
4239

43-
def _get_angle_axis_signal(self, prefix: str = "") -> SignalR[Array1D[np.float64]]:
44-
"""
45-
Override abstract and return epics signal
46-
"""
47-
if hasattr(self, "angle_axis"):
48-
return self.angle_axis
40+
def _create_angle_axis_signal(
41+
self, prefix: str = ""
42+
) -> SignalR[Array1D[np.float64]]:
4943
return epics_signal_r(Array1D[np.float64], prefix + "Y_SCALE_RBV")
5044

5145
@property

0 commit comments

Comments
 (0)