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: Use a text slider widget #7186

Merged

Conversation

GuillaumeFavelier
Copy link
Contributor

@GuillaumeFavelier GuillaumeFavelier commented Jan 9, 2020

This PR uses the new text widget introduced recently in PyVista and avoid the creation of a custom slider.

This is refactoring, the features won't change.

It's an item of #7162

@GuillaumeFavelier
Copy link
Contributor Author

GuillaumeFavelier commented Jan 9, 2020

Travis is happy except the job on macOS. I restarted it.

@codecov
Copy link

codecov bot commented Jan 9, 2020

Codecov Report

Merging #7186 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7186      +/-   ##
==========================================
- Coverage   89.76%   89.76%   -0.01%     
==========================================
  Files         445      445              
  Lines       79741    79723      -18     
  Branches    12755    12753       -2     
==========================================
- Hits        71579    71561      -18     
  Misses       5369     5369              
  Partials     2793     2793

@GuillaumeFavelier GuillaumeFavelier changed the title WIP: Use a text slider widget MRG: Use a text slider widget Jan 9, 2020
@GuillaumeFavelier
Copy link
Contributor Author

What do you think @agramfort , @larsoner ?

@agramfort
Copy link
Member

works like a charm

@agramfort agramfort merged commit e497189 into mne-tools:master Jan 9, 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 use_text_slider_widget branch June 11, 2020 09:37
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