Results 71 comments of ArturTomczak

Someone reported this, and it looked like a mistake to me. Glad we have vigilant reviewers! If that's supposed to solve "Wall that hosts a Window" (Wall that's partOf Window...

Valid point @NickNisbet. I wonder why the relation name is optional: https://github.com/buildingSMART/IDS/blob/dd7b0689a7ddd6e62133d0eaba2f662f743eee9c/Schema/ids.xsd#L55C53-L55C71

Good point. Let's ignore SetPointValue then and focus on Bounds only. I would not assume that Lower/UpperBound has to always match IDS min/maxInclusive. Instead, the bounds in IFC should be...

Before touching the tests, I want to know from @Moult why [fail-any_matching_value_in_a_bounded_property_will_pass_4_4](https://github.com/buildingSMART/IDS/blob/development/Documentation/ImplementersDocumentation/TestCases/property/fail-any_matching_value_in_a_bounded_property_will_pass_4_4.ifc) actually works (fails) in ifctester.

> ... when specifying simpleValues, the only three Real values that will pass are: 5, 1, 3 (I'm simplifying for the fact the actual tests involve unit conversions). Any other...

**Summary:** need to add a test with multiple patterns in one restriction. They should be evaluated with OR, not AND (unlike other restrictions).

> It actually passes since the least significant digit of -1000001.0000010001 is beyond the precision of a IEEE74 double so is actually equivalent with the passing value. Good point. The...

yes, however, I was talking about the table in docs, where we should probably mention this double precision aspect. Because the tolerance.md didn't exist on the development branch, I edited...

I think this is ready for closing, @andyward?

afaik, this is allowed in IDS 1.0: ``` ``` as well as other data types: ~~boolean~~, integer, date, dateTime, duration and time.