Like pandas, we should wrap bottlene

Some further thoughts: I would probably implement these as met

closed via <a class="issue-link js-issue-link" data-error-text="Failed to load title"

Wrap bottleneck for fast moving window aggregations about xarray HOT 4 CLOSED

pydata commented on May 19, 2024

Wrap bottleneck for fast moving window aggregations

from xarray.

shoyer commented on May 19, 2024

Wrapping bottleneck for operations like nanmean was fixed in #304.

from xarray.

shoyer commented on May 19, 2024

Some further thoughts:

I would probably implement these as methods on Dataset/DataArray (e.g., ds.rolling_mean(dim='time', window=5) instead of functions. This is different from pandas, but seems slightly more in line with how everything else in pandas/xray operates.
I don't have a strong opinion whether we use the name rolling (like pandas) or moving/move (like bottleneck). Maybe rolling is slightly more obvious?
These functions from bottleneck could be easily extended to work on dask arrays by using using ghost: http://dask.pydata.org/en/latest/ghost.html

from xarray.

shoyer commented on May 19, 2024

There's been some related discussion about a better API for the rolling aggregation functions in pandas recently: pandas-dev/pandas#10702

from xarray.

jhamman commented on May 19, 2024

closed via #668

from xarray.