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.

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

Merged
merged 11 commits into from
Jul 31, 2018

Conversation

Nigusse
Copy link
Contributor

@Nigusse Nigusse commented Jul 10, 2018

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.

  • Title of PR should be user-synopsis style (clearly understandable in a standalone changelog context)
  • At least one of the following appropriate labels must be added to this PR to be consumed into the changelog:
    • Defect: This pull request repairs a github defect issue. The github issue should be referenced in the PR description

Review Checklist

This will not be exhaustively relevant to every PR.

  • If defect, results of running current develop vs this branch should exhibit the fix
  • CI status: all green or justified
  • Performance: CI Linux results include performance check -- verify this
  • Unit Test(s)
  • IDD changes:
    • Verify naming conventions and styles, memos and notes and defaults
    • Open windows IDF Editor with modified IDD to check for errors
    • If transition, add rules to spreadsheet
    • If transition, add transition source
    • If transition, update idfs

@Nigusse Nigusse added Defect Includes code to repair a defect in EnergyPlus Documentation Related primarily on the LaTeX-based EnergyPlus documentation IDDChange Code changes impact the IDD file (cannot be merged after IO freeze) labels Jul 10, 2018
@Nigusse
Copy link
Contributor Author

Nigusse commented Jul 10, 2018

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.

4431_boliernomcap_eio_diffs

@Nigusse
Copy link
Contributor Author

Nigusse commented Jul 10, 2018

Some of failed example files has HVACTemplate that require ExpandObjects. Modified ExpandObjects FORTRAN code used to create Boiler:HotWater object.

@Nigusse
Copy link
Contributor Author

Nigusse commented Jul 11, 2018

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.

@nrel-bot-3
Copy link

@Nigusse @lgentile it has been 14 days since this pull request was last updated.

@Myoldmopar
Copy link
Member

@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.

@Nigusse
Copy link
Contributor Author

Nigusse commented Jul 30, 2018

@Myoldmopar Thank you for transitioning the performance test files. I missed those. Thank you again for reviewing and merging.

@mjwitte
Copy link
Contributor

mjwitte commented Jul 30, 2018

@Myoldmopar Will you be adding the transition source code?

@Myoldmopar
Copy link
Member

Yes...I'm working on the transition for the run period stuff now.

@Myoldmopar
Copy link
Member

Added transition rule, tested it out and it worked fine. CI should be happy and I think this will be ready to drop in.

@mjwitte mjwitte removed the Documentation Related primarily on the LaTeX-based EnergyPlus documentation label Jul 30, 2018
@mjwitte mjwitte added this to the EnergyPlus 9.0 milestone Jul 30, 2018
@Myoldmopar Myoldmopar merged commit fee1ef0 into develop Jul 31, 2018
@Myoldmopar Myoldmopar deleted the 158922394_Issue#4431 branch July 31, 2018 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Includes code to repair a defect in EnergyPlus IDDChange Code changes impact the IDD file (cannot be merged after IO freeze)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants