Skip to content

Conversation

@GuillaumeFavelier
Copy link
Contributor

This PR fixes the focalpoint issue when hemi='both' in _TimeViewer. It should be placed between the hemispheres.

master PR
2020-03-05_1920x1080 2020-03-05_1920x1080

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

@codecov
Copy link

codecov bot commented Mar 5, 2020

Codecov Report

Merging #7390 into master will increase coverage by 0.01%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master    #7390      +/-   ##
==========================================
+ Coverage   89.99%   90.01%   +0.01%     
==========================================
  Files         453      453              
  Lines       82098    82111      +13     
  Branches    12997    12999       +2     
==========================================
+ Hits        73887    73911      +24     
+ Misses       5385     5378       -7     
+ Partials     2826     2822       -4

@GuillaumeFavelier GuillaumeFavelier changed the title Add reset_camera() primitive in _Renderer MRG: Add reset_camera() primitive in _Renderer Mar 5, 2020
@GuillaumeFavelier
Copy link
Contributor Author

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

@agramfort agramfort merged commit c7af214 into mne-tools:master Mar 5, 2020
@agramfort
Copy link
Member

thx @GuillaumeFavelier !

@GuillaumeFavelier GuillaumeFavelier mentioned this pull request Mar 5, 2020
86 tasks
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* Add reset_camera() primitive

* Add reset_camera to test_renderer

* Add safety check

* Improve safety check
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* Add reset_camera() primitive

* Add reset_camera to test_renderer

* Add safety check

* Improve safety check
@GuillaumeFavelier GuillaumeFavelier deleted the renderer_reset_camera 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