ascat icon indicating copy to clipboard operation
ascat copied to clipboard

HSAF SMDAS SWI

Open wpreimes opened this issue 4 years ago • 9 comments

Do you want to include the SMDAS data reader in this package? https://github.com/wpreimes/ascat/blob/smdas_reader/src/ascat/smdas.py

Then I'll fix some issues, clean up, write tests etc for it.

The data itself is a bit awkward, corrupt files, missing variables, deviating time stamps in metadata and filename etc. Therefore the reader is also a bit messy...

wpreimes avatar Sep 21 '20 14:09 wpreimes

Which products do you mean by SMDAS exactly?

sebhahn avatar Sep 21 '20 15:09 sebhahn

H-SAF SM-DAS-2 H-14 root zone soil moisture index product, based on ASCAT surface soil moisture data assimilation in the ECMWF Land Data Assimilation System

wpreimes avatar Sep 21 '20 15:09 wpreimes

Yes it makes sense to add a reader, but sooner or later it this should be moved to its own package here https://github.com/H-SAF

sebhahn avatar Sep 21 '20 15:09 sebhahn

Same for all other H SAF SM products

sebhahn avatar Sep 21 '20 15:09 sebhahn

If you want I can make a new package for the SMDAS data now already. Just tell me what you prefer.

wpreimes avatar Sep 21 '20 15:09 wpreimes

Probably it makes sense to start with a new package, but I wouldn't call it SMDAS.

However, first I need to think about how to structure the packages in H SAF (package per product, package per variable, ...). What would you prefer as a user?

sebhahn avatar Sep 21 '20 15:09 sebhahn

@davidfairbairn do you have an opinion how to structure the python packages on H SAF?

sebhahn avatar Sep 21 '20 15:09 sebhahn

From a user perspective it would be nice to have everything in one place. But from a developer point of view I think a package per product makes more sense than a package per variable, because of different file types, dependencies, resolutions etc. So maybe multiple smaller readers and one toolbox to collect them?

wpreimes avatar Sep 21 '20 15:09 wpreimes

I like the idea with the toolbox, but typically products are quite similar in their format, type, etc. so probably a package per variable makes more sense? surface soil moisture and root zone soil moisture separated

sebhahn avatar Sep 21 '20 19:09 sebhahn