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

Required entity type in applicability (and conciseness) #10

Closed
CBenghi opened this issue Apr 5, 2021 · 5 comments
Closed

Required entity type in applicability (and conciseness) #10

CBenghi opened this issue Apr 5, 2021 · 5 comments

Comments

@CBenghi
Copy link
Contributor

CBenghi commented Apr 5, 2021

Hi,

I see that entity are required in the definition of applicability, while the others (classification, property and material) all have minOccurs="0"

<xs:complexType name="applicabilityType">
<xs:choice maxOccurs="unbounded">
<xs:element name="entity" type="ids:entityType"/>

I know of IDSs that are classification based, rather than entity based. In those scenarios the current schema would force us to repeat the classification requirement for all possible types, resulting in very large IDS files.

I would suggest that entity is also marked-up with minOccurs="0".

Can I ask what is the benefit of forcing the declaration of entity?

@MatthiasWeise
Copy link
Contributor

Additional question related to this topic:
If I remember well entityType.name will ignore IFC inheritance, right? So, if defining a requirement for IfcBuildingElement no subtype like for instance IfcBeam will checked.

@NickNisbet
Copy link

NickNisbet commented Sep 23, 2021 via email

@berlotti
Copy link
Member

@MatthiasWeise correct

@berlotti berlotti closed this as completed Jan 4, 2022
@CBenghi
Copy link
Contributor Author

CBenghi commented Jan 13, 2022

Hi @berlotti,

I'm not sure the main question has been answered,
It's not clear to me what is the benefit of forcing the declaration of entity.

Thanks,
Claudio

@berlotti
Copy link
Member

This was changed in a later version. Hence closing this issue.

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

No branches or pull requests

4 participants