Skip to content
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

group_by_ + arrange_ #2318

Closed
mplourde opened this issue Dec 13, 2016 · 2 comments
Closed

group_by_ + arrange_ #2318

mplourde opened this issue Dec 13, 2016 · 2 comments

Comments

@mplourde
Copy link

@mplourde mplourde commented Dec 13, 2016

arrange_ used to respect groupings. Now I often find myself using the idiom do(arrange_(., ~cpm)) to order groups individually for, say, a cumulative sum along time within groups.

Would it be feasible to add an argument to arrange_ or create a new function, say, group_arrange_ to encapsulate this?

Thank you.

@hadley
Copy link
Member

@hadley hadley commented Jan 31, 2017

Yeah, I'll bring this back as an option.

@hadley
Copy link
Member

@hadley hadley commented Feb 26, 2017

Adding this will need to wait until after the NSE rewrite since additional arguments to the generic are currently captured in .dots

@hadley hadley closed this in cf04a70 Apr 4, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Jun 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants