Skip to content

Conversation

topepo
Copy link
Member

@topepo topepo commented Aug 30, 2022

As fit() currently does.

@topepo topepo requested a review from DavisVaughan August 30, 2022 19:15
Copy link
Member

@DavisVaughan DavisVaughan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is probably worth testing this against the book and the workshop notes? It seems like something that people could definitely accidentally rely on.

Also, I think you should be able to remove every usage of check_mode(), because mode is now required in all cases and is never inferred "automatically"
https://github.com/tidymodels/parsnip/search?q=check_mode

@topepo
Copy link
Member Author

topepo commented Aug 31, 2022

Checked against TMwR and revdeps. Everything looks good.

@DavisVaughan
Copy link
Member

Ok, I think we are good to move forward with this then. I'll go ahead and release workflows with this change.

@topepo topepo merged commit 301cf72 into main Aug 31, 2022
@topepo topepo deleted the fit-xy-unknown-mode branch August 31, 2022 15:13
@github-actions
Copy link

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.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants