Skip to content

Commit

Permalink
Removed unused variable warning
Browse files Browse the repository at this point in the history
  • Loading branch information
Nigusse committed Jun 26, 2023
1 parent 2f4db21 commit 7a3026a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 2 additions & 0 deletions src/EnergyPlus/ConvectionCoefficients.cc
Original file line number Diff line number Diff line change
Expand Up @@ -433,13 +433,15 @@ void InitExtConvCoeff(EnergyPlusData &state,
Real64 TSurf = TempExt + Constant::KelvinConv;
Real64 TSky = state.dataEnvrn->SkyTempKelvin;
Real64 TGround = TAir;
HSrdSurf = 0.0;

if (surface.SurfHasSurroundingSurfProperty) {
int SrdSurfsNum = surface.SurfSurroundingSurfacesNum;
if (state.dataSurface->SurroundingSurfsProperty(SrdSurfsNum).SkyTempSchNum != 0) {
TSky = ScheduleManager::GetCurrentScheduleValue(state, state.dataSurface->SurroundingSurfsProperty(SrdSurfsNum).SkyTempSchNum) +
Constant::KelvinConv;
}
HSrdSurf = SurroundingSurfacesRadCoeffAverage(state, SurfNum, TSurf, AbsExt);
}
if (surface.UseSurfPropertyGndSurfTemp) {
int gndSurfsNum = surface.SurfPropertyGndSurfIndex;
Expand Down
1 change: 0 additions & 1 deletion src/EnergyPlus/TranspiredCollector.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1860,7 +1860,6 @@ namespace TranspiredCollector {
HPlenARR.deallocate();
Real64 HExt = sum(HExtARR * Area) / A; // dummy for call to InitExteriorConvectionCoeff
HExtARR.deallocate();
Real64 HSrdSurfExt = sum(HSrdSurfARR * Area) / A; // radiation coeff for surrounding surfaces, not used now
HSrdSurfARR.deallocate();

if (state.dataEnvrn->IsRain) HExt = 1000.0;
Expand Down

5 comments on commit 7a3026a

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

Issue9880_Linearize_ExtSurf_LWR_Exchange (Nigusse) - Win64-Windows-10-VisualStudio-16: OK (2666 of 2666 tests passed, 0 test warnings)

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

Issue9880_Linearize_ExtSurf_LWR_Exchange (Nigusse) - x86_64-MacOS-10.17-clang-13.0.0: OK (3437 of 3439 tests passed, 0 test warnings)

Messages:\n

  • 2 tests had: EIO diffs.
  • 2 tests had: ESO big diffs.
  • 2 tests had: Table small diffs.

Failures:\n

regression Test Summary

  • Passed: 770
  • Failed: 2

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.

Issue9880_Linearize_ExtSurf_LWR_Exchange (Nigusse) - x86_64-Linux-Ubuntu-22.04-gcc-11.3: OK (3478 of 3480 tests passed, 0 test warnings)

Messages:\n

  • 2 tests had: EIO diffs.
  • 2 tests had: ESO big diffs.
  • 2 tests had: Table small diffs.

Failures:\n

regression Test Summary

  • Passed: 790
  • Failed: 2

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.

Issue9880_Linearize_ExtSurf_LWR_Exchange (Nigusse) - x86_64-Linux-Ubuntu-22.04-gcc-11.3-UnitTestsCoverage-Debug: OK (1893 of 1893 tests passed, 0 test warnings)

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.

Issue9880_Linearize_ExtSurf_LWR_Exchange (Nigusse) - x86_64-Linux-Ubuntu-22.04-gcc-11.3-IntegrationCoverage-Debug: OK (776 of 776 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.