Skip to content
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 option in "mne report" to get sensitivity maps of forward models #2828

Open
agramfort opened this issue Jan 30, 2016 · 16 comments
Open

add option in "mne report" to get sensitivity maps of forward models #2828

agramfort opened this issue Jan 30, 2016 · 16 comments

Comments

@agramfort
Copy link
Member

it's a good quality insurance plot for forward models.

@agramfort agramfort added the EASY label Jan 30, 2016
@agramfort
Copy link
Member Author

something along these lines (copied from a script of @dengemann)

mag_map = mne.sensitivity_map(fwd, ch_type=fwd_ch_type, mode=fwd_ori)
brain = mag_map.plot(subject=subject, time_label='Magnetometer sensitivity',
                     hemi='rh', subjects_dir=subjects_dir, **fwd_plot_args)
brain.save_image('sensitivity_mag.png')
report.add_images_to_section('sensitivity_mag.png', 'sensitivity map',
                             'forward')
os.remove('sensitivity_mag.png')

fig = plt.figure()
plt.hist(mag_map.data.ravel(),
         bins=20, label=['Magnetometers'],
         color=['b'])
plt.title('Normal orientation sensitivity')
plt.xlabel('sensitivity')
plt.ylabel('count')
plt.legend()

@jasmainak
Copy link
Member

+1

@larsoner larsoner added this to the 0.13 milestone May 11, 2016
@larsoner
Copy link
Member

larsoner commented Sep 8, 2016

@jasmainak do you have time for this in the next two weeks?

@jasmainak
Copy link
Member

okay I'll give some time for fixing MNE stuff on Monday. I've a couple of things to do.

@jasmainak jasmainak self-assigned this Sep 9, 2016
@dengemann
Copy link
Member

+1

On Fri, Sep 9, 2016 at 2:30 PM Mainak Jas notifications@github.com wrote:

okay I'll give some time for fixing MNE stuff on Monday. I've a couple of
things to do.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2828 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fiiaVcWckZu6BFXgoJmmjaOHSIgVkks5qoVFOgaJpZM4HPzH3
.

@larsoner
Copy link
Member

@jasmainak any time for this one in the next couple of days? If not I'll remove the milestone (or you can)

@jasmainak
Copy link
Member

I have a local branch @Eric89GXL but I fear it's going to take more time. I'll remove it from the milestone

@jasmainak jasmainak removed this from the 0.13 milestone Sep 16, 2016
@parulsethi
Copy link

hi,
I’m interested in solving this issue, but will need some help through it.

If I understand correctly, a Report class method needs to be added to append 'sensitivity map' to the report? and is ‘sensitivity/count histogram’ mentioned above also required to be appended?

and if a method, will the input parameters to it would be same as that of 'mne.sensitivity_map()+SourceEstimate.plot()' or something else?

@jasmainak
Copy link
Member

hi @parulsethi I'd suggest that you start from my work in progress commit here. I don't think I got it working, but it might be easier than starting from scratch.

@jasmainak
Copy link
Member

Also, make sure you understand how to generate a report. You can read the tutorial on the mne report here

@parulsethi
Copy link

@jasmainak thanks for the suggestion, I've forked from there.

@larsoner
Copy link
Member

larsoner commented Apr 4, 2018

@agramfort do we still need this? With the ease of adding sections manually it does not seem essential

@agramfort
Copy link
Member Author

agramfort commented Apr 4, 2018 via email

@larsoner larsoner added this to To do in New developer sprint 2021 via automation Feb 19, 2021
@drammock drammock moved this from To do to In progress in New developer sprint 2021 Mar 15, 2021
@sappelhoff
Copy link
Member

@drammock is this really in progress? It seems to me that @jasmainak self-assigned in 2016 :-) I think that's no longer up to date.

I think this is still "up for grabs"

@drammock drammock moved this from In progress to To do in New developer sprint 2021 Mar 19, 2021
@drammock
Copy link
Member

@drammock is this really in progress? It seems to me that @jasmainak self-assigned in 2016 :-) I think that's no longer up to date.

I think this is still "up for grabs"

I was just hoping to inspire @jasmainak to finally get to it 😆

@jasmainak
Copy link
Member

I did say, I'll get to it by Monday .......

2021? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

7 participants