-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Catboost bug #7
Comments
Fixed this issue by avoiding "memory check" for CatBoost, since CatBoost is not supported in the current version of
|
Is this also true For xgboost and lgbm? From my understanding in the Paper "tree path dependent" is the better Method For explaining model Performance and interventional is used to explain Relationships in the Data. Also "interventional" is a lot slower so wouldnt a fast tree shap Method make a lot of sense For it? |
Yes. Thanks for your suggestion! It may make sense to accelerate "interventional" as well, however the algorithms used in "tree_path_dependent" and "interventional" are totally different. It is actually much harder to accelerate "interventional" (and I actually doubt the feasibility of accelerating "interventional" from algorithm side), and thus it is out of the scope of this package. |
Catboost produces a TreeEnsemble has no "num_nodes" error with this code. Btw do you support a background dataset parameter, like in shap for "interventional" vs "tree_path_dependent"? Because if your underlying code uses the "interventional" method this might be related to this bug: shap/shap#2557
The text was updated successfully, but these errors were encountered: