irradiance.detect_clearsky: possibly redundant input 'times'
See original GitHub issueirradiance.detect_clearsky expects measured and clear-sky GHI to be input as Series, and has an input times as a DatetimeIndex.
https://github.com/pvlib/pvlib-python/blob/85eeaa795e82bb486bc58488ac70269da8c72be0/pvlib/clearsky.py#L629
What is the use case for having times as a separate input? Should the function instead use measured.index?
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (5 by maintainers)
Top Results From Across the Web
Clear sky — pvlib python 0.9.0+0.g518cc35.dirty documentation
An iterative algorithm identifies clear periods, uses the identified periods to estimate bias in the clearsky data, scales the clearsky data and repeats....
Read more >Clear-sky irradiance predictions for solar resource mapping ...
Clear-sky irradiance predictions for solar resource mapping and large-scale applications: Improved validation methodology and detailed performance analysis ...
Read more >Field measurement of clear-sky solar irradiance in Badain ...
specification of input parameters and better measure- ments of irradiances. Wang et al. [14] also achieved an excellent clear-sky shortwave ...
Read more >Validation of models that estimate the clear sky global and ...
... A clear-sky model estimates the solar irradiance that reaches the ground surface when the sky is clear without clouds. Such models were ......
Read more >Statistical Criteria for Characterizing Irradiance Time Series
Time series of irradiance are used in analyses that seek to quantify the ... analysis, the analysis inputs representing the solar resource should...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found

Fine with me to keep it open in an effort to simplify API or to close in an effort to clean the issues board. Either way not going into 0.9, so I removed the milestone.
#1074 changed
timesfrom an arg to a kwarg with defaultNone. Bothmeasuredandclearskyarguments can still bearray. I think it does no harm to keeptimesfor compatibility with any legacy code.Recommend closing this issue.