[Insight-developers] GradientToMagnitudeImageFilter name

Cory Quammen cquammen at cs.unc.edu
Tue May 3 08:38:11 EDT 2011


On Tue, May 3, 2011 at 7:47 AM, David Doria <daviddoria at gmail.com> wrote:
> On Mon, May 2, 2011 at 9:21 PM, Cory Quammen <cquammen at cs.unc.edu> wrote:
>> David,
>>
>> I think a name change makes sense. VectorMagnitudeImageFilter seems
>> like a good name to me.
>>
>> To preserve backwards compatibility, I suggest re-implementing
>> GradientToMagnitudeImageFilter as a subclass of
>> VectorMagnitudeImageFilter and moving GradientToMagnitudeImageFilter
>> to the Nonunit/Deprecated module.
>>
>> Do you want to submit a patch to gerrit with the changes?
>>
>> Cory
>
> Sure, I'll make a patch. Which module should
> VectorMagnitudeImageFilter end up in? It is currently in
> ImageGradient. I was thinking ImageIntensity?
>
> David
>

Seems reasonable to me.

Cory

-- 
Cory Quammen
Computer Integrated Systems for Microscopy and Manipulation (CISMM)
Department of Computer Science
University of North Carolina at Chapel Hill
http://www.cs.unc.edu/~cquammen


More information about the Insight-developers mailing list