-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ Fail on GRF001 ] #269
Comments
Hi @michelangelo-acca , sorry for the late reply (Implementers Assembly). We'll look into it and come back asap |
Hi @michelangelo-acca . There are two geometric representation contexts in the model:
Only the first one has a IfcMapConversion assigned:
So the coordinate operations on these are not identical. |
But does this means that we need to have multiple MapConversion(s) in the same file, even if they are the same? One for each Geometric Context? |
I'd say so. Similar to how these context have the same-valued WorldCoordinateSystem and north vector.
Alternatively you could make the 2nd context a subcontext of the 1st one. But I personally don't know enough about the agreements to know whether that's a good or bad recommendation. Because without the coordinate operations explicitly agreeing you're essentially saying that their representations are effectively on a different place on the map. (Which to me shows we shouldn't have equated context with positioning, but there is a long history going back to that). |
This will indeed cause the GRF001 rule to pass. Furthermore, in the schema, the attribute for coordinate operations in the context is allowed to be empty. Currently, for GRF001, this will be permitted only if (1) there is only one context, or (2) the attribute is empty for all contexts within the file. Is this the desired behavior? |
@michelangelo-acca is the explanations above clear enough? Can we close this issue or do you see the need for follow up actions? |
Can someone explain why this file fails on GRF001?
grf001_4x3_geo.zip
The text was updated successfully, but these errors were encountered: