Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-29 11:46:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix axis/ref_direction swap in connection geometry

Browse Source 
The a2p placement matrix stores col[0]=X (edge direction) and
col[2]=Z (face normal), but assign_connection_geometry expects
axis=Z (normal) and ref_direction=X (edge).

Generated with the assistance of an AI coding tool.
This commit is contained in:
CyrilWaechter
2026-07-30 20:55:55 +02:00
parent ac23a67e72
commit d88b201d5f
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcopenshell-python/ifcopenshell/util/boundary.py
+2 -2
View File
@@ -352,8 +352,8 @@ def _assign_connection_geometry(
) -> None: ) -> None:
"""Assign connection geometry to a boundary using the existing API.""" """Assign connection geometry to a boundary using the existing API."""
location = face_matrix[:3, 3] location = face_matrix[:3, 3]
axis = face_matrix[:3, 0] axis = face_matrix[:3, 2]
ref_direction = face_matrix[:3, 2] ref_direction = face_matrix[:3, 0]
outer_boundary = [list(coord) for coord in polygon.exterior.coords[:-1]] outer_boundary = [list(coord) for coord in polygon.exterior.coords[:-1]]
inner_boundaries = [list(interior.coords[:-1]) for interior in polygon.interiors] inner_boundaries = [list(interior.coords[:-1]) for interior in polygon.interiors]