-
Notifications
You must be signed in to change notification settings - Fork 1.1k
add spectrum.spectral_factor example #2107
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
Comments
Spectral data can also be created synthetically with a spectral model, like |
@echedey-ls thanks for your input |
That's perfect. I thought you wanted to compare the spectral mismatch of some model with the specific mismatch of a technology. |
If the data source is to provide a "ground truth" SMM value, I'd say no need. It is enough to demonstrate the model's use; validating the models against real data is out of scope IMHO. If the data source is for model inputs, I'd prefer something like the NSRDB over ground station data. It could be that pvlib already has a PSM3 or TMY file with the necessary inputs in |
@kandersolar The purpose is for model inputs rather than providing a "ground truth" SMM value. Thanks for the recommendation, I'll take a look! |
Is your feature request related to a problem? Please describe.
There is currently no example in the example gallery demonstrating the use/application of the
spectrum.spectral_factor
models.Describe the solution you'd like
I am currently working on an example that demonstrates the calculation of spectral mismatch using multiple spectral factor functions.
Additional context
Related issue: #2065
I am away for a short while so won't have the PR ready this week, so in the meantime I wanted to open this issue in case anyone has any feedback or thoughts about anything in particular they'd like to see in such an example that they feel would be beneficial.
I am still deciding a few things, such as analysis duration and data source (currently thinking one day and NREL MIDC --- any other suggestions welcome) but will create a PR once I have something concrete.
The text was updated successfully, but these errors were encountered: