NextGen-IFC icon indicating copy to clipboard operation
NextGen-IFC copied to clipboard

retire RepresentationType

Open I-Sokolov opened this issue 4 years ago • 3 comments

Description of the proposal: remove IfcProduct.RepresentationType attribute What do we win: simplify and consistent schema, remove redundant data. All the information may be obtained from classes of IfcRepresentationItems

I-Sokolov avatar Jul 06 '20 10:07 I-Sokolov

Great catch! Seems redundant information indeed.

berlotti avatar Jul 06 '20 14:07 berlotti

Hi @I-Sokolov - just to confirm - you mean IfcRepresentation.RepresentationType, not IfcProduct.RepresentationType? https://standards.buildingsmart.org/IFC/RELEASE/IFC4/ADD2_TC1/HTML/link/ifcrepresentation.htm

we should discuss, since I know that some software tries to rely on it, although it is difficult to trust (as you need to check the SET of IfcRepresentation.Items anyhow).

TLiebich avatar Jul 06 '20 15:07 TLiebich

Yes IfcRepresentation.RepresentationType (there is no IfcProduct.RepresentationType attribute) I know products relay on this... I saw crazy code in some application that tries to marriage RepresentationType and content, esecially when they conflict:)

I-Sokolov avatar Jul 08 '20 18:07 I-Sokolov