Make IfcDiff more forgiving when invalid geometry occurs. See #3755.

Moult · Moult · commit 4131e595bdb2 · 2024-04-16T10:37:15.000+02:00
diff --git a/src/ifcdiff/ifcdiff.py b/src/ifcdiff/ifcdiff.py
@@ -190,19 +190,20 @@ def summarise_shapes(self, ifc, elements):
             shape = iterator.get()
             element = ifc.by_id(shape.id)
             geometry = shape.geometry
-            shapes[element.GlobalId] = {
-                "total_verts": len(geometry.verts),
-                "sum_verts": sum(geometry.verts),
-                "min_vert": min(geometry.verts),
-                "max_vert": max(geometry.verts),
-                "matrix": tuple(shape.transformation.matrix.data),
-                "openings": sorted(
-                    [o.RelatedOpeningElement.GlobalId for o in getattr(element, "HasOpenings", []) or []]
-                ),
-                "projections": sorted(
-                    [o.RelatedFeatureElement.GlobalId for o in getattr(element, "HasProjections", []) or []]
-                ),
-            }
+            if geometry.verts:
+                shapes[element.GlobalId] = {
+                    "total_verts": len(geometry.verts),
+                    "sum_verts": sum(geometry.verts),
+                    "min_vert": min(geometry.verts),
+                    "max_vert": max(geometry.verts),
+                    "matrix": tuple(shape.transformation.matrix.data),
+                    "openings": sorted(
+                        [o.RelatedOpeningElement.GlobalId for o in getattr(element, "HasOpenings", []) or []]
+                    ),
+                    "projections": sorted(
+                        [o.RelatedFeatureElement.GlobalId for o in getattr(element, "HasProjections", []) or []]
+                    ),
+                }
             if not iterator.next():
                 break
         return shapes
