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

Updates facemask.nii.gz with better ear and mouth mask #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

miykael
Copy link

@miykael miykael commented Apr 27, 2018

I've updated the facemask.nii.gz so that it also cuts out the ears of a subject. As discussed on neurostars, cutting out the ears can be a necessary requirement for local ethic committees to consider an MRI image as "fully anonymized".

I've also cut out some more parts of the mouth area to make sure that the dental information are excluded from the defaced image.

The adaptation of the previous facemask was done manually and verified on a dataset of 12 subjects.

@chrisgorgo
Copy link
Contributor

Thanks - could you post some screenshots of the mask pre and post adjustment here? This will help document the evolution of the package.

@miykael
Copy link
Author

miykael commented Apr 27, 2018

Of course. The following figure shows regions that were only in the old facemask in yellow, regions that are now new in the facemask in blue and regions that haven't changed in green. The template mean_reg2mean.nii.gz is depicted in the background.

mask

As can be seen, the new mask cuts out the ears, removes some more regions of the jaw and adds again the region close to the frontal pole. I've added this last part again, because it lead to frontal pole removal in some of my subjects.

@chrisgorgo
Copy link
Contributor

@robertoostenveld @jasmainak would such mask be still ok for MEG skull reconstruction and electrode localization?

@chrisgorgo
Copy link
Contributor

Thanks for providing figures @miykael!

@jasmainak
Copy link

jasmainak commented Apr 29, 2018 via email

@agramfort
Copy link

no indeed. For MEG/EEG coreg we need the preauricular points and the tragus and at least the upper half of the nose.

@robertoostenveld
Copy link

Agreed with Alex. After application of this mask re-doing the coreg would indeed be difficult (or very imprecise), if not impossible (depends on the coreg method and whether a headshape is available).

IMHO coregistration should be done prior to making the de-identified anatomical in BIDS format and coreg information should be shared in the AnatomicalLandmarkCoordinates field. As such I don't see the problem of this (or any other) mask.

Considering that after conversion of dicoms to nifti's it would also be difficult to determine MR parameters; I would say there is nothing new here: some parameters have to be determined and shared prior to or during conversion.

@chrisgorgo
Copy link
Contributor

I understand the localization could be done before defacing, but it would be more convenient if we could come up with a mask that removes some ear/nose details, but keeps enough to make the localization possible. Or do you think that this is not possible?

@agramfort
Copy link

agramfort commented Apr 30, 2018 via email

@robertoostenveld
Copy link

robertoostenveld commented Apr 30, 2018 via email

@Shotgunosine
Copy link
Collaborator

@chrisfilo What about including this mask as an option until a mask that works better with MEG can be developed?

@chrisgorgo
Copy link
Contributor

chrisgorgo commented Feb 4, 2019 via email

@miykael
Copy link
Author

miykael commented Feb 4, 2019

@Shotgunosine - Happy to adapt the changes. Should I just include an option like --remove_ears, which indicates that this template should be used?

@vsoch
Copy link
Contributor

vsoch commented Feb 4, 2019

oh my. That sounds quite painful.

@Shotgunosine
Copy link
Collaborator

@miykael That sounds good. Just a heads up, I'm hoping to get #19 merged shortly, which might require some restructuring in this PR to be compatible.

@miykael
Copy link
Author

miykael commented Feb 4, 2019

@vsoch - I agree. But my current PR would overwrite the current facemask, which supposedly is not the goal :-)

@Shotgunosine - Good, then I will wait for #19 to be merged.

@Shotgunosine
Copy link
Collaborator

@miykael #19 is merged.

@muschellij2
Copy link

It seems as though this facemask.nii.gz allows you to remove ears but leaves the eyes (while removing more mouth), which seems like it'd be good to create a series of structures to combine together as I'd like the original facemask sheared face, but also remove the ears, but the proposed mask didn't remove enough of the face for my type of data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants