-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 sourcespace to Report #12848
base: main
Are you sure you want to change the base?
Add sourcespace to Report #12848
Conversation
for more information, see https://pre-commit.ci
For Another place it would be nice to add if you're motivated is in Then it would be good to pass some |
Having looked at the many possibilities, I think the best solution would be to add a figure in the This solution has the advantage of being able to also display the EEG sensors projected onto the scalp, as well as displaying only the sources used in for forward computation (as some sources from the original source space may be dropped if too close to the inner skull surface). This will give a global idea of the sensors <-> sources model. mne.viz.plot_alignment(trans=fwd["mri_head_t"], info=fwd["info"], src=fwd["src"], eeg=dict(original=0.2, projected=0.8)) We could also have a clearer view of the source space with an additional plot: |
I like both of those plots! Adding to the |
This reverts commit 7c80b70.
for more information, see https://pre-commit.ci
Reference issue (if any)
Fixes #12836
What does this implement/fix?
Add a
src
argument to theReport.add_bem
method.Additional information
I'm not sure which strategy to adopt and changes to make to
Report.parse_folder
.The
Report.parse_folder
method automatically adds BEM to theReport
ifReport.subject
is specified. However, several sourcespaces-src.fif
files might be present in the BEM folder and/or folder to parse.