buildingSMART / buildingSMART/IDS

pass-predefined_properties_are_supported_but_discouraged_1_2

Open
#105 11 comments 0 reactions 1 assignee Claimed by @CBenghi View on GitHub
discuss & decide documentation tests
Dominant language
C#
Stars
318
Forks
90
PR merge metrics
No merged PRs in 30d

Description

Personally I don't think this complies with "simple" anymore, as it was set out in project goals. Also it is ambiguous, the exact same IDS spec would also match a regular property ```Foo_Bar```.```PanelOperation``` = "SWINGING". Granted, the spec is already ambiguous because properties/quantities are both implemented as properties, but this is another step I would say.

There is no good way of implementing this generically, which means that each implementation will have to implement ```IfcDoorLiningProperties```, ```IfcDoorPanelProperties```, ```IfcPermeableCoveringProperties```, ```IfcReinforcementDefinitionProperties```, ```IfcWindowLiningProperties```, ```IfcWindowPanelProperties```. Each of these have different attributes.

If this stays in spec, I think it should be described which attributes should be supported (I am guessing the same criteria apply as for the attributes facet).

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.