-
Notifications
You must be signed in to change notification settings - Fork 165
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
Adapt to tidyr v1.0.0 #196
Conversation
Closes #195
data, | ||
test.func, formula = formula, | ||
method = method, paired = paired, p.adjust.method = "none",...) | ||
) %>% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doing this map()
inside the mutate()
, instead of outside, makes this code work for CRAN and dev versions of tidyr.
@@ -233,6 +236,7 @@ compare_means <- function(formula, data, method = "wilcox.test", | |||
by_y <- res %>% group_by(.y.) | |||
pvalue.adj <- do(by_y, .p.adjust(., method = p.adjust.method)) | |||
res <- res %>% | |||
dplyr::ungroup() %>% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An explicit ungroup()
here makes the code work for CRAN and dev versions of tidyr.
Thank you @jennybc for this contribution, highly appreciated! |
Great! So can you do a small release of ggpubr, in the near future? |
ggpubr is submitted now to CRAN |
Excellent! Thanks for doing attending to this in such a timely manner. |
You are welcome |
Closes #195
As mentioned in #195, the root issue here is that
tidyr::nest()
now returns a grouped object, which has implications for downstream manipulations via, e.g., dplyr.