Skip to content

Add option for steps in Cube rolling windows - #7236

Open
mo-AndrewCreswick wants to merge 2 commits into
SciTools:mainfrom
mo-AndrewCreswick:cube_rolling_window_step
Open

Add option for steps in Cube rolling windows#7236
mo-AndrewCreswick wants to merge 2 commits into
SciTools:mainfrom
mo-AndrewCreswick:cube_rolling_window_step

Conversation

@mo-AndrewCreswick

Copy link
Copy Markdown

Description

This adds a step argument to iris.cube.Cube.rolling_window method to make use of the full functionality available in iris.util.rolling_window. A default argument of step=1 ensures backwards compatibility.

Unit tests added to test the aggregation over rolling windows with a step between windows and also to test that coordinates are handled correctly with a step. The behaviour of the window creation with steps is tested more thoroughly in the unit tests for iris.util.rolling_window.

I'm not sure if benchmarking is required for this PR so have not added the label for now.

Checklist

Important

The Iris core developers are here to help! If anything below is unclear, just post a comment asking for help 😊


Tip

Things you can trigger on this PR:

  • Add this label to trigger benchmarks: benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts
  • Visit this URL - swapping 9999 for this PR's number - to re-trigger the CLA check:
    https://cla-assistant.io/check/SciTools/iris?pullRequest=9999

@CLAassistant

CLAassistant commented Jul 29, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants