Skip to content
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

Create IFCTunnelDep2.2.ifc #120

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

larswik
Copy link
Collaborator

@larswik larswik commented Apr 17, 2023

File

Manually edited the example file from sprint 2.1 with the following updates:

  • Aggregate IfcAlignment directly to IfcProject
  • IfcSite positioned relative to an IfcReferent instead of aggregated to IfcProject (aggregations under IfcSite same as before)
  • New (additional) 2d geometric representation context for IfcAlignment 2d representation (IfcCompositeCurve)
  • Changed so IfcAlignment has IfcSegmentedReferenceCurve as 3d representation (instead of IfcGradientCurve)
  • Added IfcCompositeCurve as 2d representation for IfcAlignment

Covered Concept Templates - Usages Included

All concept templates up to and including sprint 2.2

Miscellaneous

Forgot to add the extra context to the project.
files/project_team/IFCTunnelDep2.2.ifc Outdated Show resolved Hide resolved
larswik added 5 commits April 17, 2023 17:12
Add GUID for IfcRelPositions
IfcSite also aggregated to IfcProject
Separate IfcRelAggregates for IfcProject-IfcSite and IfcProject-IfcAlignment
IfcReferent.PredefinedType=.STATION. everywhere
IfcLinearPlacement.PlacementRelTo = IfcAlignment.ObjectPlacement
files/project_team/IFCTunnelDep2.2.ifc Outdated Show resolved Hide resolved
files/project_team/IFCTunnelDep2.2.ifc Show resolved Hide resolved
#93= IFCCARTESIANPOINT((0.0,0.0,0.0));
#94= IFCPRODUCTDEFINITIONSHAPE($,$,(#95));
#95= IFCSHAPEREPRESENTATION(#41,'Axis','Segment',(#96));
#96= IFCCURVESEGMENT(.CONTSAMEGRADIENTSAMECURVATURE.,#97,IFCPARAMETERVALUE(-92.0),IFCNONNEGATIVELENGTHMEASURE(92.0),#100);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SergejMuhic I can't remember: what was the conclusion on mixing IfcParameterValue with IfcNonNegativeLengthMeasure?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SergejMuhic I can't remember: what was the conclusion on mixing IfcParameterValue with IfcNonNegativeLengthMeasure?

Don't know? @SergejMuhic ?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implementer agreement, sort of.

@larswik
Copy link
Collaborator Author

larswik commented May 30, 2023

Added a text to the Description attribute of IfcAlignment, IfcAlignmentHorizontal and IfcAlignmentVertical regarding the incorrect use of IfcNonNegativeLengthMeasure and IfcParameterValue in the geometric representations. Reports error for sprint 2.3 of course, but since this is 2.2, can we approve this file now?
Ping @SergejMuhic and @pjanck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants