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

IfcBoxAlignment should be enumeration #724

Closed
I-Sokolov opened this issue Nov 9, 2023 · 3 comments
Closed

IfcBoxAlignment should be enumeration #724

I-Sokolov opened this issue Nov 9, 2023 · 3 comments

Comments

@I-Sokolov
Copy link

want to share here buildingSMART/IFC4.3.x-output#621

@aothms
Copy link
Collaborator

aothms commented Nov 9, 2023

This was reported several times #544 #584

I assume it has to do with backwards compatibility concerns in STEP. Like first using a freeform text field and later migrating to CSS. And us trying to be compatible again with STEP.

I think everybody agrees, but given compatibility constraints I guess it's unlikely it will get adapted.

We have something similar in RasterFormat https://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/lexical/IfcBlobTexture.htm But probably there it is intentional.

For future reference we need to understand our compatibility policy with respect to adding an ENUM item vs changing a WR literal. This is a bit too subjective now. CC @TLiebich @berlotti

@TLiebich
Copy link
Collaborator

TLiebich commented Nov 9, 2023

@aothms - the main reason back then was to follow STEP parts, if we adopted definitions from there. I don't think, that this reason is still relevant - more the question of compatibility among IFC-Versions.

@Moult
Copy link
Collaborator

Moult commented Nov 9, 2023

Closing as duplicate #544.

@Moult Moult closed this as completed Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants