-
Notifications
You must be signed in to change notification settings - Fork 250
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 new extension SlicerDeid #2145
base: main
Are you sure you want to change the base?
Conversation
I looked at the repository, and there is no information about what exactly it does and how.
What do you mean by that? |
Thank you very much for your comment. Head Computed Tomography (CT) scans contain identifiable data in Digital Imaging and Com-munications in Medicine (DICOM) metadata as well as facial features, raising privacy concerns. This demonstrates the need for effective de-identification tools to protect patient privacy. =>This is a Slicer extension removing Personally Identifiable Information (PII) from both metadata and image content. The de-tagging procedure has been shown to be effective and accurate on multiple head CT datasets, whereas morphology-based or AI-based methods have effectively reduced the visibility of faces in images without significantly affecting the diagnostic quality of the scans. Our method: The de-identification process in this study consisted of three main steps: reading the DICOM file, re-de-identifying sensitive tags from the metadata, and removing facial features from the image. First, a DICOM reader accessed and loaded the CT data, and specific metadata tags containing identifiable information were removed. For image-based de-identification, a morphology-based and artificial intelligence-based method was applied to detect and blur facial structures, enhancing privacy while preserving relevant diagnostic information. |
@payabvashlab thank you for the contribution 🙏 and also for the extra text clarifying what what the extension does. In order to give potential users some context, I think you should also include a note indicating that de-identification is a complex topic and that this extension addresses a specific use case. I think you should also include links to these documents for people who want to know more: https://pmc.ncbi.nlm.nih.gov/articles/PMC10081345/ https://dicom.nema.org/medical/dicom/current/output/html/part15.html#chapter_E |
Yes, thank you very much. References: |
@payabvashlab the description above is absolutely not sufficient. De-identification of DICOM metadata is complex topic. Your description does not provide any details of what exactly you do for "re-identifying sensitive tags from the metadata". The reference you inserted in the previous comments describes the part of the DICOM standard that defines how de-identification of DICOM metadata should be done. You must include a statement of how your de-identification procedures compare to what is defined in the standard. A description like the one you provided can create a false sense of security in the users of your tool, which can be extremely dangerous. Note that there is already a Slicer extension serving similar purpose: https://github.com/hina-shah/SlicerBatchAnonymize. There are also several python libraries that have been around for quite some time:
You need to justify why you are introducing a new extension and a new method (and if you are using any of the above, you should say so) for de-identification, potentially confusing the users and arguably increasing risks for inadvertent exposure of patient data. |
Yes, thank you very much for your comments. |
Aside from DICOM metadata de-id, you should inform the user how you are "removing facial features", and how you validated your approach. Again, there are several open source, validated and published solutions for facial de-identification - with some being not CT-specific!
What is the justification for coming up with another defacing algorithm, and offering it to the users without any validation of its robustness? If you were introducing yet another segmentation tool, I would not scrutinize your submission at all, but in this case the risk of potential damage is too significant to not require supporting evidence and rigor in describing the details of your approach, and adding very prominent warnings for the users of your tool. |
Thank you very much for your comment.
[2] Scott A. Collins, Jing Wu, Harrison X. Bai. (2020). Facial De-identification of Head CT Scans. Radiology, 296(1), doi:10.1148/radiol.2020192617 Please check. Thank you |
Yes, we also updated the warning when using our tools: |
https://github.com/payabvashlab/SlicerDeid
New extension
Tier 1
Any extension that is listed in the Extensions Catalog must fulfill these requirements.
3d-slicer-extension
GitHub topic so that it is listed here. To edit topics, click the settings icon in the right side of "About" section header and enter3d-slicer-extension
in "Topics" and click "Save changes". To learn more about topics, read https://help.github.com/en/articles/about-topicsSettings
and in repository settings uncheckWiki
,Projects
, andDiscussions
(if they are currently not used).About
in the top-right corner of the repository main page and uncheckReleases
andPackages
(if they are currently not used)Tier 3
Community-supported extensions.
Tier 5
Critically important extensions, supported by Slicer core developers. New Slicer Stable Release is released only if all Tier 5 extension packages are successfully created on all supported platforms.