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

IFC-Rule for PredefinedType in partof-Test Cases #111

Closed
hesrah opened this issue Oct 28, 2022 · 2 comments
Closed

IFC-Rule for PredefinedType in partof-Test Cases #111

hesrah opened this issue Oct 28, 2022 · 2 comments
Assignees
Labels
Milestone

Comments

@hesrah
Copy link

hesrah commented Oct 28, 2022

In the IFC sample data of

the value 'USERDEFINED' is missing?

E. g. #2=IFCINVENTORY('0eA6m4fELI9QBIhP3wiLAp',$,$,$,'BUNNY',$,$,$,$,$,$); should be #2=IFCINVENTORY('0eA6m4fELI9QBIhP3wiLAp',$,$,$,'BUNNY','USERDEFINED',$,$,$,$,$); ?

It's the IFC-Rule if ObjectType, ElementType or ProcessType has a user value PREDEFINEDTYPE has to be 'USERDEFINED'?

@MatthiasWeise
Copy link
Contributor

Agree to this comment.
Thanks to @hesrah I just noted that ObjectType and PredefinedType are used interchangeable in the entity facet. However, my interpretation is also that in case of checking ObjectType I would expect that PredefinedType = USERDEFINED.

Anyhow, I was expecting a combination of Entity and Attribut facet for user defined types (more aligned with the IFC schema), but that would not work with the partOf facet.

@CBenghi CBenghi added the tests label May 31, 2023
@CBenghi CBenghi added this to the 1.0 milestone May 31, 2023
@CBenghi CBenghi self-assigned this Apr 19, 2024
CBenghi added a commit to CBenghi/IDS that referenced this issue May 3, 2024
CBenghi added a commit that referenced this issue May 3, 2024
@CBenghi CBenghi closed this as completed May 3, 2024
@lfcastel
Copy link

lfcastel commented Jun 10, 2024

Is it possible this change has not been adapted in the documentation?

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

No branches or pull requests

4 participants