Skip to content

Conversation

@cr7pt0gr4ph7
Copy link
Contributor

@cr7pt0gr4ph7 cr7pt0gr4ph7 commented Nov 11, 2025

Extend the documentation about lists to show how to use .list.eval in combination with aggregation functions and sort_by to process the list elements.

Also fixes the objectively false statement [...] we can also use pl.all() to refer to all of the elements of the list..

@github-actions github-actions bot added documentation Improvements or additions to documentation python Related to Python Polars rust Related to Rust Polars labels Nov 11, 2025
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.13%. Comparing base (20ebc64) to head (2759a14).
⚠️ Report is 105 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #25260      +/-   ##
==========================================
+ Coverage   81.92%   82.13%   +0.20%     
==========================================
  Files        1712     1714       +2     
  Lines      237225   238960    +1735     
  Branches     3011     3018       +7     
==========================================
+ Hits       194358   196279    +1921     
+ Misses      42094    41905     -189     
- Partials      773      776       +3     

☔ 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.

@cr7pt0gr4ph7
Copy link
Contributor Author

cr7pt0gr4ph7 commented Nov 13, 2025

Note to reviewers: The current build failure is due to unrelated problems being detected by markdown-link-check, and is not related to the changes in this PR, which is ready for review (and potentially merging).

I've created a separate PR #25314 that fixes the link errors.

Update: Link errors should be fixed now that PR #25314 has been merged.

Copy link
Collaborator

@coastalwhite coastalwhite left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution. Small nit.

@cr7pt0gr4ph7
Copy link
Contributor Author

@coastalwhite Hey, think you could have a look and see if you're satisfied by the new changes? Thanks! :)

@cr7pt0gr4ph7
Copy link
Contributor Author

The PR is ready to merge on my side, provided there are no other objections/comments.

@cr7pt0gr4ph7
Copy link
Contributor Author

@c-peters @ritchie46 Any chance of getting this reviewed? :)

@ritchie46
Copy link
Member

Thanks @cr7pt0gr4ph7

@ritchie46 ritchie46 merged commit 8f6c678 into pola-rs:main Nov 28, 2025
18 checks passed
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 python Related to Python Polars rust Related to Rust Polars

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants