-
-
Notifications
You must be signed in to change notification settings - Fork 36
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]: growR: R Implementation of the Vegetation Model ModVege #6260
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:
|
|
Wordcount for |
Failed to discover a |
|
👋🏼 @kuadrat , @shubhamjain15 , @RobLBaker , this is the review thread for {growR}. 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. For best results, don't include anything else in the comment! This will create a checklist that walks through the JOSS submission 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 2-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. Thank you so much for agreeing to review this submission! |
@editorialbot remind @mikemahoney218 in 2 weeks (Setting up an automated reminder for myself to make sure this doesn't fall through the cracks 😄 ) |
Reminder set for @mikemahoney218 in 2 weeks |
Review checklist for @shubhamjain15Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
👋 @kuadrat , @shubhamjain15 , and @RobLBaker , just wanted to share that I'm going to be OOO from February 2nd through the 9th (so, Friday through the end of next week). I'll be around somewhat but will be much more delayed in responding on this issue; apologies in advance! And as always, feel free to reach out if you have any questions or concerns 😄 |
👋 @mikemahoney218, please take a look at the state of the submission (this is an automated reminder). |
Regarding license (as I can see this is an unchecked box in the review checklist of @shubhamjain15 ), please have a look at the discussion that has already occurred during pre-review and links therein: In short: At the moment it does not seem to be possible to simultaneously satisfy CRAN's and github's conventions regarding naming and structure of license files. Since the main avenue of distribution is via CRAN, the choice has been made to follow their conventions. The conclusion in JOSS reviews for other R packages has been that this complies with JOSS (see linked comment). |
Review checklist for @RobLBakerConflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
|
Really cool work! I missed the statement of need in the Documentation. I am assuming this is separate from the statement of need in the Software paper. Perhaps I'm overlooking it somewhere - if you could please point me in the right direction or confirm that these two statements of need are one and the same, I'd be happy to check that last box. |
@shubhamjain15 , I see that your checklist is all filled in -- are you finished reviewing the package? |
(and @kuadrat , if I'm not mistaken, I think ☝️ from Rob is the last outstanding issue on this review!) |
@mikemahoney218 Hello Mike! I completed my review yesterday, and in my assessment, this submission meets all the points on the check list. :) |
Thank you so much @shubhamjain15 ! |
Dear @shubhamjain15 , @RobLBaker , thank you very much for your review efforts! I'm very glad to hear that the package is received mostly positively. @RobLBaker I have extended the README with a What is this for? section on the joss branch. Once I merge this into |
Works for me - I checked that last box. Please let me know if there's anything else needed from me & happy to review more submissions, especially if they're of this quality. |
|
ID ref-zhao2020GrasslandEcosystemServices already defined |
@kuadrat Looks like you've got a duplicate entry for |
I removed the duplicate and checked all other entries for uniqueness. |
@editorialbot recommend-accept 🙏 😆 |
|
|
👋 @openjournals/ese-eics, this paper is ready to be accepted and published. Check final proof 👉📄 Download article If the paper PDF and the deposit XML files look good in openjournals/joss-papers#5015, then you can now move forward with accepting the submission by compiling again with the command |
🎉 looks like it went through that time, which means it's time for me to hand this back to the EiC for last steps. Thanks @kuadrat for the submission, and thank you so much to @shubhamjain15 and @RobLBaker for reviewing! |
Many thanks from my side to @shubhamjain15, @RobLBaker and @mikemahoney218 for putting in the time and work into this review process! |
Hi all! My job is to help wrap up this submission and do some final checks. First, one question for @shubhamjain15 and @RobLBaker: can you verify that you installed the software and got it to run, etc? I ask because it's a bit unusual to have reviews go this quickly and not require any issues being opened on the software repo. It happens occasionally, but just want to check in to make sure the process was clear to everyone given the circumstances. Thank you! |
Yes, I was able to download and install the package. It appears to function as claimed: I was able to recreate all of the analyses and results from the Tutorial and Calibration vignettes using the example data. |
@RobLBaker Thank you for your response to my unusual request! @shubhamjain15 Could you confirm the same? |
Hi Kristen! Yes I was able to install the R package and use the examples in the reference manual and tutorial to test the functionality successfully. |
Ok great! This being an easy submission to review really says a lot for @kuadrat's effort on the package. |
Here is my to do list:
|
Everything is ready to go. |
@editorialbot accept |
|
Ensure proper citation by uploading a plain text CITATION.cff file to the default branch of your repository. If using GitHub, a Cite this repository menu will appear in the About section, containing both APA and BibTeX formats. When exported to Zotero using a browser plugin, Zotero will automatically create an entry using the information contained in the .cff file. You can copy the contents for your CITATION.cff file here: CITATION.cff
If the repository is not hosted on GitHub, a .cff file can still be uploaded to set your preferred citation. Users will be able to manually copy and paste the citation. |
🐘🐘🐘 👉 Toot for this paper 👈 🐘🐘🐘 |
🚨🚨🚨 THIS IS NOT A DRILL, YOU HAVE JUST ACCEPTED A PAPER INTO JOSS! 🚨🚨🚨 Here's what you must now do:
Any issues? Notify your editorial technical team... |
Congrats on your new publication @kuadrat! Many thanks to editor @mikemahoney218 and reviewers @shubhamjain15 and @RobLBaker for your time, hard work, and expertise!! |
🎉🎉🎉 Congratulations on your paper acceptance! 🎉🎉🎉 If you would like to include a link to your paper from your README use the following code snippets:
This is how it will look in your documentation: We need your help! The Journal of Open Source Software is a community-run journal and relies upon volunteer effort. If you'd like to support us please consider doing either one (or both) of the the following:
|
Many thanks also to all of you for your voluntary work! I'm happy this went so smoothly. Maybe a remark on that note: Part of what likely helped making this submission go through without many issues is the fact that the package has been published on CRAN. This means that it has already undergone a number of automated and human-overseen quality checks before being submitted to JOSS. |
@kuadrat Thanks for that comment and that makes sense. I'll keep my eye out for that being a mitigating (and helpful!) factor on future reviews. Also if you're interested in getting involved in JOSS, please consider signing up as a reviewer! https://reviewers.joss.theoj.org/ |
Submitting author: @kuadrat (Kevin Kramer)
Repository: https://github.com/kuadrat/growR
Branch with paper.md (empty if default branch): joss
Version: growR-1.2.0-JOSS
Editor: @mikemahoney218
Reviewers: @shubhamjain15, @RobLBaker
Archive: 10.5281/zenodo.10658193
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
@shubhamjain15 & @RobLBaker, 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 @shubhamjain15
📝 Checklist for @RobLBaker
The text was updated successfully, but these errors were encountered: