Plot_confusion_matrix vertical alignment?

oriolabril · March 18, 2019, 10:52am

Hi everyone,

I started watching the online lessons yesterday and it shocked me that when plotting the confusion matrix with plot_confusion_matrix() the numbers in the matrix are only horizontally aligned but not vertically aligned. I checked the source code and I saw that horizontalalignment="center" is specified but the vertical alignment is left at its default. Is it on purpose?

I could make a pull request to add the verticalalignment="center", it should not affect the tests, break anything nor require extra tests.

sgugger · March 18, 2019, 1:36pm

I don’t think it’s on purpose, so you’re welcome to suggest a PR to fix this.

oriolabril · March 18, 2019, 2:04pm

Thanks for the fast reply. I have already suggested the PR.

austinmw · July 30, 2019, 10:51pm

I’m experiencing vertical alignment issues with plot_confusion_matrix text:

Tried all of top, bottom, center, baseline, center_baseline for verticalalignment value, but none seemed to work properly.

ashleykleinhans · August 5, 2019, 5:27pm

Me too - did you solve this?

oriolabril · August 5, 2019, 6:31pm

This looks like an issue with the y-axis limits, which is why no alignment value solves the issue.