search

tidymodels / rules

parsnip extension for rule-based models
https://rules.tidymodels.org
Other
40 stars 4 forks source link

C50 tree tidy method #70

Closed topepo closed 1 year ago

topepo commented 1 year ago

Now works with trees by showing terminal nodes in rules

hfrick commented 1 year ago

We've recently had an issue opened on broom suggesting to move the tidiers from poissonreg into broom for better discoverability. Do you have specific reasons to keep them in rules? Otherwise, I'd argue for putting these tidiers into broom.

EmilHvitfeldt commented 1 year ago

I'm going to add to this PR because I don't like the tests. I will try to add the tests that I manually used when developing the recursive code

github-actions[bot] commented 1 year ago

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.