-
Notifications
You must be signed in to change notification settings - Fork 31
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
fix_dist -> fix_parameters #733
base: main
Are you sure you want to change the base?
Conversation
This is how benchmark results would change (along with a 95% confidence interval in relative change) if 81258af is merged into main:
|
@@ -11,6 +11,10 @@ | |||
- The interface for defining delay distributions has been generalised to also cater for continuous distributions | |||
- When defining probability distributions these can now be truncated using the `tolerance` argument | |||
|
|||
## Package changes | |||
|
|||
- `fix_dist()` has been renamed to `fix_parameters()`. By @sbfnk in and reviewed by. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `fix_dist()` has been renamed to `fix_parameters()`. By @sbfnk in and reviewed by. | |
- `fix_dist()` has been renamed to `fix_parameters()`. By @sbfnk in #733 and reviewed by @jamesmbaazam. |
#' @param strategy Character; either "mean" (use the mean estimates of the | ||
#' mean and standard deviation) or "sample" (randomly sample mean and | ||
#' standard deviation from uncertainty given in the `<dist_spec>` | ||
fix_parameters <- function(x, strategy = c("mean", "sample")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix_parameters <- function(x, strategy = c("mean", "sample")) { | |
fix_dist <- function(x, strategy = c("mean", "sample")) { |
#' standard deviation from uncertainty given in the `<dist_spec>` | ||
fix_parameters <- function(x, strategy = c("mean", "sample")) { | ||
lifecycle::deprecate_warn( | ||
"1.6.0", "fix_parameters()", "fix_parameters()" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"1.6.0", "fix_parameters()", "fix_parameters()" | |
"1.6.0", "fix_dist()", "fix_parameters()" |
@@ -108,7 +108,8 @@ reference: | |||
- bound_dist | |||
- collapse | |||
- discretise | |||
- fix_dist | |||
- fix_parameters |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- fix_parameters |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. You could also add a test for deprecation of fix_dist()
.
Description
This PR closes #715.
Initial submission checklist
devtools::test()
anddevtools::check()
).devtools::document()
).lintr::lint_package()
).After the initial Pull Request