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.

Sign up for GitHub

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

Jump to bottom

Fix kernel explainer save / load method #2604

Open

WeichenXu123 wants to merge 1 commit into shap:master from WeichenXu123:fix-kernel-save-load

Contributor

WeichenXu123 commented Jun 24, 2022

Signed-off-by: Weichen Xu weichen.xu@databricks.com

Fix kernel explainer save / load method:

Issues in original KernelExplainer.save:

It saves model by calling model.save, but shap.utils._legacy.Model has no save method
It tries to save "masker", but there's no "masker" in KernelExplainer
It does not save "KernelExplainer.data" attribute, the attribute is required when loading back

Issues in original KernelExplainer.load:

Use mismatched model loader to load model
Try to load non-existent "masker" attribute
Don't not load "data" attribute and then cause calling " KernelExplainer" constructor lack of "data" argument.


          update

a9eb9a5

Signed-off-by: Weichen Xu <weichen.xu@databricks.com>

dsgibbons mentioned this pull request

Invite open PRs from slundberg/shap to contribute dsgibbons/shap#2

Closed

55 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment