8000 feat: Streaming `{Expr,LazyFrame}.rolling` by coastalwhite · Pull Request #25058 · pola-rs/polars · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@coastalwhite
Copy link
Collaborator
@coastalwhite coastalwhite commented Oct 28, 2025

This adds a native streaming implementation for LazyFrame.rolling and Expr.rolling.

Note this does not include rolling_{min,max,std,etc.} although a lot of these can be written as .min().rolling().

@github-actions github-actions bot added enhancement New feature or an improvement of an existing feature python Related to Python Polars rust Related to Rust Polars labels Oct 28, 2025
@davidia
Copy link
Contributor
davidia commented Oct 29, 2025

Just here to say thanks and offer some encouragement. pretty excited by this feature 👍

@codecov
Copy link
codecov bot commented Oct 30, 2025

Codecov Report

❌ Patch coverage is 97.31903% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.76%. Comparing base (51f8b4c) to head (4e6b559).
⚠️ Report is 32 commits behind head on main.

Files with missing lines Patch % Lines
crates/polars-stream/src/nodes/rolling_group_by.rs 95.21% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #25058      +/-   ##
==========================================
- Coverage   81.91%   81.76%   -0.16%     
==========================================
  Files        1708     1708              
  Lines      235467   235980     +513     
  Branches     3000     3005       +5     
==========================================
+ Hits       192888   192943      +55     
- Misses      41812    42269     +457     
- Partials      767      768       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coastalwhite coastalwhite marked this pull request as ready for review October 30, 2025 14:41
@coastalwhite coastalwhite changed the title feat: Streaming LazyFrame.rolling feat: Streaming {Expr,LazyFrame}.rolling Oct 30, 2025
@coastalwhite coastalwhite merged commit fa0e9d0 into pola-rs:main Oct 31, 2025
29 checks passed
@coastalwhite coastalwhite deleted the refactor/streaming-rolling-windower branch October 31, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or an improvement of an existing feature python Related to Python Polars rust Related to Rust Polars

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

3641 @coastalwhite @davidia @orlp
0