openclimatefix / Satip

Satip contains the code necessary for retrieving, transforming and storing EUMETSAT data
https://satip.readthedocs.io/
MIT License
41 stars 28 forks source link

Spatially reproject data in data conversion script #15

Open JackKelly opened 3 years ago

JackKelly commented 3 years ago

Detailed Description

The EUMETSAT data is in geostationary projection. We probably want it in transverse Mercator.

Related

jacobbieker commented 3 years ago

SatFlow also did it here: https://github.com/openclimatefix/satflow/blob/ad5d6f6c419994835977cb75322f86b171d6a341/satflow/data/utils/utils.py#L48-L96

JackKelly commented 2 years ago

On second thoughts, maybe we shouldn't spatially reproject the satellite imagery?!?

My thinking is:

The downsides I can think of are:

What do you guys think, @jacobbieker and @peterdudfield

peterdudfield commented 2 years ago

seperate CNNs is fine - lob/master/predict_pv_yield/models/conv3d/conv3d_sat_nwp.png

Does the reporjection helpe with visualisation? Or maybe that reprojection could be done for visualization

jacobbieker commented 2 years ago

Yeah, I can see that not reprojecting it could be better, especially if we end up using other derived EUMETSAT data, like cloud masks or atmospheric instability, keeping everything in the original format makes it a lot easier and yeah, removes artifacts. I'd still want to see if reprojection does do worse or not, but yeah, that could wait. It would help with visualization, but yeah, we can just reproject for that

JackKelly commented 2 years ago

Excellent, sounds like a plan! I won't bother reprojecting for now, and we can see how we get on...