mxcubecore 1.393.0__py3-none-any.whl → 1.395.0__py3-none-any.whl

mxcubecore/HardwareObjects/Marvin.py

@@ -227,7 +227,7 @@ class Marvin(AbstractSampleChanger.SampleChanger):
             self.log_filename = os.path.join(
                 tempfile.gettempdir(), "mxcube", "marvin.log"
             )
-        logging.getLogger("HWR").debug("Marvin log filename: %s" % self.log_filename)
+        self.log.debug("Marvin log filename: %s" % self.log_filename)
         AbstractSampleChanger.SampleChanger.init(self)
 
         self.update_state(HardwareObjectState.READY)
@@ -268,9 +268,9 @@ class Marvin(AbstractSampleChanger.SampleChanger):
         """Updates sample is loaded"""
         if self._sample_detected != sample_detected:
             if sample_detected:
-                logging.getLogger("HWR").debug("Sample changer: sample re-appeared")
+                self.log.debug("Sample changer: sample re-appeared")
             else:
-                logging.getLogger("HWR").debug("Sample changer: sample disappeared")
+                self.log.debug("Sample changer: sample disappeared")
 
             self._sample_detected = sample_detected
             self._info_dict["sample_detected"] = sample_detected
@@ -281,44 +281,32 @@ class Marvin(AbstractSampleChanger.SampleChanger):
         with gevent.Timeout(
             timeout, Exception("Timeout waiting for command acknowldegement")
         ):
-            logging.getLogger("HWR").debug(
-                "Sample changer: start waiting command acknowldegement"
-            )
+            self.log.debug("Sample changer: start waiting command acknowldegement")
             while not self._command_acknowledgement:
                 gevent.sleep(0.05)
-            logging.getLogger("HWR").debug(
-                "Sample changer: done waiting command acknowldegement"
-            )
+            self.log.debug("Sample changer: done waiting command acknowldegement")
 
     def wait_sample_to_disappear(self, timeout):
         with gevent.Timeout(
             timeout, Exception("Timeout waiting for sample to disappear")
         ):
-            logging.getLogger("HWR").debug(
-                "Sample changer: start waiting sample to disappear"
-            )
+            self.log.debug("Sample changer: start waiting sample to disappear")
             while self._sample_detected:
                 if self._was_mount_error:
                     self._was_mount_error = False
                     return
                 gevent.sleep(0.05)
-            logging.getLogger("HWR").debug(
-                "Sample changer: done  waiting sample to disappear"
-            )
+            self.log.debug("Sample changer: done  waiting sample to disappear")
 
     def wait_sample_to_appear(self, timeout):
         with gevent.Timeout(timeout, Exception("Timeout waiting for sample to appear")):
-            logging.getLogger("HWR").debug(
-                "Sample changer: start waiting sample to appear"
-            )
+            self.log.debug("Sample changer: start waiting sample to appear")
             while not self._sample_detected:
                 if self._was_mount_error:
                     self._was_mount_error = False
                     return
                 gevent.sleep(0.05)
-            logging.getLogger("HWR").debug(
-                "Sample changer: done  waiting sample to appear"
-            )
+            self.log.debug("Sample changer: done  waiting sample to appear")
 
     def wait_sample_on_gonio(self, timeout):
         # with gevent.Timeout(timeout, Exception("Timeout waiting for sample on gonio")):
@@ -543,7 +531,7 @@ class Marvin(AbstractSampleChanger.SampleChanger):
         self.emit("progressInit", (msg, 100, False))
 
         # 2. Set diffractometer transfer phase
-        logging.getLogger("HWR").debug(
+        self.log.debug(
             "%s %s"
             % (
                 HWR.beamline.diffractometer.get_current_phase(),
@@ -554,7 +542,7 @@ class Marvin(AbstractSampleChanger.SampleChanger):
             HWR.beamline.diffractometer.get_current_phase()
             != HWR.beamline.diffractometer.PHASE_TRANSFER
         ):
-            logging.getLogger("HWR").debug("set transfer")
+            self.log.debug("set transfer")
             HWR.beamline.diffractometer.set_phase(
                 HWR.beamline.diffractometer.PHASE_TRANSFER, 60.0
             )
@@ -569,9 +557,9 @@ class Marvin(AbstractSampleChanger.SampleChanger):
                 )
                 raise Exception("Unable to set Transfer phase")
 
-        # logging.getLogger("HWR").debug("Sample changer: Closing guillotine...")
+        # self.log.debug("Sample changer: Closing guillotine...")
         # HWR.beamline.detector.close_cover()
-        # logging.getLogger("HWR").debug("Sample changer: Guillotine closed")
+        # self.log.debug("Sample changer: Guillotine closed")
         # 3. If necessary move detector to save position
         if self._focusing_mode == "P13mode":
             if HWR.beamline.detector.distance.get_value() < 399.0:
@@ -583,9 +571,9 @@ class Marvin(AbstractSampleChanger.SampleChanger):
                 log.info("Sample changer: Detector moved to save position")
         else:
             pass
-            # logging.getLogger("HWR").debug("Sample changer: Closing guillotine...")
+            # self.log.debug("Sample changer: Closing guillotine...")
             # HWR.beamline.detector.close_cover()
-            ##logging.getLogger("HWR").debug("Sample changer: Guillotine closed")
+            ##self.log.debug("Sample changer: Guillotine closed")
 
         # 4. Executed command and wait till device is ready
         if self._focusing_mode == "P13mode":
@@ -770,14 +758,14 @@ class Marvin(AbstractSampleChanger.SampleChanger):
         for arg in args:
             arg_arr.append(arg)
 
-        logging.getLogger("HWR").debug(
+        self.log.debug(
             "Sample changer: Sending cmd with arguments: %s..." % str(arg_arr)
         )
 
         self._command_acknowledgement = False
 
         method(arg_arr)
-        logging.getLogger("HWR").debug("Sample changer: Waiting ready...")
+        self.log.debug("Sample changer: Waiting ready...")
         self.wait_command_acknowledgement(5.0)
         self._action_started = True
         gevent.sleep(5)
@@ -788,10 +776,10 @@ class Marvin(AbstractSampleChanger.SampleChanger):
             self.wait_sample_to_appear(60.0)
         else:
             self.wait_ready(120.0)
-        logging.getLogger("HWR").debug("Sample changer: Ready")
-        logging.getLogger("HWR").debug("Sample changer: Waiting veto...")
+        self.log.debug("Sample changer: Ready")
+        self.log.debug("Sample changer: Waiting veto...")
         self.waitVeto(20.0)
-        logging.getLogger("HWR").debug("Sample changer: Veto ready")
+        self.log.debug("Sample changer: Veto ready")
         # if self._is_device_busy():
         #    raise Exception("Action finished to early. Sample changer is not ready!!!")
         self.sample_is_loaded_changed(self.chan_sample_is_loaded.get_value())
@@ -871,7 +859,7 @@ class Marvin(AbstractSampleChanger.SampleChanger):
                         Container.Pin.get_sample_address(basket_no, sample_no)
                     )
         except Exception:
-            logging.getLogger("HWR").exception("")
+            self.log.exception("")
         self._set_selected_component(basket)
         self._set_selected_sample(sample)
 
@@ -1031,7 +1019,7 @@ class Marvin(AbstractSampleChanger.SampleChanger):
                     and self._action_started
                 ):
                     self._state_string = prop_value
-                    logging.getLogger("HWR").debug(
+                    self.log.debug(
                         "Sample changer: status changed: %s" % self._state_string
                     )
                     self._update_state()
@@ -1042,7 +1030,7 @@ class Marvin(AbstractSampleChanger.SampleChanger):
                         self.emit("progressStep", self._progress)
                         self._info_dict["progress"] = self._progress
                 except Exception:
-                    logging.getLogger("HWR").exception("")
+                    self.log.exception("")
             elif prop_name == "CPuck":
                 if prop_value == "1":
                     centre_puck = True

mxcubecore/HardwareObjects/Microdiff.py

@@ -396,9 +396,7 @@ class Microdiff(MiniDiff.MiniDiff):
         # None means infinite timeout
         # <=0 means default timeout
         if timeout is not None and timeout <= 0:
-            logging.getLogger("HWR").warning(
-                "DEBUG: Strange timeout value passed %s" % str(timeout)
-            )
+            self.log.warning("DEBUG: Strange timeout value passed %s" % str(timeout))
             timeout = 30
         with gevent.Timeout(
             timeout, RuntimeError("Timeout waiting for diffractometer to be ready")
@@ -454,18 +452,18 @@ class Microdiff(MiniDiff.MiniDiff):
                 diffr = self.get_object_by_role("controller").diffractometer
                 diffr.prepare("centre")
             except Exception:
-                logging.getLogger("HWR").exception("Cannot prepare centring")
+                self.log.exception("Cannot prepare centring")
 
     def set_light_in(self):
         """Set the backlight in - used by the XMLRPC calls"""
-        logging.getLogger("HWR").info("Moving backlight in")
+        self.log.info("Moving backlight in")
         light_hwobj = self.get_object_by_role("BackLightSwitch")
         light_hwobj.set_value(light_hwobj.VALUES.IN)
         self._wait_ready(20)
 
     def set_light_out(self):
         """Set the backlight out - used by the XMLRPC calls"""
-        logging.getLogger("HWR").info("Moving backlight out")
+        self.log.info("Moving backlight out")
         light_hwobj = self.get_object_by_role("BackLightSwitch")
         light_hwobj.set_value(light_hwobj.VALUES.OUT)
         self._wait_ready(20)
@@ -474,7 +472,7 @@ class Microdiff(MiniDiff.MiniDiff):
         """Set the phase"""
         _use_custom = self.get_property("use_custom_phase_script", False)
         if not self._ready():
-            logging.getLogger("HWR").exception("MD not ready - phase not set.")
+            self.log.exception("MD not ready - phase not set.")
             return
 
         current_phase = self.get_current_phase()
@@ -790,7 +788,7 @@ class Microdiff(MiniDiff.MiniDiff):
 
         beam_pos_x, beam_pos_y = HWR.beamline.beam.get_beam_position_on_screen()
 
-        logging.getLogger("HWR").info("Starting centring procedure ...")
+        self.log.info("Starting centring procedure ...")
 
         if self.in_plate_mode():
             plateTranslation = self.get_object_by_role("plateTranslation")

mxcubecore/HardwareObjects/MicrodiffActuator.py

@@ -54,7 +54,7 @@ class MicrodiffActuator(AbstractActuator):
             tt = float(self.get_property("timeout"))
             self.timeout = tt
         except TypeError:
-            logging.getLogger("HWR").exception("")
+            self.log.exception("")
 
         if self.get_property("use_hwstate"):
             self.hwstate_attr = self.add_channel(

mxcubecore/HardwareObjects/MicrodiffBeamstop.py

@@ -1,5 +1,3 @@
-import logging
-
 from mxcubecore.BaseHardwareObjects import HardwareObject
 
 """
@@ -94,7 +92,7 @@ class MicrodiffBeamstop(HardwareObject):
         try:
             pos = self.beamstop.states[pos]
         except Exception:
-            logging.getLogger("HWR").exception("")
+            self.log.exception("")
 
         if not noEmit:
             if pos:

mxcubecore/HardwareObjects/MicrodiffInOut.py

@@ -57,7 +57,7 @@ class MicrodiffInOut(HardwareObject):
             tt = float(self.get_property("timeout"))
             self.timeout = tt
         except Exception:
-            logging.getLogger("HWR").exception("")
+            self.log.exception("")
 
         if self.get_property("use_hwstate"):
             self.hwstate_attr = self.add_channel(

mxcubecore/HardwareObjects/MicrodiffKappaMotor.py

@@ -1,4 +1,3 @@
-import logging
 import time
 
 import gevent
@@ -94,8 +93,8 @@ class MicrodiffKappaMotor(ExporterMotor):
         For more info see Acta Cryst.(2011). A67, 219-228, Sandor Brockhauser et al., formula (3).
         See also MXSUP-1823.
         """
-        logging.getLogger("HWR").info("In MicrodiffKappaMotor.getNewSamplePosition")
-        logging.getLogger("HWR").info(
+        self.log.info("In MicrodiffKappaMotor.getNewSamplePosition")
+        self.log.info(
             "Input arguments: Kappa %.2f Phi %.2f sampx %.3f sampy %.3f phiy %.3f Kappa2 %.2f Phi2 %.2f"
             % (kappaAngle1, phiAngle1, sampx, sampy, phiy, kappaAngle2, phiAngle2)
         )
@@ -123,7 +122,7 @@ class MicrodiffKappaMotor(ExporterMotor):
         new_motor_pos["sampx"] = float(-t_end[0])
         new_motor_pos["sampy"] = float(-t_end[1])
         new_motor_pos["phiy"] = float(-t_end[2])
-        logging.getLogger("HWR").info("New motor positions: %r" % new_motor_pos)
+        self.log.info("New motor positions: %r" % new_motor_pos)
         return new_motor_pos
 
     def rotation_invariant(self, v):

mxcubecore/HardwareObjects/MiniDiff.py

@@ -142,13 +142,13 @@ class MiniDiff(HardwareObject):
             try:
                 self.lightWago = hwr.get_hardware_object(wl_prop)
             except Exception:
-                logging.getLogger("HWR").exception("")
+                self.log.exception("")
 
         if self.phiMotor is not None:
             self.connect(self.phiMotor, "stateChanged", self.phiMotorStateChanged)
             self.connect(self.phiMotor, "valueChanged", self.emit_diffractometer_moved)
         else:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: phi motor is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -157,7 +157,7 @@ class MiniDiff(HardwareObject):
             self.connect(self.phizMotor, "valueChanged", self.phizMotorMoved)
             self.connect(self.phizMotor, "valueChanged", self.emit_diffractometer_moved)
         else:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: phiz motor is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -166,7 +166,7 @@ class MiniDiff(HardwareObject):
             self.connect(self.phiyMotor, "valueChanged", self.phiyMotorMoved)
             self.connect(self.phiyMotor, "valueChanged", self.emit_diffractometer_moved)
         else:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: phiy motor is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -182,7 +182,7 @@ class MiniDiff(HardwareObject):
             )
             self.connect(self.zoomMotor, "stateChanged", self.zoomMotorStateChanged)
         else:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: zoom motor is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -195,7 +195,7 @@ class MiniDiff(HardwareObject):
                 self.sampleXMotor, "valueChanged", self.emit_diffractometer_moved
             )
         else:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: sampx motor is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -208,13 +208,13 @@ class MiniDiff(HardwareObject):
                 self.sampleYMotor, "valueChanged", self.emit_diffractometer_moved
             )
         else:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: sampx motor is not defined in minidiff equipment %s",
                 str(self.name),
             )
 
         if HWR.beamline.sample_changer is None:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "MiniDiff: sample changer is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -226,13 +226,13 @@ class MiniDiff(HardwareObject):
                     self.sampleChangerSampleIsLoaded,
                 )
             except Exception:
-                logging.getLogger("HWR").exception(
+                self.log.exception(
                     "MiniDiff: could not connect to sample changer smart magnet"
                 )
         if self.lightWago is not None:
             self.connect(self.lightWago, "wagoStateChanged", self.wagoLightStateChanged)
         else:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "MiniDiff: wago light is not defined in minidiff equipment %s",
                 str(self.name),
             )
@@ -249,13 +249,11 @@ class MiniDiff(HardwareObject):
         self._set_rotation_axis_position(value, motor_name="phiz")
 
     def _set_rotation_axis_position(self, value: float, motor_name="phiz"):
-        logging.getLogger("HWR").info(
-            f"Setting rotation axis ({motor_name}) position to {value}"
-        )
+        self.log.info(f"Setting rotation axis ({motor_name}) position to {value}")
 
         try:
             fname = self.get_xml_path()
-            logging.getLogger("HWR").info(f"Updating {fname}")
+            self.log.info(f"Updating {fname}")
 
             tree = ET.parse(fname)
             motor_tag = (
@@ -266,10 +264,10 @@ class MiniDiff(HardwareObject):
             motor_tag.text = str(value)
             tree.write(fname)
         except:
-            logging.getLogger("HWR").info(f"Could not update {fname}")
+            self.log.info(f"Could not update {fname}")
             # raise
         else:
-            logging.getLogger("HWR").info(f"Wrote {fname}")
+            self.log.info(f"Wrote {fname}")
 
         if motor_name == "phiz":
             self.centringPhiz = sample_centring.CentringMotor(
@@ -285,13 +283,11 @@ class MiniDiff(HardwareObject):
         )
 
         try:
-            logging.getLogger("HWR").info(f"Setting MD Alignment reference position")
+            self.log.info(f"Setting MD Alignment reference position")
             print(f" script name {script_name} value {value}")
             self.run_script(f"{script_name}, {value}")
         except:
-            logging.getLogger("HWR").exception(
-                f"Setting MD Alignment reference position failed"
-            )
+            self.log.exception(f"Setting MD Alignment reference position failed")
             raise
 
     # Contained Objects
@@ -550,9 +546,7 @@ class MiniDiff(HardwareObject):
             self.centringSampley.set_value(sampy)
             self.centringPhiy.set_value(-phiy)
         except Exception:
-            logging.getLogger("HWR").exception(
-                "MiniDiff: could not center to beam, aborting"
-            )
+            self.log.exception("MiniDiff: could not center to beam, aborting")
 
     def getAvailableCentringMethods(self):
         return self.centringMethods.keys()
@@ -565,7 +559,7 @@ class MiniDiff(HardwareObject):
             else:
                 time.sleep(0.5)
                 self.wait_ready(60)
-                logging.getLogger("HWR").info("Using MD script for sample centring")
+                self.log.info("Using MD script for sample centring")
                 self.run_script("sample_centering")
                 time.sleep(0.5)
                 self.wait_ready(120)
@@ -577,23 +571,17 @@ class MiniDiff(HardwareObject):
                         res_centering[0].endswith("sample_centering.java")
                         and res_centering[6] == "-1"
                     ):
-                        logging.getLogger("HWR").exception(
-                            "MiniDiff: problem while centring"
-                        )
+                        self.log.exception("MiniDiff: problem while centring")
                         self.emitCentringFailed()
                     else:
-                        logging.getLogger("HWR").info(
+                        self.log.info(
                             "MiniDiff: centring went fine with %s" % str(res_centering)
                         )
                 except:
-                    logging.getLogger("HWR").exception(
-                        "MD script for sample centering had a problem"
-                    )
+                    self.log.exception("MD script for sample centering had a problem")
 
         except KeyError as diag:
-            logging.getLogger("HWR").error(
-                "MiniDiff: unknown centring method (%s)" % str(diag)
-            )
+            self.log.error("MiniDiff: unknown centring method (%s)" % str(diag))
             self.emitCentringFailed()
         else:
             try:
@@ -603,7 +591,7 @@ class MiniDiff(HardwareObject):
                     pass
 
             except Exception:
-                logging.getLogger("HWR").exception("MiniDiff: problem while centring")
+                self.log.exception("MiniDiff: problem while centring")
                 self.emitCentringFailed()
 
     def run_standard_centring(self, method, sample_info):
@@ -613,15 +601,13 @@ class MiniDiff(HardwareObject):
             self.wait_ready(30)
             fun = self.centringMethods[method]
         except KeyError as diag:
-            logging.getLogger("HWR").error(
-                "MiniDiff: unknown centring method (%s)" % str(diag)
-            )
+            self.log.error("MiniDiff: unknown centring method (%s)" % str(diag))
             self.emitCentringFailed()
         else:
             try:
                 fun(sample_info)
             except Exception:
-                logging.getLogger("HWR").exception("MiniDiff: problem while centring")
+                self.log.exception("MiniDiff: problem while centring")
                 self.emitCentringFailed()
 
     def start_centring_method(self, method, sample_info=None):
@@ -636,7 +622,7 @@ class MiniDiff(HardwareObject):
             return
 
         if self.current_centring_procedure is not None:
-            logging.getLogger("HWR").error(
+            self.log.error(
                 "MiniDiff: already in centring method %s" % self.currentCentringMethod
             )
             return
@@ -656,11 +642,9 @@ class MiniDiff(HardwareObject):
             try:
                 self.current_centring_procedure.kill(block=True)
             except Exception:
-                logging.getLogger("HWR").exception(
-                    "MiniDiff: problem aborting the centring method"
-                )
+                self.log.exception("MiniDiff: problem aborting the centring method")
 
-            logging.getLogger("HWR").exception("MiniDiff: Centring canceled")
+            self.log.exception("MiniDiff: Centring canceled")
 
             try:
                 fun = self.cancel_centring_methods[self.currentCentringMethod]
@@ -689,7 +673,7 @@ class MiniDiff(HardwareObject):
         self.accept_centring()
 
     def start_manual_centring(self, sample_info=None):
-        logging.getLogger("HWR").info("Starting centring procedure ...")
+        self.log.info("Starting centring procedure ...")
 
         beam_pos_x, beam_pos_y = HWR.beamline.beam.get_beam_position_on_screen()
 
@@ -913,7 +897,7 @@ class MiniDiff(HardwareObject):
         # save position in MD2 software
         self.save_centring_positions()
 
-        logging.getLogger("HWR").info("DEBUG %s" % self.get_centring_status())
+        self.log.info("DEBUG %s" % self.get_centring_status())
         logging.getLogger("user_level_log").info("Centring successful")
 
     def reject_centring(self):
@@ -955,12 +939,12 @@ class MiniDiff(HardwareObject):
             self.currentCentringMethod = None
             self.current_centring_procedure = None
         else:
-            logging.getLogger("HWR").debug(
+            self.log.debug(
                 "MiniDiff: trying to emit centringSuccessful outside of a centring"
             )
 
     def emitProgressMessage(self, msg=None):
-        # logging.getLogger("HWR").debug("%s: %s", self.name, msg)
+        # self.log.debug("%s: %s", self.name, msg)
         self.emit("progressMessage", (msg,))
 
     def get_centring_status(self):
@@ -1035,9 +1019,7 @@ class MiniDiff(HardwareObject):
         try:
             self.centringStatus["images"] = snapshotsProcedure.get()
         except Exception:
-            logging.getLogger("HWR").exception(
-                "MiniDiff: could not take crystal snapshots"
-            )
+            self.log.exception("MiniDiff: could not take crystal snapshots")
             self.emit("centringSnapshots", (False,))
             self.emitProgressMessage("")
         else:
@@ -1067,9 +1049,7 @@ class MiniDiff(HardwareObject):
                 try:
                     data = GonioHeadConfiguration(**chip_def)
                 except ValidationError:
-                    logging.getLogger("HWR").exception(
-                        "Validation error in %s" % chip_def_fpath
-                    )
+                    self.log.exception("Validation error in %s" % chip_def_fpath)
 
         return data
 
@@ -1086,9 +1066,7 @@ class MiniDiff(HardwareObject):
                 try:
                     GonioHeadConfiguration(**data)
                 except ValidationError:
-                    logging.getLogger("HWR").exception(
-                        "Validation error in %s" % chip_def_fpath
-                    )
+                    self.log.exception("Validation error in %s" % chip_def_fpath)
 
                 _f.write(json.dumps(data, indent=4))
 

mxcubecore/HardwareObjects/MotorWPositions.py

@@ -66,9 +66,9 @@ class MotorWPositions(AbstractMotor, Device):
             role = roles[0]
             self.motor = self.get_object_by_role(role)
         except KeyError:
-            logging.getLogger("HWR").error("MotorWPositions: motor not defined")
+            self.log.error("MotorWPositions: motor not defined")
 
-            logging.getLogger("HWR").exception("")
+            self.log.exception("")
             return
         try:
             self.delta = self["deltas"].get_property(role)
@@ -108,7 +108,7 @@ class MotorWPositions(AbstractMotor, Device):
         try:
             self.motor.set_value(self.predefined_positions[position_name])
         except Exception:
-            logging.getLogger("HWR").exception("MotorWPositions: invalid position name")
+            self.log.exception("MotorWPositions: invalid position name")
 
     def setNewPredefinedPosition(self, positionName, positionOffset):
         raise NotImplementedError

mxcubecore/HardwareObjects/PlateManipulator.py

@@ -242,7 +242,7 @@ class PlateManipulator(SampleChanger):
                     self.num_drops,
                 ) = cmd_get_config.get_value()
             except Exception:
-                logging.getLogger("HWR").exception("")
+                self.log.exception("")
         else:
             self.num_cols = self.get_property("numCols")
             self.num_rows = self.get_property("numRows")
@@ -323,7 +323,7 @@ class PlateManipulator(SampleChanger):
             self.plate_location_changed(self.chan_plate_location.get_value())
             self._on_state_changed(state)
         except AttributeError:
-            logging.getLogger("HWR").exception("")
+            self.log.exception("")
 
     def _on_state_changed(self, state):
         """