View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001789 | Slicer4 | Core: MRML | public | 2012-03-06 05:20 | 2014-03-06 06:10 |
Reporter | liuy5 | Assigned To | nicole | ||
Priority | urgent | Severity | crash | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | Slicer 4.0.1 | ||||
Target Version | Slicer 4.1.0 | Fixed in Version | Slicer 4.1.1 | ||
Summary | 0001789: Slicer crashes after cropping and saving | ||||
Description | Slicer 4 2012-03-06
Tested on multiple images | ||||
Tags | No tags attached. | ||||
The problem is not seen in r19532 64bit windows binary. |
|
Hmm, the back trace in the debugger hits the same error as in bug 1759, but I'm also getting an assert crash after I unhook the debugger, Slicer stays open, and I do File -> Exit: Is the crop volume module missing an enter() call? It has one, but is over riding the superclass, so adding in a call to the supercalss enter method and seeing if that helps. |
|
svn 19547 fixes the assert problem (could be seen if started up slicer, entered CropVolume, then tried to switch to another module). Didn't see the crash on scene close this time, will test more. |
|
Crash also happens if load a volume, go to editor to create a label volume, save scene, close scene. |
|
If I change the scene view node destructor:
it changes the crash to happen if you restore a scene view and then close the scene (crash in referencing nodes of the crosshair node), but saving the mrml file (with scene view) then closing the scene doesn't crash due to volume node image data. Still trying to figure out what the scene view saved scene thinks it's referenced/ing nodes are pointing to. |
|
this->Nodes->GetNodes()->RemoveAllItems(); seems extremelly dangerous as it removes nodes without informing scene and node observers. |
|
The RemoveAllItems calls triggers Delete on the nodes it contains and the node removed event does trigger a bit of an observer clean up. |
|
The Crop Volume error seems to be fixed as of svn 19588.... |
|
svn 19597 reverts some changes to how the scene views are saved/restored and this crash is not longer happening. Yanling, could you verify that it works for you and set this bug status to closed if so? |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2012-03-06 05:20 | liuy5 | New Issue | |
2012-03-08 11:55 | finetjul | Status | new => assigned |
2012-03-08 11:55 | finetjul | Assigned To | => nicole |
2012-03-08 11:55 | finetjul | Category | Usability => MRML |
2012-03-08 11:55 | finetjul | Target Version | => Slicer 4.1.0 |
2012-03-08 12:16 | liuy5 | Note Added: 0003810 | |
2012-03-08 12:30 | nicole | Relationship added | duplicate of 0001759 |
2012-03-08 12:38 | nicole | Note Added: 0003811 | |
2012-03-08 12:41 | nicole | Note Edited: 0003811 | |
2012-03-08 13:17 | nicole | Note Added: 0003815 | |
2012-03-09 12:22 | nicole | Relationship deleted | 0001759 |
2012-03-12 10:19 | nicole | Note Added: 0003839 | |
2012-03-12 14:47 | nicole | Note Added: 0003844 | |
2012-03-12 16:32 | finetjul | Note Added: 0003846 | |
2012-03-12 16:32 | finetjul | Note Edited: 0003846 | |
2012-03-13 06:27 | nicole | Note Added: 0003849 | |
2012-03-13 12:23 | nicole | Note Added: 0003856 | |
2012-03-14 06:37 | nicole | Note Added: 0003862 | |
2012-03-14 06:37 | nicole | Status | assigned => resolved |
2012-03-14 06:37 | nicole | Resolution | open => fixed |
2012-05-04 04:07 | liuy5 | Status | resolved => closed |
2014-03-06 06:10 | jcfr | Fixed in Version | => Slicer 4.1.1 |