Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

List of datatypes without IfcCompoundPlaneAngleMeasure #297

Open
MatthiasWeise opened this issue May 14, 2024 · 2 comments
Open

List of datatypes without IfcCompoundPlaneAngleMeasure #297

MatthiasWeise opened this issue May 14, 2024 · 2 comments
Assignees
Labels
documentation Improvements or additions to documentation tests
Milestone

Comments

@MatthiasWeise
Copy link
Contributor

Is it intentional that IfcCompoundPlaneAngleMeasure was omitted as a supported data type?
What does it mean if IfcCompoundPlaneAngleMeasure is shown in this list (https://github.com/buildingSMART/IDS/blob/6893df1669755263b2362b82752092d9c64a6e5e/Documentation/DataTypes.md):

  1. is it excluded from allowed/checkable datatypes
  2. or is it excluded from constraints?
@aothms
Copy link

aothms commented May 14, 2024

Also

Property dataTypes can be set to any values according to the following table.

Quite a few of these types are not selectable by IfcValue and hence cannot be used as property data types? Most of the enums are in fact predefined types, but others are likely not in scope at all such as IfcBoxAlignment which is only a mandatory attribute type on one other entity. Did I miss something, have I been out of the loop for too long?

IfcCompoundPlaneAngleMeasure

I guess for the same reason IfcComplexNumber is missing. Would be good to also explicitly list the types that are not supported.

I remember discussion on slack whether IfcCompoundPlaneAngleMeasure should be mapped to a double. I don't remember the outcome.

But I agree. Even if we can't constrain the value because we don't have an IDS type for it, we should be able to require it as a data type.

@CBenghi
Copy link
Contributor

CBenghi commented May 17, 2024

But I agree. Even if we can't constrain the value because we don't have an IDS type for it, we should be able to require it as a data type.

Ageed, I will update the audit tool and the documentation.

@CBenghi CBenghi self-assigned this May 17, 2024
@CBenghi CBenghi added documentation Improvements or additions to documentation tests labels May 17, 2024
@CBenghi CBenghi added this to the 1.0 milestone May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation tests
Projects
None yet
Development

No branches or pull requests

3 participants