dcmstack icon indicating copy to clipboard operation
dcmstack copied to clipboard
verified publisher icon moloney

Dicom Anonymizer

Open ssikka opened this issue 13 years ago • 2 comments

The code is barely documented and does not entirely conform to pep8.

I see this pull request as a chance to continue working on the documentation and code standards until everyone is satisfied.

ssikka avatar Nov 17 '12 22:11 ssikka

Thanks for the hard work!

I haven't looked in too much detail, but I have a couple of questions:

  1. Where do you see this integrating with dcmstack? It looks like you need to work with the original DICOM data set, not the extracted meta data. This would mean you would need to integrate this with a MetaExtractor object of some kind.

  2. What about meta data from private elements? Are you always just going to discard all private elements?

moloney avatar Nov 27 '12 22:11 moloney

Hi. It seems like you should be able to specify and ignore_rule to integrate this with the existing MetaExtractor object. Check out the examples "ignore_private" and "ignore_non_ascii_bytes" in the dcmstack.extract module.

moloney avatar Mar 01 '13 07:03 moloney