dls-dodal 1.69.0__py3-none-any.whl → 2.0.0__py3-none-any.whl

dodal/devices/oav/snapshots/grid_overlay.py

@@ -19,12 +19,12 @@ def _add_parallel_lines_to_image(
     orientation=Orientation.horizontal,
 ):
     """Draws horizontal or vertical parallel lines on a given image.
-    Draws a line of a given length and orientation starting from a given point; then \
-    draws a given number of parallel lines equally spaced with a given spacing. \
-    If the line is horizontal, the start point corresponds to left end of the initial \
-    line and the other parallel lines will be drawn below the initial line; if \
-    vertical, the start point corresponds to the top end of the initial line and the \
-    other parallel lines will be drawn to the right of the initial line. (0,0) is the \
+    Draws a line of a given length and orientation starting from a given point; then
+    draws a given number of parallel lines equally spaced with a given spacing.
+    If the line is horizontal, the start point corresponds to left end of the initial
+    line and the other parallel lines will be drawn below the initial line; if
+    vertical, the start point corresponds to the top end of the initial line and the
+    other parallel lines will be drawn to the right of the initial line. (0,0) is the
     top left of the image.
 
     Args:
@@ -32,11 +32,12 @@ def _add_parallel_lines_to_image(
         start_x (int): The x coordinate (in pixels) of the start of the initial line.
         start_y (int): The y coordinate (in pixels) of the start of the initial line.
         line_length (int): The length of each of the parallel lines in pixels.
-        spacing (float): The spacing, in pixels, between each parallel line. Strictly, \
+        spacing (float): The spacing, in pixels, between each parallel line. Strictly,
             there are spacing-1 pixels between each line
         num_lines (int): The total number of parallel lines to draw.
-        orientation (Orientation): The orientation (horizontal or vertical) of the \
-            parallel lines to draw."""
+        orientation (Orientation): The orientation (horizontal or vertical) of the
+            parallel lines to draw.
+    """
     lines = [
         (
             (

dodal/devices/oav/snapshots/snapshot_image_processing.py

@@ -8,12 +8,12 @@ CROSSHAIR_FILL_COLOUR = "White"
 
 
 def draw_crosshair(image: Image.Image, beam_x: int, beam_y: int):
-    """
-    Draw a crosshair at the beam centre coordinates specified.
+    """Draw a crosshair at the beam centre coordinates specified.
+
     Args:
-        image: The image to draw the crosshair onto. This is mutated.
-        beam_x: The x-coordinate of the crosshair (pixels)
-        beam_y: The y-coordinate of the crosshair (pixels)
+        image (Image): The image to draw the crosshair onto. This is mutated.
+        beam_x (int): The x-coordinate of the crosshair (pixels).
+        beam_y (int): The y-coordinate of the crosshair (pixels).
     """
     draw = ImageDraw.Draw(image)
     outline_width = 1
@@ -51,16 +51,18 @@ def compute_beam_centre_pixel_xy_for_mm_position(
     beam_pos_at_origin_px: Pixel,
     microns_per_pixel: tuple[float, float],
 ) -> Pixel:
-    """
-    Compute the location of the beam centre in pixels on a reference image.
+    """Compute the location of the beam centre in pixels on a reference image.
+
     Args:
-        sample_pos_mm: x, y location of the sample in mm relative to when the reference image
-            was taken.
-        beam_pos_at_origin_px: x, y position of the beam centre in the reference image (pixels)
-        microns_per_pixel: x, y scaling factor relating the sample position to the position in the image.
-    Returns:
-        x, y location of the beam centre (pixels)
+        sample_pos_mm (tuple[float, float]): x, y location of the sample in mm relative
+            to when the reference image was taken.
+        beam_pos_at_origin_px (Pixel): x, y position of the beam centre in the reference
+            image (pixels).
+        microns_per_pixel (tuple[float, float]): x, y scaling factor relating the sample
+            position to the position in the image.
 
+    Returns:
+        Pixel: x, y location of the beam centre (pixels).
     """
 
     def centre(sample_pos, beam_pos, um_per_px) -> int:

dodal/devices/oav/utils.py

@@ -41,9 +41,7 @@ def bottom_right_from_top_left(
 
 
 class EdgeOutputArrayImageType(IntEnum):
-    """
-    Enum to store the types of image to tweak the output array. We use Original.
-    """
+    """Enum to store the types of image to tweak the output array. We use Original."""
 
     ORIGINAL = 0
     GREYSCALE = 1
@@ -57,8 +55,9 @@ def get_move_required_so_that_beam_is_at_pixel(
     pixel: Pixel,
     oav: OAV,
 ) -> Generator[Msg, None, np.ndarray]:
-    """Calculate the required move so that the given pixel is in the centre of the beam."""
-
+    """Calculate the required move so that the given pixel is in the centre of the
+    beam.
+    """
     current_motor_xyz = np.array(
         [
             (yield from bps.rd(gonio.x)),
@@ -95,7 +94,7 @@ def calculate_x_y_z_of_pixel(
     microns_per_pixel: tuple[float, float],
     xyz_direction: tuple[int, int, int],
 ) -> np.ndarray:
-    """Get the x, y, z position of a pixel in mm"""
+    """Get the x, y, z position of a pixel in mm."""
     beam_distance_px: Pixel = calculate_beam_distance(beam_centre, *pixel)
 
     return current_x_y_z + camera_coordinates_to_xyz_mm(
@@ -123,13 +122,11 @@ def wait_for_tip_to_be_found(
 
 
 def convert_to_gray_and_blur(data: cv2.typing.MatLike) -> cv2.typing.MatLike:
+    """Preprocess the image array data (convert to grayscale and apply a gaussian blur)
+    Image is converted to grayscale (using a weighted mean as green contributes more to
+    brightness) as we aren't interested in data relating to colour. A blur is then
+    applied to mitigate errors due to rogue hot pixels.
     """
-    Preprocess the image array data (convert to grayscale and apply a gaussian blur)
-    Image is converted to grayscale (using a weighted mean as green contributes more to brightness)
-    as we aren't interested in data relating to colour. A blur is then applied to mitigate
-    errors due to rogue hot pixels.
-    """
-
     # kernel size describes how many of the neighbouring pixels are used for the blur,
     # higher kernal size means more of a blur effect
     kernel_size = (7, 7)

dodal/devices/p45.py

@@ -3,9 +3,7 @@ from ophyd_async.epics.motor import Motor
 
 
 class SampleY(StandardReadable):
-    """
-    Motors for controlling the sample's y position and stretch in the y axis.
-    """
+    """Motors for controlling the sample's y position and stretch in the y axis."""
 
     def __init__(self, prefix: str, name: str = ""):
         with self.add_children_as_readables():
@@ -17,9 +15,7 @@ class SampleY(StandardReadable):
 
 
 class SampleTheta(StandardReadable):
-    """
-    Motors for controlling the sample's theta position and skew
-    """
+    """Motors for controlling the sample's theta position and skew."""
 
     def __init__(self, prefix: str, name: str = ""):
         with self.add_children_as_readables():
@@ -31,9 +27,7 @@ class SampleTheta(StandardReadable):
 
 
 class TomoStageWithStretchAndSkew(StandardReadable):
-    """
-    Grouping of motors for the P45 tomography stage
-    """
+    """Grouping of motors for the P45 tomography stage."""
 
     def __init__(self, prefix: str, name: str = ""):
         with self.add_children_as_readables():

dodal/devices/pgm.py

@@ -8,8 +8,14 @@ from ophyd_async.epics.motor import Motor
 
 
 class PlaneGratingMonochromator(StandardReadable):
-    """
-    Plane grating monochromator, it is use in soft x-ray beamline to generate monochromic beam.
+    """Plane grating monochromator, it is use in soft x-ray beamline to generate
+    monochromic beam.
+
+    Args:
+        prefix (str): Beamline specific part of the PV.
+        grating (type[StrictEnum]): The Enum for the grating table.
+        grating_pv (str): The suffix PV part of grating PV.
+        name (str, optional): Name of the device.
     """
 
     def __init__(
@@ -19,18 +25,6 @@ class PlaneGratingMonochromator(StandardReadable):
         grating_pv: str = "GRATINGSELECT:SELECT",
         name: str = "",
     ) -> None:
-        """
-        Parameters
-        ----------
-        prefix:
-            Beamline specific part of the PV
-        grating:
-            The Enum for the grating table.
-        grating_pv:
-            The suffix pv part of grating Pv
-        name:
-            Name of the device
-        """
         with self.add_children_as_readables():
             self.energy = Motor(prefix + "ENERGY")
         with self.add_children_as_readables(StandardReadableFormat.CONFIG_SIGNAL):

dodal/devices/pressure_jump_cell.py

@@ -2,14 +2,21 @@ import asyncio
 from enum import IntEnum
 from typing import Generic, TypeVar
 
-from bluesky.protocols import Movable
+from bluesky.protocols import Movable, Stoppable, Triggerable
 from ophyd_async.core import (
     AsyncStatus,
+    DeviceMock,
     DeviceVector,
+    Reference,
     SignalR,
     StandardReadable,
     StandardReadableFormat,
     StrictEnum,
+    callback_on_mock_put,
+    default_mock_class,
+    set_and_wait_for_other_value,
+    set_mock_value,
+    wait_for_value,
 )
 from ophyd_async.epics.core import epics_signal_r, epics_signal_rw
 
@@ -22,11 +29,6 @@ class PumpState(StrictEnum):
     AUTO_POSITION = "Auto Position"
 
 
-class StopState(StrictEnum):
-    CONTINUE = "CONTINUE"
-    STOP = "STOP"
-
-
 class ValveControlRequest(StrictEnum):
     OPEN = "Open"
     CLOSE = "Close"
@@ -99,10 +101,8 @@ class ValveControl(
 
 
 class AllValvesControl(StandardReadable):
-    """
-    The default IOC for this device only controls
-    specific valves. Other valves are under manual
-    control.
+    """The default IOC for this device only controls specific valves. Other valves are
+    under manual control.
     """
 
     def __init__(
@@ -163,12 +163,10 @@ class Pump(StandardReadable):
 
 
 class PressureTransducer(StandardReadable):
-    """
-    Pressure transducer for a high pressure X-ray cell.
+    """Pressure transducer for a high pressure X-ray cell.
     This is the chamber and there are three of them.
     1 is the start, 3 is where the sample is.
     NOTE: the distinction between the adc prefix and the cell prefix is kept here.
-
     """
 
     def __init__(
@@ -201,34 +199,99 @@ class PressureTransducer(StandardReadable):
         super().__init__(name)
 
 
-class PressureJumpCellController(StandardReadable):
-    """
-    Top-level control for a fixed pressure or pressure jumps.
-    """
+class DoJump(StandardReadable, Triggerable):
+    def __init__(
+        self,
+        prefix: str,
+        busy_signal: Reference,
+        timeout_signal: Reference,
+        name: str = "",
+    ):
+        self._busy_signal = busy_signal
+        self._timeout_signal = timeout_signal
 
-    def __init__(self, prefix: str, name: str = "") -> None:
         with self.add_children_as_readables():
-            self.stop = epics_signal_rw(StopState, f"{prefix}STOP")
+            self.set_jump = epics_signal_rw(bool, f"{prefix}SETJUMP")
+
+        self._name = name
+        super().__init__(name)
+
+    @AsyncStatus.wrap
+    async def trigger(self):
+        await set_and_wait_for_other_value(
+            self.set_jump, True, self._busy_signal(), True
+        )
+
+        timeout_value = await self._timeout_signal().get_value()
+        await wait_for_value(self._busy_signal(), False, timeout_value)
 
-            self.target_pressure = epics_signal_rw(float, f"{prefix}TARGET")
-            self.timeout = epics_signal_rw(float, f"{prefix}TIMER.HIGH")
-            self.go = epics_signal_rw(bool, f"{prefix}GO")
 
-            self.from_pressure = epics_signal_rw(float, f"{prefix}JUMPF")
-            self.to_pressure = epics_signal_rw(float, f"{prefix}JUMPT")
-            self.jump_ready = epics_signal_rw(bool, f"{prefix}SETJUMP")
+class PressureJumpCellController(StandardReadable, Movable, Stoppable):
+    """Top-level control for a fixed pressure or pressure jumps."""
 
+    def __init__(self, prefix: str, name: str = "") -> None:
+        with self.add_children_as_readables():
+            # Common
+            self.busy = epics_signal_r(bool, f"{prefix}GOTOBUSY")
             self.result = epics_signal_r(str, f"{prefix}RESULT")
+            self.timeout = epics_signal_rw(float, f"{prefix}TIMER.HIGH")
 
+            # Constant pressure
+            self.target_pressure = epics_signal_rw(int, f"{prefix}TARGET")
+            self.go = epics_signal_rw(bool, f"{prefix}GO")
+
+            # Pressure jump
+            self.from_pressure = epics_signal_rw(int, f"{prefix}JUMPF")
+            self.to_pressure = epics_signal_rw(int, f"{prefix}JUMPT")
+            self.do_jump = DoJump(
+                prefix, Reference(self.busy), Reference(self.timeout), name
+            )
+
+            # Internal
+            self._stop = epics_signal_rw(bool, f"{prefix}STOP")
             self._name = name
+
         super().__init__(name)
 
+    @AsyncStatus.wrap
+    async def set(self, value: int):
+        """Sets the desired pressure waiting for the device to complete the
+        operation.
+        """
+        timeout = await self.timeout.get_value()
+
+        await self.target_pressure.set(value)
+        await set_and_wait_for_other_value(self.go, True, self.busy, True)
+
+        await wait_for_value(self.busy, False, timeout)  # Change complete
 
+    @AsyncStatus.wrap
+    async def stop(self, success=True):
+        await self._stop.set(True)
+
+
+class BusyMock(DeviceMock["PressureJumpCell"]):
+    async def connect(self, device) -> None:
+        async def busy(*_, **__):
+            async def busy_idle():
+                await asyncio.sleep(0)
+                set_mock_value(device.control.busy, True)
+                await asyncio.sleep(0)
+                set_mock_value(device.control.busy, False)
+
+            asyncio.create_task(busy_idle())
+
+        callback_on_mock_put(device.control.go, busy)
+        callback_on_mock_put(device.control.do_jump.set_jump, busy)
+
+
+@default_mock_class(BusyMock)
 class PressureJumpCell(StandardReadable):
-    """
-    High pressure X-ray cell, used to apply pressure or pressure jumps to a sample.
-    prefix: str
-        The prefix of beamline - SPECIAL - unusual that the cell prefix is computed separately
+    """High pressure X-ray cell, used to apply pressure or pressure jumps to a sample.
+
+    Args:
+        prefix (str): The prefix of beamline - SPECIAL - unusual that the cell prefix is
+            computed separately.
     """
 
     def __init__(
@@ -241,9 +304,7 @@ class PressureJumpCell(StandardReadable):
         self.all_valves_control = AllValvesControl(f"{prefix}{cell_prefix}", name)
         self.pump = Pump(f"{prefix}{cell_prefix}", name)
 
-        self.controller = PressureJumpCellController(
-            f"{prefix}{cell_prefix}CTRL:", name
-        )
+        self.control = PressureJumpCellController(f"{prefix}{cell_prefix}CTRL:", name)
 
         with self.add_children_as_readables():
             self.pressure_transducers: DeviceVector[PressureTransducer] = DeviceVector(

dodal/devices/qbpm.py

@@ -3,9 +3,7 @@ from ophyd_async.epics.core import epics_signal_r
 
 
 class QBPM(StandardReadable):
-    """
-    A beam position monitor that gives a position and intensity of the beam.
-    """
+    """A beam position monitor that gives a position and intensity of the beam."""
 
     def __init__(
         self,

dodal/devices/robot.py

@@ -87,6 +87,11 @@ class BartRobot(StandardReadable, Movable[SampleLocation]):
     # How far the gonio position can be out before loading will fail
     LOAD_TOLERANCE_MM = 0.02
 
+    # mode constants for CRYO_MODE - TODO when https://jira.diamond.ac.uk/browse/I03-1056
+    # is completed these should be made into a proper enum
+    CRYO_MODE_WARM = 0.0
+    CRYO_MODE_CRYO = 1.0
+
     def __init__(self, prefix: str, name: str = "") -> None:
         with self.add_children_as_readables(StandardReadableFormat.HINTED_SIGNAL):
             self.barcode = epics_signal_r(str, prefix + "BARCODE")
@@ -194,12 +199,15 @@ class BartRobot(StandardReadable, Movable[SampleLocation]):
 
     @AsyncStatus.wrap
     async def set(self, value: SampleLocation):
-        """
-        Perform a sample load from the specified sample location
+        """Perform a sample load from the specified sample location.
+
         Args:
-            value: The pin and puck to load, or SAMPLE_LOCATION_EMPTY to unload the sample.
+            value (SampleLocation): The pin and puck to load, or SAMPLE_LOCATION_EMPTY
+                to unload the sample.
+
         Raises:
-            RobotLoadError if a timeout occurs, or if an error occurs loading the sample.
+            RobotLoadError: If a timeout occurs, or if an error occurs loading the
+                sample.
         """
         try:
             if value != SAMPLE_LOCATION_EMPTY:

dodal/devices/s4_slit_gaps.py

@@ -3,7 +3,7 @@ from ophyd_async.epics.motor import Motor
 
 
 class S4SlitGaps(StandardReadable):
-    """Note that the S4 slits have a different PV fromat to other beamline slits"""
+    """Note that the S4 slits have a different PV fromat to other beamline slits."""
 
     def __init__(self, prefix: str, name="") -> None:
         with self.add_children_as_readables():

dodal/devices/selectable_source.py

@@ -14,7 +14,8 @@ T = TypeVar("T")
 
 def get_obj_from_selected_source(selected_source: SelectedSource, s1: T, s2: T) -> T:
     """Util function that maps enum values for SelectedSource to two objects. It then
-    returns one of the objects that corrosponds to the selected_source value."""
+    returns one of the objects that corrosponds to the selected_source value.
+    """
     match selected_source:
         case SelectedSource.SOURCE1:
             return s1
@@ -24,7 +25,9 @@ def get_obj_from_selected_source(selected_source: SelectedSource, s1: T, s2: T)
 
 class SourceSelector(StandardReadable, Movable[SelectedSource]):
     """Device that holds a selected_source signal enum of SelectedSource. Useful for
-    beamlines with multiple sources to coordinate which energy source or shutter to use."""
+    beamlines with multiple sources to coordinate which energy source or shutter to
+    use.
+    """
 
     def __init__(self, name: str = ""):
         with self.add_children_as_readables():

dodal/devices/slits.py

@@ -19,8 +19,7 @@ class MinimalSlits(StandardReadable):
 
 
 class Slits(MinimalSlits):
-    """
-    Representation of a 4-blade set of slits. Allows control/readout of the gap
+    """Representation of a 4-blade set of slits. Allows control/readout of the gap
     between each pair of blades.
     """
 
@@ -40,9 +39,7 @@ class Slits(MinimalSlits):
 
 
 class SlitsY(StandardReadable):
-    """
-    Representation of a 2-blade slits.
-    """
+    """Representation of a 2-blade slits."""
 
     def __init__(
         self,

dodal/devices/smargon.py

@@ -68,7 +68,7 @@ class DeferMoves(StrictEnum):
 
 
 class CombinedMove(TypedDict, total=False):
-    """A move on multiple axes at once using a deferred move"""
+    """A move on multiple axes at once using a deferred move."""
 
     x: float | None
     y: float | None
@@ -79,8 +79,7 @@ class CombinedMove(TypedDict, total=False):
 
 
 class Smargon(XYZOmegaStage, Movable):
-    """
-    Real motors added to allow stops following pin load (e.g. real_x1.stop() )
+    """Real motors added to allow stops following pin load (e.g. real_x1.stop() )
     X1 and X2 real motors provide compound chi motion as well as the compound X travel,
     increasing the gap between x1 and x2 changes chi, moving together changes virtual x.
     Robot loading can nudge these and lead to errors.

dodal/devices/temperture_controller/lakeshore/lakeshore.py

@@ -24,31 +24,28 @@ class Heater336Settings(StrictEnum):
 
 
 class Lakeshore(LakeshoreBaseIO, StandardReadable, Movable[float]):
-    """
-    Device for controlling and reading from a Lakeshore temperature controller.
+    """Device for controlling and reading from a Lakeshore temperature controller.
     It supports multiple channels and PID control.
 
-    Attributes
-    ----------
-    temperature : LakeshoreBaseIO
-        Temperature IO interface.
-    PID : PIDBaseIO
-        PID IO interface.
-    control_channel : derived_signal_rw
-        Signal for selecting the control channel,
-        optional readback as hinted signal
-        (default readback channel is the same as control channel).
-
-    temperature_high_limit: soft_signal_rw
-        Signal to store the soft high temperature limit.
-    temperature_low_limit: soft_signal_rw
-        Signal to store the soft low temperature limit.
-
-
-    Methods
-    -------
-    set(value: float)
-        Set the temperature setpoint for the selected control channel.
+    Attributes:
+        temperature (LakeshoreBaseIO): Temperature IO interface.
+        PID (PIDBaseIO): PID IO interface.
+        control_channel (derived_signal_rw): Signal for selecting the control channel,
+            optional readback as hinted signal (default readback channel is the same
+            as control channel).
+        temperature_high_limit (soft_signal_rw): Signal to store the soft high
+            temperature limit.
+        temperature_low_limit (soft_signal_rw): Signal to store the soft low
+            temperature limit.
+
+    Args:
+        prefix (str): The EPICS prefix for the device.
+        no_channels (int): Number of temperature channels.
+        heater_setting (type[SignalDatatypeT]): Enum type for heater settings.
+        control_channel (int, optional): The initial control channel (default is 1).
+        single_control_channel (bool, optional): Whether to use a single control
+            channel (default is False).
+        name (str, optional): Name of the device.
     """
 
     def __init__(
@@ -60,22 +57,6 @@ class Lakeshore(LakeshoreBaseIO, StandardReadable, Movable[float]):
         single_control_channel: bool = False,
         name: str = "",
     ):
-        """
-        Parameters
-        ----------
-        prefix : str
-            The EPICS prefix for the device.
-        no_channels : int
-            Number of temperature channels.
-        heater_setting : type[SignalDatatypeT]
-            Enum type for heater settings.
-        control_channel : int, optional
-            The initial control channel (default is 1).
-        single_control_channel : bool, optional
-            Whether to use a single control channel (default is False).
-        name : str, optional
-            Name of the device.
-        """
         self._control_channel = soft_signal_rw(int, initial_value=control_channel)
         self.temperature_high_limit = soft_signal_rw(float, initial_value=400)
         self.temperature_low_limit = soft_signal_rw(float, initial_value=0)
@@ -113,9 +94,7 @@ class Lakeshore(LakeshoreBaseIO, StandardReadable, Movable[float]):
 
     @AsyncStatus.wrap
     async def set(self, value: float) -> None:
-        """
-        Set the temperature setpoint for the active control channel.
-        """
+        """Set the temperature setpoint for the active control channel."""
         high, low = await gather(
             self.temperature_high_limit.get_value(),
             self.temperature_low_limit.get_value(),
@@ -149,22 +128,20 @@ class Lakeshore(LakeshoreBaseIO, StandardReadable, Movable[float]):
 
 
 class Lakeshore336(Lakeshore):
+    """Lakeshore 336 temperature controller. With 4 readback and control channels.
+    Heater settings are: Off, Low, Medium, High.
+
+    Args:
+        prefix (str): The EPICS prefix for the device.
+        control_channel (int, optional): The initial control channel (default is 1).
+    """
+
     def __init__(
         self,
         prefix: str,
         control_channel: int = 1,
         name: str = "",
     ):
-        """
-        Lakeshore 336 temperature controller. With 4 readback and control channels.
-        Heater settings are: Off, Low, Medium, High.
-        Parameters
-        ----------
-        prefix : str
-            The EPICS prefix for the device.
-        control_channel : int, optional
-            The initial control channel (default is 1).
-        """
         super().__init__(
             prefix=prefix,
             num_readback_channel=4,
@@ -176,24 +153,21 @@ class Lakeshore336(Lakeshore):
 
 
 class Lakeshore340(Lakeshore):
+    """Lakeshore 340 temperature controller. With 4 readback channels and a single
+    control channel.
+    Heater settings are in power from 0 to 5. 0 is 0 watt, 5 is 50 watt.
+
+    Args:
+        prefix (str): The EPICS prefix for the device.
+        control_channel (int, optional): The initial control channel (default is 1).
+    """
+
     def __init__(
         self,
         prefix: str,
         control_channel: int = 1,
         name: str = "",
     ):
-        """Lakeshore 340 temperature controller. With 4 readback channels and a single
-        control channel.
-        Heater settings are in power from 0 to 5. 0 is 0 watt, 5 is 50 watt.
-
-        Parameters
-        ----------
-        prefix : str
-            The EPICS prefix for the device.
-        control_channel : int, optional
-            The initial control channel (default is 1).
-        """
-
         super().__init__(
             prefix=prefix,
             num_readback_channel=4,