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

The Perez diffuse model should not be forcing the horizon coefficient up to zero #1238

Closed
cpr-chas opened this issue May 21, 2021 · 2 comments · Fixed by #1239
Closed

The Perez diffuse model should not be forcing the horizon coefficient up to zero #1238

cpr-chas opened this issue May 21, 2021 · 2 comments · Fixed by #1239

Comments

@cpr-chas
Copy link
Contributor

cpr-chas commented May 21, 2021

The perez model in irradiance.py forces F2, and thus the horizon component of diffuse, to be non-negative. This restriction should not happen. F2 and the horizon coefficient should be allowed to be negative and to reduce overall diffuse.

According to the original paper at https://www.osti.gov/servlets/purl/7024029
Section III.2 states this explicitly for the horizon component:
"(2) The horizon brightening coefficient, F2, is negative for overcast and low E occurrences -- indicative of brightening of the zenithal region of the sky for these conditions. This becomes positive past intermediate conditions and increases substantially with clearness."

We observed a higher than expected POAI, coming from poa diffuse, on cloudy days at certain sites.
Expected:
Horizon (burgundy) can be less than zero and sky diffuse (green) is less than isotropic (blue)
image

Observed from PVLib:
Horizon is prevented from being negative and sky diffuse ends up higher than isotropic.
image

Repro'd on PVLib 0.8.1

See added test case in the PR for this repro case.

@cwhanse
Copy link
Member

cwhanse commented May 21, 2021

It appears that the equivalent of this line was in the original port from Matlab to python. In the Matlab function F2 is not bounded below by 0. It seems likely that this is a cut-and-paste mistake in the original port. I will check the later references (1, 2 and 3) and see if there is any reversion of that statement about F2 in the 1988 reference.

@cwhanse
Copy link
Member

cwhanse commented May 21, 2021

I agree with the issue; F2 >= 0 was never a condition included in the Perez model. The references are consistent that F2 < 0 was ia possibility intentionally included. From [3], section 3.2.1 "The negative value for F2 is traceable to a relative brightening of the zenithal region for overcast conditions the physical nature of which is well understood..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants