Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-25 01:50:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

ifc5d.qto.IfcOpenshell - skip None values

Browse Source 
Just for consistency, in all other cases besides get_segment_length we were only preserving floats anyway
This commit is contained in:
Andrej730
2025-03-14 12:39:59 +05:00
parent 4c53c2ab1f
commit 1d447e6e23
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifc5d/ifc5d/qto.py
+10 -2
View File
@@ -304,7 +304,9 @@ class IfcOpenShell(QtoCalculator):
results[element].setdefault(name, {}) results[element].setdefault(name, {})
for quantity, formula in quantities.items(): for quantity, formula in quantities.items():
if formula == "get_segment_length": if formula == "get_segment_length":
results[element][name][quantity] = cls.get_segment_length(element) value = cls.get_segment_length(element)
if value is None:
continue
elif formula == "get_weight": elif formula == "get_weight":
value = cls.get_weight(element, geometry) value = cls.get_weight(element, geometry)
if value is None: if value is None:
@@ -334,7 +336,13 @@ class IfcOpenShell(QtoCalculator):
return iterators return iterators
@classmethod @classmethod
def get_segment_length(cls, element: ifcopenshell.entity_instance) -> float: def get_segment_length(cls, element: ifcopenshell.entity_instance) -> Union[float, None]:
"""Get segment length.
:param element: IFC element entity.
:return: ``float`` segment length in project units
or ``None`` if element doesn't have a representation or it's not supported.
"""
rep = ifcopenshell.util.representation.get_representation(element, "Model", "Body", "MODEL_VIEW") rep = ifcopenshell.util.representation.get_representation(element, "Model", "Body", "MODEL_VIEW")
if rep and len(rep.Items or []) == 1 and rep.Items[0].is_a("IfcExtrudedAreaSolid"): if rep and len(rep.Items or []) == 1 and rep.Items[0].is_a("IfcExtrudedAreaSolid"):
item = rep.Items[0] item = rep.Items[0]