-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Doc: fixed missing behavior explaination of sort=false for groupby issue #47529 #52736
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
Doc: fixed missing behavior explaination of sort=false for groupby issue #47529 #52736
Conversation
Reviewer suggestions: @phofl, @noatamir, @jorisvandenbossche and @MarcoGorelli |
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.
thanks for your pr
doc/source/user_guide/groupby.rst
Outdated
@@ -205,7 +205,7 @@ By default the group keys are sorted during the ``groupby`` operation. You may h | |||
df2.groupby(["X"], sort=False).sum() | |||
|
|||
|
|||
Note that ``groupby`` will preserve the order in which *observations* are sorted *within* each group. | |||
Note that ``groupby`` will preserve the order in which *observations* are sorted *within* each group . |
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.
why make this change?
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.
In the comments of the issue it was mentioned to add also a description in the "User Guide".
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.
@raanasn - it looks like you're adding an unnecessary space before the period here.
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.
exactly - if you revert this the rest look good to me
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.
Typo, thanks!!
Co-authored-by: Marco Edward Gorelli <[email protected]>
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.
Looks good to me
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.
lgtm
Thanks @raanasn! |
Changes were made by @hubschne and @raanasn at the python conference 2023 Berlin workshop