Skip to content

Commit

Permalink
Space IV.5 - Another attempt to fix linux unit test failure
Browse files Browse the repository at this point in the history
  • Loading branch information
@mjwitte
mjwitte committed Sep 6, 2024
1 parent 40fe62c commit c2616e6
Showing 1 changed file with 17 additions and 12 deletions.
29 changes: 17 additions & 12 deletions tst/EnergyPlus/unit/SizingManager.unit.cc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4151,7 +4151,7 @@ TEST_F(EnergyPlusFixture, SizingManager_ZoneSizing_Coincident_NonAir_10x_Latent_
" ElectricEquipment,",
" Space 1 Latent ElecEq, !- Name",
" Space 1, !- Zone or ZoneList Name",
" Morning, !- Schedule Name",
" Morning2, !- Schedule Name",
" EquipmentLevel, !- Design Level Calculation Method",
" 1000.0, !- Design Level {W}",
" , !- Watts per Zone Floor Area {W/m2}",
Expand All @@ -4170,6 +4170,18 @@ TEST_F(EnergyPlusFixture, SizingManager_ZoneSizing_Coincident_NonAir_10x_Latent_
" Until: 12:00,1.0, !- Field 11",
" Until: 24:00,0.0; !- Field 11",

" Schedule:Compact,",
" Morning2, !- Name",
" , !- Schedule Type Limits Name",
" Through: 12/31, !- Field 9",
" For: Alldays, !- Field 10",
" Until: 8:00,0.0, !- Field 11",
" Until: 9:00,0.5, !- Field 11",
" Until: 11:00,0.8, !- Field 11",
" Until: 12:00,1.0, !- Field 11",
" Until: 13:00,0.9, !- Field 11",
" Until: 24:00,0.0; !- Field 11",

" ElectricEquipment,",
" Space 2 ElecEq, !- Name",
" Space 2, !- Zone or ZoneList Name",
Expand Down Expand Up @@ -4224,16 +4236,9 @@ TEST_F(EnergyPlusFixture, SizingManager_ZoneSizing_Coincident_NonAir_10x_Latent_
SimulationManager::ManageSimulation(*state);

int space1Num = 1;
int curSimDay = 1;
auto &calSpSiz = state->dataSize->CalcSpaceSizing(curSimDay, space1Num);

EXPECT_NEAR(calSpSiz.LatentCoolLoadSeq(57), 7500.00, 0.01);
EXPECT_NEAR(calSpSiz.LatentCoolLoadSeq(58), 8333.33, 0.01);
EXPECT_NEAR(calSpSiz.LatentCoolLoadSeq(59), 9166.66, 0.01);
EXPECT_NEAR(calSpSiz.LatentCoolLoadSeq(60), 10000.00, 0.01);
EXPECT_NEAR(calSpSiz.LatentCoolLoadSeq(61), 10000.00, 0.01);
EXPECT_TRUE(calSpSiz.LatentCoolLoadSeq(60) > calSpSiz.LatentCoolLoadSeq(59));
EXPECT_TRUE(calSpSiz.LatentCoolLoadSeq(61) == calSpSiz.LatentCoolLoadSeq(60));
auto &calFinalSpSiz = state->dataSize->CalcFinalSpaceSizing(space1Num);
EXPECT_EQ(calFinalSpSiz.TimeStepNumAtLatentCoolMax, 72);

EXPECT_EQ("18989.14", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpHtCalcDesLd, "SPACE 1"));
EXPECT_EQ("0.527", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpHtCalcDesAirFlow, "SPACE 1"));
EXPECT_EQ("0.527", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpHtUserDesAirFlow, "SPACE 1"));
Expand All @@ -4245,7 +4250,7 @@ TEST_F(EnergyPlusFixture, SizingManager_ZoneSizing_Coincident_NonAir_10x_Latent_
EXPECT_EQ("0.669", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpClUserDesAirFlow, "SPACE 1"));
EXPECT_EQ("CHICAGO_IL_USA ANNUAL COOLING 1% DESIGN CONDITIONS DB/MCWB",
OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpClDesDay, "SPACE 1"));
EXPECT_EQ("7/21 10:20:00", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpClPkTime, "SPACE 1"));
EXPECT_EQ("7/21 12:00:00", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpClPkTime, "SPACE 1"));

EXPECT_EQ("6311.95", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpHtCalcDesLd, "SPACE 2"));
EXPECT_EQ("0.175", OutputReportPredefined::RetrievePreDefTableEntry(*state, state->dataOutRptPredefined->pdchSpHtCalcDesAirFlow, "SPACE 2"));
Expand Down

4 comments on commit c2616e6

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

SpacePart4Followup (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (2900 of 2901 tests passed, 0 test warnings)

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1591
  • Failed: 1

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.

SpacePart4Followup (mjwitte) - Win64-Windows-10-VisualStudio-16: OK (2879 of 2879 tests passed, 0 test warnings)

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.

SpacePart4Followup (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-RelWithDebInfo: OK (799 of 799 tests passed, 0 test warnings)

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.

SpacePart4Followup (mjwitte) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-RelWithDebInfo: OK (2084 of 2085 tests passed, 0 test warnings)

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1591
  • Failed: 1

Build Badge Test Badge Coverage Badge

Please sign in to comment.