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

extract_parameter_set_dials method #53

Closed
topepo opened this issue Oct 23, 2024 · 3 comments
Closed

extract_parameter_set_dials method #53

topepo opened this issue Oct 23, 2024 · 3 comments

Comments

@topepo
Copy link
Member

topepo commented Oct 23, 2024

tune is showing errors of

Error in UseMethod("extract_parameter_set_dials"): no applicable method for 'extract_parameter_set_dials' applied to an object of class "tailor"

I know it hasn't been implemented yet. I'll add skips in the tests for tailor >= 0.0.0.9002. Can you bump the version number once that method is implemented?

topepo added a commit to tidymodels/tune that referenced this issue Oct 23, 2024
@simonpcouch
Copy link
Contributor

You may need to install #51!

@simonpcouch
Copy link
Contributor

Ah, looks like I forgot to update that Remotes field in tune.

topepo added a commit to tidymodels/tune that referenced this issue Oct 23, 2024
* add type checking files

* remove newly unneeded checking functions

* snapshot updates from tidymodels/recipes#1381

* updates files

* basic replacements

* type checker replacements

* tidymodels/tailor#53

* Update R/checks.R

Co-authored-by: Simon P. Couch <[email protected]>

* add remote to get proper error messages

* typo

* update remotes?

* only test snapshots with more recent version of R *with* rankdeficient

---------

Co-authored-by: Simon P. Couch <[email protected]>
@simonpcouch
Copy link
Contributor

This has been resolved in the meantime.

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

No branches or pull requests

2 participants