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

Change how model repos are named #66

Open
jacobbieker opened this issue Apr 10, 2024 · 4 comments
Open

Change how model repos are named #66

jacobbieker opened this issue Apr 10, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@jacobbieker
Copy link
Member

Currently, our model repos on Hugging Face are named in maybe confusing ways. pvnet_v2 is the UK regional forecast model, for example. I propose a potentially clearer, more expandable way of naming our models and differentiating them:

<model_type>_<area>_<national/region/site>
i.e. pvnet_v2 would become pvnet_uk_region
PVNet for sites would be pvnet_uk_site
pvnet_india -> pvnet_india_region
windnet_india -> windnet_india_region
etc.
Summation model would be
pvnet_uk_summation

@jacobbieker jacobbieker added the enhancement New feature or request label Apr 10, 2024
@jacobbieker
Copy link
Member Author

@peterdudfield @dfulu @zakwatts does this make sense? Might make it clearer what our models are doing

@peterdudfield
Copy link
Contributor

I think that would be sensible to do, if you change the repo name will the production system break?

@jacobbieker
Copy link
Member Author

No, as we'll update them when we push a new model, so it doesn't make a difference then, we can leave the current one on pvnet2 just new ones would be on a different repo

@zakwatts
Copy link
Contributor

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants