-
Notifications
You must be signed in to change notification settings - Fork 2.5k
docs: Explain aggregation & sorting of lists #25260
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
03f79d1 to
7626982
Compare
|
Update: Link errors should be fixed now that PR #25314 has been merged. |
Correct formatting and line breaks in the documentation.
7626982 to
d739879
Compare
coastalwhite
left a comment
There was a problem hiding this 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.
|
@coastalwhite Hey, think you could have a look and see if you're satisfied by the new changes? Thanks! :) |
|
The PR is ready to merge on my side, provided there are no other objections/comments. |
|
@c-peters @ritchie46 Any chance of getting this reviewed? :) |
|
Thanks @cr7pt0gr4ph7 |
Extend the documentation about lists to show how to use
.list.evalin combination with aggregation functions andsort_byto 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..