View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002016Slicer4Module DICOMpublic2012-05-10 17:072012-10-26 10:06
Reporterinorton Assigned Topieper  
PrioritynormalSeveritymajorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version 
Target VersionSlicer 4.2.0Fixed in VersionSlicer 4.2.0 
Summary0002016: Multi-load appears not to work.
Description

I selected a study with 6 image volumes in the top pane.

(Every time it waits 30 seconds "Checking DICOMSlicerDataBundlePlugin"??)

I clicked checkbox next to 5 of those volumes and clicked "Load to Slicer".

Nothing happens.. This message shows up in the interactor every time I click:

Traceback (most recent call last):
File "C:\Program Files\Slicer 4.1.0-2012-05-09\bin\Python\DICOMLib\DICOMWidgets.py", line 239, in loadCheckedLoadables
self.loadableTable.updateCheckstate()
File "C:\Program Files\Slicer 4.1.0-2012-05-09\bin\Python\DICOMLib\DICOMWidgets.py", line 322, in updateCheckstate
self.loadables[row].selected = item.checkState()
AttributeError: 'NoneType' object has no attribute 'checkState'

TagsNo tags attached.

Activities

pieper

pieper

2012-07-26 07:24

administrator   ~0005256

Should be fixed with recent updates to dicom module. Can you check again? If fixed, please close. If not fixed, can you provide details on data that led to this (or the actual data if that is possible).

Regarding the performance, there is still one wave of optimization to be done on the ctkDICOMDatabase. I added the issue here https://github.com/commontk/CTK/issues/216
jcfr

jcfr

2012-10-25 12:37

administrator   ~0006766

Reminder sent to: inorton

Would it be possible to provide Steve with some feedback. If no feedback is given by Friday afternoon or if the problem persists, the issue will be re-targeted for 4.3.
inorton

inorton

2012-10-25 13:34

developer   ~0006777

Slicer hung at "Checking MultiVolumeImporterPlugin" for 45 minutes the first time I tried this. Tried several other sets of images - typical 10-12 series each. All imported successfully, but after clicking on the PatientID it hangs at the above plugin check. Cancelling works, but then there are no volumes to select.
jcfr

jcfr

2012-10-25 13:37

administrator   ~0006778

Reminder sent to: fedorov, pieper

Andriy, Steve,

45mins seems to be quite a long time.

Is there a way to speed things up ?

inorton

inorton

2012-10-25 13:50

developer   ~0006782

To clarify, if I click a single series, then it goes fast enough and I get options to load image. The hang only occurs when clicking on the SubjectID row.
jcfr

jcfr

2012-10-25 14:01

administrator   ~0006784

Do you still the error reported in the original description ?
inorton

inorton

2012-10-25 14:08

developer   ~0006785

I can't test the original description because of the hang - the multiple volume selection checkboxes are not available in the lower right panel after cancelling.
jcfr

jcfr

2012-10-25 14:13

administrator   ~0006786

Given the fact Steve is travelling until Monday. Re-targeting for 4.3.

2012-10-25 14:45

 

Screen shot 2012-10-25 at 18.44.33 .png (260,106 bytes)
fedorov

fedorov

2012-10-25 14:46

developer   ~0006798

I tested with a study that has 15 series (see attached screenshot), and was unable to reproduce the issue -- all series were parsed within 1-2 minutes.

Please include the study that allows to reproduce the problem.
jcfr

jcfr

2012-10-25 14:50

administrator   ~0006799

Last edited: 2012-10-25 14:50

Andriy> The assignee shouldn't be changed, instead it should be set to "feedback". Thanks
inorton

inorton

2012-10-26 09:27

developer   ~0006810

We looked at the data and Andriy has a fix - the plugin had an error so it never cleaned up the status window.

With that out of the way, can confirm the original description is fixed.
inorton

inorton

2012-10-26 09:29

developer   ~0006811

Thanks.
fedorov

fedorov

2012-10-26 09:30

developer   ~0006812

fixed in http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21256
fedorov

fedorov

2012-10-26 09:30

developer   ~0006813

fixed in http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21256

Issue History

Date Modified Username Field Change
2012-05-10 17:07 inorton New Issue
2012-05-10 17:07 inorton Status new => assigned
2012-05-10 17:07 inorton Assigned To => pieper
2012-05-10 17:07 inorton Description Updated
2012-05-18 06:26 pieper Target Version => Slicer 4.2.0 AHM Summer 2012
2012-07-26 07:24 pieper Note Added: 0005256
2012-07-26 07:24 pieper Status assigned => feedback
2012-07-26 07:24 pieper Resolution open => fixed
2012-10-25 12:37 jcfr Note Added: 0006766
2012-10-25 13:34 inorton Note Added: 0006777
2012-10-25 13:37 jcfr Note Added: 0006778
2012-10-25 13:50 inorton Note Added: 0006782
2012-10-25 14:01 jcfr Note Added: 0006784
2012-10-25 14:08 inorton Note Added: 0006785
2012-10-25 14:12 jcfr Target Version Slicer 4.2.0 - coming release => Slicer 4.3.0
2012-10-25 14:13 jcfr Note Added: 0006786
2012-10-25 14:13 jcfr Status feedback => assigned
2012-10-25 14:45 fedorov File Added: Screen shot 2012-10-25 at 18.44.33 .png
2012-10-25 14:46 fedorov Note Added: 0006798
2012-10-25 14:46 fedorov Assigned To pieper => inorton
2012-10-25 14:46 fedorov Status assigned => feedback
2012-10-25 14:49 jcfr Status feedback => assigned
2012-10-25 14:49 jcfr Assigned To inorton => pieper
2012-10-25 14:50 jcfr Note Added: 0006799
2012-10-25 14:50 jcfr Status assigned => feedback
2012-10-25 14:50 jcfr Note Edited: 0006799
2012-10-26 09:27 inorton Note Added: 0006810
2012-10-26 09:29 inorton Note Added: 0006811
2012-10-26 09:29 inorton Status feedback => closed
2012-10-26 09:29 inorton Fixed in Version => Slicer 4.2.0 - coming release
2012-10-26 09:30 fedorov Note Added: 0006812
2012-10-26 09:30 fedorov Status closed => feedback
2012-10-26 09:30 fedorov Resolution fixed => reopened
2012-10-26 09:30 fedorov Note Added: 0006813
2012-10-26 09:30 fedorov Status feedback => closed
2012-10-26 09:30 fedorov Resolution reopened => fixed
2012-10-26 10:06 jcfr Target Version Slicer 4.3.0 => Slicer 4.2.0 - coming release