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

RI007_Stationing-property #64

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

JTurbakiewicz
Copy link
Contributor

No description provided.

Copy link
Contributor

@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 wonder: do the test files fail only the rules proposed in this PR?

Copy link
Contributor

Choose a reason for hiding this comment

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

All scenarios should not have the same name.

@implementer-agreement
@RI
Feature: RI007 - Stationing property
The rule verifies, that station information can be exported in IFC using Pset_Stationing.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The rule verifies, that station information can be exported in IFC using Pset_Stationing.
The rule verifies that station information is present in IFC using Pset_Stationing.

Comment on lines +18 to +19
And Its attribute RelatingPropertyDefinition
And Its attribute Name
Copy link
Contributor

Choose a reason for hiding this comment

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

This would allow a IfcQuantitySet to pass as well. We need to reconsider this.

Comment on lines +22 to +39
Scenario: Agreement on alignment stationing

Given A file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3"
And An IfcReferent
And Its attribute IsDefinedBy
And Its attribute RelatingPropertyDefinition
And Its attribute HasProperties
Then The value of attribute Name must be Station

Scenario: Agreement on alignment stationing

Given A file with Schema Identifier "IFC4X3_TC1" or "IFC4X3_ADD1" or "IFC4X3"
And An IfcReferent
And Its attribute IsDefinedBy
And Its attribute RelatingPropertyDefinition
And Its attribute HasProperties
And Its attribute NominalValue
Then The value must exist
Copy link
Contributor

Choose a reason for hiding this comment

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

I do not agree with splitting this up in independent scenarios.

For example, any attached property with NominalValue non empty would pass the bottom test. The requirement was that the property named Station would have a non empty value.

Comment on lines +121 to +126
@dataclass
class MissingValueError:
related: ifcopenshell.entity_instance

def __str__(self):
return f"On instance {misc.fmt(self.related)} the value doesn't exist"
Copy link
Contributor

Choose a reason for hiding this comment

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

The value of what?

@evandroAlfieri
Copy link
Collaborator

This rule is i) partially test-specific (cannot be part of the Validation Service); ii) partially touching on topics currently under discussion by the Implementers Forum. For these reasons, my suggestion is to put the development of this rule on hold. I can inform when the IF reaches a conclusion on this topic.

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.

None yet

3 participants