buildingSMART / buildingSMART/NextGen-IFC

Overlap between IfcAdvancedBrep, IfcFacetedBrep, IfcTessellatedFaceSet and its subtypes

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

Description

The following entities have some similarities:
IfcAdvancedBrep
IfcAdvancedBrepWithVoids
IfcFacetedBrep
IfcFacetedBrepWithVoids
IfcTessellatedFaceSet
IfcPolygonalFaceSet
IfcTriangulatedFaceSet
Therefore the question is if some of these entities can be merged. IfcAdvancedBrep (IfcAdvancedBrepWithVoids) are clearly semantically the most rich concepts. If these entities would be replaced by the others semantic information will be lost, the most close here would be IfcPolygonalFaceSet. Also the other way around removing IfcPolygonalFaceSet or IfcTriangulatedFaceSet would prevent storing some of the basic geometry representations and a clean way to define individual triangle / face color assignment.

IfcFacetedBrep allows advanced representations of how faces are build up, via curved polygons, trimmed polygons etc. as does its subtype entity supporting openings. Therefore this entity has extra value over tessellated representations. The other way around tessellated entities are able to represent knowledge in much more condense and easy manner to process.

The bottom line is that all these 7 entities have their own clear benefits and my opinion would be to keep them separate and not merge them.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the seven IFC entities named in the issue and their subtype definitions. No file, test, or entry point is identified; a contribution would first need a decided scope for whether they remain separate or are merged, plus criteria for validating the schema change.

Written by the indexing model from the issue text.

Assessment

Domain
computer-graphics
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.