-
Notifications
You must be signed in to change notification settings - Fork 8
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
requesting rephrasing of error message for function check_multispp
#131
Comments
Hi @katharinehogan thank you so much for the suggestion. Just to clarify, are you suggesting the message be: Thoughts from others on this wording? @laurenmh @itcarroll And does anyone want to create a Pull Request? |
Hi @mbjones sorry, I completely forgot to reply to this until here I am doing a different codyn analysis and remembering. And yes, I think that rephrasing would be more clear. Just something to switch the focus from "species" to the replicates column. |
Sounds great. We could really use some help from the community that uses |
As someone with decent R programming knowledge but no experience creating/maintaining packages, what sorts of contributions are most useful? |
From our CONTRIBUTING.md document: 🎉
We'd also appreciate help with testing and releasing new versions of the package, and any other maintenance tasks. If you are familiar with R enough to use the package, then it's a straightforward process to contributing to maintaining the code. Sometimes it's pretty small changes, but incredibly useful to keep the package operational and valuable. And, I think anyone using the package might have some great ideas on how to extend it to make it even more useful to the community. |
Hello,
First of all, thank you for doing the extra work of publicly documenting and maintaining this package. I was able to easily debug something that otherwise would have been more difficult (and appreciate the extra work regardless). I think this is the same request as the issue posted on 10/9/2015 by @laurenmh.
I would like to suggest clarifying or at least revisiting the error message returned by
check_multispp
. It currently returns"One or more replicates consists of only a single species; please remove these replicates prior to calculations"
. After debugging directly fromcheck_multispp
(rather than thesynchrony
function that originally threw the error), I found the issue was that one site in myreplicate.var=
argument column was only sampled one time.synchrony
ran just fine once I removed that site.I was initially really confused by the error message saying that one replicate had only one species. Phrasing it as # of species made me think I needed to
group_by(year,site) %>% add_count(species)
to see where there was only a single species, which did not solve my issue. I would like to suggest the error message be rephrased to something like"occurs only once"
instead of"consists of only a single species"
.In hindsight I can see the reasoning for mentioning species, and my suggestion might not be better - but overall I found the reference to species misleading, since it's not clear what exactly
check_multispp
is counting in the background. Thank you for reading and considering my suggestion.The text was updated successfully, but these errors were encountered: