Skip to content

Commit 8ee9c9f

Browse files
Fix for subregion with multiple objects (#4424)
1 parent aa81e07 commit 8ee9c9f

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

_unittest/test_98_Icepak.py

+5-1
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,11 @@ def test_03_AssignPCBRegion(self):
131131
pcb_mesh_region.assignment.padding_types = "Absolute Offset"
132132
assert pcb_mesh_region.assignment.padding_values == ["2mm"] * 6
133133
assert pcb_mesh_region.assignment.padding_types == ["Absolute Offset"] * 6
134-
subregion = self.aedtapp.modeler.create_subregion([50, 50, 50, 50, 100, 100], "Percentage Offset", "PCB")
134+
assert self.aedtapp.modeler.create_subregion([50, 50, 50, 50, 100, 100], "Percentage Offset", "PCB")
135+
box = self.aedtapp.modeler.create_box([0, 0, 0], [1, 2, 3])
136+
assert self.aedtapp.modeler.create_subregion(
137+
[50, 50, 50, 50, 100, 100], "Percentage Offset", ["PCB", box.name]
138+
)
135139
else:
136140
box = self.aedtapp.modeler.create_box([0, 0, 0], [1, 2, 3])
137141
pcb_mesh_region.Objects = box.name

pyaedt/modeler/cad/Primitives.py

+1-3
Original file line numberDiff line numberDiff line change
@@ -6144,9 +6144,7 @@ def _parse_region_args(self, pad_value, pad_type, region_name, parts, region_typ
61446144
parts = [parts]
61456145
normal_parts = [p for p in parts if p in self._app.modeler.objects_by_name]
61466146
submodel_parts = [p for p in parts if p in self._app.modeler.user_defined_components]
6147-
normal_parts = ",".join(normal_parts)
6148-
submodel_parts = ",".join(submodel_parts)
6149-
arg += [["NAME:SubRegionPartNames", normal_parts], ["NAME:SubRegionSubmodelNames", submodel_parts]]
6147+
arg += [["NAME:SubRegionPartNames"] + normal_parts, ["NAME:SubRegionSubmodelNames"] + submodel_parts]
61506148
flags = "NonModel#Wireframe"
61516149
arg2 = [
61526150
"NAME:Attributes",

0 commit comments

Comments
 (0)