Let Xarray handle `decode_times`

zarr-developers / VirtualiZarr

Create virtual Zarr stores from archival data files using xarray syntax

https://virtualizarr.readthedocs.io/en/stable/api.html

Apache License 2.0

124 stars 24 forks source link

Let Xarray handle `decode_times` #232

Closed norlandrhagen closed 3 months ago

norlandrhagen commented 3 months ago

[x] (https://github.com/zarr-developers/VirtualiZarr/issues/168)
[x] Tests added
[x] Tests passing
[x] Full type hint coverage
[x] Changes are documented in docs/releases.rst
[x] New functionality has documentation

TomNicholas commented 3 months ago

@norlandrhagen I merged in the changes from main but there are failing tests when I try this locally...

TomNicholas commented 3 months ago

Thanks for fixing that @norlandrhagen ! Let's just add a note to the release notes to explain to users what we have just changed, then this should be good to merge.

Also FYI @jsignell