Page MenuHomePhabricator

Standardize/Rework DICOM module naming
Closed, ResolvedPublic

Description

Currently the naming of DICOM related modules is inconsistence (e.g. in upper case / lower case):

  • DICOMQI
  • DICOMReader
  • DicomRT
    • DicomRTIO (autoload module)
  • DicomUI

Following things should be regarded:

  1. We should either choose "DICOM" or "Dicom" but should not mix both
  2. I would propose to rename DICOMReader to have a wider scope that includes also writer, because they will come sooner or later (sooner If i look at the requests). E.g.:
    • DICOMIO
    • DICOMImage
    • DICOMImageIO (because at its heart it covers only Image IODs or derived stuff)
  3. I would propose to rename DICOMRT into RT (and rename its autoload module into DICOMRTIO). The Reason is that the module itself has nothing to do with DICOM but with data structures and mappers for RT data. Only the autoload module has something to do with io at all. DICOMRT itself is the partner of RTUI and that should be resembled in its naming like we do with other modules.

Revisions and Commits

rMITK MITK
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

kislinsk triaged this task as Wishlist priority.Jan 9 2019, 12:14 PM
kislinsk added a subscriber: kislinsk.
  1. DICOM
  2. IO suffix
  3. Yes!
  4. Surprise! Remove dependency to other auto-load modules. k thx bye

Cleaned up naming of DICOM related modules

  • DICOMReader -> DICOM
  • DICOMReaderService -> DICOMImageIO (moved to DICOM/autoload/DICOMImageIO; same style like other io autoload modules)
  • DicomRT -> RT
  • DicomRTIO -> DICOMRTIO
floca added a revision: Restricted Differential Revision.Jul 25 2020, 5:23 PM