buildingSMART / buildingSMART/IFC4.x-development

Can we make Layer set use "Name" as an attribute not "LayerSetName"

Open
#1,066 5 comments 0 reactions 0 assignees View on GitHub
after-NWI allocated-core iso-compatibility
Dominant language
Python
Stars
234
Forks
123
Avg merge
15h 4m
Merged PRs (30d)
5

Description

Reported https://forums.buildingsmart.org/t/can-we-make-layer-set-use-name-as-an-attribute-not-layersetname/3528

This is one of those things in the codebase I put a comment going "what's going on IFC".

Quoted:

Everything else uses "Name" except for IfcMaterialLayerSet and it's inconsistent :)

Contributor guide

Open the contributing guide

Research direction

Start with the linked buildingSMART forum discussion and the IFC4.x specification entry for IfcMaterialLayerSet. Determine whether changing LayerSetName to Name is compatible with the surrounding IFC naming rules and identify any affected references. Done means the naming decision and its specification impact are resolved consistently.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.