Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MRG: Set _Brain window title #7396

Merged
merged 3 commits into from
Mar 6, 2020

Conversation

GuillaumeFavelier
Copy link
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
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

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
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
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
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