deeplycloudy / xlma-python

A future, Python-based version of xlma?
MIT License
6 stars 6 forks source link

Add timedelta units and changed fill values in integers #18

Closed vbalderdash closed 2 years ago

vbalderdash commented 2 years ago

Patched time bugs with dataset creation and netcdf writing in lmatracer yml build in Python 3.7.10

deeplycloudy commented 2 years ago

For the record, what version of xarray are you using? I wonder if something changed in the allowable fill values in more recent versions.

vbalderdash commented 2 years ago

xarray v0.19.0

deeplycloudy commented 2 years ago

Well, that's completely current! Perhaps it's a regression in a newer version, but I don't see anything in the xarray release notes about it. My lmatracer environment is on 0.16.1, for what it's worth. Anyway, thanks for identifying the problem and fix. Merging!