Romulo
- Solved bug 295
+ Removed the "destroyed" flag
+ List/Tree widget is now cleared in the destructor
+ Destruction order was the cause for the crash
- Made destructors virtual in the base classes
+ It could contribute to memory leaks
+ Other class hierarchies should be checked as well
-Solved bug 295
+ SelectedItemChangedInTree() event was called after the window had been closed and the object instance had been destroyed.
+ Horrible solution: added a static "destroyed" flag to indicate that the object instance has been destroyed.
* Flag will be valid for all active instances, since it's static.
+ In desperate need of a better solution!
* Need to find the correct location to put it, though.
We need to find a better way to fix this.
The problem arises when a target depends on a ggo.h but does not depends on the
ggo.c.
When a target depends on a ggo.c it is explicitly listed in its dependencies
including the custom command that generates it.
When the ggo.h is include in a cxx included in a target CMake has no way to
know it and does not include the custom command that generates it.
In this patch I manually add the dependency between the cxx and the ggo.h.
Romulo:
- Finally solved bug 243 (moving linked images)
+ when propagating, vvMainWindow::UpdateLinkedNavigation was always using slicer[0] as the reference. Now, the reference vvSlicer
is passed as an argument from vvSlicerManager::UpdateLinkedNavigation.
- More corrections wrt bug 283 - cursor rendering... (accidentally removed from the bug tracker)
+ better treatment of cursor rendering when images are linked
+ there are still corrections to be made wrt to the corner annotations.
dsarrut [Tue, 22 Mar 2011 16:55:12 +0000 (16:55 +0000)]
Romulo:
- Corrected bug when the size of the dicom template should be used as the reference for the resampling, instead of the spacing
- Added a command line option to for the size as the reference (--useSizeAsReference - default=off)
dsarrut [Tue, 22 Mar 2011 16:25:50 +0000 (16:25 +0000)]
Romulo:
- Corrected bug when the size of the dicom template should be used as the reference for the resampling, instead of the spacing
- Added a command line option to for the size as the reference (--useSizeAsReference - default=off)
srit [Mon, 21 Mar 2011 16:09:05 +0000 (16:09 +0000)]
Romulo:
- set Link/Link All functionality
+ when linking two images: take the one in the first combo box (left) as the reference for viewing adjustments
+ when linking all images: take the last manipulated image as the reference. The last manipulated image is the one with which the user last interacted using the mouse (mouseclick, mousewheel, but NOT mousemove)
srit [Thu, 17 Mar 2011 14:43:08 +0000 (14:43 +0000)]
Romulo:
1) Perform image resampling if the spacing of the MHD file doesn't match the slice thickness found in the reference dicom series. Use the dicom spacing as the output.
2) Remove files from the dicom series if the number of slices in the MHD doesn't match the number of files in the image (it may be smaller due to rounding of image resampling). An error is still generated if there are more slices than files, though.
- issue: VV somehow still detects all images (Open Dicom), but the number of files and the series itself are correctly read.
3) Warnings are printed in both cases above if "verbose" is enabled.
srit [Thu, 17 Mar 2011 14:42:13 +0000 (14:42 +0000)]
Romulo: changed rounding of image resampling to floor() because, from a clinical point of view, it seems to be better to miss a slice than to add information that didn't exist in the original image.