-
Notifications
You must be signed in to change notification settings - Fork 394
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
Coil leaving air humidity ratio in Coil Sizing Summary shows wrong unit (temperature) #7571
Conversation
1 similar comment
Whoa, I wasn't expecting 32 files changed, but I see now it's heavily just comment changes. There are a few conflicts in some files that got overhauled in the plant refactor. Unless you specifically made functional changes in those files I think the safest thing would just be to accept the |
@Myoldmopar I'm actually working on the merge as we speak. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed a few table diffs. Confirmed that defect file is fixed. Unit tests all run. Added an entry in output rules (technically not a structural change but good to highlight this anyway since column headings change). Will merge after CI finishes.
Thanks for the review! |
Pull request overview
Fixes Imperial conversion of coil leaving air humidity ratio in Coil Sizing Summary shows temperature #7565
Coil Leaving Air Humidity Ratio in Coil Sizing Summary was using Celsius as unit
Standardized around kgWater/kgDryAir throughout the code base
Updated UnitConv accordingly.
Adding units for Design Inlet/Outlet Air Humidity Ratio in WaterCoils.cc
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Reviewer
This will not be exhaustively relevant to every PR.