Page MenuHomePhabricator

Remove old deprecated property names of DICOM information
Open, NormalPublic

Description

We still generate the deprecated properties with DICOM information like "modality" or "dicom.series.*". I think they where marked and communicated long enough as deprecated.

Therefore I think we should rework the DICOMReader to not store the properties with the old names.

The backwards compatibility in the rest of the mitk code should stay to be able to handle old scene files and co.

Event Timeline

floca triaged this task as Normal priority.Feb 12 2020, 11:50 AM
floca created this task.

FYI: When changing/removing properties there's always the chance that old mitk scene files are suddenly incompatible as properties are stored in them. If that's an issue, property aliases may be a solution for a legacy mapping between property names.

I know. I was just talking about removing the code from our DICOMReader, that gernerates these deprecated properties. The rest could stay as is, so scene files should should not be feel an impact. And we also have the mapping code in place.

kislinsk added a project: Auto-closed.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs

floca edited projects, added Next Milestone; removed Auto-closed.