dplyover icon indicating copy to clipboard operation
dplyover copied to clipboard

support for .keep (experimental mutate argument)

Open nefosl opened this issue 4 years ago • 1 comments

Thank you for this handy add-on and for pointing me to it in my Stack Overflow question a few months ago. The dplyover logo is great! dplyover::across2 has worked well for me to do simple division over pairs of columns within dplyr::mutate. I often use the .keep argument of dplyr::mutate (currently experimental); do you have any plans to make this work with dplyover?

nefosl avatar Mar 20 '21 12:03 nefosl

Thanks for your feedback and the suggestion to support dplyr::mutate's .keep argument in across2. Without relying on {dplyr}'s internals, which I try to avoid, it might be a bit tricky. Even if I get it working, I think it won't be really performant. I will definitely look into it, but with a low priority, since a call to dplyr::select before or after dplyr::mutate will get you the same result, probably with a better performance. Or am I overlooking something? Are there cases where one call to dplyr::select won't solve the problem?

Btw, I didn't expect that someone is already using the package. Note that there are changes to the code which are not reflected in the version number, since I haven't released the package yet (this will change after the first release). Once the first version is released, I'd recommend to reinstall the package.

TimTeaFan avatar Mar 21 '21 22:03 TimTeaFan