buildingSMART / buildingSMART/IDS

Interpretation of PROHIBITED for property requirement :ballot_box:

Open
#206 19 comments 0 reactions 0 assignees View on GitHub
schema
Dominant language
C#
Stars
318
Forks
90
PR merge metrics
No merged PRs in 30d

Description

**Problem:**

PROHIBITED seems to be interpreted as negation, but is missleading from an engineering point of view due to the mandatory datatype settings.

Current interpretation of PROHIBITED:
`NOT EXISTS (Pset=="My_Pset" AND Property=="My_Property" AND IfcMeasure=="IFCLABEL")
`
My expectation as engineer would however be more like this: (if I would like to exclude a property I do not care about the datatype)
`NOT EXISTS (Pset=="My_Pset" AND Property=="My_Property")`

**Proposed solution:**

- make datatype optional (or ignore in PROHIBITED requirements)
- add logical statements to documentation for interpretation of PROHIBITED, REQUIRED and OPTIONAL
(for OPTIONAL: IF Pset AND Property THEN Datatype AND Value)

P.S.:
REQUIRED (minOccurs=1) is clear, although I would expect maxOccurs=1 instead of unbounded because the required property should be present once, but not more than once, and asking for a number of properties with a pattern does not make sense to me.

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.