Skip to content

Add EvokedField.plotter#12005

Merged
larsoner merged 4 commits intomne-tools:mainfrom
wmvanvliet:evoked_field_plotter
Sep 20, 2023
Merged

Add EvokedField.plotter#12005
larsoner merged 4 commits intomne-tools:mainfrom
wmvanvliet:evoked_field_plotter

Conversation

@wmvanvliet
Copy link
Copy Markdown
Contributor

This should fix the doc build. It follows a similar approach as the Brain object, namely to expose a .plotter so that things like set_3d_title work also when passing EvokedField objects.

@larsoner
Copy link
Copy Markdown
Member

Can you touch the two failing examples in this PR? It causes CircleCI to execute them so we can verify that they don't fail and still look okay

@larsoner larsoner requested a review from agramfort as a code owner September 20, 2023 15:20
@larsoner
Copy link
Copy Markdown
Member

I guess it was just one failing example, pushed. Will examine once it renders and mark for merge-when-green, thanks in advance @wmvanvliet !

@larsoner larsoner enabled auto-merge (squash) September 20, 2023 15:50
@larsoner larsoner merged commit 034e9dc into mne-tools:main Sep 20, 2023
snwnde pushed a commit to snwnde/mne-python that referenced this pull request Mar 20, 2024
Co-authored-by: Eric Larson <larson.eric.d@gmail.com>
@wmvanvliet wmvanvliet deleted the evoked_field_plotter branch July 18, 2024 06:11
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

Successfully merging this pull request may close these issues.

2 participants