Skip to content

Commit

Permalink
Merge pull request #10775 from NREL/10774-std-62-1-errors
Browse files Browse the repository at this point in the history
Fix Std. 62.1 VRP calculations for Secondary Recirculation Systems
  • Loading branch information
Myoldmopar authored Dec 4, 2024
2 parents 27ec210 + a960bf8 commit f6e696a
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions src/EnergyPlus/SizingManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1331,23 +1331,23 @@ void ManageSystemVentilationAdjustments(EnergyPlusData &state)
Real64 Ez_Clg = thisTermUnitFinalZoneSizing.ZoneADEffCooling; // user input in Zone Air Distribution design spec object
Real64 Fc_Clg = 1.0 - (1.0 - Ez_Clg) * (1.0 - Er) * (1 - Ep_Clg);
state.dataSize->FcByZoneCool(termUnitSizingIndex) = Fc_Clg;
state.dataSize->EvzByZoneCool(termUnitSizingIndex) =
(Fa_Clg + state.dataSize->XsBySysCool(AirLoopNum) * Fb_Clg - state.dataSize->ZdzClgByZone(termUnitSizingIndex) * Fc_Clg) /
Fa_Clg;
state.dataSize->EvzByZoneCool(termUnitSizingIndex) = (Fa_Clg + state.dataSize->XsBySysCool(AirLoopNum) * Fb_Clg -
state.dataSize->ZdzClgByZone(termUnitSizingIndex) * Ep_Clg * Fc_Clg) /
Fa_Clg;
// note that SimAirServingZones::LimitZoneVentEff is intended only for single path per I/O ref

// find Evz for heating
Real64 Ep_Htg = thisTermUnitFinalZoneSizing.ZonePrimaryAirFractionHtg; // as adjusted in ManageSystemSizingAdjustments();
Real64 Fa_Htg = Ep_Htg + (1.0 - Ep_Htg) * Er;
state.dataSize->FaByZoneHeat(termUnitSizingIndex) = Fa_Htg;
Real64 Fb_Htg = Ep_Htg;
state.dataSize->FbByZoneCool(termUnitSizingIndex) = Fb_Htg;
state.dataSize->FbByZoneHeat(termUnitSizingIndex) = Fb_Htg;
Real64 Ez_Htg = thisTermUnitFinalZoneSizing.ZoneADEffHeating; // user input in Zone Air Distribution design spec object
Real64 Fc_Htg = 1.0 - (1.0 - Ez_Htg) * (1.0 - Er) * (1 - Ep_Htg);
state.dataSize->FcByZoneHeat(termUnitSizingIndex) = Fc_Htg;
state.dataSize->EvzByZoneHeat(termUnitSizingIndex) =
(Fa_Htg + state.dataSize->XsBySysHeat(AirLoopNum) * Fb_Htg - state.dataSize->ZdzHtgByZone(termUnitSizingIndex) * Fc_Htg) /
Fa_Htg;
state.dataSize->EvzByZoneHeat(termUnitSizingIndex) = (Fa_Htg + state.dataSize->XsBySysHeat(AirLoopNum) * Fb_Htg -
state.dataSize->ZdzHtgByZone(termUnitSizingIndex) * Ep_Htg * Fc_Htg) /
Fa_Htg;

} else { // 62.1 ventilation rate procedure - single path zone
state.dataSize->EvzByZoneCool(termUnitSizingIndex) =
Expand Down

3 comments on commit f6e696a

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

develop (Myoldmopar) - x86_64-Linux-Ubuntu-24.04-gcc-13.2: OK (2917 of 2917 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.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-24.04-gcc-13.2-UnitTestsCoverage-RelWithDebInfo: Tests Failed (16 of 2099 tests passed, 0 test warnings)

Failures:\n

API Test Summary

  • Passed: 9
  • Failed: 1
  • notrun: 5

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

develop (Myoldmopar) - x86_64-Linux-Ubuntu-24.04-gcc-13.2-IntegrationCoverage-RelWithDebInfo: Tests Failed (2 of 801 tests passed, 0 test warnings)

Failures:\n

integration Test Summary

  • Passed: 2
  • Failed: 799

Build Badge Test Badge Coverage Badge

Please sign in to comment.