search

buildingSMART / NextGen-IFC

62 stars 4 forks source link

retire RepresentationType #57

Open I-Sokolov opened 4 years ago

I-Sokolov commented 4 years ago

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

berlotti commented 4 years ago

Great catch! Seems redundant information indeed.

TLiebich commented 4 years ago

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).

I-Sokolov commented 4 years ago

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:)