Difference between revisions of "Slicer-3.6-QA"
From Slicer Wiki
| Line 110: | Line 110: | ||
|style="color:white; background:black"|Coal||style="width:33%"| Cast ||complete||yes and no||yes and yes||style="background:red"| 14.6% || 0 ||Nicole Aucoin||||Cast Image|| | |style="color:white; background:black"|Coal||style="width:33%"| Cast ||complete||yes and no||yes and yes||style="background:red"| 14.6% || 0 ||Nicole Aucoin||||Cast Image|| | ||
|- | |- | ||
| − | |style="background: | + | |style="background:gold"|Gold||style="width:33%"| VotingBinaryHoleFillingImageFilter ||complete||yes and yes||yes and yes|| 100% || 0 ||Bill Lorensen||||Voting Binary Hole Filling|| |
|- | |- | ||
|style="background:white"|??||style="width:33%"| ModelMaker ||complete||yes and yes||yes and yes|| 57.4% || 1 ||Nicole Aucoin||||same|| | |style="background:white"|??||style="width:33%"| ModelMaker ||complete||yes and yes||yes and yes|| 57.4% || 1 ||Nicole Aucoin||||same|| | ||
| Line 120: | Line 120: | ||
|style="background:hotpink"| Hazard||style="width:33%"| MergeModels ||no||yes and yes||yes and yes|| 62.7% || 0 ||Nicole Aucoin, Daniel Haehn||||same|| | |style="background:hotpink"| Hazard||style="width:33%"| MergeModels ||no||yes and yes||yes and yes|| 62.7% || 0 ||Nicole Aucoin, Daniel Haehn||||same|| | ||
|- | |- | ||
| − | |style="background: | + | |style="background:darkorange"|Clay||style="width:33%"| GrayscaleFillHoleImageFilter ||complete||yes and yes||yes and yes|| 57.1% || 0 ||Bill Lorensen||||Grayscale Fill Hole|| |
|- | |- | ||
| − | |style="background: | + | |style="background:darkgoldenrod"|Bronze||style="width:33%"| GrayscaleGrindPeakImageFilter ||complete||yes and yes||yes and yes|| 64.5% || 0 ||Bill Lorensen||||Grayscale Grind Peak|| |
|- | |- | ||
|style="background:hotpink"| Hazard||style="width:33%"| LabelMapSmoothing ||no||yes and no||yes and yes|| 87.9% || 0 ||Dirk Padfield, Josh Cates, Ross Whitaker||||n/a|| | |style="background:hotpink"| Hazard||style="width:33%"| LabelMapSmoothing ||no||yes and no||yes and yes|| 87.9% || 0 ||Dirk Padfield, Josh Cates, Ross Whitaker||||n/a|| | ||
|- | |- | ||
| − | | | + | |sstyle="color:white; background:black"|Coal||style="width:33%"| ImageLabelCombine ||complete||yes and no||yes and no||style="background:red"| 7.5% || 50 ||Alex Yarmarkovich||||same|| |
|- | |- | ||
|style="background:hotpink"| Hazard||style="width:33%"| ResampleVolume2 ||no||yes and yes||yes and yes||style="background:red"| 46.8% || 0 ||Francois Budin||||Resample Scalar/Vector/DWI Volume|| | |style="background:hotpink"| Hazard||style="width:33%"| ResampleVolume2 ||no||yes and yes||yes and yes||style="background:red"| 46.8% || 0 ||Francois Budin||||Resample Scalar/Vector/DWI Volume|| | ||
Revision as of 02:48, 28 April 2010
Home < Slicer-3.6-QAReturn to Slicer 3.6 documentation
- This page contains our assessment of the Slicer 3.6 modules
- See also the module culling event at the end of April 2010
Contents
Testing Status
Core Modules
| Score | Name | Documentation | Help [1] | Acknowledgment [2] | Current Owner |
|---|---|---|---|---|---|
| ?? | Camera | complete | yes, no link | no | Sebastien Barre |
| ?? | Volumes | complete | yes and yes | yes and yes | Steve Pieper |
| ?? | Diffusion Editor | complete | yes and yes | no | CF Westin |
| ?? | Models | complete | yes and yes | yes and yes | Alex Yarmarkovich |
| ?? | Measurements | complete | yes and yes | yes and yes | Nicole Aucoin |
| ?? | Fiducials | complete | yes and yes | yes and yes | Nicole Aucoin |
| ?? | Data | complete | yes and yes | yes and yes | Steve Pieper |
| ?? | Slices | complete | yes and yes | yes and yes | Jim Miller |
| ?? | Colors | complete | yes and yes | yes and yes | Nicole Aucoin |
| ?? | Interactive Editor | complete | no and yes | yes and no | Steve Pieper |
| ?? | ROI Module | complete | no and yes | no and yes | Alex Yarmarkovich |
| ?? | Volume Rendering | complete | yes and yes | no and yes | Yanling Liu |
| ?? | PET/CT Fusion | complete | yes and yes | yes and yes | Wendy Plesniak |
Application CLI Modules
| Score | Name | Documentation | Help [1] | Acknowledgment [2] | Test coverage [3] | valgrind errors | Current Owner | Comments [4] | 3.6 Documentation Name | |
|---|---|---|---|---|---|---|---|---|---|---|
| Gold | my module | complete | yes and yes | yes, yes, yes | 80% | 0 | ||||
| Hazard | DiffusionWeightedTest | no | no | yes and yes | 69.8% | 0 | CF Westin | n/a | ||
| Silver | DiffusionTensorEstimation | complete | yes and no | yes and yes | 79.7% | 9 | CF Westin | same | ||
| Hazard | DiffusionTensorMathematics | no | yes and no | yes and yes | 56.2% | 5 | CF Westin | Diffusion Tensor Scalar Measurements | ||
| Gold | OrientImage | complete | yes and yes | yes and yes | 85.7% | 0 | Bill Lorensen | OrientImages | ||
| Bronze | LinearRegistration | complete | yes and no | yes and yes | 66.7% | 4 | Daniel Blezek | same | ||
| Coal | RigidRegistration | complete | yes and no | yes and yes | 1.0% | 92 | Daniel Blezek | same | ||
| ?? | RegisterImages | complete | yes and yes | yes and yes | Casey Goodlett, Stephen Aylward | same | ||||
| Coal | Fast Affine Registration | complete | yes and no | yes and yes | 1.4% | 82 | Daniel Blezek | same | ||
| Gold | BSplineDeformableRegistration | complete | yes and yes | yes and yes | 82.3% | 6 | Bill Lorensen | Fast Nonrigid BSpline Registration | ||
| Hazard | TestGridTransformRegistration | no | no | yes and yes | 90.7% | 0 | Yinglin Lee | n/a | ||
| Bronze | CheckerBoard | complete | yes and yes | yes and yes | 67.4% | 0 | Bill Lorensen | CheckerBoard Filter | ||
| Clay | ResampleVolume | complete | yes and yes | yes and yes | 57.1% | 0 | Bill Lorensen | same | ||
| Hazard | PolyDataToLabelmap | no | yes and yes | yes and yes | 87.9% | 0 | Nicole Aucoin, Xiaodong Tao | n/a | ||
| Clay | GaussianBlurImageFilter | complete | yes and no | yes and yes | 55.9% | 0 | Julien Jomier, Stephen Aylward | Gaussian Blur | ||
| Gold | ConfidenceConnected | complete | yes and no | yes and yes | 87.5% | 0 | Jim Miller | Simple region growing | ||
| Hazard | ExecutionModelTour | no | yes and yes | yes and yes | 87.2% | 0 | Daniel Blezek, Bill Lorensen | n/a | ||
| Hazard | ImageReadDicomWrite | no | yes and yes | yes and yes | 86.1% | 0 | Bill Lorensen | Create a Dicom Series | ||
| Bronze | CurvatureAnisotropicDiffusion | complete | yes and yes | yes and yes | 64.3% | 0 | Bill Lorensen | same | ||
| Bronze | GradientAnisotropicDiffusion | complete | yes and yes | yes and yes | 63.4% | 0 | Bill Lorensen | GradientAnisotropicFilter | ||
| Clay | MedianImageFilter | complete | yes and yes | yes and yes | 51.1% | 0 | Bill Lorensen | same | ||
| Bronze | HistogramMatching | complete | yes and yes | yes and yes | 65.0% | 0 | Bill Lorensen | same | ||
| Hazard | OtsuThresholdImageFilter | no | yes and yes | yes and yes | 100% | 0 | Bill Lorensen | OtsuThreshold | ||
| Gold | OtsuThresholdSegmentation | complete | yes and no | yes and yes | 90.9% | 0 | Bill Lorensen | same | ||
| Silver | Subtract | complete | yes and yes | yes and yes | 71.7% | 0 | Bill Lorensen | SubtractImages | ||
| Hazard | Multiply | no | yes and no | yes and yes | 77.6% | 0 | Bill Lorensen | n/a | ||
| Gold | Add | complete | yes and yes | yes and yes | 84.9% | 0 | Bill Lorensen | AddImages | ||
| Bronze | Threshold | complete | yes and no | yes and yes | 55.8% | 0 | Nicole Aucoin | ThresholdImage | ||
| Gold | Mask | complete | yes and no | yes and yes | 80.0% | 0 | Nicole Aucoin | Mask Image | ||
| Coal | Cast | complete | yes and no | yes and yes | 14.6% | 0 | Nicole Aucoin | Cast Image | ||
| Gold | VotingBinaryHoleFillingImageFilter | complete | yes and yes | yes and yes | 100% | 0 | Bill Lorensen | Voting Binary Hole Filling | ||
| ?? | ModelMaker | complete | yes and yes | yes and yes | 57.4% | 1 | Nicole Aucoin | same | ||
| Hazard | MultipleModelsExample | no | yes and no | yes and yes | 1.9% | 50 | Nicole Aucoin | n/a | ||
| Hazard | GrayscaleModelMaker | no | yes and no | yes and yes | 82.1% | 1 | Bill Lorensen | same | ||
| Hazard | MergeModels | no | yes and yes | yes and yes | 62.7% | 0 | Nicole Aucoin, Daniel Haehn | same | ||
| Clay | GrayscaleFillHoleImageFilter | complete | yes and yes | yes and yes | 57.1% | 0 | Bill Lorensen | Grayscale Fill Hole | ||
| Bronze | GrayscaleGrindPeakImageFilter | complete | yes and yes | yes and yes | 64.5% | 0 | Bill Lorensen | Grayscale Grind Peak | ||
| Hazard | LabelMapSmoothing | no | yes and no | yes and yes | 87.9% | 0 | Dirk Padfield, Josh Cates, Ross Whitaker | n/a | ||
| Coal | ImageLabelCombine | complete | yes and no | yes and no | 7.5% | 50 | Alex Yarmarkovich | same | ||
| Hazard | ResampleVolume2 | no | yes and yes | yes and yes | 46.8% | 0 | Francois Budin | Resample Scalar/Vector/DWI Volume | ||
| Hazard | ZeroCrossingBasedEdgeDetectionImageFilter | no | yes and no | yes and yes | 62.5% | 0 | Elliot Uvero | n/a | ||
| Hazard | FreesurferSurfaceSectionExtraction | no | yes and no | yes and yes | 60.0% | 62 | Sylvain Bouix | n/a/ | ||
| ?? | BRAINSDemonWarp | complete | yes and no | yes and no | 0 | Hans Johnson, Greg Harris | same | |||
| ?? | BRAINSFit | complete | yes and yes | yes and yes | 0 | Eun Young Kim | same | |||
| ?? | BRAINSMush | complete | no | no | 0 | Hans Johnson | same | |||
| Hazard | BRAINSResample | no | yes and yes | yes and no | 0 | Hans Johnson | n/a | |||
| Hazard | BRAINSROIAuto | no | yes and no | yes and no | 0 | Hans Johnson | n/a | |||
| Hazard | DiffusionTensorEstimation BatchMake | no | yes and no | yes and no | 0 | Lauren O'Donnell | n/a | |||
| ?? | EMSegment BatchMake | complete | yes and yes | yes and yes | 0 | Julien Jomier, Stephen Aylward, Brad Davis | EM Segmenter batch | |||
| ?? | Gaussian Blur BatchMake | complete | yes and yes | yes and yes | 0 | Julien Jomier, Stephen Aylward, Julien Finet | Gaussian Blur batch | |||
| ?? | RegisterImages BatchMake | complete | yes and yes | yes and yes | 0 | Stephen Aylward, Julien Finet | Register Images batch | |||
| ?? | ResampleScalarVolume BatchMake | complete | yes and yes | yes and yes | 0 | Julien Finet | Resample Volume batch | |||
| Hazard | Diffeomorphic Demons Algorithm | no | no | yes and yes | 0 | Tom Vercauteren, Ender Konukoglu, Kilian Pohl | n/a | |||
| ?? | ResampleDTIVolume | complete | yes and yes | yes and yes | 0 | Francois Budin | same | |||
| ?? | Rician LMMSE Image Filter | complete | yes and no | yes and yes | 0 | Antonio Tristan Vega, Santiago Aja Fernandez, Marc Niethammer | same | |||
| ?? | Unbiased Non Local Means filter for DWI | complete | yes and no | yes and yes | 0 | Antonio Tristán Vega, Santiago Aja-Fernandez | n/a | |||
| ?? | Joint Rician LMMSE Image Filter | complete | yes and no | yes and yes | 0 | Antonio Tristán Vega, Santiago Aja Fernández | same | |||
| Hazard | Extract Skeleton | no | yes and no | no and yes | 0 | Pierre Seroul, Martin Styner, Guido Gerig, Stephen Aylward | n/a | |||
| ?? | MRI Bias Field Correction | complete | yes and yes | yes and yes | 0 | Sylvain Jaume | same | |||
| Hazard | N4ITK MRI Bias Correction | no | yes and no | yes and yes | 0 | Nick Tustison, Andriy Fedorov | same | |||
| ?? | ACPC Transform | complete | yes and yes | yes and yes | 0 | Nicole Aucoin | same | |||
| ?? | Mesh Contour Segmentation | complete | yes and yes | yes and yes | 0 | Peter Karasev, Karol Chudy, Allen Tannenbaum | same | |||
| ?? | Utah BSpline Deformable registration | complete | yes and no | yes and yes | 0 | Sam Gerber, Jim Miller, Steve Pieper, Ross Whitaker | n/a | |||
| Hazard | Calculate Volume Statistics | no | no | no | 0 | Tri Ngo | n/a | |||
| Hazard | Generate Connectivity Map | no | no | no | 0 | Tri Ngo | n/a | |||
| Hazard | ROI Tract Filter | no | no | no | 0 | Tri Ngo | n/a | |||
| Hazard | Stochastic Tractography Filter | no | no | no | 0 | Tri Ngo | n/a | |||
| Hazard | MIDAS Data and Slicer Tutorials | no | yes and no | yes and no | 0 | Patrick Reynolds, Stephen Aylward | n/a |
Testing Partition
Most Slicer modules have a GUI component and a Data Processing component.
Testing GUI components is still a challenge, so we will focus here on testing the Data Processing components. This can be done in most cases by partitioning the module into a GUI section and a Data Processing section, where the second one usually takes the form of a C++ class (although that is not a requirement).
The data processing section can be tested by using standard CTest/CMake mechanisms. Basically by adding ADD_TEST() entries to the CMakeLists.txt file of the module.
Luis Ibanez' scoring system
The following scoring will be applied to the data processing sections of all modules:
| Score | Code Coverage | Valgrind Errors | Documentation | Tutorial |
|---|---|---|---|---|
| Gold | > 80% | 0 | yes | yes |
| Silver | > 70% | < 10 | yes | yes |
| Bronze | > 60% | < 50 | yes | yes |
| Clay | > 50% | < 100 | yes | yes |
| Coal | < 50% | > 100 | yes | yes |
| Hazard | unknown | unknown | no | no |
The code coverage and Valgrind error must be the ones reported on the Nightly Slicer Dashboard. Anecdotal data is not acceptable.
