NatLabRockies / NatLabRockies/EnergyPlus

Reflective shading surfaces not sensitive to transmittance schedule

Open
#4,579 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

PriorityLow SeverityMedium
Dominant language
C++
Stars
1.6k
Forks
490
Avg merge
6d 21h
Merged PRs (30d)
22

Description

Helpdesk ticket 9696
User compares file with reflective shading surface with transmittance schedule always=1.0 to the same file with the shading surface removed. Results should be the same, but the case with the fully transmitting surface shows non-zero reflected solar.

Test files:
Base-ddays-v8.2.idf - with the surface, using ShadowCalculation=AverageOverDaysInFrequency
Evo-ddays-v8.2.idf - without the surface, AverageOverDaysInFrequency
Base-ddays-timestepshading-v8.2.idf - with the surface, TimestepFrequency
Evo-ddays-timestepshading-v8.2.idf - without the surface, TimestepFrequency

The two timesteshading files have a varying transmittance schedule, so the reflected solar should come and go in the Base case, and always be zero in the Evo case. Note that these cases currently crash in v8.2.0 - ref issue #4578

Relevant output variables:
BLOC1:ZONE1_WALL_2_0_0:Surface Outside Face Incident Sky Diffuse Surface Reflected Solar Radiation Rate per Area
BLOC1:ZONE1_WALL_2_0_0:Surface Outside Face Incident Beam To Beam Surface Reflected Solar Radiation Rate per Area
BLOC1:ZONE1_WALL_2_0_0:Surface Outside Face Incident Beam To Diffuse Surface Reflected Solar Radiation Rate per Area

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the behavior with Base-ddays-v8.2.idf, Evo-ddays-v8.2.idf, Base-ddays-timestepshading-v8.2.idf, and Evo-ddays-timestepshading-v8.2.idf, accounting for the crash tracked in issue #4578. Compare the listed reflected-solar output variables; done means a fully transmitting reflective surface matches the case without the surface and timestep-scheduled reflection changes as described.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.