Skip to content

Support explaining DNNs using SHAP#1694

Merged
shendiaomo merged 6 commits intosql-machine-learning:developfrom
shendiaomo:explain_dnn
Jan 14, 2020
Merged

Support explaining DNNs using SHAP#1694
shendiaomo merged 6 commits intosql-machine-learning:developfrom
shendiaomo:explain_dnn

Conversation

@shendiaomo
Copy link
Copy Markdown
Collaborator

This is based on the discussion in shap/shap#280
The screenshot looks like:
image

wangkuiyi
wangkuiyi previously approved these changes Jan 13, 2020
wangkuiyi
wangkuiyi previously approved these changes Jan 14, 2020
@tonyyang-svail
Copy link
Copy Markdown
Collaborator

Hi @shendiaomo, thanks for submitting this PR. Could you please add SQLs in the demo as our test cases?

@typhoonzero
Copy link
Copy Markdown
Collaborator

Maybe also test case like explain_example.py and call it in the test train_predict_test.py

wangkuiyi
wangkuiyi previously approved these changes Jan 14, 2020
@shendiaomo
Copy link
Copy Markdown
Collaborator Author

Hi @shendiaomo, thanks for submitting this PR. Could you please add SQLs in the demo as our test cases?

Done.

@shendiaomo
Copy link
Copy Markdown
Collaborator Author

Maybe also test case like explain_example.py and call it in the test train_predict_test.py

Later in another PR?

@shendiaomo shendiaomo requested a review from wangkuiyi January 14, 2020 15:13
@shendiaomo shendiaomo merged commit 9132741 into sql-machine-learning:develop Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants