-
Notifications
You must be signed in to change notification settings - Fork 55
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
Default units for measurements: missing definitions for IfcThermalTransmittance and others #242
Comments
It's fair, but if you can add the row to the document, there's an automation process that reuses it for the audit tool. |
We are running into a similar situation and struggle to use this table in our tool. As the datatype is something like When this table was discussed the idea was (if I remember well) to focus on most frequently used measurements. Meanwhile I doubt that this is really needed (@Pasi and @ Jiri Hietanen: please double check from your side) .
I am in favor of supporting all measurements and propose to use the table from the IFC documentation (see this CSV: UnitMeasurement.csv). It does not include the dimensional exponents, but I wonder if we need that information here. If we want to keep the limitation, I would at least propose to extend IfcThermalTransmittanceMeasure:
|
Hello @MatthiasWeise, I've taken a list of measures from the schemas and updated the markdown file of units. Have a look. |
Check of values such as "[20<=Value<=100]" requires to know the unit for a given measurement (meter for length, kg for mass etc.).
My understanding is that this unit is documented here: https://github.com/buildingSMART/IDS/blob/master/Documentation/units.md
Some measurements such as IfcThermalTransmittance are however missing.
Is it fair to assume that SI-base units are always used?
The text was updated successfully, but these errors were encountered: