ibell / coolprop

Deprecated version - go to
https://github.com/CoolProp/CoolProp
MIT License
24 stars 16 forks source link

Units, units, units #63

Closed jowr closed 10 years ago

jowr commented 10 years ago

Hi all

What is the status of our current unit handling? Am I wrong or is there something missing?

Why is thermal conductivity handled different than the rest? At least in Python, PropsU returns kW/m/K and not W/m/K.

Should this be fixed in Python or in Units.cpp? It also a little funny that only thermal conductivity is treated in Units.cpp...

jowr commented 10 years ago

Since the function does not do that much, I excluded it from the incompressibles for now.

jowr commented 10 years ago

OK, I did not see that there is no break; statement. Hence, conversion actually happens.