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
Fix Zone Contaminant Source or Sink CO2 Gain Volume Flow Rate output #7909
Conversation
@@ -5265,6 +5265,7 @@ namespace InternalHeatGains { | |||
e.HWPower = 0.0; | |||
e.SteamPower = 0.0; | |||
e.BaseHeatPower = 0.0; | |||
e.CO2Rate = 0.0; |
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.
This was forgotten here. Note that not all variables in ZnRpt
are set to zero here. Just the ones that are accumulated with +=
in this function.
Note the increase in unit test coverage.
|
No example files use this report variable? Should add it to one or more? |
Looked over the function where this change was made. This change looks like it includes everything. |
Unit tests run to completion locally. |
The 1 unit test failure appears unrelated and this specific failure is occurring more often. |
@mjwitte this all looks good and is ready to merge. What do you think about adding the report to one of the example files? |
EXPECT_EQ(DataHeatBalance::ZnRpt(1).SteamPower, 300.0); | ||
EXPECT_EQ(DataHeatBalance::ZnRpt(1).BaseHeatPower, 1500.0); | ||
EXPECT_EQ(DataHeatBalance::ZnRpt(1).CO2Rate, 0.0001125); | ||
} |
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.
Why isn't this showing that little error with no line feed after the last bracket?
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.
Oh, because it's not showing line 1006.
@rraustad Added output variables to one test file and corrected the docs for some of the output variables. |
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.
- Reports added to example file
- Review doc changes, built IO Ref and reviewed changes
- CI reports show expected results
Pull request overview
The #7905 defect file should report exactly the same values in columns B and C:
CO2PEOPLE:Contaminant Source or Sink CO2 Gain Volume Flow Rate m3/s
CLASSROOM:Zone Contaminant Source or Sink CO2 Gain Volume Flow Rate m3/s
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
No diffs expected, because Zone Contaminant Source or Sink CO2 Gain Volume Flow Rate is not used in any of the testfiles.
Added outputs to HeatPumpIAQP_GenericContamControl.idf
If IDD requires transition, transition source, rules, ExpandObjects, and IDFs must be updated, and add IDDChange labelIf structural output changes, add to output rules file and add OutputChange labelIf adding/removing any LaTeX docs or figures, update that document's CMakeLists file dependenciesReviewer
This will not be exhaustively relevant to every PR.
If branch is behind develop, merge develop and build locally to check for side effects of the mergeIf feature, test running new feature, try creative ways to break itCheck any new function arguments for performance impactsVerify IDF naming conventions and styles, memos and notes and defaultsIf new idf included, locally check the err file and other outputs