openclimatefix / uk-pvnet-app

Application for running PVNet in production
MIT License
1 stars 4 forks source link

Change how model repos are named #66

Closed jacobbieker closed 3 months ago

jacobbieker commented 7 months ago

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:

\_\\_\ 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 commented 7 months ago

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

peterdudfield commented 7 months ago

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

jacobbieker commented 7 months ago

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 commented 7 months ago

Looks good to me.

peterdudfield commented 3 months ago

Ill close this, as we have renamed them in HF