Skip to content

954 prediction explanations doc #981

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

Merged
merged 25 commits into from
Jul 28, 2020
Merged

Conversation

freddyaboulton
Copy link
Contributor

@freddyaboulton freddyaboulton commented Jul 27, 2020

Pull Request Description

Fixes #954 by adding explain_prediction to the api reference and by adding example usage to the Model Understanding tutorial in the user guide.

Docs Change

image


After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@codecov
Copy link

codecov bot commented Jul 27, 2020

Codecov Report

Merging #981 into main will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #981   +/-   ##
=======================================
  Coverage   99.85%   99.85%           
=======================================
  Files         178      179    +1     
  Lines        9282     9283    +1     
=======================================
+ Hits         9269     9270    +1     
  Misses         13       13           
Impacted Files Coverage Δ
...alml/pipelines/prediction_explanations/__init__.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad5a695...f16795c. Read the comment docs.

@freddyaboulton freddyaboulton marked this pull request as ready for review July 27, 2020 21:21
@freddyaboulton freddyaboulton requested review from dsherry and jeremyliweishih and removed request for dsherry and jeremyliweishih July 27, 2020 21:21
Copy link
Contributor

@angela97lin angela97lin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well-explained! LGTM :D

Not blocking but do we need the evalml/tests/pipeline_tests/explanations_tests/__init__.py empty file?

@freddyaboulton
Copy link
Contributor Author

@angela97lin Thanks! Good point, I'll delete in this PR!

@freddyaboulton freddyaboulton merged commit c828bed into main Jul 28, 2020
@freddyaboulton freddyaboulton deleted the 954-prediction-explanations-doc branch July 28, 2020 15:11
"\n",
"In the example below, we explain the prediction for the third data point in the data set. We see that the `worst concave points` feature increased the estimated probability that the tumor is malignant by 20% while the `worst radius` feature decreased the probability the tumor is malignant by 5%.\n",
"\n",
"The interpretation of the table is the same for regression problems - but the SHAP value now corresponds to the change in the estimated value of the dependent variable rather than a change in probability. For multiclass classification problems, a table will be output for each possible class."
Copy link
Contributor

@dsherry dsherry Jul 28, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@freddyaboulton I just checked this out. This looks great! My only thought was this paragraph could go after the example since its not directly connected to the example.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dsherry ! I like the suggestion and I'll make the change in #986

@angela97lin angela97lin mentioned this pull request Jul 31, 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.

Write tutorial of prediction explanations feature
3 participants