Skip to content

Conversation

@GuillaumeFavelier
Copy link
Contributor

@GuillaumeFavelier GuillaumeFavelier commented Jan 17, 2020

After merging #7219, the API to fetch hemi data has changed in _Brain. The result is that although the values for the auto-scaling are correct, the colorbar is not changed. This PR updates the old method used in update_auto_scaling().

Before PR
2020-01-17_1920x1080 2020-01-17_1920x1080

Copy link
Member

@larsoner larsoner left a comment

Choose a reason for hiding this comment

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

LGTM and works well here, +1 for merge

@codecov
Copy link

codecov bot commented Jan 17, 2020

Codecov Report

Merging #7226 into master will increase coverage by <.01%.
The diff coverage is 68.75%.

@@            Coverage Diff             @@
##           master    #7226      +/-   ##
==========================================
+ Coverage   89.76%   89.77%   +<.01%     
==========================================
  Files         445      445              
  Lines       80127    80127              
  Branches    12817    12817              
==========================================
+ Hits        71925    71932       +7     
+ Misses       5385     5378       -7     
  Partials     2817     2817

@larsoner larsoner merged commit a1dea20 into mne-tools:master Jan 17, 2020
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 fix_brain_auto_scaling branch June 11, 2020 09:39
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