satpy icon indicating copy to clipboard operation
satpy copied to clipboard

Add OSI-SAF reader

Open gerritholl opened this issue 2 years ago • 3 comments

Feature Request

Is your feature request related to a problem? Please describe.

We are currently using an unmaintained custom reader for OSI-SAF ice maps to produce NinJoTIFF images for our users.

Describe the solution you'd like

We would like to integrate this reader into Satpy.

Describe any changes to existing user workflow

No changes to existing user workflow foreseen.

Developers would have one more reader to adapt the next times Satpy fundamentals change.

Additional context

The alternative would be to keep the status quo (custom reader in DWD only). Advantage: less maintenance for Satpy developers. Disadvantage: more maintenance for DWD (+ partners) and lack of opportunity for others to use the reader.

gerritholl avatar Dec 27 '21 13:12 gerritholl

I was actually considering the need for an OSI-SAF reader and had it filed under "whenever I have time" on my to-do list. Happy to lend a hand if you do decide to implement this in Satpy.

simonrp84 avatar Dec 27 '21 14:12 simonrp84

There is some starting point at https://github.com/ch-k/satpy/blob/feature-osisaf-iceconc-nc-reader/satpy/readers/osisaf_conc_eumetcast_nc.py (I believe the author there is a copy-paste error, to be clarified), but lacking unit tests and in need of some cleanup.

gerritholl avatar Dec 27 '21 14:12 gerritholl

The file osisaf_conc_eumetcast_nc.py is based on ghrsst_l3c_sst.py. It's copy-paste with some adjustments. It was enough to supply the Polarstern ship on its mission to the north in 2019. But for sure, there is some cwork to do.

ch-k avatar Dec 29 '21 13:12 ch-k