Romulo:
- Changed CMakeLists.txt for tests
+ $CLITKDIR/tests mirrors $CLITKDIR, but contains only test code
+ CLITK_DATA_PATH must be specified if BUILD_TESTING is on
* test code can directly access the variable (macro)
+ Test executables generated in $CLITKDIR/build/tests/bin
Romulo:
- 1st version of the LineProfile tool
+ clitkLineProfile -i IMAGE --p0 POINT --p1 POINT
+ POINT can, in principle, be in 2D or 3D, but must match the image dimension (needs testing still)
+ output to std out
* [INDEX] VALUE
* index is comma separated list of coordinates
Moved vvImageReader and vvImageWriter to clitkCommon for use in tools. Removed vvConstants.h, the typedef is now in vvImageReader. Made vvImageReader Qt independent since the inheritance of QThread did not seem useful.
Romulo:
- Including automated test procedures in clitk
- Addition of test results
+ These files are updated with the result of all tests when "make test" is executed
Romulo:
- Including automated test procedures in clitk
- Addition of test data
+ build/tests is the default working directory set by CMake when running tests
Romulo:
- Including automated test procedures in clitk
- CMakeLists.txt now includes the "tests" subdir
- tests subdir contains building instructions and source code for all tests
+ subdir structure mirrors that of $CLITK3HOME
+ tests/CMakeLists.txt contains building instructions
- see also build/tests and build/Testing (committed separately)
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)