[GitHub] [arrow] thisisnic commented on pull request #13786: ARROW-11699: [R] Implement dplyr::across()

[GitBox <gi...@apache.org>]

thisisnic commented on PR #13786:
URL: https://github.com/apache/arrow/pull/13786#issuecomment-1203884371

   Will come back to this once I'm back from holiday, but as a reminder to my future self, where I was up to:
   * I have a minimal working example that runs successfully in `mutate()`
   * I wanted to incorporate the `.names` argument from `across()` and so had refactor the column naming code out into its own function, to pass that into later
   * I tried to implement `across()` in `summarise()` - this broke things with the `.groups` argument and I don't yet understand what/why, other than that `.groups` works differently with Datasets versus Tables - there may be some context in https://github.com/apache/arrow/issues/11184
   * it's also worth considering whether it's worth just focussing on `across()` for `mutate()` as a first step, and extending to `summarize()` later


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org