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, DOC: tutorial tweaks #7109

Merged
merged 3 commits into from Dec 3, 2019
Merged

Conversation

drammock
Copy link
Member

@drammock drammock commented Dec 3, 2019

minor tweaks to the preprocessing overview tutorial:

  • small extra text explaining why image plots look blocky (few epochs)
  • apply baseline when extracting artifact epochs
  • picks a nicer thumbnail image

@codecov
Copy link

codecov bot commented Dec 3, 2019

Codecov Report

Merging #7109 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7109      +/-   ##
==========================================
- Coverage   89.78%   89.77%   -0.01%     
==========================================
  Files         444      444              
  Lines       78211    78211              
  Branches    12672    12672              
==========================================
- Hits        70222    70216       -6     
- Misses       5174     5177       +3     
- Partials     2815     2818       +3

@drammock
Copy link
Member Author

drammock commented Dec 3, 2019

here's the rendering: https://17045-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/preprocessing/plot_10_preprocessing_overview.html

Azure/Travis failures are unrelated and addressed by #7108

@drammock drammock changed the title DOC: tutorial tweaks MRG, DOC: tutorial tweaks Dec 3, 2019
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.

+1 for MRG provided CIs are happy. thx @drammock

@larsoner
Copy link
Member

larsoner commented Dec 3, 2019

CIs are happy other than the ones fixed in #7108 so I'll merge. Thanks @drammock

@larsoner larsoner merged commit de3e189 into mne-tools:master Dec 3, 2019
@drammock drammock deleted the tutorial-tweaks branch December 3, 2019 22:36
christian-oreilly pushed a commit to christian-oreilly/mne-python that referenced this pull request Dec 8, 2019
* better artifact plots; better thumbnail choice

* keep it pedagogical

* change thumbnail [ci skip]
agramfort pushed a commit that referenced this pull request Dec 14, 2019
* Fixing issues with plot_surface()

* Setting transparent to True instead of False.

* PR review edits.

* PR review edits.

* Improved error management for missing pysufer.

* Update _3d.py

* Update test_3d.py

* Fixing test requires conditions.

* Document this PR change.

* Update doc/changes/latest.inc

Co-Authored-By: Eric Larson <larson.eric.d@gmail.com>

* fix : don't require git to install master (#7078)

* [MRG] Fix plot_vector_source_estimates (#7084)

* Initial fix

* Update mesh is not necessary

* Choose to modify scale_factor as post-post-process

* Add simple test for fix

* Use mean of hemi's width

* MRG, BUG: Fix running subprocesses in Jupyter notebooks (#7086)

* BUG: Fix running subprocesses in Jupyter notebooks

* FIX: Flake [ci skip]

* FIX: Adding use_scalebars as a parameter. (#7091)

* Adding use_scalebars as a parameter.

* Fixed to adhere to mne-coding standards.

* Fixing formatting issues.

* Fixing show_scalebars in viz ica from the original naming of  use_scalebars.

* Update mne/viz/ica.py

Co-Authored-By: Daniel McCloy <dan.mccloy@gmail.com>

* Update mne/viz/raw.py

Co-Authored-By: Daniel McCloy <dan.mccloy@gmail.com>

* DOC: Correct version [ci skip]

* [MRG] Implement iterative reweighted TF-MxNE (#5075)

* ir scheme

* Squash commits

* messed up my rebase

* Address comments Alex

* flake

* implement filtering and write more readable math code

* fix pydocstyle

* High pass filter, remove upsampling, get 4 sources

* TFMxnE > TF-MxNE + rm comment example

* MAINT: Ensure that meas_date=None round-trips if `anonymize_info` is used (#7090)

* BUG: Write out meas_date=None

* FIX: Maybe better?

* DOC: Document

* [WIP] Make equalize_channels capable of operating on Forward, Covariance and CrossSpectralDensity (#7029)

* Add more capabilites to equalize_channels

* Add unit tests for improved equalize_channels functionality

* Fix docstring of equalize_channels

* Add Info capabilities to equalize_channels

* Fix Forward.pick_channels docstring

* Update combine_evokeds unit test

* Fix edge case of pick_channels_cov

* Add dedicated pick_channels_cov unit test

* Add copy parameter to equalize_channels

* Make copy=False the default

* Make grand_average also equalize TFR channels

* Make copy=True the default in equalize_channels

* PEP8

* Update docstring

* Update what's new

* Update unit tests

* Update links to methods in whats_new

* Spell out new supported object types

* Fix links in whats_new again

* Remove attribute links

* MRG, DOC: revise tutorial: overview of Epoching (#7011)

* rename file

* WIP tutorial overhaul [ci skip]

* link targets

* finish tutorial revisions

* fix glossary ref

* formatting/flow tweaks [skip travis]

* fix codespell

* address neurolaunch review comments

* [MRG] Refactor mayavi/mlab out report (#7008)

* Start refactoring of report

* Fix figure_size bug

* Fix undefined function bug

* Remove unnecessary 3d functions from public API

* Refactor _fig_to_img

* Unify 3d backend testing

* Fix 3d backend testing variable

* Minor refactoring

* Improve _check_3d_figure

* TST: Trigger plot_make_report.py

* TST: Trigger plot_make_report.py

* Fix typo

* Fix bug with figure not closed

* TST: Trigger plot_make_report.py

* Integrate reviews

* crop raw to save memory (#7100)

* MRG, DOC: Update contrib guide (#7097)

* WIP update contributing guide

* refine bug reporting guidelines

* add git setup diagram

* mention dangers of PYTHONPATH

* fix typo

* more on PYTHONPATH

* clarify dev environment setup steps

* tweaks

* remove self-referential crossref

* add "origin"

* ENH: Adding support of GridSearchCV for mne.decoding.LinearModel (#7089)

* ENH: Adding support of GridSearchCV for mne.decoding.LinearModel

* (1) adds the GridSearchCV tests for LinearModel class. (2) adds Classification/Regression/GridSearchCV estimators' tests for get_coef function.

* fixed some flake8 errors.

* speeds up the tests.

* tweaks some tests.

* adds an entry to changelog.

* (1) generates the random values under RandomState. (2) uses `with block` to catch exceptions.

* BUG: CTF - set meas_date (#7104)

* BUG: CTF - set meas_date

* Comment: not to revisit testing post potential mne-c updates

* Whats New

* MRG, DOC: revise tutorial: epochs visualization (#7102)

* delete redundant tutorial

* add explanation of dropped epoch & crossref to drop_log section

* add crossref target

* overhaul epochs viz tutorial

* fix sidebar

* avoid warning

* update crossrefs

* formatting & flow tweaks

* rename file for proper ordering

* fix crossrefs

* make plots look better

* fix codespell

* [MRG] Improve Renderer API (#6761)

* Fix minor bug in sphere()

* Fix dimension in tube()

* Update doc for color

* Add color management as str

* Find better default value for tube() radius

* Correct colorConvertex syntax

* Fix import nesting

* Refactor colro code

* Rework text2d() (x,y) parameter

* Refactor _parse_str_color to _check_color

* Add some tests

* Improve doc

* Add the radius parameter for pyvista

* Fix import

* FIX: Avoid deadlocking (#7103)

* DOC: Spelling (#7106)

* ENH: Add mne sys_info command (#7105)

* ENH: Add mne sys_info command

* FIX: Install

* MRG, DOC: tutorial tweaks (#7109)

* better artifact plots; better thumbnail choice

* keep it pedagogical

* change thumbnail [ci skip]

* MAINT: Test on 3.8 (#7113)

* better info message for rank computation [skip circle] (#7110)

* FIX: Fix for old build (#7120)

* MRG, MAINT: Make calculate_head_pos_ctf public (#7117)

* MAINT: Make calculate_head_pos_ctf public

* FIX: Doc

* DOC: Fix

* FIX: Docstring

* MRG, FIX: Fix sklearn import (#7121)

* ENH: refactor _handle_event_colors (#7111)

* refactor _handle_event_colors

* restore warning

* fix tests

* fix circle

* add test

* minor refactor

* FIX: Allow unknown (#7119)

* MRG, FIX: scaling in summarize_clusters_stc (#7125)

* remove hardcoded 1e3; improve docstring

* touch tutorials to trigger rendering

* update what's new

* actually use milliseconds

* Document this PR change.

* Document this PR change.

* Update doc/changes/latest.inc

Co-Authored-By: Eric Larson <larson.eric.d@gmail.com>

* BUG: CTF - set meas_date (#7104)

* BUG: CTF - set meas_date

* Comment: not to revisit testing post potential mne-c updates

* Whats New

* MRG, FIX: scaling in summarize_clusters_stc (#7125)

* remove hardcoded 1e3; improve docstring

* touch tutorials to trigger rendering

* update what's new

* actually use milliseconds

* Fix doc CI warning about missing target.

* more

* nest check
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* better artifact plots; better thumbnail choice

* keep it pedagogical

* change thumbnail [ci skip]
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* Fixing issues with plot_surface()

* Setting transparent to True instead of False.

* PR review edits.

* PR review edits.

* Improved error management for missing pysufer.

* Update _3d.py

* Update test_3d.py

* Fixing test requires conditions.

* Document this PR change.

* Update doc/changes/latest.inc

Co-Authored-By: Eric Larson <larson.eric.d@gmail.com>

* fix : don't require git to install master (mne-tools#7078)

* [MRG] Fix plot_vector_source_estimates (mne-tools#7084)

* Initial fix

* Update mesh is not necessary

* Choose to modify scale_factor as post-post-process

* Add simple test for fix

* Use mean of hemi's width

* MRG, BUG: Fix running subprocesses in Jupyter notebooks (mne-tools#7086)

* BUG: Fix running subprocesses in Jupyter notebooks

* FIX: Flake [ci skip]

* FIX: Adding use_scalebars as a parameter. (mne-tools#7091)

* Adding use_scalebars as a parameter.

* Fixed to adhere to mne-coding standards.

* Fixing formatting issues.

* Fixing show_scalebars in viz ica from the original naming of  use_scalebars.

* Update mne/viz/ica.py

Co-Authored-By: Daniel McCloy <dan.mccloy@gmail.com>

* Update mne/viz/raw.py

Co-Authored-By: Daniel McCloy <dan.mccloy@gmail.com>

* DOC: Correct version [ci skip]

* [MRG] Implement iterative reweighted TF-MxNE (mne-tools#5075)

* ir scheme

* Squash commits

* messed up my rebase

* Address comments Alex

* flake

* implement filtering and write more readable math code

* fix pydocstyle

* High pass filter, remove upsampling, get 4 sources

* TFMxnE > TF-MxNE + rm comment example

* MAINT: Ensure that meas_date=None round-trips if `anonymize_info` is used (mne-tools#7090)

* BUG: Write out meas_date=None

* FIX: Maybe better?

* DOC: Document

* [WIP] Make equalize_channels capable of operating on Forward, Covariance and CrossSpectralDensity (mne-tools#7029)

* Add more capabilites to equalize_channels

* Add unit tests for improved equalize_channels functionality

* Fix docstring of equalize_channels

* Add Info capabilities to equalize_channels

* Fix Forward.pick_channels docstring

* Update combine_evokeds unit test

* Fix edge case of pick_channels_cov

* Add dedicated pick_channels_cov unit test

* Add copy parameter to equalize_channels

* Make copy=False the default

* Make grand_average also equalize TFR channels

* Make copy=True the default in equalize_channels

* PEP8

* Update docstring

* Update what's new

* Update unit tests

* Update links to methods in whats_new

* Spell out new supported object types

* Fix links in whats_new again

* Remove attribute links

* MRG, DOC: revise tutorial: overview of Epoching (mne-tools#7011)

* rename file

* WIP tutorial overhaul [ci skip]

* link targets

* finish tutorial revisions

* fix glossary ref

* formatting/flow tweaks [skip travis]

* fix codespell

* address neurolaunch review comments

* [MRG] Refactor mayavi/mlab out report (mne-tools#7008)

* Start refactoring of report

* Fix figure_size bug

* Fix undefined function bug

* Remove unnecessary 3d functions from public API

* Refactor _fig_to_img

* Unify 3d backend testing

* Fix 3d backend testing variable

* Minor refactoring

* Improve _check_3d_figure

* TST: Trigger plot_make_report.py

* TST: Trigger plot_make_report.py

* Fix typo

* Fix bug with figure not closed

* TST: Trigger plot_make_report.py

* Integrate reviews

* crop raw to save memory (mne-tools#7100)

* MRG, DOC: Update contrib guide (mne-tools#7097)

* WIP update contributing guide

* refine bug reporting guidelines

* add git setup diagram

* mention dangers of PYTHONPATH

* fix typo

* more on PYTHONPATH

* clarify dev environment setup steps

* tweaks

* remove self-referential crossref

* add "origin"

* ENH: Adding support of GridSearchCV for mne.decoding.LinearModel (mne-tools#7089)

* ENH: Adding support of GridSearchCV for mne.decoding.LinearModel

* (1) adds the GridSearchCV tests for LinearModel class. (2) adds Classification/Regression/GridSearchCV estimators' tests for get_coef function.

* fixed some flake8 errors.

* speeds up the tests.

* tweaks some tests.

* adds an entry to changelog.

* (1) generates the random values under RandomState. (2) uses `with block` to catch exceptions.

* BUG: CTF - set meas_date (mne-tools#7104)

* BUG: CTF - set meas_date

* Comment: not to revisit testing post potential mne-c updates

* Whats New

* MRG, DOC: revise tutorial: epochs visualization (mne-tools#7102)

* delete redundant tutorial

* add explanation of dropped epoch & crossref to drop_log section

* add crossref target

* overhaul epochs viz tutorial

* fix sidebar

* avoid warning

* update crossrefs

* formatting & flow tweaks

* rename file for proper ordering

* fix crossrefs

* make plots look better

* fix codespell

* [MRG] Improve Renderer API (mne-tools#6761)

* Fix minor bug in sphere()

* Fix dimension in tube()

* Update doc for color

* Add color management as str

* Find better default value for tube() radius

* Correct colorConvertex syntax

* Fix import nesting

* Refactor colro code

* Rework text2d() (x,y) parameter

* Refactor _parse_str_color to _check_color

* Add some tests

* Improve doc

* Add the radius parameter for pyvista

* Fix import

* FIX: Avoid deadlocking (mne-tools#7103)

* DOC: Spelling (mne-tools#7106)

* ENH: Add mne sys_info command (mne-tools#7105)

* ENH: Add mne sys_info command

* FIX: Install

* MRG, DOC: tutorial tweaks (mne-tools#7109)

* better artifact plots; better thumbnail choice

* keep it pedagogical

* change thumbnail [ci skip]

* MAINT: Test on 3.8 (mne-tools#7113)

* better info message for rank computation [skip circle] (mne-tools#7110)

* FIX: Fix for old build (mne-tools#7120)

* MRG, MAINT: Make calculate_head_pos_ctf public (mne-tools#7117)

* MAINT: Make calculate_head_pos_ctf public

* FIX: Doc

* DOC: Fix

* FIX: Docstring

* MRG, FIX: Fix sklearn import (mne-tools#7121)

* ENH: refactor _handle_event_colors (mne-tools#7111)

* refactor _handle_event_colors

* restore warning

* fix tests

* fix circle

* add test

* minor refactor

* FIX: Allow unknown (mne-tools#7119)

* MRG, FIX: scaling in summarize_clusters_stc (mne-tools#7125)

* remove hardcoded 1e3; improve docstring

* touch tutorials to trigger rendering

* update what's new

* actually use milliseconds

* Document this PR change.

* Document this PR change.

* Update doc/changes/latest.inc

Co-Authored-By: Eric Larson <larson.eric.d@gmail.com>

* BUG: CTF - set meas_date (mne-tools#7104)

* BUG: CTF - set meas_date

* Comment: not to revisit testing post potential mne-c updates

* Whats New

* MRG, FIX: scaling in summarize_clusters_stc (mne-tools#7125)

* remove hardcoded 1e3; improve docstring

* touch tutorials to trigger rendering

* update what's new

* actually use milliseconds

* Fix doc CI warning about missing target.

* more

* nest check
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* better artifact plots; better thumbnail choice

* keep it pedagogical

* change thumbnail [ci skip]
AdoNunes pushed a commit to AdoNunes/mne-python that referenced this pull request Apr 6, 2020
* Fixing issues with plot_surface()

* Setting transparent to True instead of False.

* PR review edits.

* PR review edits.

* Improved error management for missing pysufer.

* Update _3d.py

* Update test_3d.py

* Fixing test requires conditions.

* Document this PR change.

* Update doc/changes/latest.inc

Co-Authored-By: Eric Larson <larson.eric.d@gmail.com>

* fix : don't require git to install master (mne-tools#7078)

* [MRG] Fix plot_vector_source_estimates (mne-tools#7084)

* Initial fix

* Update mesh is not necessary

* Choose to modify scale_factor as post-post-process

* Add simple test for fix

* Use mean of hemi's width

* MRG, BUG: Fix running subprocesses in Jupyter notebooks (mne-tools#7086)

* BUG: Fix running subprocesses in Jupyter notebooks

* FIX: Flake [ci skip]

* FIX: Adding use_scalebars as a parameter. (mne-tools#7091)

* Adding use_scalebars as a parameter.

* Fixed to adhere to mne-coding standards.

* Fixing formatting issues.

* Fixing show_scalebars in viz ica from the original naming of  use_scalebars.

* Update mne/viz/ica.py

Co-Authored-By: Daniel McCloy <dan.mccloy@gmail.com>

* Update mne/viz/raw.py

Co-Authored-By: Daniel McCloy <dan.mccloy@gmail.com>

* DOC: Correct version [ci skip]

* [MRG] Implement iterative reweighted TF-MxNE (mne-tools#5075)

* ir scheme

* Squash commits

* messed up my rebase

* Address comments Alex

* flake

* implement filtering and write more readable math code

* fix pydocstyle

* High pass filter, remove upsampling, get 4 sources

* TFMxnE > TF-MxNE + rm comment example

* MAINT: Ensure that meas_date=None round-trips if `anonymize_info` is used (mne-tools#7090)

* BUG: Write out meas_date=None

* FIX: Maybe better?

* DOC: Document

* [WIP] Make equalize_channels capable of operating on Forward, Covariance and CrossSpectralDensity (mne-tools#7029)

* Add more capabilites to equalize_channels

* Add unit tests for improved equalize_channels functionality

* Fix docstring of equalize_channels

* Add Info capabilities to equalize_channels

* Fix Forward.pick_channels docstring

* Update combine_evokeds unit test

* Fix edge case of pick_channels_cov

* Add dedicated pick_channels_cov unit test

* Add copy parameter to equalize_channels

* Make copy=False the default

* Make grand_average also equalize TFR channels

* Make copy=True the default in equalize_channels

* PEP8

* Update docstring

* Update what's new

* Update unit tests

* Update links to methods in whats_new

* Spell out new supported object types

* Fix links in whats_new again

* Remove attribute links

* MRG, DOC: revise tutorial: overview of Epoching (mne-tools#7011)

* rename file

* WIP tutorial overhaul [ci skip]

* link targets

* finish tutorial revisions

* fix glossary ref

* formatting/flow tweaks [skip travis]

* fix codespell

* address neurolaunch review comments

* [MRG] Refactor mayavi/mlab out report (mne-tools#7008)

* Start refactoring of report

* Fix figure_size bug

* Fix undefined function bug

* Remove unnecessary 3d functions from public API

* Refactor _fig_to_img

* Unify 3d backend testing

* Fix 3d backend testing variable

* Minor refactoring

* Improve _check_3d_figure

* TST: Trigger plot_make_report.py

* TST: Trigger plot_make_report.py

* Fix typo

* Fix bug with figure not closed

* TST: Trigger plot_make_report.py

* Integrate reviews

* crop raw to save memory (mne-tools#7100)

* MRG, DOC: Update contrib guide (mne-tools#7097)

* WIP update contributing guide

* refine bug reporting guidelines

* add git setup diagram

* mention dangers of PYTHONPATH

* fix typo

* more on PYTHONPATH

* clarify dev environment setup steps

* tweaks

* remove self-referential crossref

* add "origin"

* ENH: Adding support of GridSearchCV for mne.decoding.LinearModel (mne-tools#7089)

* ENH: Adding support of GridSearchCV for mne.decoding.LinearModel

* (1) adds the GridSearchCV tests for LinearModel class. (2) adds Classification/Regression/GridSearchCV estimators' tests for get_coef function.

* fixed some flake8 errors.

* speeds up the tests.

* tweaks some tests.

* adds an entry to changelog.

* (1) generates the random values under RandomState. (2) uses `with block` to catch exceptions.

* BUG: CTF - set meas_date (mne-tools#7104)

* BUG: CTF - set meas_date

* Comment: not to revisit testing post potential mne-c updates

* Whats New

* MRG, DOC: revise tutorial: epochs visualization (mne-tools#7102)

* delete redundant tutorial

* add explanation of dropped epoch & crossref to drop_log section

* add crossref target

* overhaul epochs viz tutorial

* fix sidebar

* avoid warning

* update crossrefs

* formatting & flow tweaks

* rename file for proper ordering

* fix crossrefs

* make plots look better

* fix codespell

* [MRG] Improve Renderer API (mne-tools#6761)

* Fix minor bug in sphere()

* Fix dimension in tube()

* Update doc for color

* Add color management as str

* Find better default value for tube() radius

* Correct colorConvertex syntax

* Fix import nesting

* Refactor colro code

* Rework text2d() (x,y) parameter

* Refactor _parse_str_color to _check_color

* Add some tests

* Improve doc

* Add the radius parameter for pyvista

* Fix import

* FIX: Avoid deadlocking (mne-tools#7103)

* DOC: Spelling (mne-tools#7106)

* ENH: Add mne sys_info command (mne-tools#7105)

* ENH: Add mne sys_info command

* FIX: Install

* MRG, DOC: tutorial tweaks (mne-tools#7109)

* better artifact plots; better thumbnail choice

* keep it pedagogical

* change thumbnail [ci skip]

* MAINT: Test on 3.8 (mne-tools#7113)

* better info message for rank computation [skip circle] (mne-tools#7110)

* FIX: Fix for old build (mne-tools#7120)

* MRG, MAINT: Make calculate_head_pos_ctf public (mne-tools#7117)

* MAINT: Make calculate_head_pos_ctf public

* FIX: Doc

* DOC: Fix

* FIX: Docstring

* MRG, FIX: Fix sklearn import (mne-tools#7121)

* ENH: refactor _handle_event_colors (mne-tools#7111)

* refactor _handle_event_colors

* restore warning

* fix tests

* fix circle

* add test

* minor refactor

* FIX: Allow unknown (mne-tools#7119)

* MRG, FIX: scaling in summarize_clusters_stc (mne-tools#7125)

* remove hardcoded 1e3; improve docstring

* touch tutorials to trigger rendering

* update what's new

* actually use milliseconds

* Document this PR change.

* Document this PR change.

* Update doc/changes/latest.inc

Co-Authored-By: Eric Larson <larson.eric.d@gmail.com>

* BUG: CTF - set meas_date (mne-tools#7104)

* BUG: CTF - set meas_date

* Comment: not to revisit testing post potential mne-c updates

* Whats New

* MRG, FIX: scaling in summarize_clusters_stc (mne-tools#7125)

* remove hardcoded 1e3; improve docstring

* touch tutorials to trigger rendering

* update what's new

* actually use milliseconds

* Fix doc CI warning about missing target.

* more

* nest check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants