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

[Test Case]: Track Structures Turnout Panel #41

Open
wants to merge 32 commits into
base: develop
Choose a base branch
from

Conversation

czapplitec
Copy link
Collaborator

@czapplitec czapplitec commented Feb 17, 2022

Summary

Outline of the content and tasks addressed by this pull request, don't forget to link the relevant issue(s)

Purpose

Please mark the relevant tasks that this pull request creates or updates: #28

Exchanges

  • Creation of new exchange
  • Documentation of exchange

Test Cases

  • Creation of new test case
  • Test case summary
  • Itemised roots & test case imports
  • Usages, constraints & logic
  • Validation criteria & Results
  • Dataset Content & Documentation

General

  • CI updates
  • Repository documentation
  • Bug fixes

@czapplitec czapplitec self-assigned this Feb 17, 2022
@czapplitec czapplitec changed the title Test instruction track structures turnout panel [Test Case]: Track Structures Turnout Panel Feb 22, 2022
@czapplitec czapplitec added the test case issues & pull requests relating to test cases label Feb 22, 2022
Copy link
Member

@pjanck pjanck left a comment

Choose a reason for hiding this comment

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

Good start!

E2a-TRST/TSTP/Dataset/README.md Outdated Show resolved Hide resolved
E2a-TRST/TSTP/Dataset/README.md Outdated Show resolved Hide resolved
A track turnout panel is modelled as an instance of IfcElementAssembly with Predefined Type TURNOUTPANEL.
It is decomposed by elements including IfcRail.RAIL, IfcTrackElement.SLEEPER, IfcRail.CHECKRAIL, IfcTrackElement.FROG, IfcMechnicalFastner.RAILFASTNING, IfcMechnicalFastner.RAILJOINT.
All the placement of these elements should be based on the placement of the turnout panel, and the turnout panel as an assembly is placed based on the alignment.
It is required to define the explicit and start and end stationing of the turnout panel.
Copy link
Member

Choose a reason for hiding this comment

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

  • typo: explicit and start?
  • perhaps a mention that these should be IfcReferents would be beneficial

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated, thanks

E2a-TRST/TSTP/README.md Outdated Show resolved Hide resolved

- IfcElementAssembly
- *There exists 1 IfcElementAssembly with PredefinedType set to TRACKTURNOUTPANEL*
- *Each IfcElementAssembly/TRACKTURNOUTPANEL shall have a linear placement relative to the alignment*
Copy link
Member

Choose a reason for hiding this comment

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

The sketch Turnout_Placement.png disagrees with this statement.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This contraint is removed

- *There exists 1 IfcElementAssembly with PredefinedType set to TRACKTURNOUTPANEL*
- *Each IfcElementAssembly/TRACKTURNOUTPANEL shall have a linear placement relative to the alignment*
- *Each IfcElementAssembly/TRACKTURNOUTPANEL is spatially contained in an IfcRailwayPart/TRACKSUPERSTRUCTURE*
- *Each IfcElementAssembly/TRACKTURNOUTPANEL shall have a product span placement*
Copy link
Member

Choose a reason for hiding this comment

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

What is product span placement? Can you point me to the documentation?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I may have missed somewhere in the readme: Regarding E2a-TRST/TSTP/Dataset/Sleeper: is it expected to have the 3D model according to the real situation or the simplified situation?

Stated in Dataset/README.md line 30:

  • The geometry of IfcTrackElement.SLEEPER can be simplified and placed according to the drawing in Fig. 6.

Copy link
Member

@pjanck pjanck left a comment

Choose a reason for hiding this comment

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

I may have missed somewhere in the readme: Regarding E2a-TRST/TSTP/Dataset/Sleeper: is it expected to have the 3D model according to the real situation or the simplified situation?

@czapplitec czapplitec marked this pull request as ready for review March 21, 2022 21:58
@czapplitec czapplitec requested a review from larswik March 22, 2022 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test case issues & pull requests relating to test cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants