Closed jeffmax closed 12 years ago
Thanks, Jeff. I will merge this shortly, although I will probably replace delete with remove to make it consistent with the other ruby-dicom methods (although I realize that it would perhaps be more consistent with ruby itself to use delete* method names).
Sounds good to me Chris, I was going to go with remove, but it conflicted with the API function to remove an attribute from being anonymized.
I think actually that I should try to make ruby-dicom more consistent with Ruby itself, and therefore use delete for method names instead of remove.
In anyway, I will probably make some changes to the Anonymizer class and move more functionality to accessors instead of having our own methods for adding and removing stuff.
Thanks for the commit, I'll go ahead and merge it.
Chris,
This is a minor change that adds a function to add a tag to be completely removed during anonymization. Originally I had the list populated with a bunch of default tags that should be removed, but this approach seemed more flexible. I can provide you with a list that I am currently using, but I have found there are usually quite a few tags that would not make sense to be in an anonymized file. Some tags give away the patient's sex just by being present in the file.
Jeff