-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Add exclude parameter to raw.plot_psd() #9519
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
Conversation
drammock
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add changelog in API or new section?
I doubt this will break old code, so I'd put it as a new feature.
Add .. versionadded :: 0.24.0 tag?
sure, why not?
Add note somewhere (docstring?) to show how to plot the PSD for all channels (including bads)?
See my suggestion
agramfort
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
diff LGTM !
no objection to merge once CIs are green.
|
@drammock any idea what is wrong with the docstring? It ends with a ".", so I don't understand the error message. |
missing space between |
|
Ultraslow is timing out, otherwise looking good. |
|
thx @cbrnr ! |
a1938c5 to
f1ccc1c
Compare
|
Just pushed a hotfix |
f1ccc1c to
e371fe5
Compare
Fixes #9441.
This adds an
excludeparameter toraw.plot_psd(), which makes it possible to plot the PSD of all channels (including bads) usingraw.plot_psd(exclude=[]).Questions/To Do:
.. versionadded :: 0.24.0tag?