gjoseph92 / stackstac

Turn a STAC catalog into a dask-based xarray
https://stackstac.readthedocs.io
MIT License
238 stars 49 forks source link

Pandas compat #143

Closed TomAugspurger closed 2 years ago

TomAugspurger commented 2 years ago

pandas has deprecated using the Float64Index constructor directly, in favor of Index(..., dtype="float64").

I also opened https://github.com/pandas-dev/pandas/issues/46484, to track support for a version of RangeIndex that can support floats.

gjoseph92 commented 2 years ago

Thanks @TomAugspurger!

TomAugspurger commented 2 years ago

@gjoseph92 do you have any bandwidth to cut a release with this? Our next set of environments will include a pandas where this deprecation warning is present. No worries if not, we're able to silence specific warnings if necessary.

gjoseph92 commented 2 years ago

Sure, I can do that in the next few days.

gjoseph92 commented 2 years ago

Also just FYI @TomAugspurger, you might want to keep dask and distributed at 2022.1.1 for now. There are a number of stability issues with newer distributed versions that haven't been resolved yet (xref https://github.com/dask/community/issues/238).

TomAugspurger commented 2 years ago

Thanks for the heads up. Via https://github.com/conda-forge/pangeo-dask-feedstock/blob/main/recipe/meta.yaml we're pinned to 2022.2.1.

I've been using it for a week or so and haven't noticed anything anecdotally. I'll do a bit more load testing and will report on https://github.com/dask/community/issues/238 if I spot issues.