-
Notifications
You must be signed in to change notification settings - Fork 34
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
Default value for IfcAxis2PlacementLinear.Axis is not defined #125
Comments
A solution has been proposed in buildingSMART/IFC4.3.x-development#732 (comment). To summarize the proposal:
Is there sufficient agreement to draft an Implementation Agreement for https://standards.buildingsmart.org/documents/Implementation/IFC_Implementation_Agreements/. I'm guessing implementation agreements are a very difficult thing to do since the last two listed are from 2014 and are still proposals in progress. If there is agreement on the path forward, I willing to help bring this proposal to a formal conclusion. |
Based on recent discussions in jwouellette/TPF-5_372-Unit_Test_Suite#15, perhaps the problem and solution should be restated: Problem See further discussion and illustrations in buildingSMART/IFC4.3.x-development#732 Solution(s)
Axis = The exact direction of the local Z Axis. If Axis is omitted, the direction is taken perpendicular to the tangent of the basis curve in the plane of the basis curve. Require schema changes?
Require documentation changes?
|
When trying to compute placements for comparison with https://github.com/buildingSMART/IFC4.x-IF/tree/main/IFC-files/Linear-placement-sleepers/ACCA#readme the following issue is noted.
Problem
The default value of IfcAxis2PlacementLinear.Axis is not defined. Using IfcAxis2Placement3D for inspiration, you could say that the default Axis is (0,0,1). However, IfcAxis2PlacementLinear.RefDirection, when omitted, is taken from the curve tangent at IfcAxis2PlacementLinear.Location so you could infer that Axis should be derived from the IfcPointByDistanceExpression.BasisCurve as the direction perpendicular to the curve in the "Distance Along - Elevation" plane.
See further discussion and illustrations in buildingSMART/IFC4.3.x-development#732
Additionally, the question as to whether both or neither IfcAxis2PlacementLinear.Axis and RefDirection are needed similar to IfcAxis2Placement3D requirements. See buildingSMART/IFC4.3.x-development#731.
Solution(s)
Require schema changes?
✓
noRequire documentation changes?
✓
yesThe text was updated successfully, but these errors were encountered: