MantisBT - ITK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0009745ITKpublic2009-10-20 13:232010-10-21 12:31
ReporterLuis Ibanez 
Assigned ToLuis Ibanez 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Resolution Date
Sprint
Sprint Status
Summary0009745: MattesMutualInformation doesn't work on Binary Images
DescriptionDespite the fact that Mutual Information is a suitable metric for registering binary images, the current implementation of MattesMutualInformation in ITK doesn't support this type of images. Internal computations assume that more than the joint histogram always has more than two bins by two bins.
Steps To Reproduce
Additional InformationReported in the ITK users mailing list by Serena Fabri
http://www.itk.org/pipermail/insight-users/2009-October/033022.html [^]

also reported by Hans Johnson during tcons.
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-10-20 13:23Luis IbanezNew Issue
2009-10-20 13:23Luis IbanezStatusnew => assigned
2009-10-20 13:23Luis IbanezAssigned To => Luis Ibanez
2009-10-21 11:56Luis IbanezNote Added: 0018142
2009-10-21 12:03Luis IbanezNote Added: 0018144
2009-11-11 12:15Luis IbanezNote Added: 0018389
2009-11-11 12:16Luis IbanezNote Added: 0018390
2009-11-11 12:16Luis IbanezStatusassigned => resolved
2009-11-11 12:16Luis IbanezResolutionopen => fixed
2010-10-21 12:31Gabe HartStatusresolved => closed

Notes
(0018142)
Luis Ibanez   
2009-10-21 11:56   
There are two implementations of MattesMutualInformation currently in the toolkit.

A) Code/Algorithms/itkMattesMutualInformationImageToImageMetric.h

B) Code/Review/itkOptMattesMutualInformationImageToImageMetric.h

Implementation (A) is used by default, and implementation (B) will be used if the CMake flag:ITK_USE_OPTIMIZED_REGISTRATION_METHODS is ON.
(0018144)
Luis Ibanez   
2009-10-21 12:03   
The following two commits fix case (A) (from note above).

http://public.kitware.com/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMattesMutualInformationImageToImageMetric.txx?root=Insight&r1=1.63&r2=1.64&sortby=date [^]

http://public.kitware.com/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMattesMutualInformationImageToImageMetric.h?root=Insight&r1=1.27&r2=1.28&sortby=date [^]
(0018389)
Luis Ibanez   
2009-11-11 12:15   
The following commits fixed case (B)

http://public.kitware.com/cgi-bin/viewcvs.cgi/Code/Review/itkOptMattesMutualInformationImageToImageMetric.txx?root=Insight&r1=1.33&r2=1.34&sortby=date [^]

http://public.kitware.com/cgi-bin/viewcvs.cgi/Code/Review/itkOptMattesMutualInformationImageToImageMetric.h?root=Insight&r1=1.8&r2=1.9&sortby=date [^]

http://public.kitware.com/cgi-bin/viewcvs.cgi/Code/Review/itkOptMattesMutualInformationImageToImageMetric.h?root=Insight&r1=1.7&r2=1.8&sortby=date [^]
(0018390)
Luis Ibanez   
2009-11-11 12:16   
Tests for binary images were added and are passing in the dashboard.

http://public.kitware.com/cgi-bin/viewcvs.cgi/Examples/Registration/CMakeLists.txt?root=Insight&r1=1.189&r2=1.190&sortby=date [^]

http://public.kitware.com/cgi-bin/viewcvs.cgi/Examples/Registration/CMakeLists.txt?root=Insight&r1=1.187&r2=1.188&sortby=date [^]