buildingSMART / buildingSMART/NextGen-IFC

IfcLoop and IfcVertex are not ABSTRACT?

Open
#65 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
TeX
Stars
64
Forks
4
PR merge metrics
No merged PRs in 30d

Description

Description of the proposal:
Make `IfcLoop` and `IfcVertex` entities abstract.

They do not possess any attributes, therefore I don't see any reason to see them in a physical file. E.g.:

> #90 = IFCLOOP();

Therefore: make them `ABSTRACT`.

What do we win: clearer schema.

What do we loose: nothing.

Schema impact: backward compatible, I would even suggest updating the old schemas as well.

Instance model impact: No chance to submit a file with entities like `IfcLoop` and/or `IfcVertex`.

Backwards compatible: Yes.

Automatic migration possible: ?

Additional implications: ?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.