View Issue Details [ Jump to Notes ] [ Print ]
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 
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.
TagsNo tags attached.
Resolution Date
Sprint
Sprint Status
Attached Files

 Relationships
related to 0008482closedkentwilliams RecursiveMultiResolutionPyramid needs to center of mass of original image 

  Notes
(0000464)
Luis Ibanez (manager)
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 (developer)
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 (manager)
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 (developer)
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.

 Issue History
Date Modified Username Field Change
2009-03-05 13:53 Tom Vercauteren Note Added: 0015570
2009-03-05 13:53 Tom Vercauteren Status closed => feedback
2009-03-05 13:53 Tom Vercauteren Resolution fixed => reopened
2009-03-05 13:54 Tom Vercauteren Relationship added related to 0008482
2009-04-21 09:39 Luis Ibanez Note Added: 0016106
2009-04-21 12:03 Tom Vercauteren Note Added: 0016122
2009-04-21 12:03 Tom Vercauteren Status feedback => closed
2009-04-21 12:03 Tom Vercauteren Resolution reopened => fixed

Copyright © 2000 - 2018 MantisBT Team