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

Soiling detection methods #184

Open
cwhanse opened this issue Feb 1, 2023 · 4 comments
Open

Soiling detection methods #184

cwhanse opened this issue Feb 1, 2023 · 4 comments

Comments

@cwhanse
Copy link
Member

cwhanse commented Feb 1, 2023

Here's an interesting approach: https://doi.org/10.1016/j.isci.2021.102165

@cwhanse
Copy link
Member Author

cwhanse commented Apr 24, 2023

@abhisheksparikh
Copy link

Here's an interesting approach: https://doi.org/10.1016/j.isci.2021.102165

Since we have the soiling module in pvlib, should this be in pvlib-python? Also, the SRR method's code has a github repository - I wonder if Michael would be interested in getting it in pvanalytics (or pvlib).

@kandersolar
Copy link
Member

Since we have the soiling module in pvlib, should this be in pvlib-python?

IMHO: methods of predicting soiling ratio for performance modeling purposes belong in pvlib-python and methods of detecting and/or extracting soiling from measured performance belong elsewhere (perhaps here in pvanalytics).

Also, the SRR method's code has a github repository - I wonder if Michael would be interested in getting it in pvanalytics (or pvlib).

Maybe you are talking about https://github.com/NREL/pv_soiling? That repository is not actively developed, but SRR is also implemented in RdTools: RTD and GH. Since there's already a maintained (and better) version elsewhere, I don't see much value in having a duplicate implementation of it here. And I very much doubt Mike et al. would be interested in transferring the "official" version from RdTools to pvanalytics.

@abhisheksparikh
Copy link

Since we have the soiling module in pvlib, should this be in pvlib-python?

IMHO: methods of predicting soiling ratio for performance modeling purposes belong in pvlib-python and methods of detecting and/or extracting soiling from measured performance belong elsewhere (perhaps here in pvanalytics).

Yes, I agree - but then, what method should we use/explore to detect soiling from the performance data? I am referring to Table 6. of https://doi.org/10.1016/j.isci.2021.102165.

Also, the SRR method's code has a github repository - I wonder if Michael would be interested in getting it in pvanalytics (or pvlib).

Maybe you are talking about https://github.com/NREL/pv_soiling? That repository is not actively developed, but SRR is also implemented in RdTools: RTD and GH. Since there's already a maintained (and better) version elsewhere, I don't see much value in having a duplicate implementation of it here. And I very much doubt Mike et al. would be interested in transferring the "official" version from RdTools to pvanalytics.

Okay, that makes sense.

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

3 participants