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 {epiparameter} usage #100

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Update {epiparameter} usage #100

merged 1 commit into from
Aug 23, 2024

Conversation

joshwlambert
Copy link
Member

This PR updates the {superspreading} package to use the latest function and class names from the {epiparameter} package (a suggested dependency of {superspreading}).

Since epiverse-trace/epiparameter#360 the <epidist> class was renamed <epiparameter> (see epiverse-trace/epiparameter#325). This came with functions that used the previous naming being removed, and epidist_db() became deprecated and replaced by epiparameter_db().

This PR replaces the use of epidist_db() with epiparameter_db(), renames get_epidist_params() to get_epiparameter_params() and replaces assertions that check the class from <epidist> to <epiparameter>. Error messages are updated, as is function documentation.

This is not a breaking change to {superspreading} as only internal functions are renamed or have arguments changed, and error messages are the only user-facing change. The vignettes are updated to call epiparameter_db().

@joshwlambert joshwlambert merged commit 9dd23e0 into main Aug 23, 2024
8 checks passed
@joshwlambert joshwlambert deleted the updt_epiparameter branch August 23, 2024 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant