Page MenuHomePhabricator

Dicom editor can not load dicom segmentation objects
Closed, ResolvedPublic

Description

When parsing dicom folders which contain dicom segmentation objects they will be correctly listed. However if double clicked MITK will crash.

This seems to be happening in

void DicomEventHandler::OnSignalAddSeriesToDataManager(const ctkEvent& ctkEvent)

where a couple of special cases are handled and then a default reader selection is used instead of using mime types.

Event Timeline

What about this task? Could we add these changes to the master branch?

Probably, the dicom folder read implementation was a bit cobbled together and could probably use some more thought.

I would have (minor) concerns with directly merging it. The comments raised for the last commit.

nolden edited projects, added MITK (2018-04); removed MITK.
nolden added subscribers: kislinsk, nolden.

I would have (minor) concerns with directly merging it. The comments raised for the last commit.

I agree with Ralf's concerns. If I oversee this correctly we could merge this without changing the node naming convention. @kislinsk : would be great to have this in the release. It's actually a crash.

It's not clear to me if this mergable as-is or not?

It's not clear to me if this mergable as-is or not?

In my opion not. But I can help to patch it that way.

As far as I can see (had to dig through the commits) All changes of this branch, have already been cherry picked to the master at some point. (Don't know if they are also already in the current beta brunch).

My complaints have been fixed in context of T25441: Name DataNodes with DICOM data appropriately by using dicom tags, that I will now merge into the master.
Thus I would propose to close this task. Availablilty in the beta must still be checked, if wanted.

Deleted branch T23343-enable-microservice-based-reading-for-dicom-editor.