Closed whedon closed 4 years ago
Hello human, I'm @whedon, a robot that can help you with some common editorial tasks. @acolum, @pholme it looks like you're currently assigned to review this paper :tada:.
:star: Important :star:
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:
@whedon commands
For example, to regenerate the paper pdf after making changes in the paper's md or bib files, type:
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
Just finished my review - everything looks great!
Thank you @acolum! 👋 @pholme let me know if you need something else to complete the review.
👋 @pholme, are you still able to review this? Let me know.
@whedon add @hagberg as reviewer
OK, @hagberg is now a reviewer
@hagberg how is the review going?
Hi @hagberg, did you end up opening the issues to ask the authors to change issues you saw during the review? If yes, can you link them here? Thanks!
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
PDF failed to compile for issue #1731 with the following error:
Error reading bibliography ./paper.bib (line 39, column 1): unexpected "y" expecting space, ",", white space or "}" Error running filter pandoc-citeproc: Filter returned error status 1 Looks like we failed to compile the PDF
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
Hi @lpantano - Do you know if there is any way to embed a video into the pdf file? One of the final things I want to show is a tool to build an animation, but at present I can't find any way to embed it.
@hagberg
I've added the functional testing (created by @tinghf ) into the repository under EoN/tests
I've also added a discussion of related packages into the paper.
Let me know of anything else you'd like improved.
Looks good. Issues are closed.
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
Hi @lpantano - Do you know if there is any way to embed a video into the pdf file? One of the final things I want to show is a tool to build an animation, but at present I can't find any way to embed it.
I will need to ask about this, not sure this is possible.
I’m fairly sure the answer is no
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@lpantano @danielskatz
Okay - I've removed the animation from the paper.md file. Should be good to go now.
@whedon check references
Attempting to check references...
OK DOIs
- 10.1098/rsif.2011.0403 is OK
- 10.1103/PhysRevE.80.020901 is OK
- 10.6084/m9.figshare.1164194 is OK
MISSING DOIs
- https://doi.org/10.1016/j.cpc.2017.06.007 may be missing for title: Optimized Gillespie algorithms for the simulation of Markovian epidemic processes on large and heterogeneous networks
- https://doi.org/10.1090/s0002-9947-1945-0013857-4 may be missing for title: Markoff chains–denumerable case
- https://doi.org/10.18637/jss.v084.i08 may be missing for title: EpiModel: An R Package for Mathematical Modeling of Infectious Disease over Networks
- https://doi.org/10.1093/comnet/cnv021 may be missing for title: Complex contagions and hybrid phase transitions
- https://doi.org/10.1101/297267 may be missing for title: FAVITES: simultaneous simulation of transmission networks, phylogenetic trees and sequences
- https://doi.org/10.1186/s13362-019-0058-7 may be missing for title: A monotonic relationship between the variability of the infectious period and final size in pairwise epidemic modelling
- https://doi.org/10.1515/9781400841356.497 may be missing for title: A simple model of global cascades on random networks
- https://doi.org/10.1086/jar.33.4.3629752 may be missing for title: An information flow model for conflict and fission in small groups
INVALID DOIs
- None
@joelmiller, it seems there are missing DOIs, can you double check this? Thanks!
@whedon check references
Attempting to check references...
OK DOIs
- 10.1016/j.physa.2006.06.018 is OK
- 10.1016/j.cpc.2017.06.007 is OK
- 10.1090/S0002-9947-1945-0013857-4 is OK
- 10.18637/jss.v084.i08 is OK
- 10.1007/978-3-319-50806-1 is OK
- 10.1021/j100540a008 is OK
- 10.25080/issn.2575-9752 is OK
- 10.1093/comnet/cnv021 is OK
- 10.1098/rsif.2011.0403 is OK
- 10.1103/PhysRevE.80.020901 is OK
- 10.1093/bioinformatics/bty921 is OK
- 10.6084/m9.figshare.1164194 is OK
- 10.1103/PhysRevLett.103.058701 is OK
- 10.1186/s13362-019-0058-7 is OK
- 10.1073/pnas.082090499 is OK
- 10.1086/jar.33.4.3629752 is OK
MISSING DOIs
- None
INVALID DOIs
- None
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
@whedon generate pdf
Attempting PDF compilation. Reticulating splines etc...
Submitting author: @joelmiller (Joel Miller) Repository: https://github.com/springer-math/Mathematics-of-Epidemics-on-Networks Version: v1.1 Editor: @lpantano Reviewers: @acolum, @pholme, @hagberg Archive: 10.5281/zenodo.3572756
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
@acolum & @pholme, 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 @lpantano know.
✨ Please try and complete your review in the next two weeks ✨
Review checklist for @acolum
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @pholme
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper
Review checklist for @hagberg
Conflict of interest
Code of Conduct
General checks
Functionality
Documentation
Software paper