buildingSMART / IDS

Computer interpretable (XML) standard to define Information Delivery Specifications for BIM (mainly used for IFC)
Other
167 stars 52 forks source link

IDS Audit Tool: Wrong error message for Pset_FootCommon.LoadBearing and Pset_FurnitureTypeCommon.Reference #257

Open MatthiasWeise opened 2 months ago

MatthiasWeise commented 2 months ago

Both properties exist in IFC4, but are wrongly reported as an error. It is most likely due to an update in IFC4.0.2.0. My understanding of IFC4 in IDS is that it is based on latest official version, IFC4.0.2.1.

Error 103: Invalid property baseName LoadBearing (accepted values are Reference,Status) in the context of Ifc4 on 'simpleValue' element at line

CBenghi commented 2 months ago

Hello @andyward, any idea if XBIM has ways to control the minor versioning of Xbim.Essentials to generate the right metadata for the IDS Audit tool?

CBenghi commented 2 months ago

Marked for discussion and documentation in order to discuss the expected schema versions, and document it.

andyward commented 2 months ago

Hello @andyward, any idea if XBIM has ways to control the minor versioning of Xbim.Essentials to generate the right metadata for the IDS Audit tool?

Hi @CBenghi

My understanding is that IDS only supports 'Official' schemas, not Retired/Withdrawn. So I don't see how we'd need to validate for specific IFC schema minor versions, since currently:

IFC2x3 = IFC2x3 TC2 = 2.3.0.1 IFC4 = IFC4 ADD2 TC1 = 4.0.2.1 IFC4x3 = IFC 4.3 ADD2 = 4.3.2.0

I'd guess the issue is that xbim.Properties (used in ids-lib) must not have been updated based on the IFC4 pset xml definitions in the latest addendums. We updated it for 4x3 ADD2 recently but it's quite possible it never gott updated for IFC4 ADD2+

CBenghi commented 2 months ago

Hi Andy, You're quite right, I read this quickly and mixed up in my mind schemas and properties. I'll review the details tomorrow. Thanks