Page MenuHomePhabricator

Update DicomToNRRD cmdApp to new reader system
Closed, ResolvedPublic

Description

The DicomToNRRD cmdApp uses legacy readers. We need to update it to using a new dicom reader in order to preserve properties.

Event Timeline

goch added a project: Restricted Project.
goch added a parent task: Restricted Maniphest Task.

Ausgehend von der Diskussion um das Einlese verhalten habe ich mir überlegt, dass es vielleicht sinnvoll ist die Apps namentlich und vom scope noch mal sauberer zu trennen.

Ich würde vorschlagen, dass das was du gemacht hast eigentlich eine "IOUtilApp" ist.
Denn eigentlich geht es gar nicht um DICOM2NRRD sondern dass wir die Funktionalität der Workbench/IOUtil (Daten zu laden und wieder in einem Format zu speichern) auch in einem Kommandozeilen tool anbieten.

Bei einem echten DICOM2NRRD Tool könnten wir dan einfach direkt den neuen DICOMAutoselector nehmen und darüber einlesen. Dann kann man auch die explizite angabe der Dateien etc. (was Marco angemerkt hat lösen).

Ich würde beide Funktionalitäten und Usecases nicht krampfhaft in ein Tool reinpressen.

I agree, however I see this as part of T22714 as that bug will reorganize the common cmdApps anyway.

goch removed goch as the assignee of this task.
goch claimed this task.

Deleted branch T22801-update-dicom2nrrd.