-
Notifications
You must be signed in to change notification settings - Fork 42
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
Feature suggestion: Extract splits from tune results as a resampling object #947
Comments
Could you say a little bit more about why it is that you'd need to extract the splits from the Note to self: FWIW, we did find a use for a similar helper in |
Well. In my pipelines I usually have one process for fitting resamples & tuning and sometimes I only save the tune_resamples object and not the rset... But, then "ups" I need the rset too because I want to check something and I didnt save it. {tailor} could increase the probability of this issue. Furthermore, I want to try AutoGuon inference approach and this function could help. |
Gotcha, thanks for the reply! I will leave this open as we can see some use cases for this, though it may not be at the top of our to-do for a bit. |
If you guide me on what tests do you like to include, I would make a proper PR, so we can merge it. |
Feature suggestion
Now that we have the new {tailor} package for post-processing in titydmodels, I find myself in the need to reuse the splits from tune_results as a resampling object.
I believe this new extract_resamples function (or whatever name you prefer) could improve the interactive usage of tidymodels.
Here a minimal reproducible example to demonstrate its use:
This implementation seems to give identical results for my vfold_cv example, but I guess other rset type of objects should be tested.
The text was updated successfully, but these errors were encountered: