-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Adjust for capacity #3
Comments
Hi @peterdudfield, I would like to work on this. |
Hi @peterdudfield , I have figured out a way to adjust all sites to 1 kwp and the times predictions by these results. I'm not sure if I should open a PR with the existing code in data.py, or update my PR here in #66 . Could you help me out here? |
I would try to make a seperate PR, then things can be reviews and merged as they need to. |
Okay that works, thank you |
Adjust all sites to 1 kwp and the times predictions by these results
Need to adjust site here to 1 kwp and then times prediction here
There is some argument to letting the model use this information and 10 MW sites will behaviour differently to 1KW as they will be more spread out geograhpically. However, I think until we trained with more examples like that, its worth having the simpliest of all scaling to 1KW sites.
The text was updated successfully, but these errors were encountered: