buildingSMART / buildingSMART/IFC4.x-development

IfcRepresentationMap : MappingOrigin is unclear

Open
#815 3 comments 0 reactions 0 assignees View on GitHub
after-NWI iso-Ed
Dominant language
Python
Stars
234
Forks
123
Avg merge
15h 4m
Merged PRs (30d)
5

Description

An old issue been part since forever but

Description in documentation to anyone not native in english is a bit unclear.
"defines the position about which the mapped representation is mapped."

Either
a. Placement describes a transformation to place geometry to its placement and is used as is.
b. Placement is the geometry's placement and is combined with mapping target using inverse placement.
Last time we tested this both versions were used by importers.

1. A better description would be nice. Currently there is no sample included in documentation that uses mapping source which would define the correct use. Change one of the columns in [file](https://standards.buildingsmart.org/IFC/RELEASE/IFC4_3/HTML/annex_e/mapped-geometric-shape/mapped-shape-with-multiple-items.html)
2. Make this optional. This placement is effective irrelevant as the mapping target tarformation is always used (unless this class is used in something else). Now everyone writes unnecessary default placement always.
3. Deprecate this if there is no actual use

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.