Skip to content

docs: Warn about non-deterministic lazy row indexes - #28622

Open
Whning0513 wants to merge 1 commit into
pola-rs:mainfrom
Whning0513:docs/28294-lazy-row-index
Open

docs: Warn about non-deterministic lazy row indexes#28622
Whning0513 wants to merge 1 commit into
pola-rs:mainfrom
Whning0513:docs/28294-lazy-row-index

Conversation

@Whning0513

Copy link
Copy Markdown

Fixes #28294.

Warn that adding a row index to a non-deterministic LazyFrame can make the index relationship to other columns unpredictable.

Copilot AI review requested due to automatic review settings July 31, 2026 09:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added documentation Improvements or additions to documentation python Related to Python Polars rust Related to Rust Polars first-contribution First contribution by user title needs formatting labels Jul 31, 2026
@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.36%. Comparing base (36e414b) to head (f73d1af).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #28622      +/-   ##
==========================================
- Coverage   81.71%   81.36%   -0.36%     
==========================================
  Files        1870     1870              
  Lines      262992   262972      -20     
  Branches     3217     3217              
==========================================
- Hits       214917   213977     -940     
- Misses      47243    48163     +920     
  Partials      832      832              

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Whning0513 Whning0513 changed the title docs: warn about non-deterministic lazy row indexes docs: Warn about non-deterministic lazy row indexes Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation first-contribution First contribution by user python Related to Python Polars rust Related to Rust Polars

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add warning to documentation of with_row_index about non-deterministic inputs

2 participants