Skip to content

Commit

Permalink
10634 Address Custom Check Error
Browse files Browse the repository at this point in the history
Something about the logical flag wasn't what the custom check wanted.  This is an attempt to fix that.
  • Loading branch information
RKStrand committed Aug 9, 2024
1 parent c95600b commit 8f7c5de
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/EnergyPlus/CondenserLoopTowers.cc
Original file line number Diff line number Diff line change
Expand Up @@ -4695,7 +4695,7 @@ namespace CondenserLoopTowers {

// Do not RETURN here if flow rate is less than SmallMassFlow. Check basin heater and then RETURN.

bool returnFlagSet;
bool returnFlagSet = false;
this->checkMassFlowAndLoad(state, MyLoad, returnFlagSet);
if (returnFlagSet) return;

Expand Down Expand Up @@ -4970,7 +4970,7 @@ namespace CondenserLoopTowers {
// Do not RETURN here if flow rate is less than SmallMassFlow. Check basin heater and then RETURN.
if (state.dataPlnt->PlantLoop(this->plantLoc.loopNum).LoopSide(this->plantLoc.loopSideNum).FlowLock == DataPlant::FlowLock::Unlocked)
return; // TODO: WTF
bool returnFlagSet;
bool returnFlagSet = false;
this->checkMassFlowAndLoad(state, MyLoad, returnFlagSet);
if (returnFlagSet) return;

Expand Down Expand Up @@ -5203,7 +5203,7 @@ namespace CondenserLoopTowers {
if (state.dataPlnt->PlantLoop(this->plantLoc.loopNum).LoopSide(this->plantLoc.loopSideNum).FlowLock == DataPlant::FlowLock::Unlocked)
return; // TODO: WTF

bool returnFlagSet;
bool returnFlagSet = false;
this->checkMassFlowAndLoad(state, MyLoad, returnFlagSet);
if (returnFlagSet) return;

Expand Down Expand Up @@ -6457,7 +6457,6 @@ namespace CondenserLoopTowers {

void CoolingTower::checkMassFlowAndLoad(EnergyPlusData &state, Real64 const MyLoad, bool &returnFlagSet)
{
returnFlagSet = false;
// MassFlowTolerance is a parameter to indicate a no flow condition
if (this->WaterMassFlowRate <= DataBranchAirLoopPlant::MassFlowTolerance) {
// for multiple cells, we assume that it's a common basin
Expand Down

4 comments on commit 8f7c5de

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

10634CondenserNotFollowingPlanOpScheme (RKStrand) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3684 of 3695 tests passed, 4 test warnings)

Messages:\n

  • 4 tests had: ESO small diffs.
  • 5 tests had: MTR small diffs.
  • 7 tests had: EIO diffs.
  • 11 tests had: ESO big diffs.
  • 10 tests had: MTR big diffs.
  • 10 tests had: Table big diffs.
  • 10 tests had: Table string diffs.
  • 1 test had: Table small diffs.
  • 1 test had: ERR diffs.

Failures:\n

regression Test Summary

  • Passed: 800
  • Failed: 11

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.

10634CondenserNotFollowingPlanOpScheme (RKStrand) - x86_64-MacOS-10.18-clang-15.0.0: OK (3643 of 3654 tests passed, 4 test warnings)

Messages:\n

  • 4 tests had: ESO small diffs.
  • 5 tests had: MTR small diffs.
  • 7 tests had: EIO diffs.
  • 11 tests had: ESO big diffs.
  • 10 tests had: MTR big diffs.
  • 1 test had: Table small diffs.
  • 10 tests had: Table big diffs.
  • 10 tests had: Table string diffs.
  • 1 test had: ERR diffs.

Failures:\n

regression Test Summary

  • Passed: 780
  • Failed: 11

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

10634CondenserNotFollowingPlanOpScheme (RKStrand) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (795 of 795 tests passed, 0 test warnings)

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

10634CondenserNotFollowingPlanOpScheme (RKStrand) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (2070 of 2070 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.