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
switch to cli in various functions #1244
Conversation
cli::cli_abort( | ||
c( | ||
"!" = "Please supply at least one variable specification.", | ||
"i" = "See {.help [?selections](recipes::selections)} \\ | ||
for more information." |
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.
I didn't add tests for this, as it is technically deprecated. I opened an issue to properly handle that here #1243
"the data passed using `training` will be ", | ||
"ignored." | ||
) | ||
if (!is.null(rec$template)) { |
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.
This was the bug that is mentioned in the PR. $training
never worked.
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.
Looking sharp!
Co-authored-by: Simon P. Couch <simonpatrickcouch@gmail.com>
Merge commit '26a7ed8ddcffd9fefe5956b8d97d5a94a62f2778' #Conflicts: # NEWS.md
This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex https://reprex.tidyverse.org) and link to this issue. |
ref: #1237
This PR is a little bitter since there was a couple of use-cases that didn't neatly fit anywhere else.
A bug was found, and fixed, which was about how retraining a recipe will ignore new data, this now throws a warning as intended