Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gendaylit - normRGB #506

Closed
Funerus opened this issue Mar 28, 2024 · 2 comments
Closed

gendaylit - normRGB #506

Funerus opened this issue Mar 28, 2024 · 2 comments

Comments

@Funerus
Copy link

Funerus commented Mar 28, 2024

Hey everyone,

I've been going through the gendaylit command as defined within the main code of bifacial_radiance.

Specifically, when defining the ground glow for each wavelength channel, the reflectance of each channel is normalized with normval = R*.263 + G*.655 + B.082 leading to:
skyfunc glow ground_glow
0
0
4 R/normval G/normval B/normval 0

Since I am interested in solar irradiance and not illuminance, do I still need to normalize each RGB reflectance value?

Regardless, by default the sky is defined to be grey, so I would assume that normalizing ground reflectance is unnecessary?

Any insights are highly appreciated,
Cheers,
Alex

@cdeline
Copy link
Contributor

cdeline commented Aug 23, 2024

Hi Alex (sorry for the delay)

I'm sorry that we're not going to be the best ones to give you an authoritative answer on this question. I would refer to the gendaylit man pages where this topic is discussed. I believe that the treatment of irradiance in this way is self-consistent because the definition of the sky brightness is in Wm-2, and the measurement of incident irradiance is in Wm-2. My recollection was that defining the ground glow based on this default illuminance value will allow calculation back to Wm-2 using Radiance's default white light efficacy of 179 lm/W.

@cdeline
Copy link
Contributor

cdeline commented Sep 17, 2024

I'm going to close this issue as resolved - please reply back if it should be reopened.

@cdeline cdeline closed this as completed Sep 17, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants