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

Update confusion matrix to give both counts and normalized values in one figure #86

Open
nbagherzadeh opened this issue Jul 1, 2022 · 2 comments

Comments

@nbagherzadeh
Copy link
Contributor

We currently have the option to generate row-, and column-normalized confusion matrices, or non-normalized confusion matrices.

Having an option to generate normalized CM that also shows the counts are very helpful to better understand class-imbalances.

Example below:

image

@hannahilea
Copy link
Contributor

hannahilea commented Jul 1, 2022

Thoughts about making it a (/the) default implementation? I.e., update our current row- and column- normalized cm to include counts? Might be useful to have it visible by default rather than having yet another kwarg... In which case the action items would be:

  • for row- and column-normalized cm, add count above (or below) the current normalized value
  • for non-normalized cm, do nothing (?)

While we're at it, do we want to make the row- and column-normalized outputs percents? We don't currently do that...

@hannahilea
Copy link
Contributor

Discussed with @nbagherzadeh; barring other opinions, we think this could/should be the default: all normalized plots also show a count above the fraction. For now, not converting to percent.

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

No branches or pull requests

2 participants