Fix #2523. Extrusion depth now reflects perpendicular depth

src/blenderbim/blenderbim/bim/module/model/wall.py

@@ -242,7 +242,9 @@ class ChangeExtrusionDepth(bpy.types.Operator, tool.Ifc.Operator):
             extrusion = tool.Model.get_extrusion(representation)
             if not extrusion:
                 return
-            extrusion.Depth = self.depth
+            x, y, z = extrusion.ExtrudedDirection.DirectionRatios
+            x_angle = Vector((0, 1)).angle_signed(Vector((y, z)))
+            extrusion.Depth = self.depth * (1 / cos(x_angle))
             if element.is_a("IfcWall"):
                 for rel in element.ConnectedFrom:
                     if rel.is_a() == "IfcRelConnectsElements":
@@ -272,6 +274,7 @@ class ChangeExtrusionXAngle(bpy.types.Operator, tool.Ifc.Operator):
         wall_objs = []
         other_objs = []
         x_angle = radians(self.x_angle)
+        unit_scale = ifcopenshell.util.unit.calculate_unit_scale(tool.Ifc.get())
         for obj in context.selected_objects:
             element = tool.Ifc.get_entity(obj)
             if not element:
@@ -282,6 +285,10 @@ class ChangeExtrusionXAngle(bpy.types.Operator, tool.Ifc.Operator):
             extrusion = tool.Model.get_extrusion(representation)
             if not extrusion:
                 return
+            x, y, z = extrusion.ExtrudedDirection.DirectionRatios
+            existing_x_angle = Vector((0, 1)).angle_signed(Vector((y, z)))
+            perpendicular_depth = extrusion.Depth / (1 / cos(existing_x_angle))
+            extrusion.Depth = perpendicular_depth * (1 / cos(x_angle))
             extrusion.ExtrudedDirection.DirectionRatios = (0.0, sin(x_angle), cos(x_angle))
             if element.is_a("IfcWall"):
                 wall_objs.append(obj)
@@ -631,9 +638,7 @@ class DumbWallGenerator:
             "geometry.add_wall_representation",
             tool.Ifc.get(),
             context=self.body_context,
-            thickness=self.layers["thickness"]
+            thickness=self.layers["thickness"],
-            * 1
-            / cos(self.x_angle),  # 1/cos(self.x_angle) maintains the correct MaterialLayer thickness upon rotation
             offset=self.layers["offset"],
             length=self.length,
             height=self.height,
@@ -1143,9 +1148,7 @@ class DumbWallJoiner:
             height=height,
             x_angle=x_angle,
             offset=layers["offset"],
-            thickness=layers["thickness"]
+            thickness=layers["thickness"],
-            * 1
-            / cos(x_angle),  # 1/cos(x_angle) maintains the correct MaterialLayer thickness upon rotation
             clippings=self.clippings,
             booleans=tool.Model.get_manual_booleans(element),
         )
@@ -1206,12 +1209,12 @@ class DumbWallJoiner:
         while True:
             if item.is_a("IfcExtrudedAreaSolid"):
                 results["item"] = item
-                results["height"] = item.Depth * self.unit_scale
                 x, y, z = item.ExtrudedDirection.DirectionRatios
                 if not tool.Cad.is_x(x, 0) or not tool.Cad.is_x(y, 0) or not tool.Cad.is_x(z, 1):
                     results["direction"] = Vector(item.ExtrudedDirection.DirectionRatios)
                     results["x_angle"] = Vector((0, 1)).angle_signed(Vector((y, z)))
                     results["is_sloped"] = True
+                results["height"] = (item.Depth * self.unit_scale) / (1 / cos(results["x_angle"]))
                 break
             elif item.is_a("IfcBooleanClippingResult"):
                 item = item.FirstOperand

src/ifcopenshell-python/ifcopenshell/api/geometry/add_slab_representation.py

@@ -59,7 +59,7 @@ class Usecase:
             self.file.createIfcArbitraryClosedProfileDef("AREA", None, curve),
             None,
             extrusion_direction,
-            self.convert_si_to_unit(self.settings["depth"]),
+            self.convert_si_to_unit(self.settings["depth"]) * 1 / cos(self.settings["x_angle"]),
         )
         if self.settings["clippings"]:
             return self.apply_clippings(extrusion)

src/ifcopenshell-python/ifcopenshell/api/geometry/add_wall_representation.py

@@ -51,6 +51,7 @@ class Usecase:
     def create_item(self):
         length = self.convert_si_to_unit(self.settings["length"])
         thickness = self.convert_si_to_unit(self.settings["thickness"])
+        thickness *= 1 / cos(self.settings["x_angle"])
         points = (
             (0.0, 0.0),
             (0.0, thickness),
@@ -76,7 +77,7 @@ class Usecase:
                 self.file.createIfcDirection((1.0, 0.0, 0.0)),
             ),
             extrusion_direction,
-            self.convert_si_to_unit(self.settings["height"]),
+            self.convert_si_to_unit(self.settings["height"]) * (1 / cos(self.settings["x_angle"])),
         )
         if self.settings["booleans"]:
             extrusion = self.apply_booleans(extrusion)

src/ifcopenshell-python/ifcopenshell/api/geometry/disconnect_path.py

@@ -37,17 +37,17 @@ class Usecase:
             connections = [
                 r
                 for r in self.settings["element"].ConnectedTo
-                if r.RelatingConnectionType == self.settings["connection_type"]
+                if r.is_a("IfcRelConnectsPathElements") and r.RelatingConnectionType == self.settings["connection_type"]
             ] + [
                 r
                 for r in self.settings["element"].ConnectedFrom
-                if r.RelatedConnectionType == self.settings["connection_type"]
+                if r.is_a("IfcRelConnectsPathElements") and r.RelatedConnectionType == self.settings["connection_type"]
             ]
         else:
             connections = [
                 r
                 for r in self.settings["relating_element"].ConnectedTo
-                if r.RelatedElement == self.settings["related_element"]
+                if r.is_a("IfcRelConnectsPathElements") and r.RelatedElement == self.settings["related_element"]
             ]
 
         for connection in set(connections):