Peter Dudfield

PR: https://github.com/openclimatefix/predict_pv_yield/pull/73

https://github.com/openclimatefix/predict_pv_yield/blob/issue/85-perciever-nwp/predict_pv_yield/models/perceiver/perceiver_conv3d_nwp_sat.py This model does now

I dont have a direct comparision to running it without the cnn. results: https://github.com/openclimatefix/predict_pv_yield/blob/issue/85-perciever-nwp/experiments/2021-09/2021-09-28/experiments.txt

yea, I think it would be a useful to do. Could also see if 01, ... N layers of CNN are useful?

code is ready- just need to run the model

This has been run - did not make much difference. Need to run it twice, (with and without) to provide some numbers

Hi @mfrasco, thanks for getting involved. @JackKelly yes we are still planning to use pydocstyle. This has been implemented for [Nowcasting_dataset](https://github.com/openclimatefix/nowcasting_dataset) and [Nowcasting_utils](https://github.com/openclimatefix/nowcasting_utils). And yes 'predict_pv_yield' does need updating It...

Thanks very much @mfrasco

Maybe be linked with this - https://github.com/openclimatefix/nowcasting_dataset/issues/610 Obviously both data sets cant cover exactly the same, so just have to decide does topology > satellite, or satellite > topology

I can see why that would be convenient - It potentially goes against our principle of 'module' data sources which so far has been useful. One solution could be to...