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

Fix bug where appended assets sometimes duplicated geometry contexts

Browse Source
This commit is contained in:
Dion Moult
2022-09-03 22:16:35 +10:00
parent 56393ef358
commit af4cb22756
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcopenshell-python/ifcopenshell/api/project/append_asset.py
+6 -6
View File
@@ -52,14 +52,14 @@ class Usecase:
if [e for e in self.file.by_type("IfcMaterial") if e.Name == self.settings["element"].Name]: if [e for e in self.file.by_type("IfcMaterial") if e.Name == self.settings["element"].Name]:
return return
self.whitelisted_inverse_attributes = {"IfcMaterial": ["HasProperties", "HasRepresentation"]} self.whitelisted_inverse_attributes = {"IfcMaterial": ["HasProperties", "HasRepresentation"]}
self.existing_contexts = self.file.by_type("IfcGeometricRepresentationContext")
element = self.add_element(self.settings["element"]) element = self.add_element(self.settings["element"])
if not element.HasRepresentation: if not element.HasRepresentation:
return element return element
self.existing_contexts = self.file.by_type("IfcGeometricRepresentationContext")
added_contexts = [ added_contexts = [
e e
for e in self.file.traverse(element.HasRepresentation[0]) for e in self.file.traverse(element.HasRepresentation[0])
if e.is_a() == "IfcGeometricRepresentationSubContext" if e.is_a("IfcGeometricRepresentationContext")
] ]
for added_context in added_contexts: for added_context in added_contexts:
equivalent_existing_context = self.get_equivalent_existing_context(added_context) equivalent_existing_context = self.get_equivalent_existing_context(added_context)
@@ -93,9 +93,9 @@ class Usecase:
"IfcMaterialDefinition": ["HasExternalReferences", "HasProperties"], "IfcMaterialDefinition": ["HasExternalReferences", "HasProperties"],
"IfcRepresentationItem": ["StyledByItem"], "IfcRepresentationItem": ["StyledByItem"],
} }
element = self.add_element(self.settings["element"])
self.existing_contexts = self.file.by_type("IfcGeometricRepresentationContext") self.existing_contexts = self.file.by_type("IfcGeometricRepresentationContext")
added_contexts = [e for e in self.file.traverse(element) if e.is_a() == "IfcGeometricRepresentationContext"] element = self.add_element(self.settings["element"])
added_contexts = [e for e in self.file.traverse(element) if e.is_a("IfcGeometricRepresentationContext")]
for added_context in added_contexts: for added_context in added_contexts:
equivalent_existing_context = self.get_equivalent_existing_context(added_context) equivalent_existing_context = self.get_equivalent_existing_context(added_context)
if not equivalent_existing_context: if not equivalent_existing_context:
@@ -117,9 +117,9 @@ class Usecase:
"IfcMaterialDefinition": ["HasExternalReferences", "HasProperties"], "IfcMaterialDefinition": ["HasExternalReferences", "HasProperties"],
"IfcRepresentationItem": ["StyledByItem"], "IfcRepresentationItem": ["StyledByItem"],
} }
element = self.add_element(self.settings["element"])
self.existing_contexts = self.file.by_type("IfcGeometricRepresentationContext") self.existing_contexts = self.file.by_type("IfcGeometricRepresentationContext")
added_contexts = [e for e in self.file.traverse(element) if e.is_a() == "IfcGeometricRepresentationContext"] element = self.add_element(self.settings["element"])
added_contexts = [e for e in self.file.traverse(element) if e.is_a("IfcGeometricRepresentationContext")]
for added_context in added_contexts: for added_context in added_contexts:
equivalent_existing_context = self.get_equivalent_existing_context(added_context) equivalent_existing_context = self.get_equivalent_existing_context(added_context)
if not equivalent_existing_context: if not equivalent_existing_context: