-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
N-D rolling #819
Comments
@forman - The main reason this isn't supported yet is that we haven't implemented it yet. I recently added the At this point, I don't know of any plans to extend this functionality although we could discuss further. The complexity of N-D rolling aggregations does increase a fair bit and we would ideally like to let a lower level package (e.g. bottleneck) handle most of that. Since you seem to have a tangible application for the N-D rolling feature, maybe this is something you want to contribute to? |
Thanks for the prompt reply! Once we have decided to use xarray for our project(s) and once we familiarized with its internals, we'll be happy to contribute and support you! Currently we all feel a bit dizzy about the many options we have and how to decide which way to go: Create our own library using xarray or build on UK MetOffice's Iris, Apache OCW, or Max-Planck-Institute's CDO, etc. |
Hello together, additional to forman's comments I would like to note that for the xarray rolling operation a stride would be useful in some use-cases. I have understood, that no work is currently planned, but wanted to leave this remark for further contributions. I.e.
with
Thanks for xarray! It's great! |
@magonser We could add support for strides, but for builtin operations like |
In order to maintain a list of currently relevant issues, we mark issues as stale after a period of inactivity |
Dear xarray Team,
We just discovered xarray and it seems to be a fantastic candidate to serve as a core library for our climate data toolbox we are about to implement. While investigating the API we recognized that the
windows
kwargs inis limited to a single
dim=window_size
entry. Are there any plans to make it rolling in N-D?This could be very useful for efficient gap filling, filtering or other methodologies that use grid cell neighbourhoods in multiple dimensions.
Actually, I also asked myself why the
groupby
andresample
methods don't take an N-Ddim
argument. This would allow for performing not only a temporal resampling but also a spatial resampling in the lat/lon plane or even a spatio-temporal resampling (including up- and downsampling in either dim).Anyway, thanks for xarray!
Regards
Norman
The text was updated successfully, but these errors were encountered: