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

Spatial Composition restraint #35

Open
jwouellette opened this issue Mar 13, 2020 · 1 comment
Open

Spatial Composition restraint #35

jwouellette opened this issue Mar 13, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@jwouellette
Copy link

Please refer to:
https://forums.buildingsmart.org/t/spatial-composition-restraints/1129

Description of the proposal:
Remove legacy composition type

Describe how it contributes to the objectives set in https://github.com/buildingSMART/NextGen-IFC/wiki/Towards-a-technology-independent-IFC:

What do we win:

What do we loose

Schema impact:

Instance model impact:

Backwards compatible:

Automatic migration possible:

Additional implications:

Note that not all points need to be satisfied!
Backwards compatibility and file size are not concerns.

@jwouellette jwouellette added the enhancement New feature or request label Mar 13, 2020
@Moult
Copy link

Moult commented Mar 23, 2020

My understanding of this issue is that after the composition type is removed, it implies that you can nest more than three levels deep of a single spatial element, e.g: IfcSite > IfcBuilding > IfcBuilding > IfcBuilding > IfcBuilding > IfcBuilding > IfcBuildingStorey becomes legal.

+1. I had a shot at filling out the fields:

Wins: more flexibility in spatial tree. No need for vendors to apply non-automated rule checking (as there are no WHERE rules) on three level deep composition type. Also less confusion about whether a building complex is an IfcSite or a IfcBuilding.COMPLEX.

Losses: Possibly some BIM authors might go crazy and create silly unnecessarily nested trees. Deep trees are baaaaaad. It also creates confusion about the currently non-abstract IfcFacility and IfcFacilityPart proposals. Also, if I wanted to know how many building complexes I have, the query becomes slightly harder to write.

Backwards compatible: No

Automatic migration: Yes

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

No branches or pull requests

2 participants