The Wayback Machine - https://web.archive.org/web/20220525210937/https://github.com/scikit-learn/scikit-learn/pull/20753
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

ENH Adds im_kw to ConfusionMatrixDisplay #20753

Merged
merged 7 commits into from Mar 7, 2022

Conversation

thomasjpfan
Copy link
Member

@thomasjpfan thomasjpfan commented Aug 15, 2021

Reference Issues/PRs

Fixes #16048

What does this implement/fix? Explain your changes.

This PR adds im_kw to ConfusionMatrixDisplay, which enables kwargs like vmin or vmax to be passed into plot.

Copy link
Contributor

@glemaitre glemaitre left a comment

LGTM

@thomasjpfan thomasjpfan added this to the 1.1 milestone Sep 11, 2021
@thomasjpfan thomasjpfan added the Quick Review label Feb 22, 2022
@jjerphan jjerphan removed the Quick Review label Mar 7, 2022
doc/whats_new/v1.1.rst Outdated Show resolved Hide resolved
@jjerphan jjerphan merged commit 13174b1 into scikit-learn:main Mar 7, 2022
27 checks passed
Diwakar-Gupta pushed a commit to Diwakar-Gupta/scikit-learn that referenced this issue Mar 7, 2022
thomasjpfan added a commit to thomasjpfan/scikit-learn that referenced this issue Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants