mxcubecore 1.392.0__py3-none-any.whl → 1.394.0__py3-none-any.whl

mxcubecore/HardwareObjects/ALBA/ALBAMiniDiff.py

@@ -44,7 +44,6 @@ Specific HwObj for M2D2 diffractometer @ ALBA
 - None
 """
 
-import logging
 import time
 
 import gevent
@@ -81,7 +80,7 @@ class ALBAMiniDiff(GenericDiffractometer):
         self.super_hwobj = self.get_object_by_role("beamline-supervisor")
 
         if self.centring_hwobj is None:
-            logging.getLogger("HWR").debug("ALBAMinidiff: Centring math is not defined")
+            self.log.debug("ALBAMinidiff: Centring math is not defined")
 
         if self.super_hwobj is not None:
             self.connect(
@@ -110,7 +109,7 @@ class ALBAMiniDiff(GenericDiffractometer):
             )
             self.connect(self.phi_motor_hwobj, "valueChanged", self.phi_motor_moved)
         else:
-            logging.getLogger("HWR").error("ALBAMiniDiff: Phi motor is not defined")
+            self.log.error("ALBAMiniDiff: Phi motor is not defined")
 
         if self.phiz_motor_hwobj is not None:
             self.connect(
@@ -118,7 +117,7 @@ class ALBAMiniDiff(GenericDiffractometer):
             )
             self.connect(self.phiz_motor_hwobj, "valueChanged", self.phiz_motor_moved)
         else:
-            logging.getLogger("HWR").error("ALBAMiniDiff: Phiz motor is not defined")
+            self.log.error("ALBAMiniDiff: Phiz motor is not defined")
 
         if self.phiy_motor_hwobj is not None:
             self.connect(
@@ -126,7 +125,7 @@ class ALBAMiniDiff(GenericDiffractometer):
             )
             self.connect(self.phiy_motor_hwobj, "valueChanged", self.phiy_motor_moved)
         else:
-            logging.getLogger("HWR").error("ALBAMiniDiff: Phiy motor is not defined")
+            self.log.error("ALBAMiniDiff: Phiy motor is not defined")
 
         if self.zoom_motor_hwobj is not None:
             self.connect(
@@ -141,7 +140,7 @@ class ALBAMiniDiff(GenericDiffractometer):
                 self.zoom_motor_hwobj, "stateChanged", self.zoom_motor_state_changed
             )
         else:
-            logging.getLogger("HWR").error("ALBAMiniDiff: Zoom motor is not defined")
+            self.log.error("ALBAMiniDiff: Zoom motor is not defined")
 
         if self.sample_x_motor_hwobj is not None:
             self.connect(
@@ -153,7 +152,7 @@ class ALBAMiniDiff(GenericDiffractometer):
                 self.sample_x_motor_hwobj, "valueChanged", self.sampleX_motor_moved
             )
         else:
-            logging.getLogger("HWR").error("ALBAMiniDiff: Sampx motor is not defined")
+            self.log.error("ALBAMiniDiff: Sampx motor is not defined")
 
         if self.sample_y_motor_hwobj is not None:
             self.connect(
@@ -165,7 +164,7 @@ class ALBAMiniDiff(GenericDiffractometer):
                 self.sample_y_motor_hwobj, "valueChanged", self.sampleY_motor_moved
             )
         else:
-            logging.getLogger("HWR").error("ALBAMiniDiff: Sampx motor is not defined")
+            self.log.error("ALBAMiniDiff: Sampx motor is not defined")
 
         if self.focus_motor_hwobj is not None:
             self.connect(self.focus_motor_hwobj, "valueChanged", self.focus_motor_moved)
@@ -187,7 +186,7 @@ class ALBAMiniDiff(GenericDiffractometer):
             state = DiffractometerState.tostring(DiffractometerState.Ready)
 
         if state != self.current_state:
-            logging.getLogger("HWR").debug(
+            self.log.debug(
                 "ALBAMinidiff: State changed %s (was: %s)"
                 % (str(state), self.current_state)
             )
@@ -242,9 +241,9 @@ class ALBAMiniDiff(GenericDiffractometer):
 
         y = xy["Y"] * self.pixels_per_mm_y + self.zoom_centre["y"]
 
-        # logging.getLogger("HWR").debug("  motor_positions_to_screen ")
-        # logging.getLogger("HWR").debug(" positions = %s " % str(centred_positions_dict))
-        # logging.getLogger("HWR").debug(" x,y = %s, %s " % (x,y))
+        # self.log.debug("  motor_positions_to_screen ")
+        # self.log.debug(" positions = %s " % str(centred_positions_dict))
+        # self.log.debug(" x,y = %s, %s " % (x,y))
 
         return x, y
 
@@ -297,9 +296,7 @@ class ALBAMiniDiff(GenericDiffractometer):
         if self.prepare_centring():
             GenericDiffractometer.start_manual_centring(self, *args, **kwargs)
         else:
-            logging.getLogger("HWR").info(
-                " Failed to prepare diffractometer for centring"
-            )
+            self.log.info(" Failed to prepare diffractometer for centring")
             self.invalidate_centring()
 
     def start_auto_centring(self, *args, **kwargs):
@@ -310,9 +307,7 @@ class ALBAMiniDiff(GenericDiffractometer):
         if self.prepare_centring():
             GenericDiffractometer.start_auto_centring(self, *args, **kwargs)
         else:
-            logging.getLogger("HWR").info(
-                " Failed to prepare diffractometer for centring"
-            )
+            self.log.info(" Failed to prepare diffractometer for centring")
             self.invalidate_centring()
 
     def prepare_centring(self):
@@ -320,12 +315,10 @@ class ALBAMiniDiff(GenericDiffractometer):
         Prepare beamline for to sample_view phase.
         """
         if not self.is_sample_view_phase():
-            logging.getLogger("HWR").info(
-                " Not in sample view phase. Asking supervisor to go"
-            )
+            self.log.info(" Not in sample view phase. Asking supervisor to go")
             success = self.go_sample_view()
             if not success:
-                logging.getLogger("HWR").info("Cannot set SAMPLE VIEW phase")
+                self.log.info("Cannot set SAMPLE VIEW phase")
                 return False
 
         return True
@@ -341,10 +334,10 @@ class ALBAMiniDiff(GenericDiffractometer):
 
     #    # go to sample_view phase
     #    if not self.is_sample_view_phase():
-    #        logging.getLogger("HWR").info(" Not in sample view phase. Asking supervisor to go")
+    #        self.log.info(" Not in sample view phase. Asking supervisor to go")
     #        success = self.go_sample_view()
     #        if not success:
-    #            logging.getLogger("HWR").info("Cannot set SAMPLE VIEW phase")
+    #            self.log.info("Cannot set SAMPLE VIEW phase")
     #            return False
 
     #    phi_init_position = self.phi_motor_hwobj.get_value()
@@ -366,7 +359,7 @@ class ALBAMiniDiff(GenericDiffractometer):
     #            if click < 2:
     #                self.phi_motor_hwobj.set_value_relative(-90, timeout=None)
 
-    #    #logging.getLogger("HWR").info(" Returning phi to initial position %s" % phi_init_position)
+    #    #self.log.info(" Returning phi to initial position %s" % phi_init_position)
     #    #self.phi_motor_hwobj.set_value(phi_init_position, timeout=None)
     #
     #    return self.centring_hwobj.centeredPosition(return_by_name=False)
@@ -388,7 +381,7 @@ class ALBAMiniDiff(GenericDiffractometer):
         while True:
             super_state = str(self.super_hwobj.get_state()).upper()
             if super_state != "MOVING":
-                logging.getLogger("HWR").debug(
+                self.log.debug(
                     "ALBAMinidiff: go_sample_view done . super_state is %s"
                     % super_state
                 )
@@ -550,7 +543,7 @@ class ALBAMiniDiff(GenericDiffractometer):
         elif phase == "Centring":
             self.super_hwobj.go_sample_view()
         else:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "Diffractometer set_phase asked for un-handled phase: %s" % phase
             )
 

mxcubecore/HardwareObjects/ALBA/ALBAPilatus.py

@@ -19,7 +19,6 @@
 
 from __future__ import print_function
 
-import logging
 import time
 
 from PyTango.gevent import DeviceProxy
@@ -64,7 +63,7 @@ class ALBAPilatus(AbstractDetector, HardwareObject):
             self.latency_time = None
 
         if self.latency_time is None:
-            logging.getLogger("HWR").debug(
+            self.log.debug(
                 "Cannot obtain latency time from Pilatus XML. Using %s"
                 % self.default_latency_time
             )
@@ -112,7 +111,7 @@ class ALBAPilatus(AbstractDetector, HardwareObject):
             beam_x = self.beamx_chan.get_value()
             beam_y = self.beamy_chan.get_value()
         except Exception:
-            pass
+            self.log.exception("")
         return beam_x, beam_y
 
     def get_manufacturer(self):
@@ -161,7 +160,7 @@ class ALBAPilatus(AbstractDetector, HardwareObject):
         kev_diff = abs(det_energy - currentenergy)
 
         if kev_diff > 1.2:
-            logging.getLogger("HWR").debug(
+            self.log.debug(
                 "programming energy_threshold on pilatus to: %s" % currentenergy
             )
             # if self.wait_standby():
@@ -197,19 +196,17 @@ class ALBAPilatus(AbstractDetector, HardwareObject):
         trig_mode = "EXTERNAL_TRIGGER"
         # latency_time = 0.003
 
-        logging.getLogger("HWR").debug(
+        self.log.debug(
             " Preparing detector (dev=%s) for data collection" % self.devname
         )
 
-        logging.getLogger("HWR").debug("    /saving directory: %s" % basedir)
-        logging.getLogger("HWR").debug("    /prefix          : %s" % prefix)
-        logging.getLogger("HWR").debug("    /saving_format   : %s" % fileformat)
-        logging.getLogger("HWR").debug("    /trigger_mode    : %s" % trig_mode)
-        logging.getLogger("HWR").debug("    /acq_nb_frames   : %s" % nb_frames)
-        logging.getLogger("HWR").debug(
-            "    /acq_expo_time   : %s" % str(exp_time - self.latency_time)
-        )
-        logging.getLogger("HWR").debug("    /latency_time    : %s" % self.latency_time)
+        self.log.debug("    /saving directory: %s" % basedir)
+        self.log.debug("    /prefix          : %s" % prefix)
+        self.log.debug("    /saving_format   : %s" % fileformat)
+        self.log.debug("    /trigger_mode    : %s" % trig_mode)
+        self.log.debug("    /acq_nb_frames   : %s" % nb_frames)
+        self.log.debug("    /acq_expo_time   : %s" % str(exp_time - self.latency_time))
+        self.log.debug("    /latency_time    : %s" % self.latency_time)
 
         self.device.write_attribute("saving_mode", "AUTO_FRAME")
         self.device.write_attribute("saving_directory", basedir)
@@ -234,7 +231,7 @@ class ALBAPilatus(AbstractDetector, HardwareObject):
         return True
 
     def prepare_collection(self, nb_frames, first_img_no):
-        logging.getLogger("HWR").debug(
+        self.log.debug(
             "ALBAPilatus. preparing collection. nb_images: %s, first_no: %s"
             % (nb_frames, first_img_no)
         )

mxcubecore/HardwareObjects/ALBA/ALBASession.py

@@ -1,4 +1,3 @@
-import logging
 import os
 import time
 
@@ -53,9 +52,7 @@ class ALBASession(Session.Session):
         # else:
         #    thedir = os.path.join(thedir, 'ARCHIVE')
 
-        logging.getLogger("HWR").debug(
-            "ALBASession. returning archive directory: %s" % archive_dir
-        )
+        self.log.debug("ALBASession. returning archive directory: %s" % archive_dir)
         return archive_dir
 
     def set_ldap_homedir(self, homedir):

mxcubecore/HardwareObjects/ALBA/ALBAZoomMotor.py

@@ -56,8 +56,6 @@ Example Hardware Object XML file :
 </object>
 """
 
-import logging
-
 import PyTango
 
 from mxcubecore import BaseHardwareObjects
@@ -79,7 +77,7 @@ class ALBAZoomMotor(BaseHardwareObjects.Device, AbstractMotor):
         super().__init__(name)
 
     def init(self):
-        logging.getLogger("HWR").debug("Initializing zoom motor IOR")
+        self.log.debug("Initializing zoom motor IOR")
         self.positionChannel = self.get_channel_object("position")
         self.stateChannel = self.get_channel_object("state")
         self.labelsChannel = self.get_channel_object("labels")
@@ -98,22 +96,22 @@ class ALBAZoomMotor(BaseHardwareObjects.Device, AbstractMotor):
             #    retlist.append(int(pos))
             pos = str(label.replace(":", " "))
             retlist.append(pos)
-        logging.getLogger("HWR").debug("Zoom positions list: %s" % repr(retlist))
+        self.log.debug("Zoom positions list: %s" % repr(retlist))
         new_retlist = []
         for n, e in enumerate(retlist):
             name = e.split()
             new_retlist.append("%s %s" % (n + 1, name[0]))
-        logging.getLogger("HWR").debug("Zoom positions list: %s" % repr(new_retlist))
+        self.log.debug("Zoom positions list: %s" % repr(new_retlist))
 
         # retlist = ["z1 1","z2 2"]
-        # logging.getLogger("HWR").debug("Zoom positions list: %s" % repr(retlist))
+        # self.log.debug("Zoom positions list: %s" % repr(retlist))
         return new_retlist
 
     def moveToPosition(self, posno):
         no = posno.split()[0]
-        logging.getLogger("HWR").debug("type %s" % type(no))
+        self.log.debug("type %s" % type(no))
         #        no = posno
-        logging.getLogger("HWR").debug("Moving to position %s" % no)
+        self.log.debug("Moving to position %s" % no)
         state = self.positionChannel.set_value(int(no))
 
     def motorIsMoving(self):
@@ -148,16 +146,14 @@ class ALBAZoomMotor(BaseHardwareObjects.Device, AbstractMotor):
         try:
             n = int(self.positionChannel.get_value())
             value = "%s z%s" % (n, n)
-            logging.getLogger("HWR").debug(
-                "get_current_position_name: %s" % repr(value)
-            )
+            self.log.debug("get_current_position_name: %s" % repr(value))
             return value
         except Exception:
-            logging.getLogger("HWR").debug("cannot get name zoom value")
+            self.log.debug("cannot get name zoom value")
             return None
 
     def stateChanged(self, state):
-        logging.getLogger("HWR").debug("stateChanged emitted: %s" % state)
+        self.log.debug("stateChanged emitted: %s" % state)
         the_state = self.get_state()
         if the_state != self.currentstate:
             self.currentstate = the_state
@@ -167,7 +163,7 @@ class ALBAZoomMotor(BaseHardwareObjects.Device, AbstractMotor):
         previous_position = self.currentposition
         self.currentposition = self.get_current_position_name()
         if self.currentposition != previous_position:
-            logging.getLogger("HWR").debug(
+            self.log.debug(
                 "predefinedPositionChanged emitted: %s" % self.currentposition
             )
             self.emit("predefinedPositionChanged", (self.currentposition, 0))

mxcubecore/HardwareObjects/ALBA/ALBAZoomMotorAutoBrightness.py

@@ -49,8 +49,6 @@ Example Hardware Object XML file :
 </object>
 """
 
-import logging
-
 from mxcubecore import BaseHardwareObjects
 from mxcubecore.HardwareObjects.abstract.AbstractMotor import AbstractMotor
 
@@ -70,7 +68,7 @@ class ALBAZoomMotorAutoBrightness(BaseHardwareObjects.HardwareObject, AbstractMo
         super().__init__(name)
 
     def init(self):
-        logging.getLogger("HWR").debug("Initializing zoom motor autobrightness IOR")
+        self.log.debug("Initializing zoom motor autobrightness IOR")
 
         self.zoom = self.get_object_by_role("zoom")
         self.blight = self.get_object_by_role("blight")
@@ -80,12 +78,12 @@ class ALBAZoomMotorAutoBrightness(BaseHardwareObjects.HardwareObject, AbstractMo
 
     def get_predefined_positions_list(self):
         retlist = self.zoom.get_predefined_positions_list()
-        logging.getLogger("HWR").debug("Zoom positions list: %s" % repr(retlist))
+        self.log.debug("Zoom positions list: %s" % repr(retlist))
         return retlist
 
     def moveToPosition(self, posno):
         # no = posno.split()[0]
-        # logging.getLogger("HWR").debug("Moving to position %s" % no)
+        # self.log.debug("Moving to position %s" % no)
 
         # self.blight.moveToPosition(posno)
         self.zoom.moveToPosition(posno)
@@ -124,22 +122,20 @@ class ALBAZoomMotorAutoBrightness(BaseHardwareObjects.HardwareObject, AbstractMo
     def get_current_position_name(self):
         #        n = int(self.positionChannel.get_value())
         #        value = "%s z%s" % (n, n)
-        #        logging.getLogger("HWR").debug("get_current_position_name: %s" % repr(value))
+        #        self.log.debug("get_current_position_name: %s" % repr(value))
         #        return value
         return self.zoom.get_current_position_name()
 
     def stateChanged(self, state):
-        logging.getLogger("HWR").debug("stateChanged emitted: %s" % state)
+        self.log.debug("stateChanged emitted: %s" % state)
         self.emit("stateChanged", (self.get_state(),))
 
     def positionChanged(self, currentposition):
         currentposition = self.get_current_position_name()
-        logging.getLogger("HWR").debug(
-            "predefinedPositionChanged emitted: %s" % currentposition
-        )
+        self.log.debug("predefinedPositionChanged emitted: %s" % currentposition)
         # Update light brightness step-by-step
         posno = currentposition.split()[0]
-        logging.getLogger("HWR").debug("Moving brightness to: %s" % posno)
+        self.log.debug("Moving brightness to: %s" % posno)
 
         self.blight.moveToPosition(posno)
 

mxcubecore/HardwareObjects/ALBA/XalocMiniDiff.py

@@ -1,4 +1,3 @@
-import logging
 import time
 
 import gevent
@@ -15,7 +14,7 @@ class XalocMiniDiff(GenericDiffractometer):
         self.calibration = self.get_object_by_role("calibration")
         self.centring_hwobj = self.get_object_by_role("centring")
         if self.centring_hwobj is None:
-            logging.getLogger("HWR").debug("EMBLMinidiff: Centring math is not defined")
+            self.log.debug("EMBLMinidiff: Centring math is not defined")
 
         self.cmd_start_auto_focus = self.get_command_object("startAutoFocus")
 
@@ -33,7 +32,7 @@ class XalocMiniDiff(GenericDiffractometer):
             )
             self.connect(self.phi_motor_hwobj, "valueChanged", self.phi_motor_moved)
         else:
-            logging.getLogger("HWR").error("EMBLMiniDiff: Phi motor is not defined")
+            self.log.error("EMBLMiniDiff: Phi motor is not defined")
 
         if self.phiz_motor_hwobj is not None:
             self.connect(
@@ -41,7 +40,7 @@ class XalocMiniDiff(GenericDiffractometer):
             )
             self.connect(self.phiz_motor_hwobj, "valueChanged", self.phiz_motor_moved)
         else:
-            logging.getLogger("HWR").error("EMBLMiniDiff: Phiz motor is not defined")
+            self.log.error("EMBLMiniDiff: Phiz motor is not defined")
 
         if self.phiy_motor_hwobj is not None:
             self.connect(
@@ -49,7 +48,7 @@ class XalocMiniDiff(GenericDiffractometer):
             )
             self.connect(self.phiy_motor_hwobj, "valueChanged", self.phiy_motor_moved)
         else:
-            logging.getLogger("HWR").error("EMBLMiniDiff: Phiy motor is not defined")
+            self.log.error("EMBLMiniDiff: Phiy motor is not defined")
 
         if self.zoom_motor_hwobj is not None:
             self.connect(
@@ -64,7 +63,7 @@ class XalocMiniDiff(GenericDiffractometer):
                 self.zoom_motor_hwobj, "stateChanged", self.zoom_motor_state_changed
             )
         else:
-            logging.getLogger("HWR").error("EMBLMiniDiff: Zoom motor is not defined")
+            self.log.error("EMBLMiniDiff: Zoom motor is not defined")
 
         if self.sample_x_motor_hwobj is not None:
             self.connect(
@@ -76,7 +75,7 @@ class XalocMiniDiff(GenericDiffractometer):
                 self.sample_x_motor_hwobj, "valueChanged", self.sampleX_motor_moved
             )
         else:
-            logging.getLogger("HWR").error("EMBLMiniDiff: Sampx motor is not defined")
+            self.log.error("EMBLMiniDiff: Sampx motor is not defined")
 
         if self.sample_y_motor_hwobj is not None:
             self.connect(
@@ -88,7 +87,7 @@ class XalocMiniDiff(GenericDiffractometer):
                 self.sample_y_motor_hwobj, "valueChanged", self.sampleY_motor_moved
             )
         else:
-            logging.getLogger("HWR").error("EMBLMiniDiff: Sampx motor is not defined")
+            self.log.error("EMBLMiniDiff: Sampx motor is not defined")
 
         if self.focus_motor_hwobj is not None:
             self.connect(self.focus_motor_hwobj, "valueChanged", self.focus_motor_moved)

mxcubecore/HardwareObjects/BeamInfo.py

@@ -19,8 +19,6 @@ beamPosChanged
 -----------------------------------------------------------------------
 """
 
-import logging
-
 from mxcubecore.BaseHardwareObjects import HardwareObject
 
 
@@ -66,20 +64,20 @@ class BeamInfo(HardwareObject):
                 self.aperture_hwobj, "apertureChanged", self.aperture_pos_changed
             )
         else:
-            logging.getLogger("HWR").debug("BeamInfo: Aperture hwobj not defined")
+            self.log.debug("BeamInfo: Aperture hwobj not defined")
 
         self.slits_hwobj = self.get_object_by_role("slits")
         if self.slits_hwobj is not None:
             self.connect(self.slits_hwobj, "gapSizeChanged", self.slits_gap_changed)
         else:
-            logging.getLogger("HWR").debug("BeamInfo: Slits hwobj not defined")
+            self.log.debug("BeamInfo: Slits hwobj not defined")
 
         if self.beam_definer is not None:
             self.connect(
                 self.beam_definer, "definerPosChanged", self.definer_pos_changed
             )
         else:
-            logging.getLogger("HWR").debug("BeamInfo: Beam definer hwobj not defined")
+            self.log.debug("BeamInfo: Beam definer hwobj not defined")
 
         default_beam_divergence_vertical = None
         default_beam_divergence_horizontal = None
@@ -91,7 +89,7 @@ class BeamInfo(HardwareObject):
                 self.get_property("beam_divergence_horizontal")
             )
         except Exception:
-            pass
+            self.log.exception("")
         self.default_beam_divergence = [
             default_beam_divergence_horizontal,
             default_beam_divergence_vertical,

mxcubecore/HardwareObjects/Beamline.py

@@ -42,7 +42,6 @@ __copyright__ = """ Copyright © 2019 by the MXCuBE collaboration """
 __license__ = "LGPLv3+"
 __author__ = "Rasmus H Fogh"
 
-import logging
 
 from mxcubecore.BaseHardwareObjects import (
     ConfiguredObject,
@@ -330,7 +329,7 @@ class Beamline(HardwareObject):
         if acquisition_type != "default":
             dd0 = self.config.default_acquisition_parameters.get(acquisition_type)
             if dd0 is None:
-                logging.getLogger("HWR").warning(
+                self.log.warning(
                     "No separate parameters for acquisition type: %s - using default."
                     % acquisition_type
                 )
@@ -362,7 +361,7 @@ class Beamline(HardwareObject):
         try:
             acq_parameters.resolution = self.resolution.get_value()
         except Exception:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "get_default_acquisition_parameters: "
                 "No current resolution, setting to 0.0"
             )
@@ -371,7 +370,7 @@ class Beamline(HardwareObject):
         try:
             acq_parameters.energy = self.energy.get_value()
         except Exception:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "get_default_acquisition_parameters: No current energy, setting to 0.0"
             )
             acq_parameters.energy = 0.0
@@ -379,7 +378,7 @@ class Beamline(HardwareObject):
         try:
             acq_parameters.transmission = self.transmission.get_value()
         except Exception:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "get_default_acquisition_parameters: "
                 "No current transmission, setting to 0.0"
             )
@@ -390,7 +389,7 @@ class Beamline(HardwareObject):
         try:
             acq_parameters.detector_binning_mode = self.detector.get_binning_mode()
         except Exception:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "get_default_acquisition_parameters: "
                 "Could not get detector mode, setting to ''"
             )
@@ -399,7 +398,7 @@ class Beamline(HardwareObject):
         try:
             acq_parameters.detector_roi_mode = self.detector.get_roi_mode()
         except Exception:
-            logging.getLogger("HWR").warning(
+            self.log.warning(
                 "get_default_acquisition_parameters: "
                 "Could not get roi mode, setting to ''"
             )

mxcubecore/HardwareObjects/BeamlineActions.py

@@ -53,9 +53,7 @@ class ControllerCommand(CommandObject):
                 res = cmd_execution.get()
                 res = res if res else ""
             except Exception:
-                logging.getLogger("HWR").exception(
-                    "%s: execution failed", str(self.name())
-                )
+                self.log.exception("%s: execution failed", str(self.name()))
                 self.emit("commandFailed", (str(self.name()),))
             else:
                 if isinstance(res, gevent.GreenletExit):
@@ -199,7 +197,7 @@ class BeamlineActions(HardwareObject):
                     "Command with name %s already exists"
                     % command["command"].split(".")[-1]
                 )
-                logging.getLogger("HWR").warning(msg)
+                self.log.warning(msg)
                 continue
 
             if command["type"] == "annotated":
@@ -292,9 +290,7 @@ class BeamlineActions(HardwareObject):
         try:
             result = greenlet.get()
         except Exception:
-            logging.getLogger("HWR").exception(
-                "%s: execution failed", self._current_command.cmd_name
-            )
+            self.log.exception("%s: execution failed", self._current_command.cmd_name)
             cmd_obj.emit("commandFailed", (self._current_command.cmd_name,))
         else:
             self._current_command.set_last_result(result)

mxcubecore/HardwareObjects/BlissHutchTrigger.py

@@ -44,9 +44,7 @@ class BlissHutchTrigger(BaseHardwareObjects.HardwareObject):
             self.device = PyTango.gevent.DeviceProxy(self.get_property("pss_tangoname"))
         except PyTango.DevFailed as traceback:
             last_error = traceback[-1]
-            logging.getLogger("HWR").error(
-                "%s: %s", str(self.name()), last_error["desc"]
-            )
+            self.log.error("%s: %s", str(self.name()), last_error["desc"])
             self.device = None
 
         self.pollingTask = None
@@ -60,6 +58,8 @@ class BlissHutchTrigger(BaseHardwareObjects.HardwareObject):
             self.card, self.channel = map(int, PSSinfo.split("/"))
         except Exception:
             logging.getLogger().error("%s: cannot find PSS number", self.name())
+
+            self.log.exception("")
             return
 
         if self.device is not None:

mxcubecore/HardwareObjects/BlissMotorWPositions.py

@@ -1,4 +1,3 @@
-import logging
 from warnings import warn
 
 from BlissMotor import BlissMotor
@@ -22,9 +21,7 @@ class BlissMotorWPositions(BlissMotor):
         try:
             positions = self["positions"]
         except Exception:
-            logging.getLogger("HWR").error(
-                "%s does not define positions.", str(self.name())
-            )
+            self.log.error("%s does not define positions.", str(self.name()))
         else:
             for definedPosition in positions:
                 positionUsername = definedPosition.get_property("username")
@@ -32,7 +29,7 @@ class BlissMotorWPositions(BlissMotor):
                 try:
                     offset = float(definedPosition.get_property("offset"))
                 except Exception:
-                    logging.getLogger("HWR").warning(
+                    self.log.warning(
                         "%s, ignoring position %s: invalid offset.",
                         str(self.name()),
                         positionUsername,
@@ -93,7 +90,7 @@ class BlissMotorWPositions(BlissMotor):
         try:
             self.set_value(self.predefinedPositions[positionName])
         except Exception:
-            logging.getLogger("HWR").exception(
+            self.log.exception(
                 "Cannot move motor %s: invalid position name.", str(self.username)
             )
 
@@ -116,4 +113,4 @@ class BlissMotorWPositions(BlissMotor):
             self.predefinedPositions[str(positionName)] = float(positionOffset)
             self.sortPredefinedPositionsList()
         except Exception:
-            logging.getLogger("HWR").exception("Cannot set new predefined position")
+            self.log.exception("Cannot set new predefined position")