Removed transform of XTIME because it doesn't conform to Xarray unit standards

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

56 stars 16 forks source link

Removed transform of XTIME because it doesn't conform to Xarray unit standards #145

Closed lpilz closed 8 months ago

lpilz commented 8 months ago

Change Summary

I removed the transform of XTIME in _decode_times so that the time coordinate is only constructed from the Times data variable and nothing else is changed.

Related issue number

Closes #131

Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable

lpilz commented 8 months ago

@jthielen Any comments? Otherwise I'll merge it beginning of next week.