Skip to content

Commit

Permalink
OA Details - some cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
mjwitte committed Apr 12, 2021
1 parent b693d92 commit bb5cacd
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 3 additions & 0 deletions src/EnergyPlus/OutputReportTabular.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6237,6 +6237,9 @@ void FillRemainingPredefinedEntries(EnergyPlusData &state)
if (!(state.dataAirflowNetwork->SimulateAirflowNetwork == AirflowNetwork::AirflowNetworkControlMultizone ||
state.dataAirflowNetwork->SimulateAirflowNetwork == AirflowNetwork::AirflowNetworkControlMultiADS)) {
ZonePreDefRep(iZone).AFNInfilVolTotalStdDen = 0.0;
ZonePreDefRep(iZone).AFNVentVolTotalStdDen = 0.0;
ZonePreDefRep(iZone).AFNInfilVolTotalOccStdDen = 0.0;
ZonePreDefRep(iZone).AFNVentVolTotalOccStdDen = 0.0;
ZonePreDefRep(iZone).AFNInfilVolMin = 0.0;
ZonePreDefRep(iZone).AFNInfilVolTotalOcc = 0.0;
}
Expand Down
1 change: 0 additions & 1 deletion src/EnergyPlus/SystemReports.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5069,7 +5069,6 @@ void ReportMaxVentilationLoads(EnergyPlusData &state)
state.dataSysRpts->SysNatVentFlow(sysNum) * TimeStepSys * DataGlobalConstants::SecInHour;

// set time mechanical+natural ventilation is below, at, or above target Voz-dyn
// MJWToDo - InfilVolume should be NatVentVolume or similar after this is split in AFN
Real64 totMechNatVentVolFlowStdRho = mechVentFlow + state.dataSysRpts->SysNatVentFlow(sysNum);

Real64 targetFlowVoz = state.dataSysRpts->SysTargetVentilationFlowVoz(sysNum);
Expand Down

5 comments on commit bb5cacd

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

8475-Separate-AFN-infiltration-from-natural-ventilation (mjwitte) - x86_64-MacOS-10.15-clang-11.0.0: OK (3056 of 3080 tests passed, 11 test warnings)

Messages:\n

  • 33 tests had: AUD diffs.
  • 31 tests had: RDD diffs.
  • 14 tests had: ESO small diffs.
  • 3 tests had: ESO big diffs.
  • 22 tests had: Table big diffs.
  • 11 tests had: EIO diffs.
  • 2 tests had: Table small diffs.
  • 10 tests had: MTR small diffs.

Failures:\n

regression Test Summary

  • Passed: 698
  • Failed: 24

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

8475-Separate-AFN-infiltration-from-natural-ventilation (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: OK (1634 of 1634 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.

8475-Separate-AFN-infiltration-from-natural-ventilation (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: OK (3096 of 3120 tests passed, 11 test warnings)

Messages:\n

  • 33 tests had: AUD diffs.
  • 31 tests had: RDD diffs.
  • 13 tests had: ESO small diffs.
  • 22 tests had: Table big diffs.
  • 3 tests had: ESO big diffs.
  • 10 tests had: MTR small diffs.
  • 8 tests had: EIO diffs.
  • 1 test had: Table small diffs.

Failures:\n

regression Test Summary

  • Passed: 718
  • Failed: 24

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

8475-Separate-AFN-infiltration-from-natural-ventilation (mjwitte) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: Coverage Too Low

Failures:\n

integration Test Summary

  • Passed: 726
  • Timeout: 1

Build Badge Test Badge Coverage 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.

8475-Separate-AFN-infiltration-from-natural-ventilation (mjwitte) - Win64-Windows-10-VisualStudio-16: OK (2329 of 2329 tests passed, 0 test warnings)

Build Badge Test Badge

Please sign in to comment.