InsightSoftwareConsortium / ITK

Insight Toolkit (ITK) -- Official Repository. ITK builds on a proven, spatially-oriented architecture for processing, segmentation, and registration of scientific images in two, three, or more dimensions.
https://itk.org
Apache License 2.0
1.4k stars 662 forks source link

Split handling of standard conforming DICOM from non-standard DICOM #4496

Open thewtex opened 5 months ago

thewtex commented 5 months ago

As suggested by @pieper in https://github.com/InsightSoftwareConsortium/ITK/pull/4134#issuecomment-1691734111

My general preference would be to have the core reader code only handle completely standard dicom so that it is readable and maintainable. Any special code to handle malformed or vendor specific data should be detected and handled in dedicated classes, ideally with diagnostic info available so users know they have odd data and can report the issues to whoever created it.