[Insight-developers] Change to itkMultiResolutionPyramidImageFilter.txx

[Bradley Lowekamp]

I am giving these changes a closer look. I don' think they were the  
correct thing to do as they made a fundamental change in the behavior  
of the Pyramids.

http://www.itk.org/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMultiResolutionPyramidImageFilter.txx?root=Insight&r1=1.25&r2=1.26&sortby=date

Changing from the Shrinker (which used to be an optimized nearest  
neighbor) to using a linear interpolated Resample seems wrong to me.

I don't fully understand what the "center of mass" bug is. I think a  
more documentation and a diagram is really needed, to describe the  
behavior of resampling, and shrinking. This will prevent another bug  
from happening again when the current behavior is not liked.

Brad

ps
I could talk more about optimization of itkShrinkImageFilter has been  
reduced too, but I think that is another subject.


On Mar 25, 2009, at 4:33 PM, Bill Lorensen wrote:

> Luis,
>
> This change
>
> http://www.itk.org/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMultiResolutionPyramidImageFilter.txx?root=Insight&r1=1.29&r2=1.30&sortby=date
>
> should be added to the 3.10 release. This undoes the backward
> compatibility I broke with a bad checkin on Dec 17
> http://www.itk.org/cgi-bin/viewcvs.cgi/Code/Algorithms/itkMultiResolutionPyramidImageFilter.txx?root=Insight&r1=1.26&r2=1.27&sortby=date
>
> Bill
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/listinfo/insight-developers

========================================================
Bradley Lowekamp
Lockheed Martin Contractor for
Office of High Performance Computing and Communications
National Library of Medicine
blowekamp at mail.nih.gov


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20090326/40b0b7f2/attachment.htm>