Skip to content

Delegate notebook detection to PyVista#7240

Merged
agramfort merged 1 commit intomne-tools:masterfrom
GuillaumeFavelier:pyvista_spyder_plot_xhemi
Jan 22, 2020
Merged

Delegate notebook detection to PyVista#7240
agramfort merged 1 commit intomne-tools:masterfrom
GuillaumeFavelier:pyvista_spyder_plot_xhemi

Conversation

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor

This PR delegates the notebook environment detection to the 3d backend instead of using unreliable internal routine _check_notebook().

Fixes #7239.

@larsoner
Copy link
Copy Markdown
Member

@SophieHerbst can you try this and see if it fixes your problem?

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 22, 2020

Codecov Report

Merging #7240 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #7240      +/-   ##
=========================================
+ Coverage   89.66%   89.7%   +0.03%     
=========================================
  Files         445     445              
  Lines       79391   80159     +768     
  Branches    12822   12823       +1     
=========================================
+ Hits        71188   71908     +720     
- Misses       5342    5427      +85     
+ Partials     2861    2824      -37

@SophieHerbst
Copy link
Copy Markdown
Contributor

It works –– awesome! Thank you for the help!

@agramfort agramfort merged commit 89f4538 into mne-tools:master Jan 22, 2020
@agramfort
Copy link
Copy Markdown
Member

thx @GuillaumeFavelier and @SophieHerbst for the team work !

AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
@GuillaumeFavelier GuillaumeFavelier deleted the pyvista_spyder_plot_xhemi branch June 11, 2020 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

error when trying to use pyvista to plot stc

4 participants