Skip to content

Commit

Permalink
Coil sizing report - protect against errors for fan not found
Browse files Browse the repository at this point in the history
  • Loading branch information
mjwitte committed Feb 14, 2018
1 parent aa64ea0 commit 1ee7c8d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/EnergyPlus/ReportCoilSelection.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1577,6 +1577,9 @@ ReportCoilSelection::setCoilSupplyFanInfo(
int const & fanIndex
)
{
if ( fanName == "" ) {
return;
}
int index = getIndexForOrCreateDataObjFromCoilName( coilName,coilType );
auto & c( coilSelectionDataObjs[ index ] );
c->fanAssociatedWithCoilName = fanName;
Expand Down

8 comments on commit 1ee7c8d

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

Coil-Report-Table (mjwitte) - x86_64-MacOS-10.9-clang: OK (2815 of 2831 tests passed, 605 test warnings)

Messages:

  • 614 tests had: AUD diffs.
  • 9 tests had: EIO diffs.
  • 9 tests had: ESO big diffs.
  • 8 tests had: Table big diffs.
  • 99 tests had: ERR diffs.
  • 1 test had: Table small diffs.
  • 1 test had: MTR big diffs.

Failures:

regression Test Summary

  • Passed: 605
  • Failed: 9

WaterCoilsTest Test Summary

  • Passed: 1
  • SEGFAULT: 7

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.

Coil-Report-Table (mjwitte) - i386-Windows-7-VisualStudio-14: OK (2824 of 2831 tests passed, 0 test warnings)

Failures:

WaterCoilsTest Test Summary

  • Passed: 1
  • Failed: 7

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.

Coil-Report-Table (mjwitte) - Win64-Windows-7-VisualStudio-14: OK (2824 of 2831 tests passed, 0 test warnings)

Failures:

WaterCoilsTest Test Summary

  • Passed: 1
  • Failed: 7

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.

Coil-Report-Table (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2855 of 2871 tests passed, 609 test warnings)

Messages:

  • 618 tests had: AUD diffs.
  • 9 tests had: EIO diffs.
  • 9 tests had: ESO big diffs.
  • 8 tests had: Table big diffs.
  • 99 tests had: ERR diffs.
  • 1 test had: Table small diffs.
  • 1 test had: MTR big diffs.

Failures:

regression Test Summary

  • Passed: 625
  • Failed: 9

WaterCoilsTest Test Summary

  • Passed: 1
  • SEGFAULT: 7

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.

Coil-Report-Table (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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

Coil-Report-Table (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1594 of 1601 tests passed, 0 test warnings)

Failures:

WaterCoilsTest Test Summary

  • Passed: 1
  • OTHER_FAULT: 7

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.

Coil-Report-Table (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2199 of 2220 tests passed, 0 test warnings)

Failures:

integration Test Summary

  • Passed: 605
  • Failed: 14

WaterCoilsTest Test Summary

  • Passed: 1
  • OTHER_FAULT: 7

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.

Coil-Report-Table (mjwitte) - x86_64-Linux-Ubuntu-14.04-custom_check: Build Failed

Build Badge

Please sign in to comment.