xarray-contrib / flox

Fast & furious GroupBy operations for dask.array
https://flox.readthedocs.io
Apache License 2.0
123 stars 16 forks source link

import `normalize_axis_index` from `numpy.lib` on `numpy>=2` #364

Closed keewis closed 5 months ago

keewis commented 5 months ago

This currently emits a warning, not an error (i.e. not that urgent), but I did notice this a couple of times in xarray's CI.

I'm not too happy about having to move the definition of module_available above the import (isort might revert that?), but I didn't know whether it would be fine to move that function to a separate module (and I was too lazy to try figuring out a good name for that module).

dcherian commented 5 months ago

Thanks for doing this!