View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003209 | Slicer4 | Module DICOM | public | 2013-07-09 08:38 | 2017-06-07 23:27 |
Reporter | fedorov | Assigned To | pieper | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | |||||
Target Version | Slicer 4.4.0 | Fixed in Version | Slicer 4.3.0 | ||
Summary | 0003209: Crash importing the directory | ||||
Description | Trying to import this directory: https://www.dropbox.com/sh/io6wfpvcuw3wew2/C_rbq4mvsM I have a crash Processing /Users/fedorov/Downloads/TESTSlicer/PCAMPMRI-00685_20040222_1113/SCANS/8/DICOM/PCAMPMRI-00685.MR.M2197_BD_Pelvis_w.8.27.20040222.111314.cqg9ee.dcm | ||||
Additional Information | r22153 | ||||
Tags | No tags attached. | ||||
This might be because I have multiple instances of Slicer running. I cannot close them at the moment, will update if confirmed. |
|
I was able to import and load this data on my machine so maybe it si something to do with you having two instances running. There is another bug on that topic, so we can mark this as duplicate if so. I'd like to figure out the exact circumstances that lead to the crash though, since I haven't ever been able to replicate it on my machine with multiple instances running. |
|
I have the crash consistently with only one instance of Slicer running, r22257, here's the dataset: https://www.dropbox.com/s/12n4kts4icseagr/ProstatePhantom_MR.tgz |
|
Processing /Users/fedorov/ImageData/MR-US/ProstatePhantom/MR/ANDRIY_PROSTATE_TEST.MR.INTERVENTIONAL_PROSTATE.2.1.20130501.075723.y5v654.dcm error: [/Users/fedorov/github/Slicer-Superbuild-Release-ITK4/Slicer-build/bin/Slicer.app/Contents/MacOS/./Slicer] exit abnormally - Report the problem. |
|
I tried this on a developer build and it did not crash (see attached image). Was this done on your build? I wonder if it works on the nightly. |
|
2013-08-06 07:21
|
|
We'll need to try debugging on your machine if possible. |
|
yes: can crash the nightly! |
|
The crash seems to happen for the following scenario: 1) study is imported into DICOM db, images are not copied to the import directory |
|
Better exception handling added in r22273 The underlying issue that causes the exception to be thrown is reported here: |
|
This time, I get the exception message in the screenshot attached. I assume this is expected behavior pending CTK issue resolution, right? If so, let's keep this bug open until the underlying CTK issue is resolved? |
|
2013-08-09 14:53
|
|
Steve: there seems to be a side-effect of fixing that issue. The popup box telling how many patients/studies were imported is not focal and is hidden behind the directory selection dialog window. Or maybe that directory selection dialog should be closed after the directory is selected? First time it happened, I didn't realize what actually happened and had to kill Slicer to regain control. |
|
Has this recurred? I could not reproduce it. |
|
Nope, can't reproduce that modal window issue. |
|
Changed severity from crash => minor since Slicer doesn't crash anymore. See 0001832 |
|
Andrey - can we close this issue now? |
|
Slicer: 2145-support-for-installing-extension-from-file 8c53d089 2013-08-09 10:12:42 Details Diff |
BUG: 0001832 0003140 0003209 avoid crashes by catching exceptions Some libraries and systems throw exceptions, but Qt's default event processing does not catch them[1]. For example, slicer could crash when ctkDICOMItem is pointed to a file that does not exist. Wrapping event processing in try/catch should handle other conditions, like out of memory exceptions thrown from inside render operations (e.g. due to overly complex glyph rendering). [1] http://qt-project.org/doc/qt-4.8/exceptionsafety.html http://stackoverflow.com/questions/10665655/how-to-log-uncatched-exceptions-of-a-qapplication git-svn-id: http://svn.slicer.org/Slicer4/trunk@22273 3bd1e089-480b-0410-8dfb-8563597acbee |
||
mod - Base/QTGUI/qSlicerApplication.cxx | Diff File | ||
mod - Base/QTGUI/qSlicerApplication.h | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-07-09 08:38 | fedorov | New Issue | |
2013-07-09 08:38 | fedorov | Status | new => assigned |
2013-07-09 08:38 | fedorov | Assigned To | => pieper |
2013-07-09 09:37 | fedorov | Note Added: 0008965 | |
2013-07-09 09:46 | pieper | Target Version | => Slicer 4.3.0 |
2013-07-09 11:46 | pieper | Note Added: 0008968 | |
2013-08-05 07:29 | fedorov | Note Added: 0009380 | |
2013-08-05 07:29 | fedorov | Note Added: 0009381 | |
2013-08-06 07:21 | pieper | Note Added: 0009394 | |
2013-08-06 07:21 | pieper | File Added: Screen Shot 2013-08-06 at 11.19.07 AM.png | |
2013-08-06 10:03 | pieper | Note Added: 0009409 | |
2013-08-06 10:03 | pieper | Status | assigned => feedback |
2013-08-06 12:33 | fedorov | Note Added: 0009416 | |
2013-08-06 13:57 | fedorov | Note Added: 0009418 | |
2013-08-09 06:20 | pieper | Note Added: 0009436 | |
2013-08-09 06:20 | pieper | Status | feedback => resolved |
2013-08-09 06:20 | pieper | Fixed in Version | => Slicer 4.3.0 |
2013-08-09 06:20 | pieper | Resolution | open => fixed |
2013-08-09 06:20 | pieper | Relationship added | related to 0001832 |
2013-08-09 14:53 | fedorov | Note Added: 0009451 | |
2013-08-09 14:53 | fedorov | File Added: Screen Shot 2013-08-09 at 18.44.22 .png | |
2013-08-09 14:53 | fedorov | Status | resolved => feedback |
2013-08-09 14:53 | fedorov | Resolution | fixed => reopened |
2013-08-11 08:39 | fedorov | Note Added: 0009462 | |
2013-08-19 13:40 | pieper | Note Added: 0009522 | |
2013-08-19 15:19 | fedorov | Note Added: 0009526 | |
2013-08-30 12:45 | jcfr | Target Version | Slicer 4.3.0 => Slicer 4.3.1 |
2013-09-10 12:23 | jcfr | Severity | crash => minor |
2013-09-10 12:24 | jcfr | Note Added: 0009923 | |
2013-09-11 12:19 | fedorov | Status | feedback => assigned |
2013-09-17 12:21 | jcfr | Target Version | Slicer 4.3.1 => Slicer 4.3.2 |
2014-03-06 10:15 | nicole | Target Version | Slicer 4.3.2 => Slicer 4.4.0 |
2014-03-06 11:08 | pieper | Note Added: 0011260 | |
2014-03-06 12:35 | fedorov | Status | assigned => closed |
2014-03-06 12:35 | fedorov | Resolution | reopened => fixed |
2017-06-07 23:27 | pieper | Changeset attached | => Slicer 2145-support-for-installing-extension-from-file 8c53d089 |