Skip to content

Commit

Permalink
Accumulate outputs in the new branch, fix single outlier
Browse files Browse the repository at this point in the history
  • Loading branch information
Yujie Xu committed Dec 20, 2023
1 parent 010fd51 commit b5b9122
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/EnergyPlus/HVACVariableRefrigerantFlow.cc
Original file line number Diff line number Diff line change
Expand Up @@ -11869,8 +11869,13 @@ void VRFCondenserEquipment::CalcVRFCondenser_FluidTCtrl(EnergyPlusData &state)
Real64 CompEvaporatingPWRSpdMaxCurrentTsuc =
this->RatedCompPower * CurveValue(state, this->OUCoolingPWRFT(NumOfCompSpdInput), Tdischarge, this->EvaporatingTemp);
if ((Q_c_OU * C_cap_operation) > CompEvaporatingCAPSpdMaxCurrentTsuc + CompEvaporatingPWRSpdMaxCurrentTsuc) {
Q_c_OU = CompEvaporatingCAPSpdMaxCurrentTsuc;
CompSpdActual = this->CompressorSpeed(NumOfCompSpdInput);
Ncomp = this->RatedCompPower * CurveValue(state, this->OUCoolingPWRFT(NumOfCompSpdInput), Tdischarge, this->EvaporatingTemp);
Ncomp = CompEvaporatingPWRSpdMaxCurrentTsuc;
m_air = this->OUAirFlowRate * RhoAir;
SH_OU = this->SH;
this->VRFOU_TeTc(
state, HXOpMode::EvapMode, Q_c_OU, SH_OU, m_air, OutdoorDryBulb, OutdoorHumRat, OutdoorPressure, Tfs, this->EvaporatingTemp);
} else if ((Q_c_OU * C_cap_operation) <= CompEvaporatingCAPSpdMin) {
// Required heating load is smaller than the min heating capacity

Expand Down

5 comments on commit b5b9122

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

vrfHeating3 (Unknown) - x86_64-MacOS-10.17-clang-14.0.0: OK (3537 of 3546 tests passed, 0 test warnings)

Messages:\n

  • 4 tests had: EIO diffs.
  • 4 tests had: ERR diffs.
  • 4 tests had: ESO big diffs.
  • 4 tests had: Table big diffs.
  • 2 tests had: MTR big diffs.

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1481
  • Failed: 5

regression Test Summary

  • Passed: 782
  • Failed: 4

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.

vrfHeating3 (Unknown) - Win64-Windows-10-VisualStudio-16: OK (2754 of 2759 tests passed, 0 test warnings)

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1479
  • Failed: 5

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.

vrfHeating3 (Unknown) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3578 of 3587 tests passed, 0 test warnings)

Messages:\n

  • 4 tests had: EIO diffs.
  • 4 tests had: ERR diffs.
  • 4 tests had: ESO big diffs.
  • 4 tests had: Table big diffs.
  • 2 tests had: MTR big diffs.

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1481
  • Failed: 5

regression Test Summary

  • Passed: 802
  • Failed: 4

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.

vrfHeating3 (Unknown) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (1967 of 1972 tests passed, 0 test warnings)

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1481
  • Failed: 5

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.

vrfHeating3 (Unknown) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (790 of 790 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.