xarray-contrib / xwrf

A lightweight interface for working with the Weather Research and Forecasting (WRF) model output in Xarray.
https://xwrf.readthedocs.io/
Apache License 2.0
59 stars 16 forks source link

More comprehensive unit harmonization #116

Closed lpilz closed 1 year ago

lpilz commented 1 year ago

Change Summary

Unit harmonization is improved by:

Related issue number

Checklist

lpilz commented 1 year ago

There are some units which I didn't quite know how to translate:

lpilz commented 1 year ago

Small ping on this PR :)

jthielen commented 1 year ago

As far as the units of uncertain translation go:

lpilz commented 1 year ago

Hey, happy holidays! Sorry for the long delay on my part now... Didn't have too much time to spend on this until now due to AGU.

I added the changes, however I don't quite know what you would like to see here.

add new tests for each of the new categories of units handled in the expanded unit_harmonization_map

What are the new categories you're referring to?

lpilz commented 1 year ago

Small ping here :)

jthielen commented 1 year ago

Thanks for the ping on this, and sorry for the very long delay! Unfortunately I'm a bit swamped with prelim-related stuff at the moment, so I won't be able to get back to this until next week. But, hopefully will be able to then.

lpilz commented 1 year ago

Hey :) just a small reping. If and whenever you find the time of course!

lpilz commented 1 year ago

Hey @jthielen, I'm just going to go ahead and merge this if you don't mind.

jthielen commented 1 year ago

Hey @jthielen, I'm just going to go ahead and merge this if you don't mind.

Yes, please go ahead! Sorry for my continued inattentiveness. Hopefully things will be getting better soon.