[ITK-dev] [ITK] ConnectedThreshold

[Bill Lorensen]

I agree with Brad. Not worth the effort nor breaking the API.


On Thu, May 8, 2014 at 10:13 AM, Bradley Lowekamp
<blowekamp at mail.nih.gov> wrote:
> Dirk,
>
> The name has never bothered me. It does not sound like a productive change to me, and I think it will cause confusion and delay with our users.
>
> Brad
>
> On May 8, 2014, at 8:17 AM, Padfield, Dirk R (GE Global Research) <padfield at research.ge.com> wrote:
>
>> Hi All,
>>
>> When looking through the ITK region growing algorithms, I noticed that one of them has a different naming convention:
>>
>> itkConnectedThresholdImageFilter
>>
>> itkConfidenceConnectedImageFilter
>> itkIsolatedConnectedImageFilter
>> itkNeighborhoodConnectedImageFilter
>> itkVectorConfidenceConnectedImageFilter
>>
>> All of them end in "Connected" except "ConnectedThreshold".  I can't think of any good justification for this except that "this is how it's always been."
>>
>> What do you think about renaming it to "ThresholdConnected", and then creating a (deprecated) shell with the old name that just calls the new one?
>>
>> Thanks,
>> Dirk
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Kitware offers ITK Training Courses, for more information visit:
>> http://kitware.com/products/protraining.php
>>
>> 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
>> _______________________________________________
>> Community mailing list
>> Community at itk.org
>> http://public.kitware.com/cgi-bin/mailman/listinfo/community
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/protraining.php
>
> 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



-- 
Unpaid intern in BillsBasement at noware dot com