Skip to content

Commit

Permalink
Merge pull request #9296 from NREL/9228_OutputTableAnnual
Browse files Browse the repository at this point in the history
#9228 - Tabular file not generated with only Output:Table:Annual
  • Loading branch information
mjwitte authored Feb 26, 2022
2 parents 4fa778e + 872443d commit 58b5b32
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/EnergyPlus/OutputReportTabularAnnual.cc
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,9 @@ void GetInputTabularAnnual(EnergyPlusData &state)

objCount = state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, currentModuleObject);
if (objCount > 0) {

state.dataOutRptTab->WriteTabularFiles = true;

// if not a run period using weather do not create reports
if (!state.dataGlobal->DoWeathSim) {
ShowWarningError(state,
Expand Down
1 change: 1 addition & 0 deletions src/EnergyPlus/SimulationManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1653,6 +1653,7 @@ namespace SimulationManager {
ReportingRequested = (state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Table:SummaryReports") > 0 ||
state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Table:TimeBins") > 0 ||
state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Table:Monthly") > 0 ||
state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Table:Annual") > 0 ||
state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Variable") > 0 ||
state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Meter") > 0 ||
state.dataInputProcessing->inputProcessor->getNumObjectsFound(state, "Output:Meter:MeterFileOnly") > 0 ||
Expand Down
2 changes: 2 additions & 0 deletions tst/EnergyPlus/unit/OutputReportTabularAnnual.unit.cc
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,9 @@ TEST_F(EnergyPlusFixture, OutputReportTabularAnnual_GetInput)

state->dataGlobal->DoWeathSim = true;

EXPECT_FALSE(state->dataOutRptTab->WriteTabularFiles);
GetInputTabularAnnual(*state);
EXPECT_TRUE(state->dataOutRptTab->WriteTabularFiles);

EXPECT_EQ(state->dataOutputReportTabularAnnual->annualTables.size(), 1u);

Expand Down

5 comments on commit 58b5b32

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: OK (2508 of 2508 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (mjwitte) - Win64-Windows-10-VisualStudio-16: OK (2465 of 2465 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (mjwitte) - x86_64-MacOS-10.15-clang-11.0.0: OK (2487 of 2487 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: OK (1744 of 1744 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: OK (746 of 746 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.