Andy Ward

Hopefully not 'inevitable' given how many folks use xbim! What version of Essentials are you using? Can you provide a reproduction - or 'csproj' file?

Was this not by design? See https://github.com/buildingSMART/IDS/issues/278 - I feel like it should maybe have an underscore separator or similar so a) it looks less like an error b) will...

>Can't comment on the tradeoffs though wrt codegen as mentioned by @andyward Regarding whitespace in an enum: It's more than code-gen - it just creates an extra bump in the...

Worth noting **SetPointValue** was only added in IFC4, and my reading is it's intended for 'operational values' rather than design-time measures. As well as these examples on [Spatial Pset_SpaceThermalLoad](https://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/lexical/Pset_SpaceThermalLoad.htm) I've...

@atomczak Agreed we should ignore SetPointValue. > the bounds in IFC should be within the IDS min/max This makes sense. In the case of open bounded IDS requirements, isn't it...

My reading on these 'bounded property' tests is that they are designed to test **equality** against any of the three attributes of a IfcPropertyBoundedValue (UpperBoundValue, LowerBoundValue, SetPointValue). Importantly, the attributes...

Yes OK with the proposed updates to the docs. Feels like this may be a 1.1 change vs retrospective 1.0? I guess it comes back to my point above about...

Presumably as a workaround you can use the built in OR operator in a regex. e.g. using | to separate the patterns ```xml ``` But obviously it make it pretty...

> I don't see the need for the workaround you are proposing. Am I missing something? I was just assuming this might need additional support/testing in the relevant implementations. e.g....

> Are you suggesting that we add an implementer agreement to constrain this? I'm not sure how best - but it feels like an edge case that may cause discrepancies...