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

Adjust for capacity #3

Open
Tracked by #1
peterdudfield opened this issue Nov 25, 2023 · 4 comments
Open
Tracked by #1

Adjust for capacity #3

peterdudfield opened this issue Nov 25, 2023 · 4 comments

Comments

@peterdudfield
Copy link
Contributor

peterdudfield commented Nov 25, 2023

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.

This was referenced Nov 25, 2023
@0xFrama
Copy link
Contributor

0xFrama commented Feb 28, 2024

Hi @peterdudfield, I would like to work on this.
Can you be more specific on what should get done? Thank you!

@aryanbhosale
Copy link
Contributor

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?

@peterdudfield
Copy link
Contributor Author

I would try to make a seperate PR, then things can be reviews and merged as they need to.
Thank you for doing this

@aryanbhosale
Copy link
Contributor

I would try to make a seperate PR, then things can be reviews and merged as they need to. Thank you for doing this

Okay that works, thank you

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