Skip to content

Commit

Permalink
10299 Move a PixelCounting Check
Browse files Browse the repository at this point in the history
An error message was being skipped because of the movement of the Get part of the old routine.  anyScheduledShadingSurface was not set yet so the error message was never produced.  This change should fix this problem and eliminate the change in the .err file for SolarShadingTestGPU.idf.
  • Loading branch information
RKStrand committed Jul 22, 2024
1 parent ce283a3 commit c688048
Showing 1 changed file with 12 additions and 12 deletions.
24 changes: 12 additions & 12 deletions src/EnergyPlus/SolarShading.cc
Original file line number Diff line number Diff line change
Expand Up @@ -519,18 +519,6 @@ void GetShadowingInput(EnergyPlusData &state)
state.dataSysVars->shadingMethod = ShadingMethod::PolygonClipping;
}

if ((state.dataSysVars->shadingMethod == DataSystemVariables::ShadingMethod::PixelCounting) &&
state.dataSolarShading->anyScheduledShadingSurface) {
ShowSevereError(state, "The Shading Calculation Method of choice is \"PixelCounting\"; ");
ShowContinueError(state, "and there is at least one shading surface of type ");
ShowContinueError(state, "Shading:Site:Detailed, Shading:Building:Detailed, or Shading:Zone:Detailed, ");
ShowContinueError(state, "that has an active transmittance schedule value greater than zero or may vary.");
ShowContinueError(state, "With \"PixelCounting\" Shading Calculation Method, the shading surfaces will be treated as ");
ShowContinueError(state, "completely opaque (transmittance = 0) during the shading calculation, ");
ShowContinueError(state, "which may result in inaccurate or unexpected results.");
ShowContinueError(state, "It is suggested switching to another Shading Calculation Method, such as \"PolygonClipping\".");
}

aNum++;
if (NumAlphas >= aNum) {
if (Util::SameString(state.dataIPShortCut->cAlphaArgs(aNum), "Periodic")) {
Expand Down Expand Up @@ -738,6 +726,18 @@ void processShadowingInput(EnergyPlusData &state)
{
// all shadow input processing that needed zones and surfaces to already be read into data (part of fix for Defect #10299)

if ((state.dataSysVars->shadingMethod == DataSystemVariables::ShadingMethod::PixelCounting) &&
state.dataSolarShading->anyScheduledShadingSurface) {
ShowSevereError(state, "The Shading Calculation Method of choice is \"PixelCounting\"; ");
ShowContinueError(state, "and there is at least one shading surface of type ");
ShowContinueError(state, "Shading:Site:Detailed, Shading:Building:Detailed, or Shading:Zone:Detailed, ");
ShowContinueError(state, "that has an active transmittance schedule value greater than zero or may vary.");
ShowContinueError(state, "With \"PixelCounting\" Shading Calculation Method, the shading surfaces will be treated as ");
ShowContinueError(state, "completely opaque (transmittance = 0) during the shading calculation, ");
ShowContinueError(state, "which may result in inaccurate or unexpected results.");
ShowContinueError(state, "It is suggested switching to another Shading Calculation Method, such as \"PolygonClipping\".");
}

if (state.dataSysVars->shadingMethod == DataSystemVariables::ShadingMethod::Imported) {
int ExtShadingSchedNum;
for (int SurfNum = 1; SurfNum <= state.dataSurface->TotSurfaces; ++SurfNum) {
Expand Down

5 comments on commit c688048

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

10299CheckConvexityChangeCheckForShadowCalculations (RKStrand) - x86_64-MacOS-10.18-clang-15.0.0: OK (3649 of 3650 tests passed, 787 test warnings)

Messages:\n

  • 787 tests had: EIO diffs.

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1572
  • 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.

10299CheckConvexityChangeCheckForShadowCalculations (RKStrand) - Win64-Windows-10-VisualStudio-16: OK (2854 of 2858 tests passed, 0 test warnings)

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1567
  • Failed: 4

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.

10299CheckConvexityChangeCheckForShadowCalculations (RKStrand) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3690 of 3691 tests passed, 791 test warnings)

Messages:\n

  • 791 tests had: EIO diffs.

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1572
  • Failed: 1

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.

10299CheckConvexityChangeCheckForShadowCalculations (RKStrand) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (2063 of 2066 tests passed, 0 test warnings)

Failures:\n

API Test Summary

  • Passed: 13
  • NUMERICAL: 2

EnergyPlusFixture Test Summary

  • Passed: 1572
  • Failed: 1

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.

10299CheckConvexityChangeCheckForShadowCalculations (RKStrand) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (795 of 795 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.