-
Notifications
You must be signed in to change notification settings - Fork 6
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
Mixing length and param value? #8
Comments
In principle we agreed that mixing of Length Measures and Parameter values within one IfcCurveSegment instance is not preferable. However with the latest schema we use IfcLengthMeasure (instead of IfcNonNegativeLengthMeasure) and we agree not to use IfcParameterValue for now. |
Right.. so what does that mean for these two samples? We should remove them. We only retain the stationing example? |
I think we should regenerate the geometry based on the latest schema, will be able to do this in Monday |
If you could that would be wonderful! |
@aothms By chance I've been looking at these examples to understand how to model alignments. https://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/concepts/Product_Shape/Product_Geometric_Representation/Alignment_Geometry/Alignment_Geometry_-_Segments/content.html states that
I don't see the zero length segments (business or geometric) in these examples. Are they there? Also, https://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/concepts/Object_Composition/Aggregation/Alignment_Aggregation_To_Project/content.html states that
This is not the case in two of the examples. If there is an omission can it be addressed? |
@RickBrice you're fully correct. We'll have to see how it goes. There is a bit of a conflict between really wanting to include some examples that demonstrate the new concepts in 4.3, but at the same time, because of the time by which these kind of non-schema usage restrictions have remained somewhat fluid, we're really struggling to find something appropriate to include. @peterrdf any updates on this by any chance? If it's too much trouble to regenerate this one it's also fine if you create something else that's a couple kilobytes in size (but preferably a little more diverse then just something from Alignment-atomic-testset). Here's what I propose. I've added two bespoke validation rules for these two cases brought up by @RickBrice. By the time of our deadline of the 7th (yes, the day after tomorrow) we'll have to remove what's not conformant. |
@aothms @peterrdf It also looks like the 4.1.5.1, https://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/concepts/Object_Connectivity/Alignment_Spatial_Reference/content.html, tells us that IfcAlignment is associated with a spatial structure with IfcRelReferencedInSpatialStructure. I assume this means IfcSite, IfcBridge, IfcRoad, or IfcRailway, but none of these refer to this concept template. Actually I can't find anything that uses this concept template. Is it a leftover from 4x3 development? |
@evandroAlfieri is more up to date on this. We have a long way to go in terms which of the concepts describe (a) possibilities (b) recommendations (c) requirements. This template is more on the possibilities side. You can exchange an independent alignment (aggregated into the project) but if you also have road or rail in your model you can express their relationship with an alignment using IfcRelReferencedInSpatialStructure. Note that IfcRelReferencedInSpatialStructure is not a tree-like relationship such as decomposition or containment in IFC. If you take a look at IfcObjectDefinition which defines the inverse called |
Nothing to add to @aothms summary. For further questions on this matter, I'd suggest the community Forum - or get in touch via email. Let's try to stick to the issue title for this discussion. Thanks |
Again @peterrdf came in and fixed the day. Models should agree now with all written agreements. Have another look when you get the time @RickBrice |
@aothms Thanks - see the revisions and they seem to align with the specifications. I'll continue to review these examples and report any findings. |
@evandroAlfieri @peterrdf I don't recall all details from the conversations. Sorry for the noise. Were there any objects against mixing Length and Param values in the curve segments parameters?
IFC4.3.x-sample-models/models/alignment-geometries-and-linear-positioning/fixed-reference-swept-area-solid/fixed-reference-swept-area-solid.ifc
Line 87 in f12921e
The text was updated successfully, but these errors were encountered: