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

Are we using measure_with_unit appropriately to track dimensional exponents? #1

Open
Moult opened this issue Mar 2, 2022 · 1 comment

Comments

@Moult
Copy link
Collaborator

Moult commented Mar 2, 2022

Also agree that your interpretation is correct. The key documentation is under the ValueComponent:

The value of the physical quantity when expressed in the specified units.

One more thing I'd raise (which can be dealt with in a separate bug) is whether or not the IFCTHERMODYNAMICTEMPERATUREMEASURE is appropriate. Especially if we start to keep track of dimensional exponents. I hunted down the original 10303-41 and here is the documentation for measure_with_unit:

https://steptools.com/stds/smrl/data/resource_docs/fundamentals_of_product_description_and_support/sys/19_schema.htm#measure_schema.measure_with_unit

Then, if you check under value_component and click on measure_value, you get this list:

https://steptools.com/stds/smrl/data/resource_docs/fundamentals_of_product_description_and_support/sys/19_schema.htm#measure_schema.measure_value

Notice how there is this suggestion under numeric measure:

19.3.28 numeric_measure EXPRESS-G
A numeric_measure is the numeric value of a physical quantity.

NOTE In order to define measure quantities that are not included as specific types in this standard a numeric_measure may be used as the value_component of a measure_with_unit, the corresponding unit_component being an si_unit or a derived_unit.

Perhaps we should've been using numeric measure all this time?

Originally posted by @Moult in buildingSMART/IFC4.3.x-development#7 (comment)

@Moult
Copy link
Collaborator Author

Moult commented Mar 2, 2022

Marking as onhold since this is a big issue.

@berlotti berlotti transferred this issue from buildingSMART/IFC4.3.x-development Mar 2, 2022
aothms pushed a commit that referenced this issue Jan 8, 2023
aothms pushed a commit that referenced this issue Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant