Skip to content

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

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