buildingSMART / buildingSMART/IFC4.x-development

Potential deprecation of IfcLogical

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

Description

IfcLogical as a data type presents a few issues:

1. Complexity of serialisation: many serialisations understand booleans, but few understand logical. This prevents IFC being easier to express (ahahah get it) in other serisalisations.
2. Ambiguity of null state. What's the difference between null and unknown?
3. Sometimes abused to provided two bits of information in a single field (database design 101, 1 field = 1 value), like in IfcMaterialLayer.

Thoughts to deprecate and change to optional IfcBoolean?

I vaguely recall that it was maintained in some places for STEP compatibility, but not sure if that is still a priority, especially if NULL is equivalent to UNKNOWN.

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.