Skip to content

Commit

Permalink
SpaceSizing - break apart more UpdateZoneSizing
Browse files Browse the repository at this point in the history
  • Loading branch information
mjwitte committed Jul 28, 2023
1 parent b1befec commit 9d9fe53
Show file tree
Hide file tree
Showing 5 changed files with 803 additions and 698 deletions.
2 changes: 1 addition & 1 deletion src/EnergyPlus/DataZoneEquipment.cc
Original file line number Diff line number Diff line change
Expand Up @@ -367,7 +367,7 @@ void GetZoneEquipmentData(EnergyPlusData &state)
// assigned to this node
if (ControlledZoneNum > 0) {
Zone(ControlledZoneNum).SystemZoneNodeNumber = thisZoneEquipConfig.ZoneNode;
if (state.dataHeatBal->doSpaceHeatBalance) {
if (state.dataHeatBal->doSpaceHeatBalanceSizing || state.dataHeatBal->doSpaceHeatBalanceSimulation) {

// SpaceHB TODO: For now, auto-assign the system node name for spaces
int spaceCount = 0;
Expand Down
Loading

5 comments on commit 9d9fe53

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

SpaceSizingHVAC (mjwitte) - Win64-Windows-10-VisualStudio-16: OK (2673 of 2673 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.

SpaceSizingHVAC (mjwitte) - x86_64-MacOS-10.17-clang-13.0.0: OK (3445 of 3446 tests passed, 770 test warnings)

Messages:\n

  • 771 tests had: AUD diffs.
  • 1 test had: BND diffs.
  • 1 test had: EIO diffs.
  • 1 test had: ESO big diffs.
  • 1 test had: Table big diffs.

Failures:\n

regression Test Summary

  • Passed: 771
  • Failed: 1

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.

SpaceSizingHVAC (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.3: OK (3486 of 3487 tests passed, 774 test warnings)

Messages:\n

  • 775 tests had: AUD diffs.
  • 1 test had: BND diffs.
  • 1 test had: EIO diffs.
  • 1 test had: ESO big diffs.
  • 1 test had: Table big diffs.

Failures:\n

regression Test Summary

  • Passed: 791
  • Failed: 1

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.

SpaceSizingHVAC (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.3-UnitTestsCoverage-Debug: OK (1900 of 1900 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.

SpaceSizingHVAC (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.3-IntegrationCoverage-Debug: OK (776 of 776 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.