You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
__________________________________ test_plot __________________________________
mne\time_frequency\tests\test_tfr.py:522: in test_plot
cbar = fig.get_axes()[0].CB # Fake dragging with mouse.
E AttributeError: 'AxesSubplot' object has no attribute 'CB'
This one: the order of elements in fig.get_axes() appears to have changed between matplotlib 3.3.x and 3.4rc1. Long-term this would be avoided by having all 2D figures using the new figure classes (it would be easy to store pointers to specific AxesSubplots). Short term I've added an ugly workaround to the test.
___________________________ test_plot_epochs_clicks ___________________________
mne\viz\tests\test_epochs.py:174: in test_plot_epochs_clicks
assert first_ch in fig.mne.info['bads']
E AssertionError: assert 'MEG 0113' in []
Here, clicking the channel names fires off two pick events in a row in MPL 3.4.rc1, effectively marking and then immediately unmarking the channel before a redraw happens. Upstream bug report: matplotlib/matplotlib#19576
____________________________ test_plot_ica_sources ____________________________
mne\viz\tests\test_ica.py:233: in test_plot_ica_sources
assert_array_equal(ica.exclude, [0])
E AssertionError:
E Arrays are not equal
E
E (shapes (0,), (1,) mismatch)
E x: array([], dtype=float64)
E y: array([0])
This is caused by the same issue above (duplicate pick events)
___________________________ test_plot_raw_selection ___________________________
mne\viz\tests\test_raw.py:258: in test_plot_raw_selection
assert sel_fig.lasso.ec[:, 0].sum() == 1 # one channel red
E assert 0.0 == 1
E +0.0
E -1
This is caused by the same issue above (duplicate pick events)
_________________________ test_plot_raw_child_figures _________________________
mne\viz\tests\test_raw.py:332: in test_plot_raw_child_figures
assert len(fig.mne.child_figs) == 1
E assert 0 == 1
E +0
E -1
This is (probably) caused by the same issue above (duplicate pick events)
____________________________ test_plot_raw_traces _____________________________
mne\viz\tests\test_raw.py:393: in test_plot_raw_traces
assert label in fig.mne.info['bads']
E AssertionError: assert 'MEG 0112' in []
This is caused by the same issue above (duplicate pick events)
_________________________ test_plot_topo_image_epochs _________________________
mne\viz\tests\test_topo.py:240: in test_plot_topo_image_epochs
assert num_figures_before + 1 == len(plt.get_fignums())
E assert 2 == 1
E +2
E -1
_____________________________ test_plot_tfr_topo ______________________________
mne\viz\tests\test_topo.py:268: in test_plot_tfr_topo
_fake_click(fig, fig.axes[0], (0.08, 0.65))
E Failed: DID NOT WARN. No warnings of type (<class 'RuntimeWarning'>,) was emitted. The list of emitted warnings is: [].
cannot reproduce locally so far. Probably caused by a different MPL commit than the one above.
And locally my experience on some (in the last few weeks) matplotlib master is that mne browse_raw sample_audvis_raw.fif produces something with spacing way off:
This could be causing most of the errors about counts being wrong, since those rely on clicks landing in the right places.
I usually do a bunch of pip install -ve . during git bisect when I hit these mpl bugs. Then you can see the PR that broke it and complain to them or work around it :)
@drammock I actually don't have any failure in pytest mne/time_frequency/tests/test_tfr.py or pytest mne/viz on latest matplotlib master, but I do still have the bug that mne browse_raw sample_audvis_raw.fif creates this:
And after a tiny resize it's fixed:
So we still have some event problem. Do you see this locally? If so, do you want to git bisect matplotlib and/or MNE to figure out the source? If not (you don't have the issue or don't want to debug) I can do it.
Haven't checked lately but I was also seeing the margin issue before. I can debug, but won't get to it today (still digging myself out after the sprint)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This one: the order of elements in
fig.get_axes()appears to have changed between matplotlib 3.3.x and 3.4rc1. Long-term this would be avoided by having all 2D figures using the new figure classes (it would be easy to store pointers to specific AxesSubplots). Short term I've added an ugly workaround to the test.Here, clicking the channel names fires off two pick events in a row in MPL 3.4.rc1, effectively marking and then immediately unmarking the channel before a redraw happens. Upstream bug report: matplotlib/matplotlib#19576
This is caused by the same issue above (duplicate pick events)
This is caused by the same issue above (duplicate pick events)
This is (probably) caused by the same issue above (duplicate pick events)
This is caused by the same issue above (duplicate pick events)
cannot reproduce locally so far.
git bisectlocalizes this one to MPL commit matplotlib/matplotlib@68498c6cannot reproduce locally so far. Probably caused by a different MPL commit than the one above.