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]: RMAVIS v1.0: a R Shiny application for the analysis of vegetation survey data and assignment to GB NVC communities #6682
Comments
Hello humans, I'm @editorialbot, a robot that can help you with some common editorial tasks. For a list of things I can do to help you, just type:
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
|
Software report:
Commit count by author:
|
Paper file info: 📄 Wordcount for ✅ The paper includes a |
License info: 🔴 License found: |
|
👋🏼 @ZekeMarshall, @adithirgis, @rasanderson this is the review thread for the paper. Just about all of our communications will happen here from now on. 😄 As a reviewer, the first step is to create a checklist for your review by entering
as the top of a new comment in this thread. These checklists contain the JOSS requirements. As you go over the submission, please check any items that you feel have been satisfied. The first comment in this thread also contains links to the JOSS reviewer guidelines. The JOSS review is different from most other journals. Our goal is to work with the authors to help them meet our criteria instead of merely passing judgment on the submission. As such, the reviewers are encouraged to submit issues and pull requests on the software repository. When doing so, please mention We aim for reviews to be completed within about 4 weeks. Please let me know if you require some more time. Please feel free to ping me (@mikemahoney218) if you have any questions/concerns. |
Review checklist for @rasandersonConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
@ZekeMarshall , I just noticed that you changed the license of the project a few days ago to a non open-source license. JOSS only reviews projects that are open source using an OSI-approved license (as mentioned on the first bullet point on the 'Submission requirements' checklist), which none of the creative commons licenses are (and CC recommends against using their licenses for software). Is there a reason you changed away from the MIT license? We will not be able to accept the project using the current (CC-BY) license. |
Hi @mikemahoney218 , apologies for this! I neglected to check that CCBY4.0 wasn't OSI-approved. I'll change this again in the morning when i'm in the office, likely to AAL, we just wanted to ensure attribution. PS thank you @adithirgis and @rasanderson for agreeing to review! |
Hi @mikemahoney218 , I've now changed the licence to the OSI-approved LGPL 3.0 and made a new release. Could you set the version to v0.9995? Thanks! |
Thanks @ZekeMarshall ! I believe we don't need to update the version until we accept the package -- feel free to bump it and release new ones whenever is convenient for you. |
@editorialbot generate pdf |
Hi @rasanderson & @adithirgis, I've bumped the version to v0.9996. This version includes a few minor changes and one more substantial change, namely using the NBN to form the taxonomic backbone for vascular plants as well as bryophytes, so now there is only one source for the taxonomic backbone. |
Hi @rasanderson & @adithirgis, I've bumped the version to v0.9997. This version sets R 4.4 as the minimum version in response to the reported 'non-vulnerability' in R wherein arbitrary code can be executed when de-serialising R data objects, see here, which is not in fact fixed by R 4.4.0. |
Just as a quick note: I'm going to be traveling and generally less available until May 20th. I'll still be checking GitHub and email intermittently (so feel free to reach out with any questions or concerns), but apologies if it takes me a bit longer to respond than usual! |
Thanks for letting us know and no worries at all. Have a good break! |
Thanks for letting us know. I hope to have some first comments back fairly soon, so apologies for slight delay. |
@editorialbot set https://github.com/NERC-CEH/RMAVIS as repository |
I'm sorry @ZekeMarshall, I'm afraid I can't do that. That's something only editors are allowed to do. |
Hi @mikemahoney218 , I've transferred the RMAVIS repository to our organisations Github (https://github.com/NERC-CEH/RMAVIS), would you be able to set this as the repository? |
Submitting author: @ZekeMarshall (Zeke Marshall)
Repository: https://github.com/ZekeMarshall/RMAVIS/
Branch with paper.md (empty if default branch): main
Version: v1.0
Editor: @mikemahoney218
Reviewers: @adithirgis, @rasanderson
Archive: Pending
Status
Status badge code:
Reviewers and authors:
Please avoid lengthy details of difficulties in the review thread. Instead, please create a new issue in the target repository and link to those issues (especially acceptance-blockers) by leaving comments in the review thread below. (For completists: if the target issue tracker is also on GitHub, linking the review thread in the issue or vice versa will create corresponding breadcrumb trails in the link target.)
Reviewer instructions & questions
@adithirgis & @rasanderson, your review will be checklist based. Each of you will have a separate checklist that you should update when carrying out your review.
First of all you need to run this command in a separate comment to create the checklist:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mikemahoney218 know.
✨ Please start on your review when you are able, and be sure to complete your review in the next six weeks, at the very latest ✨
Checklists
📝 Checklist for @rasanderson
The text was updated successfully, but these errors were encountered: