Skip to content

MRG: Set _Brain window title#7396

Merged
agramfort merged 3 commits intomne-tools:masterfrom
GuillaumeFavelier:brain_window_title
Mar 6, 2020
Merged

MRG: Set _Brain window title#7396
agramfort merged 3 commits intomne-tools:masterfrom
GuillaumeFavelier:brain_window_title

Conversation

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor

This PR follows #7382 and unifies the behaviour of the title parameter of _Brain.

It's of course still possible to add a title string label with set_3d_title().

It's an item of #7162

Copy link
Copy Markdown
Member

@agramfort agramfort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works !

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2020

Codecov Report

Merging #7396 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #7396      +/-   ##
==========================================
+ Coverage   90.00%   90.01%   +0.01%     
==========================================
  Files         453      453              
  Lines       82115    82223     +108     
  Branches    12999    13011      +12     
==========================================
+ Hits        73904    74015     +111     
+ Misses       5387     5382       -5     
- Partials     2824     2826       +2     

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

I had to move _set_3d_title a bit further in the code because mayavi complains when rendering to an empty figure.

@GuillaumeFavelier GuillaumeFavelier changed the title Set _Brain window title MRG: Set _Brain window title Mar 6, 2020
@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

I think this is ready to be merged @agramfort, @larsoner

@agramfort agramfort merged commit 32048ae into mne-tools:master Mar 6, 2020
@agramfort
Copy link
Copy Markdown
Member

thx @GuillaumeFavelier

AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* Change window title

* Use subject_id as header

* Move _set_3d_title
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* Change window title

* Use subject_id as header

* Move _set_3d_title
@GuillaumeFavelier GuillaumeFavelier deleted the brain_window_title branch June 11, 2020 09:51
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.

2 participants