mir_eval icon indicating copy to clipboard operation
mir_eval copied to clipboard

Fix display tests for matplotlib 3.9

Open bmcfee opened this issue 1 year ago • 4 comments

This came up in #382 , but matplotlib 3.9 is causing some regressions on display tests. From a quick read-through of the changelog, I don't see any obvious culprits.

It may have something to do with legend positioning or tick markers. :shrug:

bmcfee avatar Jun 12 '24 15:06 bmcfee

Had a chance to test this locally, it's a grab bag. Most failures are indeed due to legend location, and we can fix this by setting the loc explicitly.

Segment displays are somehow back to not setting limits properly though, so the viewport is completely wrong. These will take some work to fix.

bmcfee avatar Jun 12 '24 15:06 bmcfee

Ok, I think this issue with segments is due to this change: https://matplotlib.org/stable/api/prev_api_changes/api_changes_3.9.0.html#boxplots-now-ignore-masked-data-points - axvspan now creates Rectangle instead of Polygon patches, and for some reason that means they are now not participating in data limit calculations.

bmcfee avatar Jun 12 '24 15:06 bmcfee

Issue punted upstream to https://github.com/matplotlib/matplotlib/issues/28383

bmcfee avatar Jun 12 '24 15:06 bmcfee

Looks like the fix is merged and will be in the 3.9.1 release.

bmcfee avatar Jul 02 '24 11:07 bmcfee