Patrick Schaefer

Results 19 comments of Patrick Schaefer

@mayer79 Thanks. I've tried TreeShap as well, and unfortunately ran into another issue since it calls `lightgbm::lgb.model.dt.tree(lgb_model)` which dumps the trees as a single text string, which R can't handle...

I'm sorry, I'm not sure. I didn't test it. All I know is that `phis

I've taken a crack at adding support for `case_weight()` [72](https://github.com/tidymodels/bonsai/pull/72)

Thanks for the review @simonpcouch. Note, I am also working on a method that allows some ability to customize validation data beyond just specifying a proportion of training data. I...

Sorry, this fell off my radar. I can finish addressing the comments next week, if that works for you. Get Outlook for Android ________________________________ From: Simon P. Couch ***@***.***> Sent:...

I've fixed the spacing issues, and dealt with the intermediate file from testthat. Should I submit a new PR?

Sorry, forgot to rerun the tests and checks. Looks good to me now, let me know if there is anything else.

No problem. It might be worth opening a new issue to discuss how to deal with `linear_tree` (if you want some help). I think we can probably come up with...

@simonpcouch Was there anything else you need from me?