-
Notifications
You must be signed in to change notification settings - Fork 43
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
feature request: multiple testing correction #28
Comments
Yes, I agree this would indeed be a useful feature. I am not quite sure what would be the best default (Bonferroni, Holm, Benjamini-Hochberg, None?), but adding the possibility for the use to choose any of those makes sense. |
I just submitted a pull request with this functionality quickly added - for BH/fdr only. Prior to merging you can try it out here: https://github.com/sidderb/ggsignif |
ggsignif works very nicely and is incredibly useful with facet_wrap. If the BH/FDR option could recognize the number of facets and correct accordingly, that would be great ! |
I would also have great use for multiple testing option- |
+1 for this feature. |
Was this feature added? If so, where do I find how to use it? |
I've incorporated sidderb FDR functionality to the newest version of ggsignif. Please find at ggsignif-FDR |
If I do not miss anything, there is currently no way to include a multiple comparison correction. This would be a very useful addition.
The text was updated successfully, but these errors were encountered: