You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This seems rather concerning. It's not obvious to me where precisely the link function should be applied in TreeExplainer, if at all. As far as I can make out, the TreeExplainer uses the "model_output" inferred from the explained model rather than the user-defined "link" variable.
The text was updated successfully, but these errors were encountered:
connortann
changed the title
BUG: The TreeExplainer ignores the link argument when created via the Explainer API
BUG: TreeExplainer ignores "link" argument passed by the Explainer API
Feb 19, 2024
The current implementation of TreeExplainer entirely ignores the
link
andlinearize_link
arguments which are passed fromExplainer
here:shap/shap/explainers/_explainer.py
Line 188 in b3573bd
They are currently caught by the "deprecated options" catchall, and subsequently ignored:
shap/shap/explainers/_tree.py
Lines 63 to 72 in b3573bd
This seems rather concerning. It's not obvious to me where precisely the link function should be applied in TreeExplainer, if at all. As far as I can make out, the TreeExplainer uses the "model_output" inferred from the explained model rather than the user-defined "link" variable.
The text was updated successfully, but these errors were encountered: