mxcubecore 1.378.0__py3-none-any.whl → 1.380.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/HardwareObjects/abstract/AbstractSampleChanger.py

@@ -816,22 +816,14 @@ class SampleChanger(Container, HardwareObject):
         self._trigger_loaded_sample_changed_event(None)
 
     def _set_loaded_sample(self, sample):
-        previous_loaded = None
-
-        for smp in self.get_sample_list():
-            if smp.is_loaded():
-                previous_loaded = smp
-                break
-
-        for smp in self.get_sample_list():
-            if smp != sample:
-                smp._set_loaded(False)
+        for s in self.get_sample_list():
+            if s != sample:
+                s._set_loaded(False)
             else:
-                if self.get_loaded_sample() == smp:
-                    smp._set_loaded(True)
+                if self.get_loaded_sample() != s:
+                    s._set_loaded(True)
 
-        if previous_loaded != self.get_loaded_sample():
-            self._trigger_loaded_sample_changed_event(sample)
+        self._trigger_loaded_sample_changed_event(sample)
 
     def _set_selected_sample(self, sample):
         cur = self.get_selected_sample()

{mxcubecore-1.378.0.dist-info → mxcubecore-1.380.0.dist-info}/METADATA

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

{mxcubecore-1.378.0.dist-info → mxcubecore-1.380.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
@@ -361,7 +361,7 @@ mxcubecore/HardwareObjects/abstract/AbstractNState.py,sha256=JZvD_ZpQu5U8ycaWFii
 mxcubecore/HardwareObjects/abstract/AbstractOnlineProcessing.py,sha256=vs_fJsDDLwivu1kYHA7RKJXj1aTH5V6G2tEy7MYGw4c,34104
 mxcubecore/HardwareObjects/abstract/AbstractProcedure.py,sha256=kW2SzN5kYYCeOS2IzpzQ9VAjYAoRU7ZtyDxqrOb8Pbg,7849
 mxcubecore/HardwareObjects/abstract/AbstractResolution.py,sha256=eLoymf-i637FbwfZwOtTmKOeSqCQjE2OnpSMdl8oIEg,8524
-mxcubecore/HardwareObjects/abstract/AbstractSampleChanger.py,sha256=ZHMjR4DBmkL7H-KY2QHg-yywMT95mN6N6UuTJLN8Ovc,26956
+mxcubecore/HardwareObjects/abstract/AbstractSampleChanger.py,sha256=h9j7XF8KrRtVQpthF_424fsEIR2LqsEZLNPrLaQB4TI,26718
 mxcubecore/HardwareObjects/abstract/AbstractSampleView.py,sha256=-E9cTMW-gw2vY91OeL2V5wg1icduLeYFOX50vo2_NEA,7774
 mxcubecore/HardwareObjects/abstract/AbstractShutter.py,sha256=OeHSr3N9x-a9QmGdQEu3usN23TYspJGDnZjAkShqk3w,2798
 mxcubecore/HardwareObjects/abstract/AbstractSlits.py,sha256=1fymQFlxJ9naOVT9A8THszEkHm7VxQSGJJWPQV0v3Hs,3839
@@ -469,8 +469,8 @@ mxcubecore/utils/conversion.py,sha256=G1bk2Mi2ZwGbZa5pEeiFaKWxhSVXVGqu1L9_SioyUO
 mxcubecore/utils/qt_import.py,sha256=0lPmqok_oYQZ059kJCq7RWdg490T8YKyRvoZGyWDy4M,14486
 mxcubecore/utils/tango.py,sha256=vwEVrIrWKEFaeaJUz3xbaC7XWHY8ZeJ-pfcSrTfZPIE,2114
 mxcubecore/utils/units.py,sha256=Gh7ovTUN00XBMUoyDG5W7akCx1pROL-M6pK2z1ouemg,1361
-mxcubecore-1.378.0.dist-info/COPYING,sha256=u-Mc8zCecwyo4YoP8UulmzCiZZ_MmCLROd_NBtOcRj0,35148
-mxcubecore-1.378.0.dist-info/COPYING.LESSER,sha256=46mU2C5kSwOnkqkw9XQAJlhBL2JAf1_uCD8lVcXyMRg,7652
-mxcubecore-1.378.0.dist-info/METADATA,sha256=20z9-vDgS9Dj4Aec2KI-C9eynIfMPftgt0CGn3LqXw0,4259
-mxcubecore-1.378.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-mxcubecore-1.378.0.dist-info/RECORD,,
+mxcubecore-1.380.0.dist-info/COPYING,sha256=u-Mc8zCecwyo4YoP8UulmzCiZZ_MmCLROd_NBtOcRj0,35148
+mxcubecore-1.380.0.dist-info/COPYING.LESSER,sha256=46mU2C5kSwOnkqkw9XQAJlhBL2JAf1_uCD8lVcXyMRg,7652
+mxcubecore-1.380.0.dist-info/METADATA,sha256=6B_PPOklkVJy1qOYcNv3IOP6-2vET86xyiVn87VENqk,4259
+mxcubecore-1.380.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+mxcubecore-1.380.0.dist-info/RECORD,,