Skip to content

Commit

Permalink
Merge branch '7720_WaterHeater_Standalone_and_HPWH' into 7711_WaterTh…
Browse files Browse the repository at this point in the history
…ermalTanks_Crash
  • Loading branch information
jmarrec committed Jan 28, 2020
2 parents 52e55d1 + 4d50326 commit 15289cd
Show file tree
Hide file tree
Showing 2 changed files with 264 additions and 14 deletions.
8 changes: 4 additions & 4 deletions src/EnergyPlus/WaterThermalTanks.cc
Original file line number Diff line number Diff line change
Expand Up @@ -11334,7 +11334,7 @@ namespace WaterThermalTanks {

// PURPOSE OF THIS SUBROUTINE:
// Calculates the water heater standard ratings, such as Energy Factor and Recovery Efficiency. Results are written
// to the EIO file. Standard ratings are not calculated for storage-only tanks, i.e., MaxCapacity = 0.
// to the EIO file. Standard ratings are not calculated for storage-only tanks, i.e., MaxCapacity = 0, nor for Integrated Heat Pumps

// METHODOLOGY EMPLOYED:
// Water heater inputs are set to the specified test conditions. For HPWHs, the heating capacity and COP are assumed
Expand Down Expand Up @@ -11378,7 +11378,6 @@ namespace WaterThermalTanks {
FirstTimeFlag = true;

int TimeStepPerHour = int(1.0 / DataHVACGlobals::TimeStepSys);
int HPNum = 0;
// Simulate 24 hour test
for (int Step = 1; Step <= TimeStepPerHour * 24; ++Step) {

Expand Down Expand Up @@ -11417,7 +11416,7 @@ namespace WaterThermalTanks {

} else {

HPNum = this->HeatPumpNum;
int HPNum = this->HeatPumpNum; // Convenience variable
Real64 AmbientHumRat = 0.00717; // Humidity ratio at 67.5 F / 50% RH

// set the heat pump air- and water-side mass flow rate
Expand Down Expand Up @@ -11679,7 +11678,8 @@ namespace WaterThermalTanks {
} else {
RecoveryEfficiency = 0.0;
EnergyFactor = 0.0;
if (HPWaterHeater.empty() || !HPWaterHeater(HPNum).bIsIHP) {
// If this a regular tank, or an HPWH that's not an Integrated one
if ((this->HeatPumpNum == 0) || !HPWaterHeater(this->HeatPumpNum).bIsIHP) {
ShowWarningError("Water heater = " + this->Name +
": Recovery Efficiency and Energy Factor could not be calculated during the test for standard ratings");
ShowContinueError("Setpoint was never recovered and/or heater never turned on");
Expand Down
Loading

7 comments on commit 15289cd

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

7711_WaterThermalTanks_Crash (jmarrec) - x86_64-Linux-Ubuntu-18.04-cppcheck: OK (0 of 0 tests passed, 0 test warnings)

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

7711_WaterThermalTanks_Crash (jmarrec) - x86_64-Linux-Ubuntu-18.04-custom_check: OK (11 of 11 tests passed, 0 test warnings)

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

7711_WaterThermalTanks_Crash (jmarrec) - x86_64-Linux-Ubuntu-18.04-gcc-7.4: OK (2659 of 2659 tests passed, 0 test warnings)

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.

7711_WaterThermalTanks_Crash (jmarrec) - x86_64-Linux-Ubuntu-18.04-gcc-7.4-UnitTestsCoverage-Debug: OK (1271 of 1271 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.

7711_WaterThermalTanks_Crash (jmarrec) - x86_64-Linux-Ubuntu-18.04-gcc-7.4-IntegrationCoverage-Debug: OK (677 of 678 tests passed, 0 test warnings)

Failures:\n

integration Test Summary

  • Passed: 677
  • Timeout: 1

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

7711_WaterThermalTanks_Crash (jmarrec) - x86_64-MacOS-10.13-clang: OK (2619 of 2619 tests passed, 0 test warnings)

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.

7711_WaterThermalTanks_Crash (jmarrec) - Win64-Windows-10-VisualStudio-16: OK (2619 of 2619 tests passed, 0 test warnings)

Build Badge Test Badge

Please sign in to comment.