mne-python icon indicating copy to clipboard operation
mne-python copied to clipboard

Use spatial_colors by default in Evoked.plot()

Open hoechenberger opened this issue 2 years ago • 6 comments

When using Evoked.plot(), in fact I always want to use spatial_colors=True as this feature is so simple, yet extremely helpful.

Problem is, on the first try I almost always forget to pass the respective parameter, so then I need to amend my code and re-run it.

Also I believe that many users are probably not aware of the feature, as … well, as it's not the default.

Could we not make it such that if we have sensor locations, we use spatial_colors=True by default?

We'd still keep the parameter to allow for manual adjustment, of course.

WDYT?

cc @sappelhoff @cbrnr @alexrockhill @adam2392

hoechenberger avatar May 07 '22 18:05 hoechenberger

Fine with me!

cbrnr avatar May 07 '22 18:05 cbrnr

Fine with me as well :+1:

sappelhoff avatar May 07 '22 19:05 sappelhoff

Definite +1

alexrockhill avatar May 07 '22 20:05 alexrockhill

No objection but make sure no warning is raised by default if no channel locations are present

This maybe suggests a spatial_colors=‘auto’ by default

agramfort avatar May 07 '22 20:05 agramfort

+1 from me too.

mmagnuski avatar May 07 '22 21:05 mmagnuski

reopened by #11151

drammock avatar Sep 14 '22 15:09 drammock