-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update default solar constant #1566
Comments
|
I've been frustrated by Regarding default values: if a model was tuned for a specific value of the solar constant then I'm not sure it makes sense to go against its reference by changing the default. |
That function is a convenience wrapper for four extraterrestrial irradiance models, which all differ in how the earth-Sun distance is computed. Two of the models also specify a solar constant: 'asce' uses 1367.7 and 'spencer' uses 1366.1. 'pyephem' refers to the ephem python package, which doesn't contain a solar constant that I have found, and 'nrel' refers to SPA and there is no specified solar constant for either. So we could make the solar constant an input to this function, but I'd want to specify the values for 'asce' and 'spencer'. |
@cwhanse The constant and the method to calculate sun-earth distance are independent of each other. The constant probably only matters for the clear-sky models, where the best value to use may be the one used at the time the model was developed. Plopping in the new value everywhere would probably create a bit of a bias. |
I agree, I'm thinking to maintain consistency with the underlying reference for those two extraterrestrial irradiance models which specify a constant. |
The default solar constant used in
irradiance.get_extra_radiation
is currently 1366.1 W/m².Recent work revised the average solar constant to 1361.1 W/m² (see https://www.researchgate.net/publication/324269776_A_reevaluation_of_the_solar_constant_based_on_a_42-year_total_solar_irradiance_time_series_and_a_reconciliation_of_spaceborne_observations).
This new value seems now to be gaining broad acceptance, a.o. in certain standards.
Maybe it's a good time to update also the default value in pvlib to 1361.1 W/m².
The text was updated successfully, but these errors were encountered: