mxcubecore 1.377.0__py3-none-any.whl → 1.379.0__py3-none-any.whl

mxcubecore/HardwareObjects/Gphl/GphlWorkflowConnection.py

@@ -41,7 +41,6 @@ from py4j import (
     clientserver,
     java_gateway,
 )
-from py4j.protocol import Py4JJavaError
 
 from mxcubecore import HardwareRepository as HWR
 from mxcubecore.BaseHardwareObjects import (
@@ -96,7 +95,7 @@ class GphlWorkflowConnection(HardwareObject):
         super().__init__(name)
         # Py4J gateway to external workflow program
         self._gateway = None
-        self.msg_class_imported = False
+        self.jvm_imports_checked = False
 
         # ID for current workflow calculation
         self._enactment_id = None
@@ -221,7 +220,7 @@ class GphlWorkflowConnection(HardwareObject):
             # so we are never in state 'ON'/STANDBY
             raise RuntimeError("Workflow is already running, cannot be started")
 
-        self.msg_class_imported = False
+        self.jvm_imports_checked = False
 
         # Cannot be done in init, where the api.sessions link is not yet ready
         self.config.software_paths["GPHL_WDIR"] = os.path.join(
@@ -479,30 +478,36 @@ class GphlWorkflowConnection(HardwareObject):
         if self.get_state() is self.STATES.OFF:
             return None
 
-        if not self.msg_class_imported:
-            # Guard agaiunst Java classes being moved to different packages
-            try:
-                msg_class = (
-                    self._gateway.jvm.py4j.reflection.ReflectionUtil.classForName(
-                        "co.gphl.sdcp.astra.service.py4j.Py4jMessage"
-                    )
+        if not self.jvm_imports_checked:
+            # We need to use dir to check for the presence or absence of an imported
+            # class. hasattr/getattr don't do what is needed here, because if the
+            # attribute name is not present java_gateway receives proto.SUCCESS_PACKAGE
+            # from the Java side and instantiates a JavaPackage with the name, even
+            # though no such package exists in the JVM. See:
+            # https://github.com/py4j/py4j/blob/cb9e392d8fc5bec6b99a612e2911017900061628/py4j-python/src/py4j/java_gateway.py#L1748
+            # This looks like a py4j bug on the Java side but needs more investigation.
+
+            # The Py4jMessage class is used as the indicator class here: we assume that
+            # if it has been imported, all other unqualified Java classnames used via
+            # the default JVM view have been imported, otherwise none of them have been
+            # imported and we need to do it here.
+            if "Py4jMessage" not in dir(self._gateway.jvm):
+                for qualified_class_name in (
+                    "co.gphl.sdcp.astra.service.py4j.Py4jMessage",
+                    "co.gphl.beamline.v2_unstable.instrumentation.CentringStatus",
+                    "co.gphl.beamline.v2_unstable.domain_types.CrystalClass",
+                    "co.gphl.beamline.v2_unstable.domain_types.ChemicalElement",
+                    "co.gphl.beamline.v2_unstable.domain_types.AbsorptionEdge",
+                ):
+                    java_gateway.java_import(self._gateway.jvm, qualified_class_name)
+                logging.getLogger("HWR").warning(
+                    "Importing required unqualified class names from the JVM explicitly"
                 )
-                java_gateway.java_import(
-                    self._gateway.jvm, "co.gphl.sdcp.astra.service.py4j.Py4jMessage"
-                )
-            except Py4JJavaError:
-                msg_class = (
-                    self._gateway.jvm.py4j.reflection.ReflectionUtil.classForName(
-                        "co.gphl.sdcp.py4j.Py4jMessage"
-                    )
+                logging.getLogger("HWR").warning(
+                    "Please consider upgrading the GPhL workflow application"
                 )
-                java_gateway.java_import(
-                    self._gateway.jvm, "co.gphl.sdcp.py4j.Py4jMessage"
-                )
-            logging.getLogger("HWR").debug(
-                "GΦL workflow Py4jMessage class is: %s" % msg_class
-            )
-        self.msg_class_imported = True
+
+            self.jvm_imports_checked = True
 
         xx0 = self._decode_py4j_message(py4j_message)
         message_type = xx0.message_type
@@ -923,9 +928,7 @@ class GphlWorkflowConnection(HardwareObject):
         py4j_payload = self._payload_to_java(payload)
 
         try:
-            response = self._gateway.jvm.co.gphl.sdcp.astra.service.py4j.Py4jMessage(
-                py4j_payload, correlation_id
-            )
+            response = self._gateway.jvm.Py4jMessage(py4j_payload, correlation_id)
         except:
             self.abort_workflow(
                 message="Error creating Java message (%s) to send to workflow"
@@ -938,9 +941,7 @@ class GphlWorkflowConnection(HardwareObject):
     def _CentringDone_to_java(self, centringDone):
         jvm = self._gateway.jvm
         return jvm.astra.messagebus.messages.information.CentringDoneImpl(
-            jvm.co.gphl.beamline.v2_unstable.instrumentation.CentringStatus.valueOf(
-                centringDone.status
-            ),
+            jvm.CentringStatus.valueOf(centringDone.status),
             self.to_java_time(centringDone.timestamp),
             self._GoniostatTranslation_to_java(centringDone.goniostatTranslation),
         )
@@ -1042,9 +1043,7 @@ class GphlWorkflowConnection(HardwareObject):
         crystal_classes = selectedLattice.userCrystalClasses
         if crystal_classes:
             ccset = set(
-                jvm.co.gphl.beamline.v2_unstable.domain_types.CrystalClass.fromStringList(
-                    self.toJStringArray(crystal_classes)
-                )
+                jvm.CrystalClass.fromStringList(self.toJStringArray(crystal_classes))
             )
             builder = builder.userCrystalClasses(ccset)
         urlstrings = selectedLattice.referenceReflectionFiles
@@ -1093,9 +1092,7 @@ class GphlWorkflowConnection(HardwareObject):
         crystal_classes = userProvidedInfo.crystalClasses
         if crystal_classes:
             ccset = set(
-                jvm.co.gphl.beamline.v2_unstable.domain_types.CrystalClass.fromStringList(
-                    self.toJStringArray(crystal_classes)
-                )
+                jvm.CrystalClass.fromStringList(self.toJStringArray(crystal_classes))
             )
             builder = builder.crystalClasses(ccset)
         xx0 = userProvidedInfo.spaceGroup
@@ -1122,12 +1119,8 @@ class GphlWorkflowConnection(HardwareObject):
         if anomalousScatterer is None:
             return None
 
-        element = jvm.co.gphl.beamline.v2_unstable.domain_types.ChemicalElement.valueOf(
-            anomalousScatterer.element
-        )
-        edge = jvm.co.gphl.beamline.v2_unstable.domain_types.AbsorptionEdge.valueOf(
-            anomalousScatterer.edge
-        )
+        element = jvm.ChemicalElement.valueOf(anomalousScatterer.element)
+        edge = jvm.AbsorptionEdge.valueOf(anomalousScatterer.edge)
         return jvm.astra.messagebus.messages.domain_types.AnomalousScattererImpl(
             element, edge
         )

mxcubecore/utils/tango.py

@@ -14,7 +14,7 @@ def add_attribute_channel(
     hwo: HardwareObject,
     tango_device: str,
     attribute_name: str,
-    polling: int,
+    polling: int | None = None,
     update_callback=None,
 ) -> TangoChannel:
     """Utility function to add Tango attribute Channel to a hardware object.

{mxcubecore-1.377.0.dist-info → mxcubecore-1.379.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: mxcubecore
-Version: 1.377.0
+Version: 1.379.0
 Summary: Core libraries for the MXCuBE application
 License: LGPL-3.0-or-later
 Keywords: mxcube,mxcube3,mxcubecore

{mxcubecore-1.377.0.dist-info → mxcubecore-1.379.0.dist-info}/RECORD

@@ -187,7 +187,7 @@ mxcubecore/HardwareObjects/Gphl/CollectEmulator.py,sha256=iiH6EblDwEMdxl44BswHJE
 mxcubecore/HardwareObjects/Gphl/GphlMessages.py,sha256=3WSBmUGLka4gilfwpqXyc4SJMlOt9y2sRNrOiE0-DaM,35759
 mxcubecore/HardwareObjects/Gphl/GphlQueueEntry.py,sha256=Rwu6YL8U4knibWI9yOXBoUVuayL6O0rHQWQuEP2Hu_g,2309
 mxcubecore/HardwareObjects/Gphl/GphlWorkflow.py,sha256=vlNhXkP2Ob-bvOuRY5rYVTSgZ8ZULrSQYYhtSS9V4fA,132177
-mxcubecore/HardwareObjects/Gphl/GphlWorkflowConnection.py,sha256=qnr_Vqv4AFOLKCrUUf11HxflpO4NCGk9OYnflvLNt9s,49898
+mxcubecore/HardwareObjects/Gphl/GphlWorkflowConnection.py,sha256=s6B8BdgU6IYeGxi69QqJiM-ZUu34c-ZlaX3S5yRl4wo,50316
 mxcubecore/HardwareObjects/Gphl/Transcal2MiniKappa.py,sha256=Dzf05WSRC5vtOOluZ69f5xZrr6BmT7ITu-OfrKMqemc,9777
 mxcubecore/HardwareObjects/Gphl/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 mxcubecore/HardwareObjects/Gphl/run_mxcube_test.csh,sha256=IvrrFiuRS7c6kYD_FflVc0WINr7oOCBEryAOtMk5zqQ,1233
@@ -467,10 +467,10 @@ mxcubecore/saferef.py,sha256=lbUbzojjg1XM6gKtGKug48CD_LeuQXpZ7R3r1fMjCKw,7164
 mxcubecore/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 mxcubecore/utils/conversion.py,sha256=G1bk2Mi2ZwGbZa5pEeiFaKWxhSVXVGqu1L9_SioyUOk,4513
 mxcubecore/utils/qt_import.py,sha256=0lPmqok_oYQZ059kJCq7RWdg490T8YKyRvoZGyWDy4M,14486
-mxcubecore/utils/tango.py,sha256=lAl7Su_GgyXFEEKZ1yu_2gU18wXHVaBbGR8RYcIOVYk,2100
+mxcubecore/utils/tango.py,sha256=vwEVrIrWKEFaeaJUz3xbaC7XWHY8ZeJ-pfcSrTfZPIE,2114
 mxcubecore/utils/units.py,sha256=Gh7ovTUN00XBMUoyDG5W7akCx1pROL-M6pK2z1ouemg,1361
-mxcubecore-1.377.0.dist-info/COPYING,sha256=u-Mc8zCecwyo4YoP8UulmzCiZZ_MmCLROd_NBtOcRj0,35148
-mxcubecore-1.377.0.dist-info/COPYING.LESSER,sha256=46mU2C5kSwOnkqkw9XQAJlhBL2JAf1_uCD8lVcXyMRg,7652
-mxcubecore-1.377.0.dist-info/METADATA,sha256=w5bWtSmJvquae8bGr7aKvSBAdMXL-4WKiqjiYHvQhl8,4259
-mxcubecore-1.377.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-mxcubecore-1.377.0.dist-info/RECORD,,
+mxcubecore-1.379.0.dist-info/COPYING,sha256=u-Mc8zCecwyo4YoP8UulmzCiZZ_MmCLROd_NBtOcRj0,35148
+mxcubecore-1.379.0.dist-info/COPYING.LESSER,sha256=46mU2C5kSwOnkqkw9XQAJlhBL2JAf1_uCD8lVcXyMRg,7652
+mxcubecore-1.379.0.dist-info/METADATA,sha256=Hw8bAfncOB3DfUvGPo_2AYfoxKx2Y9soX1q6A18Pmrk,4259
+mxcubecore-1.379.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+mxcubecore-1.379.0.dist-info/RECORD,,