FIX: Mesh assignment update for 24R2 new APIs

src/ansys/aedt/core/modules/mesh_icepak.py

@@ -869,38 +869,41 @@
         list
         """
         if isinstance(self._assignment, SubRegion):
-            # try to update name
             if self.name in self._app.odesign.GetChildObject("Mesh").GetChildNames():
-                parts = []
-                subparts = []
-                if "Parts" in self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropNames():
-                    parts = self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropValue("Parts")
-                if "Submodels" in self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropNames():
-                    subparts = (
-                        self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropValue("Submodels")
+                if self._app.settings.aedt_version < "2024.2":
+                    # try to update name, APIs lacking a lot
+                    parts = []
+                    subparts = []
+                    if "Parts" in self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropNames():
+                        parts = self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropValue("Parts")
+                    if "Submodels" in self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropNames():
+                        subparts = (
+                            self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropValue("Submodels")
+                        )
+                    if not isinstance(parts, list):
+                        parts = [parts]
+                    if not isinstance(subparts, list):
+                        subparts = [subparts]
+                    parts += subparts
+                    sub_regions = self._app.modeler.non_model_objects
+                    for sr in sub_regions:
+                        p1 = []
+                        p2 = []
+                        if "Part Names" in self._app.modeler[sr].history().props:
+                            p1 = self._app.modeler[sr].history().props.get("Part Names", None)
+                            if not isinstance(p1, list):
+                                p1 = [p1]
+                        elif "Submodel Names" in self._app.modeler[sr].history().props:
+                            p2 = self._app.modeler[sr].history().props.get("Submodel Names", None)
+                            if not isinstance(p2, list):
+                                p2 = [p2]
+                        p1 += p2
+                        if "CreateSubRegion" == self._app.modeler[sr].history().command and all(p in p1 for p in parts):
+                            self._assignment.name = sr
+                else:
+                    self._assignment.name = (
+                        self._app.odesign.GetChildObject("Mesh").GetChildObject(self.name).GetPropValue("Assignment")
                     )
-                if not isinstance(parts, list):
-                    parts = [parts]
-                if not isinstance(subparts, list):
-                    subparts = [subparts]
-                parts += subparts
-                sub_regions = self._app.modeler.non_model_objects
-                for sr in sub_regions:
-                    p1 = []
-                    p2 = []
-                    history = self._app.modeler[sr].history()
-                    history_props = history.properties
-                    if "Part Names" in history_props:
-                        p1 = history_props.get("Part Names", None)
-                        if not isinstance(p1, list):
-                            p1 = [p1]
-                    elif "Submodel Names" in history_props:
-                        p2 = history_props.get("Submodel Names", None)
-                        if not isinstance(p2, list):
-                            p2 = [p2]
-                    p1 += p2
-                    if "CreateSubRegion" == history.command and all(p in p1 for p in parts):
-                        self._assignment.name = sr
             return self._assignment
         elif isinstance(self._assignment, list):
             return self._assignment

tests/system/general/test_98_Icepak.py

@@ -318,6 +318,11 @@ def test_03_AssignPCBRegion(self):
         assert self.aedtapp.mesh.meshregions_dict
         assert pcb_mesh_region.delete()
 
+    def test_07_ExportStepForWB(self):
+        file_path = self.local_scratch.path
+        file_name = "WBStepModel"
+        assert self.aedtapp.export_3d_model(file_name, file_path, ".x_t", [], ["Region", "Component_Region"])
+
     def test_04_ImportGroup(self):
         assert self.aedtapp.copyGroupFrom("Group1", "uUSB", src_project_name, self.project_path)
 
@@ -376,11 +381,6 @@ def test_05_EMLoss(self):
     def test_06_clear_linked_data(self):
         assert self.aedtapp.clear_linked_data()
 
-    def test_07_ExportStepForWB(self):
-        file_path = self.local_scratch.path
-        file_name = "WBStepModel"
-        assert self.aedtapp.export_3d_model(file_name, file_path, ".x_t", [], ["Region", "Component_Region"])
-
     def test_08_Setup(self):
         setup_name = "DomSetup"
         my_setup = self.aedtapp.create_setup(setup_name)
@@ -2007,3 +2007,16 @@ def test_82_folder_settings(self, add_app):
         fs.scale_settings.unit = "kel"
         assert fs.scale_settings.unit == "kel"
         app.close_project()
+
+    def test_83_MultipleMeshRegions(self):
+        # test issue 5485
+        self.aedtapp.insert_design("test83")
+        c1 = self.aedtapp.modeler.create_cylinder(orientation=0, origin=[0, 0, 0], radius=5, height=10)
+        c2 = self.aedtapp.modeler.create_cylinder(orientation=1, origin=[100, 100, 100], radius=2, height=15)
+        mesh_class = self.aedtapp.mesh
+        mess1 = self.aedtapp.logger.get_messages()
+        m1 = mesh_class.assign_mesh_region([c1.name])
+        m2 = mesh_class.assign_mesh_region([c2.name])
+        assert m1.assignment.name != m2.assignment.name
+        mess2 = self.aedtapp.logger.get_messages()
+        assert len(mess1.global_level) == len(mess2.global_level)