Skip to content
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

Add hide_zeros param to confusion matrix #39

Merged
merged 1 commit into from
Jul 10, 2017

Conversation

ExcaliburZero
Copy link
Contributor

Add an optional parameter to the plot_confusion_matrix function to allow the hiding of zero values in the matrix in order to make the plot easier to read in cases where there are many zero values.

Here is what a confusion matrix looks like normally (hide_zeros = False)

scikit_plot_04

Here is the same confusion matrix with zeros hidden (hide_zeros = True)

scikit_plot_06

@reiinakano
Copy link
Owner

This looks really good!

Though one more thing before I merge. Do you mind adding this to https://github.com/reiinakano/scikit-plot/blob/master/scikitplot/classifiers.py#L56 as well? Thanks!

Add an optional parameter to the plot_confusion_matrix function to allow
the hiding of zero values in the matrix in order to make the plot easier
to read in cases where there are many zero values.
@ExcaliburZero
Copy link
Contributor Author

Though one more thing before I merge. Do you mind adding this to https://github.com/reiinakano/scikit-plot/blob/master/scikitplot/classifiers.py#L56 as well? Thanks!

I have now added it there as well.

@reiinakano reiinakano merged commit 9392981 into reiinakano:master Jul 10, 2017
@reiinakano
Copy link
Owner

LGTM. Thanks!

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.

None yet

2 participants