IFC4.3.x-development icon indicating copy to clipboard operation
IFC4.3.x-development copied to clipboard

IfcRelCoversBldgElements unclear

Open CamChatHel opened this issue 3 years ago • 1 comments

Hello, It is confusing that in the description of IfcCovering you write: "..using the inverse relationship Covers pointing to the building element is then accessible via IfcRelCoversBldgElements.RelatingBuildingElement". (http://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/lexical/IfcCovering.htm) , but in the description of IfcRelCoversBldgElements you write: "This relationship is now deprecated and replaced by IfcRelAggregates.". It is the same for IfcCovering that are assigned to spaces.

Maybe correct in the first place at IfcCovering that only IfcRelAggregates is valid and IfcRelCoversBldgElements is deprecated .

CamChatHel avatar Jun 20 '22 12:06 CamChatHel

Yes, I agree, the docs should update to reflect the deprecated state of IfcRelCoversBldgElements. Aggregation is commonly used to represent layers in compound materials/elements, so it indeed makes sense for coverings as well.

I did notice though that in the Element Decomposition concept we do not list the usage of IfcWall - IfcCovering http://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/concepts/Object_Composition/Aggregation/Element_Decomposition/content.html That could also be an omission.

aothms avatar Jun 21 '22 11:06 aothms

For this topic there is also an issue when looking at the attributes of IfcCovering. Here we still have CoversElements with SET [0:1] OF IfcRelCoversBldgElements FOR RelatedCoverings. This reference can not exist if IfcRelCoversBldgElements is deprecated.

CamChatHel avatar Jan 19 '23 11:01 CamChatHel