search

NREL / EnergyPlus

EnergyPlus™ is a whole building energy simulation program that engineers, architects, and researchers use to model both energy consumption and water use in buildings.
https://energyplus.net
Other
1.15k stars 392 forks source link

ShadingProperty:Reflectance internal case issue #4508

Open Myoldmopar opened 10 years ago

Myoldmopar commented 10 years ago

From: @lklawrie email:

The problem, as I will relate it to the team, is that the Shading:Fin creates these surfaces with a special suffix in Upper/lower case.  And the module getting the ShadingProperty:Reflectance is not using the "ignore case" search on the surface names.

problem is in surface geometry line 6097.  a quick fix would be to use "FindItem" rather than FindItemInList
    SurfNum = FindItemInList(cAlphaArgs(1),SurfaceTmp%Name,TotSurfaces)
Just thought of a possible different workaround for this which I will check out tomorrow. Put \retaincase on the shading property reflectance name field in the idd

The IDD doesn't show this \retaincase workaround, and the source line doesn't seem to reflect any correction, so I think this is still an issue.

Defect file to be added to DevSupport.

mjwitte commented 9 years ago

@Myoldmopar I don't see a defect file for this. Do you have one?

Myoldmopar commented 9 years ago

I'll try to do the source fix approach.