MantisBT - ITK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000503ITKpublic2004-01-09 14:042009-04-21 12:03
ReporterLuis Ibanez 
Assigned ToLuis Ibanez 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Resolution Date
Sprint
Sprint Status
Summary0000503: MultiResolutionPyramid filter shouldn't use image spacing in the smoothin
DescriptionThe subsampling of the MultiResolutionPyramid filters require to perform smoothing first. This is done using the DiscreteGaussianImageFilter which by defaults takes the pixel spacing into consideration.
Since the smoothing sigma in the Pyramid is a function of the ratio of scaling factors between consecutive levels, the DiscreteGaussian shouldn't take image spacing into consideration.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0008482closed kentwilliams RecursiveMultiResolutionPyramid needs to center of mass of original image 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-03-05 13:53Tom VercauterenNote Added: 0015570
2009-03-05 13:53Tom VercauterenStatusclosed => feedback
2009-03-05 13:53Tom VercauterenResolutionfixed => reopened
2009-03-05 13:54Tom VercauterenRelationship addedrelated to 0008482
2009-04-21 09:39Luis IbanezNote Added: 0016106
2009-04-21 12:03Tom VercauterenNote Added: 0016122
2009-04-21 12:03Tom VercauterenStatusfeedback => closed
2009-04-21 12:03Tom VercauterenResolutionreopened => fixed

Notes
(0000464)
Luis Ibanez   
2004-01-09 14:09   
This is fixed by adding a call to
smoother->SetUseImageSpacing( false )
in the files
itkRecursiveMultiResolutionPyramidImageFilter and
itkMultiResolutionPyramidImageFilter
(0015570)
Tom Vercauteren   
2009-03-05 13:53   
The fix has been unfixed. MultiResolutionPyramidImageFilter now uses the spacing in the smoothing:
http://www.itk.org/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMultiResolutionPyramidImageFilter.txx?root=Insight&r1=1.26&r2=1.27&sortby=date [^]

Is there a reason for that?

Also the behavior is now inconsistent with the RecursiveMultiResolutionPyramidImageFilter which does not use the image spacing in the filtering:
http://www.itk.org/cgi-bin/viewcvs.cgi/Code/Algorithms/itkRecursiveMultiResolutionPyramidImageFilter.txx?root=Insight&sortby=date&view=markup [^]
(0016106)
Luis Ibanez   
2009-04-21 09:39   
This issue has been discussed in several ITK tcons,

(for example:)
http://www.itk.org/Wiki/Agenda%26Status_041709 [^]

and it is considered fixed now.

Tom:
Could you please verify if we can close this bug ?

    Thanks
(0016122)
Tom Vercauteren   
2009-04-21 12:03   
Seems fine to me:
http://www.itk.org/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMultiResolutionPyramidImageFilter.txx?root=Insight&r1=1.29&r2=1.30&sortby=date [^]

I'll close the bug.