View Issue Details

IDProjectCategoryView StatusLast Update
0004248Slicer4Module DICOMpublic2017-03-29 15:34
Reporterpinter Assigned Topinter  
PrioritynormalSeverityminorReproducibilitysometimes
Status closedResolutionfixed 
Product VersionSlicer 4.5.0-1 
Target VersionSlicer 4.7.0Fixed in VersionSlicer 4.7.0 
Summary0004248: Scalar volume DICOM export crashes after 10-12 exports
Description

User reported crash that happens after exporting 10-12 (varies) scalar volumes, and trying to export the next one. The main points:

  • When exporting dicoms by hand, Slicer consistently crashes after about 10 or 11 images (it can vary by 1 or 2). Steps to reproduce: Have >15 images in a patient node, right click image and hit export, select folder and allow export, repeat until crash after about 10 images.
  • Slicer will crash after its done putting the volumes in the folder that you selected for exporting. It crashes sometime after that.
  • in regards to the crashing bug I was mentioning, it doesn’t occur when I do it this way [using python script to create and export exportables]. Most likely it occurs when adding the exported stuff to the database (if I understand the process correctly).
Steps To Reproduce

Have >15 images in a patient node, right click image and hit export, select folder and allow export, repeat until crash after about 10 images.

Additional Information

See details in conversation
http://slicer-users.65878.n3.nabble.com/Export-Dicoms-td4030729.html

TagsNo tags attached.

Activities

pieper

pieper

2016-08-25 15:14

administrator   ~0014072

Hi Csaba -

Thanks for supporting the user and for capturing this in the tracker!

Did you ever get a copy of the final exporter script? It's odd to me that the script doesn't crash but the user interface does. Did you get any clues on that?

-Steve

pinter

pinter

2016-08-26 10:51

developer   ~0014073

I haven't received the script he wrote. I will remind him again.

The fact that using the script the crash does not occur suggests that it is caused by a step done by the tag editor widget. My guess would be the same as the user's: adding the exported data back to the database immediately after exporting.

pieper

pieper

2016-08-26 11:18

administrator   ~0014074

Interesting - do you think the issue is in the database or the tag editing widget? I'd vote for the widget - maybe it has a dangling pointer or something. Should be detectable if we can reproduce it in a debugger.

pinter

pinter

2016-08-26 11:39

developer   ~0014075

I agree. Reproducing the issue will be no doubt tedious, exporting lots of series in debug mode :) I can investigate this in a time frame depending on the priority of this issue.

pinter

pinter

2016-08-26 16:03

developer  

Export.py (2,537 bytes)
pinter

pinter

2016-08-26 16:04

developer   ~0014076

Attached user script to programmatically export volumes to DICOM

pinter

pinter

2017-03-29 14:52

developer   ~0014387

I did some fixes today around DICOM export, and also there was a complete overhaul of subject hierarchy since reporting this issue, so wanted to check this error "while I was there".
So I exported 25 MRHeads using the export dialog (with import exported data checkbox on, which is now not the default state), and there was no crash.

Please test it with the latest nightly (tomorrow or later), or close this issue. Thanks!

pinter

pinter

2017-03-29 14:52

developer   ~0014388

Oh I just realized that I was the reporter :)
Steve, should we close this?

pinter

pinter

2017-03-29 15:00

developer   ~0014389

I exported 5 more (30 in total), still no crash...

pieper

pieper

2017-03-29 15:33

administrator   ~0014390

Agreed, since there is a new implementation and we can't replicate the issue let's close this. Thanks Csaba!

Issue History

Date Modified Username Field Change
2016-08-25 09:55 pinter New Issue
2016-08-25 09:55 pinter Status new => assigned
2016-08-25 09:55 pinter Assigned To => pieper
2016-08-25 15:14 pieper Note Added: 0014072
2016-08-26 10:51 pinter Note Added: 0014073
2016-08-26 11:18 pieper Note Added: 0014074
2016-08-26 11:39 pinter Note Added: 0014075
2016-08-26 16:03 pinter File Added: Export.py
2016-08-26 16:04 pinter Note Added: 0014076
2016-10-12 04:29 jcfr Target Version Slicer 4.6.0 => Slicer 4.7.0
2016-10-13 02:03 jcfr Product Version Slicer 4.5.1 => Slicer 4.5.0-1
2017-03-29 14:52 pinter Note Added: 0014387
2017-03-29 14:52 pinter Status assigned => feedback
2017-03-29 14:52 pinter Note Added: 0014388
2017-03-29 14:52 pinter Status feedback => assigned
2017-03-29 15:00 pinter Note Added: 0014389
2017-03-29 15:33 pieper Note Added: 0014390
2017-03-29 15:34 pinter Status assigned => closed
2017-03-29 15:34 pinter Assigned To pieper => pinter
2017-03-29 15:34 pinter Resolution open => fixed
2017-03-29 15:34 pinter Fixed in Version => Slicer 4.7.0