buildingSMART / buildingSMART/IFC4.x-development
IfcSIUnitName has DEGREE_CELSIUS
Open
allocated-core
ifc-update-out
proposal
- Dominant language
- Python
- Stars
- 234
- Forks
- 123
- Avg merge
- 15h 4m
- Merged PRs (30d)
- 5
Description
The SI unit for thermodynamic temperature is Kelvin, not Degrees Celsius. Yet, degrees are in the enum, which is not quite correct. In addition, this creates a break between the mapping of 29 unit types to 30 SI units (with degrees celsius being the extra unit).
I assume this is here for pure convenience, but I thought I'd raise it nonetheless. Is this something we care about to get right, or just leave it as is?
The break in mapping makes it slightly (just slightly) inconvenience for software because we cannot fully auto derive one from the other.
Contributor guide
Assessment
This issue has not been assessed yet.