-
-
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]: individual: An R package for individual-based epidemiological models #3539
Comments
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @seabbs, @strengejacke, @SteRoe it looks like you're currently assigned to review this paper 🎉. Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post. ⭐ Important ⭐ If you haven't already, you should seriously consider unsubscribing from GitHub notifications for this (https://github.com/openjournals/joss-reviews) repository. As a reviewer, you're probably currently watching this repository which means for GitHub's default behaviour you will receive notifications (emails) for all reviews 😿 To fix this do the following two things:
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 |
|
|
@seabbs, @strengejacke, @SteRoe: Thanks for agreeing to review. Please carry out your review in this issue by updating the checklist above and giving feedback in this issue. The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. If possible create issues (and cross-reference) in the submission's repository to avoid too specific discussions in this review thread. If you have any questions or concerns please let me know. |
@mikldk: This is my first JOSS review. Filled out the check list, but I'm not sure how to submit it. Most notably I found no statement on examples and tests, so I left these checks open. Other points are fine. How to proceed? |
@SteRoe Thanks for asking! The check items are not to be submitted, once they are checked that is saved and all is good. The ones you do not think are okay/not satisfied you can address here in this issue to the author (minor things), or preferably create issues in the submission's repository and mention this review issue so that the issues are linked. |
@mikldk @SteRoe Thanks for bringing that up! Not sure how we declare these to JOSS but we have extensive automated tests (here's our latest coverage), and our documentation has an example in the tutorial (which had to be removed from the paper for brevity). |
Some other potentially useful links... Installation instructions ;) |
@giovannic Maybe, you could reference your test suite and the tutorial in a short section? |
I've added a short note to the Licensing and Availability section. |
👋 @SteRoe, please update us on how your review is going (this is an automated reminder). |
👋 @seabbs, please update us on how your review is going (this is an automated reminder). |
👋 @strengejacke, please update us on how your review is going (this is an automated reminder). |
@seabbs, @strengejacke, @SteRoe: Can you please give a brief status of your review? This is not to rush you, merely to give me an impression of the progress and time-frame. |
My points were properly replied. For me, the manuscript is fine. How to proceed? |
@mikldk Ah, ok done 👍 |
@whedon generate pdf |
|
👍 done |
@whedon generate pdf |
@giovannic The title still says "[...] for individual based epidemiological models" - without hyphen? |
@mikldk I've addressed that here mrc-ide/individual@ec4f1be |
@whedon generate pdf |
@whedon recommend-accept |
|
|
👋 @openjournals/joss-eics, this paper is ready to be accepted and published. Check final proof 👉 openjournals/joss-papers#2645 If the paper PDF and Crossref deposit XML look good in openjournals/joss-papers#2645, then you can now move forward with accepting the submission by compiling again with the flag
|
@openjournals/joss-eics : Please note that the title has changed (a hyphen added to "individual-based"). I changed the issue title here, but I am not sure if it needs to be changed elsewhere, too. |
@openjournals/dev does the title need to be changed anywhere else, or will it be fine if correct in the paper? |
I'm going to proceed assuming that is the case, since the generated XML and PDF are correct. |
@whedon accept deposit=true |
|
🐦🐦🐦 👉 Tweet 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... |
Congratulations @giovannic on your article's publication in JOSS! Many thanks to @strengejacke and @SteRoe for reviewing this, and @mikldk for editing it. |
🎉🎉🎉 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! 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:
|
Thank you @SteRoe and @strengejacke for reviewing our paper, and to @mikldk for editing our submission! |
Submitting author: @giovannic (Giovanni Charles)
Repository: https://github.com/mrc-ide/individual
Version: v0.1.6
Editor: @mikldk
Reviewer: @seabbs, @strengejacke, @SteRoe
Archive: 10.5281/zenodo.5529932
Due to the challenges of the COVID-19 pandemic, JOSS is currently operating in a "reduced service mode". You can read more about what that means in our blog post.
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
@seabbs & @strengejacke & @SteRoe, please carry out your review in this issue by updating the checklist below. If you cannot edit the checklist please:
The reviewer guidelines are available here: https://joss.readthedocs.io/en/latest/reviewer_guidelines.html. Any questions/concerns please let @mikldk 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 ✨
Review checklist for @seabbs
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @strengejacke
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @SteRoe
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
The text was updated successfully, but these errors were encountered: