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
Add exclude parameter to raw.plot_psd() #9519
Conversation
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
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
exclude
parameter 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.0
tag?