tidymodels / tidymodels/probably

Is there a way to take our `tune_results` object and coerce it into the right format to use here?

Open
#71 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

documentation feature
Dominant language
R
Stars
123
Forks
16
PR merge metrics
No merged PRs in 30d

Description

    Is there a way to take our `tune_results` object and coerce it into the right format to use here? 

In the tune_results structure, the data object in the elements of splits are the training data and these new validate functions requires the predictors to be there.

We should talk about this in the Details section (that they are not the same) and maybe have a helper function to convert the original results to something that cal_validate_ * can consume.

I don't see a simple way to keep the same resampling structure as the original tune_results object.

Originally posted by @topepo in https://github.com/tidymodels/probably/pull/63#discussion_r1060156332

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing the tune_results structure, especially the data objects inside splits, with the inputs expected by the cal_validate_* functions. Done means the Details section explains the difference and, if a helper is adopted, the original results can be converted into a consumable structure without losing the intended resampling information.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
data
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.