Skip to content
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

Update simulation functions #44

Merged
merged 11 commits into from
Jan 4, 2024
Merged

Update simulation functions #44

merged 11 commits into from
Jan 4, 2024

Conversation

joshwlambert
Copy link
Member

This PR updates the simulation functions, both exported and internal, given suggestions from the full package review (#33).

Updates include:

  • Rename min_chain_size argument to min_outbreak_size
  • Remove ... and chkDots() from exported simulation functions
  • Stricter input checking on delay distributions that can be either anonymous functions or <epidist> objects
  • Column deletion is done by name rather than by position
  • sim_linelist() documentation is updated
  • add_ct now defaults to TRUE
  • Maximum iteration limit is added to while loop when simulation branching process

Tests are updated in accordance with function updates, vignettes are updated with new argument names.

@joshwlambert joshwlambert merged commit e8bbbfa into main Jan 4, 2024
8 checks passed
@joshwlambert joshwlambert deleted the updt_sim_funcs branch January 4, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant