Add Getting Started vignette#201
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThe pull request adds tinyplot as a suggested package dependency, introduces a new bibliography reference entry, and includes a comprehensive "Getting started" vignette demonstrating ringbp package workflows encompassing model configuration, simulations, visualisations, and extinction probability analysis. Changes
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 5
🤖 Fix all issues with AI agents
In `@vignettes/ringbp.Rmd`:
- Line 91: The sentence defining isolation is missing the word "of"; update the
line that starts with "Isolation is the separation infectious individuals from
others, with the aim to prevent further transmission." to read "Isolation is the
separation of infectious individuals from others, with the aim to prevent
further transmission." so the term "Isolation" definition is grammatical and
clear.
- Around line 60-62: Remove the duplicate word "the" in the sentence defining
the offspring distribution by editing the phrase "The mean of the the offspring
distribution is the basic reproduction number (R_0)" to "The mean of the
offspring distribution is the basic reproduction number (R_0)". Update this
occurrence in the vignette text around the sentence that begins "The mean of the
the offspring distribution..." in vignettes/ringbp.Rmd so the wording is correct
and runs as a single "the".
- Line 52: Fix the duplicated word "and and" in the sentence that links to the
[`ringbp-model.Rmd` vignette]; update the sentence text to read "...model and
non-pharmaceutical interventions influence disease transmission..." (or
optionally "...model and the influence of non-pharmaceutical interventions on
disease transmission...") so the duplicate "and" is removed.
- Line 99: In the vignette text referencing event_prob_opts(), correct the
misspelled parameter name "presymtomatic_transmission" to
"presymptomatic_transmission" so the documentation matches the actual parameter;
update the sentence mentioning `asymptomatic`, `presymptomatic_transmission`,
and `symptomatic_ascertained` accordingly.
- Around line 119-129: Fix wording and a misspelling in the quarantine
explanatory paragraph: rephrase to something like "Quarantine in the {ringbp}
model is isolation of individuals regardless of infection status: contacts are
quarantined as soon as the infector is confirmed and isolated, which can occur
before contacts become symptomatic. This differs from isolation, which requires
the infectee to be symptomatic." Correct the misspelling "qurantine" to
"quarantine", ensure the code example using intervention_opts(quarantine =
FALSE) and the sentence "By default, isolation of symptomatic cases is active
and quarantine is not active." remain clear and grammatically correct; update
the paragraph surrounding the variable interventions and function
intervention_opts() accordingly.
pearsonca
left a comment
There was a problem hiding this comment.
Some minor notes; looks good overall
Co-authored-by: Sebastian Funk <sebastian.funk@lshtm.ac.uk> Co-authored-by: Carl A. B. Pearson <pearsonca@users.noreply.github.com>
This PR addresses #139 by adding a Getting started vignette (
ringbp.Rmd) to the package.The vignette gives an overview of the {ringbp} model and how to parameterise it, with an explanation of the model parameters. It runs a simple
scenario_sim()example and shows how to plot the outbreak (both for weekly and cumulative cases), and summarise the results usingextinct_prob().As suggested in #139 it contains a simplified version of the analysis in Hellewell et al.
The plotting in
ringbp.Rmduses {tinyplot} which does not include any non-base R dependencies so it is a nice lightweight dependency and makes it easy to plot the outbreak data. {tinyplot} is added toSuggestsin theDESCRIPTIONfile.A
references.bibfile is added tovignettes/, this includes the Hellewell et al. citation which is cited inringbp.Rmd.Summary by CodeRabbit
Documentation
Chores
✏️ Tip: You can customize this high-level summary in your review settings.