Skip to content

Update docs for confusion matrix methods - #1611

Merged
angela97lin merged 3 commits into
mainfrom
ange_blitz_documentation_updates
Dec 28, 2020
Merged

Update docs for confusion matrix methods#1611
angela97lin merged 3 commits into
mainfrom
ange_blitz_documentation_updates

Conversation

@angela97lin

@angela97lin angela97lin commented Dec 28, 2020

Copy link
Copy Markdown
Contributor

@angela97lin angela97lin self-assigned this Dec 28, 2020
@codecov

codecov Bot commented Dec 28, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1611 (3d0a495) into main (7138ec2) will not change coverage.
The diff coverage is 100.0%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1611   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         240      240           
  Lines       18056    18056           
=======================================
  Hits        18048    18048           
  Misses          8        8           
Impacted Files Coverage Δ
evalml/model_understanding/graphs.py 99.8% <100.0%> (ø)
...lml/tests/model_understanding_tests/test_graphs.py 100.0% <100.0%> (ø)

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 7138ec2...3d0a495. Read the comment docs.

@angela97lin
angela97lin marked this pull request as ready for review December 28, 2020 22:35

@bchen1116 bchen1116 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@angela97lin
angela97lin merged commit 54bf169 into main Dec 28, 2020
@angela97lin
angela97lin deleted the ange_blitz_documentation_updates branch December 28, 2020 23:46
@dsherry dsherry mentioned this pull request Dec 29, 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.

normalize_method can be None should be documented for graph_confusion_matrix Error message for graph_confusion_matrix is incorrectly formatted

2 participants