Closed peshence closed 4 months ago
Looking into why the whitelisting may be causing an issue with DICOM to Nifti conversion, it does seem like this is due to too aggressive anonymisation removing needed MRI image tags.
It's possible that this is due to required (type 1) attributes in the MR image module being removed. See also https://dicom.innolitics.com/ciods/mr-image/mr-image for a more readable format.
Used https://github.com/pydicom/dicom-validator to look at the intermediate anonymised DICOMs, and there were a few things that are being flagged up as missing from a fully valid MR compared to its IOD. The minimum needed to be added are:
Closing in favour of #366
291 Switch from DIY DICOM anonymiasation to Kitware DICOM Anonymizer. In the process:
Possible improvement: Convert current YAMl tag configs to JSON to make use the the existing underlying action loading functionality. We can then do regex replacements and a few other slightly more complex actions. Not sure if it's necessary at this stage.
Possible further actions, which I think shouldn't be in this PR: