Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parsing/Filling CF metadata as part of preprocessing #185

Open
jbusecke opened this issue Aug 2, 2021 · 2 comments
Open

Parsing/Filling CF metadata as part of preprocessing #185

jbusecke opened this issue Aug 2, 2021 · 2 comments

Comments

@jbusecke
Copy link
Owner

jbusecke commented Aug 2, 2021

This is a follow up on #52. Over there I made the decision to rename variables manually within this package, but I would still like to use cf conventions where possible. I am thinking of checking and maybe adding cf compliant attributes to the datasets as part of combined_preprocessing.

We have basically two options to add missing cf metadata:

  1. Manually parsing them as part of combined_preprocessing (This we can do right now, and I would suggest to do this as an intermediate step)
  2. Ultimately I would hope that this step is actually implemented further upstream in e.g. https://github.com/roocs/daops. This would work as function that queries a database of 'fixes' for the given dataset, applies those metadata fixes. But this also requires entering a LOT of data up front, so I think this would be the desired long term solution (where 1 is phased out at some point).
@jbusecke
Copy link
Owner Author

jbusecke commented Aug 2, 2021

Ah very nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants