DigitalSlideArchive / ImageDePHI

Apache License 2.0
8 stars 0 forks source link

DICOM Image Types #259

Closed manthey closed 2 weeks ago

manthey commented 2 months ago

If we have a DICOM image type that isn't in our enum, we won't redact it, but our enum is just a repetition of the image type. For flexibility, it would be better to always let these through. We can then add custom rules for deleting or keeping other image types. Otherwise, we can't redact most CT data.

manthey commented 1 month ago

@naglepuff This could just be wrapping self.image_type = WsiImageType(self.dicom_data.ImageType[WSI_IMAGE_TYPE_INDEX]) with a try / except, and, if needed setting self.image_type to WsiImageType.OTHER. Or, maybe it's be better to set it to self.dicom_data.ImageType[WSI_IMAGE_TYPE_INDEX] instead?