Skip to content

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


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.

@jbrockmendel
Copy link
Member

@Vedant-Kadam-Noobie thanks for the PR but simon and I prefer the existing phrasing.

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.

3 participants