docs: Add guide for training models for new countries#125
Conversation
Create comprehensive step-by-step guide covering Add link to guide in README.md Resolves openclimatefix#113
|
I've added a guide for training models on new countries (issue #113). Included all the steps and added country selection guidance based on your feedback. Checked the config examples against the actual codebase to make sure everything matches. Happy to make any changes if needed! |
peterdudfield
left a comment
There was a problem hiding this comment.
This looks really good. @siddharth7113 do you want to check this?
Sure, I'll take a look at this either today or tommorow. Thanks @mahendra-918 for doing this. |
peterdudfield
left a comment
There was a problem hiding this comment.
Ive just left one change, would you be able to do that @mahendra-918 and then we can get this merged
|
@peterdudfield Thanks for pointing that out! I've updated the code example to use location_id instead of gsp_id and added the link to ocf-data-sampler as requested |
| time_resolution_minutes: 30 | ||
| ``` | ||
|
|
||
| ### 3.2 Calculate Normalization Constants |
There was a problem hiding this comment.
@peterdudfield Do we need to do this for each new country?
There was a problem hiding this comment.
I think we do, we can probably use some global constant instead, but it might make the models better if we use local constants, perhaps something to test out
|
Hi @mahendra-918 I have left some comments on the PR, sorry it took long, after some minor tweaks this can be good to merge. |
|
Thanks @siddharth7113 I've addressed all the feedback in the latest commit: Documentation Cleanup: Removed the untested 'Complete Workflow' section and the manual 'GFS Download' instructions. |
|
Hii @peterdudfield @siddharth7113 just checking in to see if you have any further thoughts on the latest documentation updates. Ready to merge whenever you are! |
|
Lets get it merged, and we can always add a few more bits here and there |
|
Thanks for the support and the reviews, @peterdudfield Happy to have this merged |
Pull Request
Description
This PR adds a guide for training models on new countries, which addresses issue #113. I've created a step-by-step guide that covers all the points mentioned in the issue.
The guide walks through:
I also added a section on choosing countries based on the feedback about focusing on countries with large solar installations. I included specific data sources for countries like USA, Netherlands, Belgium, Germany, and France since those were mentioned as good starting points.
Fixes #113
Changes Made
docs/training_model_new_country.md- the main guide (about 600 lines)README.md- added a link to the new guide in the documentation sectionHow Has This Been Tested?
I went through the guide and checked:
I also ran the linter to make sure there are no formatting issues.
No data processing changes, so no plotting needed.
Checklist: