add xgb.Booster methods to feature_effects() and partial_dependence()#60
add xgb.Booster methods to feature_effects() and partial_dependence()#60btupper wants to merge 1 commit intomayer79:mainfrom
Conversation
|
Thanks for digging into this. But: PS: XGBoost is currently working on a big release where the PPS: I am currently working on a new release of {effectplots}, getting rid of the |
|
Thanks for the speedy response! Those updates sound great and we are happy to wait. I'm pretty sure I don't understand "It would be unnatural to fit on matrix data and then apply the model to data frames." They are both just arrays with column variables to my pea-brain. We often fit models with data frames and then apply with matrices (well, raster data). That said, it might not be important for me to understand. So, if feature_effects() works well with matrices, I wonder where our hang up occurs. I know that somewhere down in its innards xgboost prefers sparse matrices, and we found ourselves needing to cast those as |
|
I see. Let's craft an example where the problem pops up. In many situations, one can solve smallish problems with the This said: the methods for {ranger} objects are also not really necessary as one could pass Those for DALEX explainers or h2o models are more relevant in this respect. |
Hello,
We have been using your nice package in a white shark forecasting study where we use tidymodels to shape our workflow. We bumped into issues using
effectplotswith tidymodel workflows build withxgboost. This pull request offersfeature_effects()andpartial_dependence()methods forxbg.Boosterclass objects. We didn't make a test for it, but we can add that if you like. We hope you might consider this addition toeffectplots.Thank you,
Ben and Kyle (@kolive4)