Skip to content

Commit

Permalink
add SafeCopyPlantNode() to cooling towers
Browse files Browse the repository at this point in the history
  • Loading branch information
@EnergyArchmage
EnergyArchmage committed Aug 14, 2024
1 parent c8531d1 commit 4d61594
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/EnergyPlus/CondenserLoopTowers.cc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6344,6 +6344,7 @@ namespace CondenserLoopTowers {
// This subroutine is for passing results to the outlet water node.

// set node information
PlantUtilities::SafeCopyPlantNode(state, this->WaterInletNodeNum, this->WaterOutletNodeNum);
state.dataLoopNodes->Node(this->WaterOutletNodeNum).Temp = this->OutletWaterTemp;

if (state.dataPlnt->PlantLoop(this->plantLoc.loopNum).LoopSide(this->plantLoc.loopSideNum).FlowLock == DataPlant::FlowLock::Unlocked ||
Expand Down

5 comments on commit 4d61594

@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 (EnergyArchmage) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3687 of 3699 tests passed, 4 test warnings)

Messages:\n

  • 4 tests had: ESO small diffs.
  • 6 tests had: MTR small diffs.
  • 8 tests had: EIO diffs.
  • 12 tests had: ESO big diffs.
  • 10 tests had: MTR big diffs.
  • 1 test had: Table small diffs.
  • 11 tests had: Table big diffs.
  • 10 tests had: Table string diffs.
  • 1 test had: ERR diffs.

Failures:\n

regression Test Summary

  • Passed: 800
  • Failed: 12

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 (EnergyArchmage) - x86_64-MacOS-10.18-clang-15.0.0: OK (3646 of 3658 tests passed, 4 test warnings)

Messages:\n

  • 4 tests had: ESO small diffs.
  • 6 tests had: MTR small diffs.
  • 7 tests had: EIO diffs.
  • 12 tests had: ESO big diffs.
  • 10 tests had: MTR big diffs.
  • 11 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: 780
  • Failed: 12

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 (EnergyArchmage) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (2072 of 2072 tests passed, 0 test warnings)

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

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

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.

10634CondenserNotFollowingPlanOpScheme (EnergyArchmage) - Win64-Windows-10-VisualStudio-16: OK (2865 of 2865 tests passed, 0 test warnings)

Build Badge Test Badge

Please sign in to comment.