Page MenuHomePhabricator
Create Task
Maniphest T10148

MITK crashes while importing DICOM data
Closed, ResolvedPublic
Actions

Description

When importing diffusion weighted dicom images, MITK sometimes crashes after the processing of the data. In that case, DWI files are not created. It happened after importing large diffusion weighted data (24000 images). Sometimes the problem can be cirumvented by using the "set output folder" option.

Related Objects

Event Timeline

freitag added projects: MITK, Bugzilla.Nov 15 2011, 2:43 PM
fritzsck added a subscriber: fritzsck.Nov 15 2011, 3:30 PM

Hi Martin,
thank you for reporting this.
Tom, have you encountered that bug already?

• vanbruggen added a subscriber: • vanbruggen.Nov 16 2011, 10:13 AM

Yes, Martin imported all datasets and will give me the ones that caused the errors so I can search for the bug.

• hering added a subscriber: • hering.Nov 16 2011, 4:36 PM

The output of the application on several 'corrupt' dataset was:

ERROR: Multiple studies found. Please limit to 1 study per folder

The dcmdump shows that there are two different StudyInstanceUID in one folder.

• vanbruggen added a comment.Nov 16 2011, 4:53 PM

I will look at this bug soon

• vanbruggen added a comment.Nov 22 2011, 5:16 PM

[c9bb01]: Merge branch 'bug-10148-dicomBagi'

Merged commits:

2011-11-22 17:00:45 Thomas Van Bruggen [5531df]
added option to correct dicom datasets containing double series uid or series filenames

nolden added a subscriber: nolden.Nov 29 2011, 10:46 AM

Is this fixed?

• hering added a comment.Nov 30 2011, 2:21 PM

We could not reproduce the crash on linux machines. Could someone (Caspar?) test this on a windows machine?

The previous commit fixes the problem with importing the corrupted data:

  • The DicomImportView from DiffusionApp was extended by an (advanced) option for allowing multiple StudyInstanceUIDs inside one serie. This option must be explicitely turned ON for this kind of data.
• goch added a subscriber: • goch.Nov 30 2011, 3:14 PM

Did it in fact crash? The original report did not mention the application closing unexpectedly. But will test this with one or two datasets as soon as Thomas has moved them to the shared drive.

• vanbruggen added a comment.Nov 30 2011, 3:38 PM

is there

• goch added a comment.Nov 30 2011, 5:39 PM

Could load the data set without error. Seems fine.

• vanbruggen added a comment.Jan 13 2012, 10:46 AM

[2d634c]: Merge branch 'bug-10148-bagiDicom'

Merged commits:

2012-01-13 09:42:30 Thomas van Bruggen [ac2d7b]
Merge remote-tracking branch 'origin/bug-10148-bagiDicom' into bug-10148-bagiDicom

2012-01-11 15:32:27 Thomas Van Bruggen [ba1f05]
added remove button for list widget

2012-01-10 12:53:46 Thomas Van Bruggen [c5eb61]
os specific path for log file

2012-01-10 09:47:37 Thomas Van Bruggen [d7a3fc]
Merge remote-tracking branch 'origin/bug-10148-bagiDicom' into bug-10148-bagiDicom

2012-01-09 20:16:46 Thomas van Bruggen [6a1096]
create log file on importing large sets of dicom data for trouble shooting

2012-01-06 16:51:27 Thomas van Bruggen [d2c59d]
fixed save to disk in dicom import for linux (needs checking for windows)

2012-01-05 21:19:07 Thomas van Bruggen [f8f06d]
intermediate commit

2011-12-15 14:23:26 Thomas van Bruggen [f97409]
Merge remote-tracking branch 'origin/master'

2011-12-01 10:33:42 Thomas van Bruggen [efe5d6]
Merge remote-tracking branch 'origin/master'

• hering mentioned this in T10299: Extend the DicomImport to handle scans with missing slices.Feb 1 2012, 10:26 AM
MITK · MIC · IMSY · DKFZ