-
Notifications
You must be signed in to change notification settings - Fork 398
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
Deprecated Boiler "Design Water Outlet Temperature" input field #6816
Conversation
The table below shows hot water boiler autosized nominal capacity for selected sample of example files before and after the fix. The nominal capacity change is within 2.05%. This diffs is due to change in hot water temperature used to calculate density and specific heat value of hot water for autosizing boiler nominal capacity. |
Some of failed example files has HVACTemplate that require ExpandObjects. Modified ExpandObjects FORTRAN code used to create Boiler:HotWater object. |
Corrected field numbers in Boiler:HotWater object used with HVACTemplate:Plant:Boiler:ObjectReference object in expandobjects. This should fix the failed example file HVACTemplate-5ZoneVAVWaterCooled-ObjectReference.idf. |
@Nigusse A bunch of the performance test files were failing because they also needed to be transitioned. I made that transition, and those should all pass. I also made one tiny change to an unrelated file here (RefrigeratedWarehouse.idf). It will show diffs here just in the ERR file but it will stop the diffs from occurring in other branches. I believe this will be good to go in when CI finishes with 6f2723a. |
@Myoldmopar Thank you for transitioning the performance test files. I missed those. Thank you again for reviewing and merging. |
@Myoldmopar Will you be adding the transition source code? |
Yes...I'm working on the transition for the run period stuff now. |
Added transition rule, tested it out and it worked fine. CI should be happy and I think this will be ready to drop in. |
Addresses issue #4431
Pull request overview
This is sort of defect. Deprecated Boiler "Design Water Outlet Temperature" input field and replaced its value with hot water design sizing calculation value of 60.0C. The user input value was inconsistent with the sizing calculation procedure. Transition rules were added and the 141 example files with hot water boiler objects. Diffs are expected in 141 example files.
Work Checklist
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Review Checklist
This will not be exhaustively relevant to every PR.