mx-bluesky 1.5.10__py3-none-any.whl → 1.5.11__py3-none-any.whl

mx_bluesky/beamlines/i24/serial/fixed_target/{i24ssx_Chip_Manager_py3v1.py → i24ssx_chip_manager_py3v1.py}

@@ -459,7 +459,7 @@ def load_lite_map() -> MsgGenerator:
     SSX_LOGGER.debug("Run load stock map with 'clear' setting.")
     yield from load_stock_map("clear")
     # fmt: off
-    # Oxford_block_dict is wrong (columns and rows need to flip) added in script below to generate it automatically however kept this for backwards compatiability/reference
+    # Oxford_block_dict is wrong (columns and rows need to flip) added in script below to generate it automatically however kept this for backwards compatibility/reference
     oxford_block_dict = {   # noqa: F841
         'A1': '01', 'A2': '02', 'A3': '03', 'A4': '04', 'A5': '05', 'A6': '06', 'A7': '07', 'A8': '08',
         'B1': '16', 'B2': '15', 'B3': '14', 'B4': '13', 'B5': '12', 'B6': '11', 'B7': '10', 'B8': '09',
@@ -523,7 +523,7 @@ def load_lite_map() -> MsgGenerator:
 def moveto(place: str = "origin", pmac: PMAC = inject("pmac")) -> MsgGenerator:
     SSX_LOGGER.info(f"Move to: {place}")
     if place == Fiducials.zero:
-        SSX_LOGGER.info("Chip aspecific move.")
+        SSX_LOGGER.info("Chip moving to zero")
         yield from bps.trigger(pmac.to_xyz_zero)
         return
 
@@ -705,7 +705,7 @@ def cs_maker(pmac: PMAC = inject("pmac")) -> MsgGenerator:
 
     Skew:
     Skew is the difference between the Sz1 and Sz2 after rotation is taken out.
-    This should be measured in situ prior to expriment, ie. measure by hand using
+    This should be measured in situ prior to experiment, ie. measure by hand using
     opposite and adjacent RBV after calibration of scale factors.
     """
     chip_type = int(caget(CHIPTYPE_PV))
@@ -738,10 +738,10 @@ def cs_maker(pmac: PMAC = inject("pmac")) -> MsgGenerator:
             float(cs_info["scalez"]),
         )
         skew = float(cs_info["skew"])
-        Sx_dir, Sy_dir, Sz_dir = (
-            int(cs_info["Sx_dir"]),
-            int(cs_info["Sy_dir"]),
-            int(cs_info["Sz_dir"]),
+        sx_dir, sy_dir, sz_dir = (
+            int(cs_info["sx_dir"]),
+            int(cs_info["sy_dir"]),
+            int(cs_info["sz_dir"]),
         )
     except KeyError:
         SSX_LOGGER.error("Wrong or missing key in the cs json file.")
@@ -751,60 +751,60 @@ def cs_maker(pmac: PMAC = inject("pmac")) -> MsgGenerator:
         if val not in [1, -1]:
             raise ValueError("Wrong value for direction. Please set to either -1 or 1.")
 
-    check_dir(Sx_dir)
-    check_dir(Sy_dir)
-    check_dir(Sz_dir)
+    check_dir(sx_dir)
+    check_dir(sy_dir)
+    check_dir(sz_dir)
 
     # Rotation Around Z
-    # If stages upsidedown (I24) change sign of Sz
-    Sz1 = -1 * f1_y / fiducial_dict[chip_type][0]
-    Sz2 = f2_x / fiducial_dict[chip_type][1]
-    Sz = Sz_dir * ((Sz1 + Sz2) / 2)
-    Cz = np.sqrt(1 - Sz**2)
-    SSX_LOGGER.info(f"Sz1 , {Sz1:1.4f}, {np.degrees(np.arcsin(Sz1)):1.4f}")
-    SSX_LOGGER.info(f"Sz2 , {Sz2:1.4f}, {np.degrees(np.arcsin(Sz2)):1.4f}")
-    SSX_LOGGER.info(f"Sz , {Sz:1.4f}, {np.degrees(np.arcsin(Sz)):1.4f}")
-    SSX_LOGGER.info(f"Cz , {Cz:1.4f}, {np.degrees(np.arcsin(Cz)):1.4f}")
+    # If stages upsidedown (I24) change sign of sz
+    sz1 = -1 * f1_y / fiducial_dict[chip_type][0]
+    sz2 = f2_x / fiducial_dict[chip_type][1]
+    sz = sz_dir * ((sz1 + sz2) / 2)
+    cz = np.sqrt(1 - sz**2)
+    SSX_LOGGER.info(f"sz1 , {sz1:1.4f}, {np.degrees(np.arcsin(sz1)):1.4f}")
+    SSX_LOGGER.info(f"sz2 , {sz2:1.4f}, {np.degrees(np.arcsin(sz2)):1.4f}")
+    SSX_LOGGER.info(f"sz , {sz:1.4f}, {np.degrees(np.arcsin(sz)):1.4f}")
+    SSX_LOGGER.info(f"cz , {cz:1.4f}, {np.degrees(np.arcsin(cz)):1.4f}")
     # Rotation Around Y
-    Sy = Sy_dir * f1_z / fiducial_dict[chip_type][0]
-    Cy = np.sqrt(1 - Sy**2)
-    SSX_LOGGER.info(f"Sy , {Sy:1.4f}, {np.degrees(np.arcsin(Sy)):1.4f}")
-    SSX_LOGGER.info(f"Cy , {Cy:1.4f}, {np.degrees(np.arcsin(Cy)):1.4f}")
+    sy = sy_dir * f1_z / fiducial_dict[chip_type][0]
+    cy = np.sqrt(1 - sy**2)
+    SSX_LOGGER.info(f"sy , {sy:1.4f}, {np.degrees(np.arcsin(sy)):1.4f}")
+    SSX_LOGGER.info(f"cy , {cy:1.4f}, {np.degrees(np.arcsin(cy)):1.4f}")
     # Rotation Around X
-    # If stages upsidedown (I24) change sign of Sx
-    Sx = Sx_dir * f2_z / fiducial_dict[chip_type][1]
-    Cx = np.sqrt(1 - Sx**2)
-    SSX_LOGGER.info(f"Sx , {Sx:1.4f}, {np.degrees(np.arcsin(Sx)):1.4f}")
-    SSX_LOGGER.info(f"Cx , {Cx:1.4f}, {np.degrees(np.arcsin(Cx)):1.4f}")
+    # If stages upsidedown (I24) change sign of sx
+    sx = sx_dir * f2_z / fiducial_dict[chip_type][1]
+    cx = np.sqrt(1 - sx**2)
+    SSX_LOGGER.info(f"sx , {sx:1.4f}, {np.degrees(np.arcsin(sx)):1.4f}")
+    SSX_LOGGER.info(f"cx , {cx:1.4f}, {np.degrees(np.arcsin(cx)):1.4f}")
 
-    x1factor = mtr1_dir * scalex * (Cy * Cz)
-    y1factor = mtr2_dir * scaley * (-1.0 * Cx * Sz)
-    z1factor = mtr3_dir * scalez * Sy
+    x1factor = mtr1_dir * scalex * (cy * cz)
+    y1factor = mtr2_dir * scaley * (-1.0 * cx * sz)
+    z1factor = mtr3_dir * scalez * sy
 
-    x2factor = mtr1_dir * scalex * ((Sx * Sy * Cz) + (Cx * Sz))
-    y2factor = mtr2_dir * scaley * ((Cx * Cz) - (Sx * Sy * Sz))
-    z2factor = mtr3_dir * scalez * (-1.0 * Sx * Cy)
+    x2factor = mtr1_dir * scalex * ((sx * sy * cz) + (cx * sz))
+    y2factor = mtr2_dir * scaley * ((cx * cz) - (sx * sy * sz))
+    z2factor = mtr3_dir * scalez * (-1.0 * sx * cy)
 
-    x3factor = mtr1_dir * scalex * ((Sx * Sz) - (Cx * Sy * Cz))
-    y3factor = mtr2_dir * scaley * ((Cx * Sy * Sz) + (Sx * Cz))
-    z3factor = mtr3_dir * scalez * (Cx * Cy)
+    x3factor = mtr1_dir * scalex * ((sx * sz) - (cx * sy * cz))
+    y3factor = mtr2_dir * scaley * ((cx * sy * sz) + (sx * cz))
+    z3factor = mtr3_dir * scalez * (cx * cy)
 
     SSX_LOGGER.info(f"Skew being used is: {skew:1.4f}")
-    s1 = np.degrees(np.arcsin(Sz1))
-    s2 = np.degrees(np.arcsin(Sz2))
-    rot = np.degrees(np.arcsin((Sz1 + Sz2) / 2))
+    s1 = np.degrees(np.arcsin(sz1))
+    s2 = np.degrees(np.arcsin(sz2))
+    rot = np.degrees(np.arcsin((sz1 + sz2) / 2))
     calc_skew = (s1 - rot) - (s2 - rot)
     SSX_LOGGER.info(f"s1:{s1:1.4f} s2:{s2:1.4f} rot:{rot:1.4f}")
     SSX_LOGGER.info(f"Calculated rotation from current fiducials is: {rot:1.4f}")
     SSX_LOGGER.info(f"Calculated Skew from current fiducials is: {calc_skew:1.4f}")
     SSX_LOGGER.info("Calculated Skew has been known to have the wrong sign")
 
-    sinD = np.sin((skew / 2) * (np.pi / 180))
-    cosD = np.cos((skew / 2) * (np.pi / 180))
-    new_x1factor = (x1factor * cosD) + (y1factor * sinD)
-    new_y1factor = (x1factor * sinD) + (y1factor * cosD)
-    new_x2factor = (x2factor * cosD) + (y2factor * sinD)
-    new_y2factor = (x2factor * sinD) + (y2factor * cosD)
+    sin_d = np.sin((skew / 2) * (np.pi / 180))
+    cod_d = np.cos((skew / 2) * (np.pi / 180))
+    new_x1factor = (x1factor * cod_d) + (y1factor * sin_d)
+    new_y1factor = (x1factor * sin_d) + (y1factor * cod_d)
+    new_x2factor = (x2factor * cod_d) + (y2factor * sin_d)
+    new_y2factor = (x2factor * sin_d) + (y2factor * cod_d)
 
     cs1 = f"#1->{new_x1factor:+1.3f}X{new_y1factor:+1.3f}Y{z1factor:+1.3f}Z"
     cs2 = f"#2->{new_x2factor:+1.3f}X{new_y2factor:+1.3f}Y{z2factor:+1.3f}Z"

mx_bluesky/beamlines/i24/serial/fixed_target/{i24ssx_Chip_StartUp_py3v1.py → i24ssx_chip_startup_py3v1.py}

@@ -70,10 +70,10 @@ def pathli(l_in=None, way="typewriter", reverse=False):
 def zippum(list_1_args, list_2_args):
     list_1, type_1, reverse_1 = list_1_args
     list_2, type_2, reverse_2 = list_2_args
-    A_path = pathli(list_1, type_1, reverse_1)
-    B_path = pathli(list_2, type_2, reverse_2)
+    a_path = pathli(list_1, type_1, reverse_1)
+    b_path = pathli(list_2, type_2, reverse_2)
     zipped_list = []
-    for a, b in zip(A_path, B_path, strict=False):
+    for a, b in zip(a_path, b_path, strict=False):
         zipped_list.append(a + b)
     return zipped_list
 

mx_bluesky/beamlines/i24/serial/fixed_target/i24ssx_moveonclick.py

@@ -13,7 +13,7 @@ from dodal.devices.i24.pmac import PMAC
 from dodal.devices.oav.oav_detector import OAV
 
 from mx_bluesky.beamlines.i24.serial.fixed_target import (
-    i24ssx_Chip_Manager_py3v1 as manager,
+    i24ssx_chip_manager_py3v1 as manager,
 )
 from mx_bluesky.beamlines.i24.serial.fixed_target.ft_utils import Fiducials
 from mx_bluesky.beamlines.i24.serial.log import SSX_LOGGER
@@ -52,12 +52,12 @@ def _move_on_mouse_click_plan(
         position coordinates.
     """
     zoomcalibrator = yield from _calculate_zoom_calibrator(oav)
-    beamX, beamY = yield from _get_beam_centre(oav)
+    beam_x, beam_y = yield from _get_beam_centre(oav)
     x, y = clicked_position
-    xmove = -10 * (beamX - x) * zoomcalibrator
-    ymove = 10 * (beamY - y) * zoomcalibrator
+    xmove = -10 * (beam_x - x) * zoomcalibrator
+    ymove = 10 * (beam_y - y) * zoomcalibrator
     SSX_LOGGER.info(f"Zoom calibrator {zoomcalibrator}")
-    SSX_LOGGER.info(f"Beam centre {beamX} {beamY}")
+    SSX_LOGGER.info(f"Beam centre {beam_x} {beam_y}")
     SSX_LOGGER.info(f"Moving X and Y {xmove} {ymove}")
     xmovepmacstring = "&2#5J:" + str(xmove)
     ymovepmacstring = "&2#6J:" + str(ymove)
@@ -66,22 +66,22 @@ def _move_on_mouse_click_plan(
 
 
 # Register clicks and move chip stages
-def onMouse(event, x, y, flags, param):
+def on_mouse(event, x, y, flags, param):
     if event == cv.EVENT_LBUTTONUP:
-        RE = param[0]
+        run_engine = param[0]
         pmac = param[1]
         oav = param[2]
         SSX_LOGGER.info(f"Clicked X and Y {x} {y}")
-        RE(_move_on_mouse_click_plan(oav, pmac, (x, y)))
+        run_engine(_move_on_mouse_click_plan(oav, pmac, (x, y)))
 
 
-def update_ui(oav, frame, RE):
+def update_ui(oav, frame, run_engine):
     # Get beam x and y values
-    beamX, beamY = RE(_get_beam_centre(oav)).plan_result
+    beam_x, beam_y = run_engine(_get_beam_centre(oav)).plan_result
 
     # Overlay text and beam centre
     cv.ellipse(
-        frame, (beamX, beamY), (12, 8), 0.0, 0.0, 360, (0, 255, 255), thickness=2
+        frame, (beam_x, beam_y), (12, 8), 0.0, 0.0, 360, (0, 255, 255), thickness=2
     )
     cv.putText(
         frame,
@@ -156,13 +156,13 @@ def update_ui(oav, frame, RE):
     cv.imshow("OAV1view", frame)
 
 
-def start_viewer(oav: OAV, pmac: PMAC, RE: RunEngine, oav1: str = OAV1_CAM):
-    # Create a video caputure from OAV1
+def start_viewer(oav: OAV, pmac: PMAC, run_engine: RunEngine, oav1: str = OAV1_CAM):
+    # Create a video capture from OAV1
     cap = cv.VideoCapture(oav1)
 
     # Create window named OAV1view and set onmouse to this
     cv.namedWindow("OAV1view")
-    cv.setMouseCallback("OAV1view", onMouse, param=[RE, pmac, oav])  # type: ignore
+    cv.setMouseCallback("OAV1view", on_mouse, param=[run_engine, pmac, oav])  # type: ignore
 
     SSX_LOGGER.info("Showing camera feed. Press escape to close")
     # Read captured video and store them in success and frame
@@ -172,44 +172,44 @@ def start_viewer(oav: OAV, pmac: PMAC, RE: RunEngine, oav1: str = OAV1_CAM):
     while success:
         success, frame = cap.read()
 
-        update_ui(oav, frame, RE)
+        update_ui(oav, frame, run_engine)
 
         k = cv.waitKey(1)
         if k == 113:  # Q
-            RE(manager.moveto(Fiducials.zero, pmac))
+            run_engine(manager.moveto(Fiducials.zero, pmac))
         if k == 119:  # W
-            RE(manager.moveto(Fiducials.fid1, pmac))
+            run_engine(manager.moveto(Fiducials.fid1, pmac))
         if k == 101:  # E
-            RE(manager.moveto(Fiducials.fid2, pmac))
+            run_engine(manager.moveto(Fiducials.fid2, pmac))
         if k == 97:  # A
-            RE(bps.trigger(pmac.home, wait=True))
+            run_engine(bps.trigger(pmac.home, wait=True))
             print("Current position set as origin")
         if k == 115:  # S
-            RE(manager.fiducial(1))
+            run_engine(manager.fiducial(1))
         if k == 100:  # D
-            RE(manager.fiducial(2))
+            run_engine(manager.fiducial(2))
         if k == 99:  # C
-            RE(manager.cs_maker(pmac))
+            run_engine(manager.cs_maker(pmac))
         if k == 98:  # B
-            RE(
+            run_engine(
                 manager.block_check()
             )  # doesn't work well for blockcheck as image doesn't update
         if k == 104:  # H
-            RE(bps.abs_set(pmac.pmac_string, "&2#6J:-10", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#6J:-10", wait=True))
         if k == 110:  # N
-            RE(bps.abs_set(pmac.pmac_string, "&2#6J:10", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#6J:10", wait=True))
         if k == 109:  # M
-            RE(bps.abs_set(pmac.pmac_string, "&2#5J:-10", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#5J:-10", wait=True))
         if k == 98:  # B
-            RE(bps.abs_set(pmac.pmac_string, "&2#5J:10", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#5J:10", wait=True))
         if k == 105:  # I
-            RE(bps.abs_set(pmac.pmac_string, "&2#7J:-150", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#7J:-150", wait=True))
         if k == 111:  # O
-            RE(bps.abs_set(pmac.pmac_string, "&2#7J:150", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#7J:150", wait=True))
         if k == 117:  # U
-            RE(bps.abs_set(pmac.pmac_string, "&2#7J:-1000", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#7J:-1000", wait=True))
         if k == 112:  # P
-            RE(bps.abs_set(pmac.pmac_string, "&2#7J:1000", wait=True))
+            run_engine(bps.abs_set(pmac.pmac_string, "&2#7J:1000", wait=True))
         if k == 0x1B:  # esc
             cv.destroyWindow("OAV1view")
             print("Pressed escape. Closing window")
@@ -220,8 +220,8 @@ def start_viewer(oav: OAV, pmac: PMAC, RE: RunEngine, oav1: str = OAV1_CAM):
 
 
 if __name__ == "__main__":
-    RE = RunEngine(call_returns_result=True)
+    run_engine = RunEngine(call_returns_result=True)
     # Get devices out of dodal
     oav: OAV = i24.oav(connect_immediately=True)
     pmac: PMAC = i24.pmac(connect_immediately=True)
-    start_viewer(oav, pmac, RE)
+    start_viewer(oav, pmac, run_engine)

mx_bluesky/beamlines/i24/serial/log.py

@@ -9,7 +9,7 @@ import bluesky.plan_stubs as bps
 from bluesky.log import logger as bluesky_logger
 from bluesky.utils import MsgGenerator
 from dodal.log import DEFAULT_GRAYLOG_PORT, ophyd_async_logger
-from dodal.log import LOGGER as dodal_logger
+from dodal.log import LOGGER as DODAL_LOGGER
 
 from mx_bluesky.common.utils.log import do_default_logging_setup
 
@@ -19,7 +19,7 @@ VISIT_PATH = Path("/dls_sw/i24/etc/ssx_current_visit.txt")
 # Logging set up
 SSX_LOGGER = logging.getLogger("I24serial")
 SSX_LOGGER.addHandler(logging.NullHandler())
-SSX_LOGGER.parent = dodal_logger
+SSX_LOGGER.parent = DODAL_LOGGER
 
 
 logging_config = {
@@ -101,14 +101,14 @@ def config(
     """
     if logfile:
         logs = _get_logging_file_path() / logfile
-        fileFormatter = logging.Formatter(
+        file_formatter = logging.Formatter(
             "%(asctime)s %(levelname)s: \t(%(name)s) %(message)s",
             datefmt="%d-%m-%Y %I:%M:%S",
         )
-        FH = logging.FileHandler(logs, mode=write_mode, encoding="utf-8", delay=delayed)
-        FH.setLevel(logging.DEBUG)
-        FH.setFormatter(fileFormatter)
-        SSX_LOGGER.addHandler(FH)
+        fh = logging.FileHandler(logs, mode=write_mode, encoding="utf-8", delay=delayed)
+        fh.setLevel(logging.DEBUG)
+        fh.setFormatter(file_formatter)
+        SSX_LOGGER.addHandler(fh)
     do_default_logging_setup(
         "mx-bluesky.log",
         DEFAULT_GRAYLOG_PORT,
@@ -116,8 +116,8 @@ def config(
         integrate_all_logs=False,
     )
     # Remove dodal StreamHandler to avoid duplication of messages above debug
-    dodal_logger.removeHandler(dodal_logger.handlers[0])
-    _integrate_bluesky_logs(dodal_logger)
+    DODAL_LOGGER.removeHandler(DODAL_LOGGER.handlers[0])
+    _integrate_bluesky_logs(DODAL_LOGGER)
 
 
 def log_on_entry(func):
@@ -151,6 +151,6 @@ def clean_up_log_config_at_end() -> MsgGenerator:
     # See https://github.com/DiamondLightSource/mx-bluesky/issues/609
     for handler in SSX_LOGGER.handlers:
         SSX_LOGGER.removeHandler(handler)
-    for handler in dodal_logger.handlers:
-        dodal_logger.removeHandler(handler)
+    for handler in DODAL_LOGGER.handlers:
+        DODAL_LOGGER.removeHandler(handler)
     yield from bps.null()

mx_bluesky/beamlines/i24/serial/parameters/fixed_target/cs/cs_maker.json

@@ -3,7 +3,7 @@
     "scaley": 10011.9,
     "scalez": 10000.0,
     "skew": 0.025,
-    "Sx_dir": 1,
-    "Sy_dir": 1,
-    "Sz_dir": 1
+    "sx_dir": 1,
+    "sy_dir": 1,
+    "sz_dir": 1
 }

mx_bluesky/beamlines/i24/serial/setup_beamline/ca.py

@@ -38,15 +38,3 @@ def caput(pv, new_val):
     else:
         a = Popen(["caput", pv, str(new_val)], stdout=PIPE, stderr=PIPE)
         a_stdout, a_stderr = a.communicate()
-
-
-def evaluate(val):
-    try:
-        int(val)
-        return int(val)
-    except Exception:
-        try:
-            float(val)
-            return float(val)
-        except ValueError:
-            return val

mx_bluesky/beamlines/i24/serial/setup_beamline/pv.py

@@ -4,36 +4,17 @@ PVs
 # Now with Eiger goodness
 """
 
-import os
-import sys
-
-
-def __show__(name):
-    """Checks available variables given a string, uses first two letters"""
-    for things in globals():
-        if name[:2].lower() in things.lower():
-            print("Available:", things)
-    print()
-
-
-def __which__():
-    """Return script directory, used for finding which pv.py you are running"""
-    pathname, scriptname = os.path.split(sys.argv[0])
-    print("Current dir: " + os.path.abspath(pathname))
-    print("path to pv.py: ")
-
-
 requested_transmission = "BL24I-OP-ATTN-01:T2A:SETVAL1"
 
 
 # Eiger
 eiger_filepath = "BL24I-EA-EIGER-01:CAM:FilePath"
 eiger_filename = "BL24I-EA-EIGER-01:CAM:FileName"
-eiger_ODfilepath = "BL24I-EA-EIGER-01:OD:FilePath"
-eiger_ODfilename = "BL24I-EA-EIGER-01:OD:FileName"
-eiger_seqID = "BL24I-EA-EIGER-01:CAM:SequenceId"
+eiger_od_filepath = "BL24I-EA-EIGER-01:OD:FilePath"
+eiger_od_filename = "BL24I-EA-EIGER-01:OD:FileName"
+eiger_seq_id = "BL24I-EA-EIGER-01:CAM:SequenceId"
 eiger_numimages = "BL24I-EA-EIGER-01:CAM:NumImages"
-eiger_ODnumcapture = "BL24I-EA-EIGER-01:OD:NumCapture"
+eiger_od_num_capture = "BL24I-EA-EIGER-01:OD:NumCapture"
 eiger_numexpimage = "BL24I-EA-EIGER-01:CAM:NumExposures"
 eiger_acquiretime = "BL24I-EA-EIGER-01:CAM:AcquireTime"
 eiger_acquireperiod = "BL24I-EA-EIGER-01:CAM:AcquirePeriod"
@@ -47,37 +28,37 @@ eiger_stream = "BL24I-EA-EIGER-01:CAM:StreamEnable"
 eiger_monitor = "BL24I-EA-EIGER-01:CAM:MonitorEnable"
 eiger_datasource = "BL24I-EA-EIGER-01:CAM:DataSource"
 eiger_statuspoll = "BL24I-EA-EIGER-01:CAM:ReadStatus.SCAN"
-eiger_ROImode = "BL24I-EA-EIGER-01:CAM:ROIMode"
+eiger_roi_mode = "BL24I-EA-EIGER-01:CAM:ROIMode"
 eiger_ff = "BL24I-EA-EIGER-01:CAM:FlatfieldApplied"
 eiger_compress = "BL24I-EA-EIGER-01:CAM:FWCompression"
 eiger_compresstype = "BL24I-EA-EIGER-01:CAM:CompressionAlgo"
-eiger_ODcompress = "BL24I-EA-EIGER-01:OD:Compression"
-eiger_ODdatatype = "BL24I-EA-EIGER-01:OD:DataType"
+eiger_od_compress = "BL24I-EA-EIGER-01:OD:Compression"
+eiger_od_datatype = "BL24I-EA-EIGER-01:OD:DataType"
 eiger_bitdepthrbv = "BL24I-EA-EIGER-01:CAM:BitDepthImage_RBV"
 eiger_countmode = "BL24I-EA-EIGER-01:CAM:CountingMode"
 eiger_autosum = "BL24I-EA-EIGER-01:CAM:AutoSummation"
 eiger_hdrdetail = "BL24I-EA-EIGER-01:CAM:StreamHdrDetail"
 eiger_hdrappen = "BL24I-EA-EIGER-01:CAM:StreamHdrAppendix"
-eiger_ODcapture = "BL24I-EA-EIGER-01:OD:Capture"
+eiger_od_capture = "BL24I-EA-EIGER-01:OD:Capture"
 eiger_acquire = "BL24I-EA-EIGER-01:CAM:Acquire"
 eiger_wavelength = "BL24I-EA-EIGER-01:CAM:Wavelength"
 eiger_detdist = "BL24I-EA-EIGER-01:CAM:DetDist"
 eiger_beamx = "BL24I-EA-EIGER-01:CAM:BeamX"
 eiger_beamy = "BL24I-EA-EIGER-01:CAM:BeamY"
 eiger_omegaincr = "BL24I-EA-EIGER-01:CAM:OmegaIncr"
-eiger_ODfilenameRBV = "BL24I-EA-EIGER-01:OD:FP:FileName_RBV"
+eiger_od_filename_rbv = "BL24I-EA-EIGER-01:OD:FP:FileName_RBV"
 
 # BPMs
 qbpm1_inten = "BL24I-DI-QBPM-01:INTEN"
-qbpm1_intenN = "BL24I-DI-QBPM-01:INTEN_N"
+qbpm1_inten_n = "BL24I-DI-QBPM-01:INTEN_N"
 qbpm2_inten = "BL24I-DI-QBPM-02:INTEN"
-qbpm2_intenN = "BL24I-DI-QBPM-02:INTEN_N"
+qbpm2_inten_n = "BL24I-DI-QBPM-02:INTEN_N"
 qbpm3_inten = "BL24I-DI-QBPM-03:INTEN"
-qbpm3_intenN = "BL24I-DI-QBPM-03:INTEN_N"
+qbpm3_inten_n = "BL24I-DI-QBPM-03:INTEN_N"
 # Cividec
 cividec_x = "BL24I-AL-XBPM-01:XS"
 cividec_y = "BL24I-AL-XBPM-01:YS"
-cividec_sumI = "BL24I-EA-XBPM-01:SumAll:Sigma_RBV"
+cividec_sum_i = "BL24I-EA-XBPM-01:SumAll:Sigma_RBV"
 cividec_beamx = "BL24I-EA-XBPM-01:PosX:MeanValue_RBV"
 cividec_beamy = "BL24I-EA-XBPM-01:PosY:MeanValue_RBV"
 # Lancelot

mx_bluesky/beamlines/i24/serial/setup_beamline/pv_abstract.py

@@ -22,15 +22,15 @@ class Eiger:
     det_y_threshold = 70.0
     det_y_target = 59.0
 
-    class pv:
+    class PV:
         detector_distance = pv.eiger_detdist
         wavelength = pv.eiger_wavelength
         transmission = "BL24I-EA-PILAT-01:cam1:FilterTransm"
-        filenameRBV = pv.eiger_ODfilenameRBV
-        file_name = pv.eiger_ODfilename
-        file_path = pv.eiger_ODfilepath
+        filename_rbv = pv.eiger_od_filename_rbv
+        file_name = pv.eiger_od_filename
+        file_path = pv.eiger_od_filepath
         file_template = None
-        sequence_id = pv.eiger_seqID
+        sequence_id = pv.eiger_seq_id
         beamx = pv.eiger_beamx
         beamy = pv.eiger_beamy
         bit_depth = pv.eiger_bitdepthrbv