Skip to content

DOC: Improve clarity of GroupBy introduction sentence #61805

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Vedant-Kadam-Noobie
Copy link

This small change clarifies the introductory sentence of the GroupBy user guide, as recommended for documentation improvements. It makes the definition of the "group by" process more direct and easier for new users to understand.

This small change clarifies the introductory sentence of the GroupBy user guide, as recommended for documentation improvements. It makes the definition of the "group by" process more direct and easier for new users to understand.
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @Vedant-Kadam-Noobie for the PR

@@ -6,8 +6,7 @@
Group by: split-apply-combine
*****************************

By "group by" we are referring to a process involving one or more of the following
steps:
A "group by" operation involves some combination of the following steps:
Copy link
Member

Choose a reason for hiding this comment

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

personally I think using "some combination" is less clear and less accurate.

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

Successfully merging this pull request may close these issues.

2 participants