Skip to content

remove random forest from model eval chapter - #496

Merged
topepo merged 3 commits into
mainfrom
86-rf
Aug 31, 2026
Merged

remove random forest from model eval chapter#496
topepo merged 3 commits into
mainfrom
86-rf

Conversation

@topepo

@topepo topepo commented Aug 20, 2026

Copy link
Copy Markdown
Member

In section 4, we show how fit_resamples() works, then do the exact same thing with random forest. Immediately in the next section, we show boosted trees (very similar to RF). Given that we have a single-day workshop, this seems wasteful.

@topepo
topepo requested a review from hfrick August 20, 2026 17:29

@hfrick hfrick left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The reason for the random forest in this deck is that we wanted to show a second model since the whole reason for resamples is to be able to compare different models. (And the whole game status update shows multiple models.) I think that's a good point to reiterate at the end of this deck, however you chose to do it.

cls_res
```

## Evaluating model performance `r hexes(c("probably", "tune"))`

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is an earlier exercise that introduces probably (slide 13) so this feels a little like repetition. Is that intended?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it is

@topepo
topepo merged commit e06abd6 into main Aug 31, 2026
@topepo
topepo deleted the 86-rf branch August 31, 2026 20:29
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

Successfully merging this pull request may close these issues.

2 participants