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
animate_topomap - CSD fix #12605
animate_topomap - CSD fix #12605
Conversation
for more information, see https://pre-commit.ci
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.
looks clean ! thx @michalrzak
can you update the changelog to make the CI green? thx
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.
Moved the imports and removed now unnecessary plt.close("all")
s that I saw as well, marking for merge when green. Thanks in advance @michalrzak !
Reference Issue
Example: Fixes #12599.
What does this implement/fix?
For a detailed explanation, please refer to the associated issue.
In essence, this PR addresses the issue where
animate_topomap()
fails to function with "csd" data. The proposed solutions in the issue include a straightforward addition of "csd" to the list of permittedch_types
foranimate_topomap
, as well as a more intricate approach that consolidates thech_type
selections from bothanimate_topomap()
andplot_evoked_topomap()
.This PR implements the latter solution, as I find it to be the more comprehensive approach. However, I may have missed a specific reason for maintaining different channel selections in
animate_topomap()
andplot_evoked_topomap()
. Should such reasons exist, I am open to adjusting this PR accordingly.