Peter Dudfield

Results 286 comments of Peter Dudfield

Update solar limit to 5000 (from 4000)

Yea I know what you mean. I thought it would be an empty space of padding, but it looks like its padding on the Yaxis, so its maybe a bit...

Store when Download data

@braddf any thoughts on this?

Loading model using DGMR.from_pretrained("openclimatefix/dgmr") fails

Thanks @agijsberts for finding this. Should we pin the requirements file to `huggingface-hub==0.21.4`. Would you be interested in making a PR for this? Would you be able to expand further...

Loading model using DGMR.from_pretrained("openclimatefix/dgmr") fails

Thanks @agijsberts, yea if you were able to have a look at the `DGMR.__init__` and how the best way to resolve this is, then that would be super

Loading model using DGMR.from_pretrained("openclimatefix/dgmr") fails

Thanks @agijsberts - have a good holiday, and I look foreward to seeing the PR If Im honest Im not totlaly sure why `NowcastingModelHubMixin` is used instead of `PyTorchModelHubMixin`, but...

National Capacity

Maybe linked with #472

National Capacity

In general I would keep PR seperate, as they will take longer to development and check, the more features you have. @braddf might be able to advise here, potentially there...

Changed folder structure

#120

Changed folder structure

Ah sorry, I was just thinking refactroing with in the main satip folder

Changed folder structure

Thanks @peach280 for this. Just to let you know, OCF are no longer supporting this repo, as we have moved over to https://github.com/openclimatefix/satellite-consumer Therefore I will close this PR