Skip to content

Commit

Permalink
Loosen tolerance on temperature in unit test
Browse files Browse the repository at this point in the history
  • Loading branch information
Myoldmopar committed Jul 15, 2024
1 parent a9572ac commit 472e3e9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tst/EnergyPlus/unit/MixedAir.unit.cc
Original file line number Diff line number Diff line change
Expand Up @@ -7745,7 +7745,7 @@ TEST_F(EnergyPlusFixture, MixedAir_TemperatureError)

MixedAir::SimOAMixer(*state, state->dataAirLoop->OutsideAirSys(1).ComponentName(1), state->dataAirLoop->OutsideAirSys(1).ComponentIndex(1));

auto T_sat = Psychrometrics::PsyTsatFnHPb(*state, state->dataMixedAir->OAMixer(1).MixEnthalpy, state->dataMixedAir->OAMixer(1).MixPressure);
Real64 const T_sat = Psychrometrics::PsyTsatFnHPb(*state, state->dataMixedAir->OAMixer(1).MixEnthalpy, state->dataMixedAir->OAMixer(1).MixPressure);

// T_db must be >= T_sat at the mixed-air node to remain physical
EXPECT_TRUE(state->dataMixedAir->OAMixer(1).MixTemp >= T_sat);
Expand Down
2 changes: 1 addition & 1 deletion tst/EnergyPlus/unit/PackagedTerminalHeatPump.unit.cc
Original file line number Diff line number Diff line change
Expand Up @@ -4028,7 +4028,7 @@ TEST_F(EnergyPlusFixture, PTACDrawAirfromReturnNodeAndPlenum_Test)
// same temperature test as above commented out test (23.15327704750551), now shows 21.2 C
// how do you mix 2 air streams with T1in=31.18 and T2in=23.15 and get Tout=21.23 ??
// must be a node enthalpy issue with this unit test?
EXPECT_NEAR(21.2317, state->dataLoopNodes->Node(ATMixer1AirOutNode).Temp, 0.001);
EXPECT_NEAR(21.2316, state->dataLoopNodes->Node(ATMixer1AirOutNode).Temp, 0.005);
EXPECT_NEAR(0.324036, state->dataLoopNodes->Node(ATMixer1AirOutNode).MassFlowRate, 0.001);

// mass balance zone 1 ATMixer outlet enthalpy based on pri and sec inlet stream enthalpy
Expand Down

1 comment on commit 472e3e9

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

adjust-mixed-air-temp (Myoldmopar) - Win64-Windows-10-VisualStudio-16: OK (2853 of 2853 tests passed, 0 test warnings)

Build Badge Test Badge

Please sign in to comment.