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
roll_any() and roll_all() #14
Comments
Thanks, that's an interesting suggestion. Just so I understand what you’re thinking, here’s an example of how I’d approach it in base R:
If so, I can see it fitting within my framework and existing arguments: |
Yeah, exactly. |
I have added both
Also, the performance is significantly faster. Let me know if you have any other feedback. |
Looks great, thanks! |
Would you consider extending roll to also think about logical vectors?
roll_any()
androll_all()
are admittedly esoteric, but can sometimes be useful, and would complete the matrix of summary functions at https://github.com/r-lib/vctrs/issues/9The text was updated successfully, but these errors were encountered: