Closed editorialbot closed 1 year ago
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:
@editorialbot commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@editorialbot generate pdf
Software report:
github.com/AlDanial/cloc v 1.88 T=0.09 s (710.4 files/s, 199300.6 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
Markdown 7 413 0 7999
C/C++ Header 2 1 30 2620
XML 1 0 133 1784
R 44 497 1818 1726
Rmd 3 146 356 184
TeX 1 14 0 101
YAML 4 5 7 63
C++ 2 14 3 40
-------------------------------------------------------------------------------
SUM: 64 1090 2347 14517
-------------------------------------------------------------------------------
gitinspector failed to run statistical information for the repository
Wordcount for paper.md
is 1581
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.48550/arXiv.2211.15229 is OK
- 10.1093/aje/kwt133 is OK
- 10.1371/journal.pcbi.1008409 is OK
- 10.1007/s11222-016-9696-4 is OK
- 10.1038/s41467-021-21237-w is OK
MISSING DOIs
- None
INVALID DOIs
- None
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
Hi @rowlandseymour, @strengejacke and @bernadette-eu this is our review thread. Please feel free to raise and address any issues in the software repository, and then link back to here, or comment here for any wider issues or questions you may have for me.
@samhforbes Thanks for starting the review thread, Sam. I found this to be a substantial and well presented package. I spent yesterday playing around with it and trying to break various parts. There are lots of nice error messages and automated tests. From what I can see, it is implementing the Bayesian hierarchal model it is supposed to. The paper sets out the rationale and background, though I think the summary could be improved. There are comments throughout the repository. I've raised a couple of minor issues, and when these are done, I'm happy to recommend it be accepted.
Thank you @rowlandseymour for your feedback.
I am currently working to address your comments. @samhforbes I was wondering about good practices when dealing with these open issues/ reviewer comments on Github. Is it more preferable to create a PR (i.e. "joss_reviews") perform the changes and then merge with master? Or work directly on the master branch? Looking at other JOSS issues/reviews, I can see the former happening.
@rowlandseymour I will let you know if I have any follow-up clarifications.
Lampros Bouranis
Hi @bernadette-eu this is up to your personal workflow preferences - we see both examples.
@rowlandseymour
Thank you for your support on this work. I have addressed your comments and have replied at each issue that you have opened. Happy to address your comments further, if my responses are not satisfactory.
All the best, Lampros Bouranis
@bernadette-eu Thanks for addressing all of those comments. I should have said earlier, it's a nice package, I think you must have put a lot of work into this.
@samhforbes My review is now complete and I'm happy to recommend accepting this.
@rowlandseymour Thank you for your informative review. It's actually a 2-year effort, during the MSCA Fellowship ! :)
Hi @strengejacke I hope all is well! Please let me know if you need anything further to complete your review.
Hi @samhforbes hope you had a great summer break! I see that strengejacke has already completed most of the review and wanted to check for a progress update on that (given that things are slower during the summer). Happy to address any comments from the reviewer.
All the best, Lampros Bouranis
Sorry for being quiet the past time, because I was busy first and then on vacation. I will get back to the remaining part of the review tomorrow.
Hi @strengejacke hope you are doing well! Just checking in regarding the completion of the review. If you need anything further, do let me know.
Thanks, Lampros
Hi @strengejacke we look forward to the remainder of your review. Please let us know if there will be any issue getting this done soon.
Thanks, one minor issue and one recommendation to be addressed.
@strengejacke
Thank you for your feedback. I have addressed your comments and have replied at each issue that you have opened. Happy to address your comments further, if my responses are not satisfactory.
All the best, Lampros Bouranis
Thanks, @samhforbes I recommend accept! Sorry for the huge delay in reviewing the paper...
Fabulous, thank you both @rowlandseymour and @strengejacke
@bernadette-eu I am going to now begin the post-review stage, so there are a few things for you to do alongside the final checks I need to do. This is a nice package, well done!
@editorialbot make post-review checklist
I'm sorry human, I don't understand that. You can see what commands I support by typing:
@editorialbot commands
@editorialbot commands
Hello @samhforbes, here are the things you can ask me to do:
# List all available commands
@editorialbot commands
# Add to this issue's reviewers list
@editorialbot add @username as reviewer
# Remove from this issue's reviewers list
@editorialbot remove @username from reviewers
# Get a list of all editors's GitHub handles
@editorialbot list editors
# Assign a user as the editor of this submission
@editorialbot assign @username as editor
# Remove the editor assigned to this submission
@editorialbot remove editor
# Remind an author, a reviewer or the editor to return to a review after a
# certain period of time (supported units days and weeks)
@editorialbot remind @reviewer in 2 weeks
# Check the references of the paper for missing DOIs
@editorialbot check references
# Perform checks on the repository
@editorialbot check repository
# Adds a checklist for the reviewer using this command
@editorialbot generate my checklist
# Set a value for version
@editorialbot set v1.0.0 as version
# Set a value for branch
@editorialbot set joss-paper as branch
# Set a value for repository
@editorialbot set https://github.com/organization/repo as repository
# Set a value for the archive DOI
@editorialbot set set 10.5281/zenodo.6861996 as archive
# Mention the EiCs for the correct track
@editorialbot ping track-eic
# Generates the pdf paper
@editorialbot generate pdf
# Recommends the submission for acceptance
@editorialbot recommend-accept
# Generates a LaTeX preprint file
@editorialbot generate preprint
# Flag submission with questionable scope
@editorialbot query scope
# Get a link to the complete list of reviewers
@editorialbot list reviewers
# Creates a post-review checklist with editor and authors tasks
@editorialbot create post-review checklist
# Open the review issue
@editorialbot start review
@editorialbot set <DOI here> as archive
@editorialbot set <version here> as version
@editorialbot generate pdf
@editorialbot check references
and ask author(s) to update as needed@editorialbot recommend-accept
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@bernadette-eu I am going to now begin the post-review stage, so there are a few things for you to do alongside the final checks I need to do. This is a nice package, well done!
Hi @samhforbes ,
All changes were made in the joss_review branch. Let me merge with the master branch first and we'll generate the pdf again.
Thanks, Lampros Bouranis
Oh - thanks, sorry I missed that. If you haven't already, please check the capitalisation of references in the paper (I see Cori et al for example is all capitalised in the title, whereas others are not).
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@samhforbes below is the information on the Zenodo DOI and release version of the software for you. I think the "Additional Author Tasks After Review is Complete" list checks out - please let me know if I can provide further information. You can proceed by generating the pdf.
Version Number: v1.1.5 DOI: https://doi.org/10.5281/zenodo.8376673
@editorialbot set v1.1.5 as version
Done! version is now v1.1.5
@editorialbot set https://doi.org/10.5281/zenodo.8376673 as archive
Done! archive is now 10.5281/zenodo.8376673
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.48550/arXiv.2211.15229 is OK
- 10.1093/aje/kwt133 is OK
- 10.1371/journal.pcbi.1008409 is OK
- 10.1007/s11222-016-9696-4 is OK
- 10.1038/s41467-021-21237-w is OK
MISSING DOIs
- None
INVALID DOIs
- None
Hi @bernadette-eu please add
- the DOI for Carpenter et al (I think https://doi.org/10.18637/jss.v076.i01 is right)
Yeap, this this the correct one.
- Volume and pages for Cori et al 2013
I have fixed that reference.
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@editorialbot generate pdf
:point_right::page_facing_up: Download article proof :page_facing_up: View article proof on GitHub :page_facing_up: :point_left:
@editorialbot check references
Reference check summary (note 'MISSING' DOIs are suggestions that need verification):
OK DOIs
- 10.48550/arXiv.2211.15229 is OK
- 10.18637/jss.v076.i01 is OK
- 10.1093/aje/kwt133 is OK
- 10.1371/journal.pcbi.1008409 is OK
- 10.1007/s11222-016-9696-4 is OK
- 10.1038/s41467-021-21237-w is OK
MISSING DOIs
- None
INVALID DOIs
- None
Submitting author: !--author-handle-->@bernadette-eu<!--end-author-handle-- (Lampros Bouranis) Repository: https://github.com/bernadette-eu/Bernadette Branch with paper.md (empty if default branch): Version: v1.1.5 Editor: !--editor-->@samhforbes<!--end-editor-- Reviewers: @rowlandseymour, @strengejacke Archive: 10.5281/zenodo.8376673
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
@rowlandseymour & @strengejacke, 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 @samhforbes 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 @rowlandseymour
📝 Checklist for @strengejacke