-
Notifications
You must be signed in to change notification settings - Fork 0
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
Review for JOSS #16
Comments
Thank you so much for all of the great advice and suggestions! I have now addressed all of these as requested. Some notes below. Installation and functionality.
Manuscript
Further suggestions
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Dear @marccanby,
I have now reviewed the manuscript corresponding to the linguiphyr. I am not a linguist and the topic is a bit far from my direct expertise. Nonetheless, I provide some suggestions in the hope they will help improving the manuscript and the package.
Installation and functionality
The interface itself runs smoothly and is intuitive.
Manuscript
Further suggestions
I assume the columns in the example dataset represent languages. Would it be an idea to use glottocodes or isocodes here? This would facilitate interoperability with other datasets and packages.
References do not show up on the github page (both in-text and reference list). But this may be automatically generated when compiling the paper?
Now the installation of PAUP goes from the browser. Would it be an idea to do that from a prompt from within R? I.e. a prompt when starting linguiphyr for the first time? It would perhaps also be helpful to add a built-in check for the latest version of PAUP and check whether this corresponds to the locally installed version.
Good luck with the revisions!
Kind regards,
Sietze
The text was updated successfully, but these errors were encountered: