-
Notifications
You must be signed in to change notification settings - Fork 9
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]: Alignment Linear Positioning & Stationing #23
base: develop
Are you sure you want to change the base?
[Test Case]: Alignment Linear Positioning & Stationing #23
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start!
- horizontal | ||
- vertical | ||
|
||
and the stationing and relative positioning concepts where physical elements can be placed relative to positioning elements. In order to accomplish a semantic description of a station, property sets will be used on referents. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You didn't list Property sets (neither entities nor concepts) in the itemized roots section.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where do you see a placeholder for that? I did not see it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant IfcPropertySet
and IfcSingleProperty
(or similar) in the Itemized Roots section of the document.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really sure honestly. Were these supposed to be added? 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It says above:
property sets will be used on referents
That is the reason for my question. I'm not sure if these are supposed to be added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In "my" TIs, I listed e.g. IfcPropertySet, IfcPropertySingleValue under "Itemised roots/IFC Entities" and e.g. Property Sets For Objects under "Itemised roots/Concept templates".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far so good.
Co-authored-by: Stefan Jaud <[email protected]>
@@ -0,0 +1,26 @@ | |||
## Variations | |||
The Following occurence variations need to be checked and certified in relation to the targetted entities and concept templates: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Following occurence variations need to be checked and certified in relation to the targetted entities and concept templates: | |
The Following occurence variations need to be checked and certified in relation to the targeted entities and concept templates: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
- Entity_01 - IfcReferent/REFERENCEMARKER | ||
- Entity_02 - IfcReferent/STATION | ||
- Entity_03 - IfcReferent/POSITION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is IfcReferent/POSITION still in?
This test case utilises the attached dataset documented by the following drawings and data schedule. | ||
|
||
*This is a later step tha involved the detailed documentation of the certification dataset (model)* | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this will need to be added.
|
||
## Drawings (Visualisations) | ||
The following Drawings and visualisations describe the test case dataset to be modelled and certified. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Follows content in "Model dataset"
|
||
| Filename | Description | | ||
|-----------------------------------|-------------------------------------------| | ||
| *filename* | *short description* | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Follows content in "Model dataset"
- *IfcRelPositions* | ||
- *IfcSite* | ||
- *IfcShapeRepresentation* | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IfcPropertySet, IfcpropertySingleValue here?
- *Alignment Geometry Gradient* | ||
- *Project Global Positioning* | ||
- *Spatial Containment* | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Property Sets for Objects here?
</details> | ||
|
||
## Usages, Constraints & Logic | ||
The following itemised restrictions and constraints shall be placed on IFC Entities & Concept Templates: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What to write here and what to write under "Validation Criteria" needs to be clarified => Discussion!
Considering the aim of this test, other **optional** results, not subject to the bSI certification process, yet usefull to illustrate test results are: | ||
- Screen-shot of ... | ||
- CSV export of ... | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Incomplete :-)
--- | ||
|
||
## Validation criteria | ||
:zap: For this test case to be considered passed **all capabilities** listed in this section shall be verified, with no exception. :zap: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See previous comment regarding this section vs "Usages..."
Summary
Outline of the content and tasks addressed by this pull request, dont forget to link the relevent issue(s)
Purpose
Please mark the relevent tasks that this pull request creates or updates:
Resolves #7
Test Cases