cftime icon indicating copy to clipboard operation
cftime copied to clipboard

Future plans for the `longdouble` argument

Open spencerkclark opened this issue 3 years ago • 2 comments

In #284 @mcuntz added a longdouble argument to enable encoding times (in memory) with greater precision. The default was set to False, since otherwise it would interfere with serialization logic in downstream libraries (e.g. xarray). I just wanted to check if it is still planned to eventually change the default to True. In xarray this would be fairly straightforward to address by explicitly setting the longdouble argument to False in the time encoding logic, but before doing that I just wanted to confirm cftime's future plans.

spencerkclark avatar Oct 08 '22 19:10 spencerkclark

I think it would be good to change the default to True in a future release, as long as xarray supports it.

jswhit avatar Oct 11 '22 17:10 jswhit

Sounds good. I'll make that PR to xarray when I get a chance.

spencerkclark avatar Oct 11 '22 17:10 spencerkclark