Skip to content

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

Conversation

raanasn
Copy link
Contributor

@raanasn raanasn commented Apr 18, 2023

Changes were made by @hubschne and @raanasn at the python conference 2023 Berlin workshop

@raanasn
Copy link
Contributor Author

raanasn commented Apr 18, 2023

Reviewer suggestions: @phofl, @noatamir, @jorisvandenbossche and @MarcoGorelli

@noatamir noatamir added the Sprints Sprint Pull Requests label Apr 18, 2023
Copy link
Member

@MarcoGorelli MarcoGorelli left a 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

@@ -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 .
Copy link
Member

Choose a reason for hiding this comment

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

why make this change?

Copy link
Contributor Author

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

Copy link
Member

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.

Copy link
Member

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Typo, thanks!!

@MarcoGorelli MarcoGorelli added this to the 2.1 milestone Apr 20, 2023
Copy link
Member

@MarcoGorelli MarcoGorelli left a 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

Copy link
Member

@rhshadrach rhshadrach left a comment

Choose a reason for hiding this comment

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

lgtm

@rhshadrach rhshadrach merged commit 423fffd into pandas-dev:main Apr 21, 2023
@rhshadrach
Copy link
Member

Thanks @raanasn!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Sprints Sprint Pull Requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: missed behavior explaination of sort=False for groupby
4 participants