-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add irradiance QC algorithm from Forstinger et al. #110
Comments
cc @AdamRJensen as he's on the author list |
I uploaded the code here just to have a stable link for the paper. Ideally it would be contributed to a library, so I would be happy to contribute it here. We still have a few improvements of the multiplot that we're working on finalizing. |
Nice plot! |
Another paper describing a QC method to concurrent GHI and DHI. |
@YvesMSaintDrenan Would you be willing to add the multi-plot function to pvanalytics? I know that it's part of libinsitu, but I still think it would be a valuable contribution to pvanalytics. I'd also be happy to do the implementation, if you agree. |
I am all for it ! Just before you start please contact Raphaël who is currently updating the code to implement plotly (we want to make it possible to zoom and annotate the plots). In addition, we would like to implement the last tests of Nollas et al. (https://doi.org/10.1016/j.renene.2022.11.056) in the visual inspection. |
@YvesMSaintDrenan I imagine that as a first step, we might just want to add the basic plotting function, which takes a boolean series of flags, arrays of GHI, GHI_clear, DNI, and DHI, and solar position. How people calculate the flags and the remaining inputs can then be up to them as a first step. This would simplify this initial contribution significantly and make it rather flexible. A Class that can make it more user-friendly, e.g., calculate flags and retrieve CAMS can then be added secondarily. Would you be ok with that? @cwhanse @kanderso-nrel Are you okay with adding plotly as a dependency to the project? |
Some reasons I hesitate to include plotting functionality in pvanalytics:
As an alternative to including plotting functionality in pvanalytics itself, how about implementing them in gallery examples? Note that plotly can be used with sphinx-gallery (see docs). |
Ill-considered idea: create a new package under the pvlib organization (pvlib/pvplotting?) that embraces the junk drawer mentality and has a low barrier to entry. |
That is a very cool plot! My 2 cents along the lines of everyone has their favorite plotting package :) - the [holoviews] (https://holoviews.org/index.html) library would allow specifying the plot without specifying the plotting library. I'm not sure if the full figure would reliable work in each of the possible plotting backends (matplotlib, bokeh, and plotly), but might be something to investigate. I always use Bokeh as the backend when I'm using holoviews or hvplot. Regardless, it is a cool plot, and it would be awesome to have access to a function somewhere to generate it, especially an interactive version! |
Supporting both pvanalytics and pvlib-python? I could support that if we come up with an organizing structure. Otherwise, an example gallery makes sense to me. |
Status of QC tests integration with pvanalytics
Additionally, we should implement
|
Conference paper here. Looks to be an improvement over Long and Shi.
The text was updated successfully, but these errors were encountered: