xarray-contrib / flox

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

cohorts: Use IntervalIndex to reduce search space. #304

Closed dcherian closed 10 months ago

dcherian commented 10 months ago

This was a great improvement for the county grouping but a regression for periodic time groupings where the bounding boxes are quite large and do not reduce the search space at all. In #306 I switched to using the bitmask to reduce the search space. This is even better for the county grouping and generally better for the periodic grouping. More optimizations are possible there.