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.

Sign up for GitHub

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

Jump to bottom

mp1 saturation default T units bug #99

Open
chmarti1 opened this issue Oct 12, 2024 · 0 comments
Open

mp1 saturation default T units bug #99

chmarti1 opened this issue Oct 12, 2024 · 0 comments
Labels
bug priority

Comments

@chmarti1
Copy link
Owner

chmarti1 commented Oct 12, 2024 •
edited
Loading

There is a bug in v2.2.5 in _mp1._sat_argparse that can result in the default temperature units not being honored. If the pm.config[def_T_unit] setting is different from pm.config['unit_temperature'], saturation properties will incorrectly interpret the temperature units in unit_temperature.

This only causes a problem if the units have been changed to disagree and if users do not specify a temperature. It will be corrected in the next release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chmarti1