[Insight-developers] Naming Conventions with StatisticsRefactoring

Bill Lorensen bill.lorensen at gmail.com
Sun Apr 12 14:00:48 EDT 2009


I have been looking at this.

Thanks,

Bill

On Sun, Apr 12, 2009 at 1:50 PM, Luis Ibanez <luis.ibanez at kitware.com> wrote:
> Hi Bill,
>
> It may be worth to consider the method that we set up in the Migration Guide:
> http://www.itk.org/Wiki/Proposals:Refactoring_Statistics_Framework_2007_Migration_Users_Guide
>
> It lists class by class, what methods have been removed,
> and explain what new methods are provided in replacement.
>
> BTW: The recent renaming of classes has not been mapped in this Wiki page.
>          We probably should do that at some point.
>
>
>     Luis
>
>
> ---------------------------------------------------------------------------------
> On Sat, Apr 11, 2009 at 6:08 PM, Bill Lorensen <bill.lorensen at gmail.com> wrote:
>> Luis,
>>
>> Sounds good to me also. My observation is that it is non-trivial to
>> convert form the itkHistogram to the new histogram. I think that
>> renaming it will make this transition easier for our customers. I have
>> have a plan to help our customers to upgrade to the new statistics.
>> It's stil a half-baked plan, so I'll hold back on details for now.
>>
>> Bill
>>
>> On Sat, Apr 11, 2009 at 5:51 PM, Luis Ibanez <luis.ibanez at kitware.com> wrote:
>>> Bill,
>>>
>>> Your observation is correct.
>>>
>>> The new itkHistogram now performs as the VariableDimensionHistogram
>>> of the old repository.
>>>
>>> MeasurementHistogram sound fine to me, as long as we provide good
>>> documentation, given that a newcomer will probably expect to find
>>> "itkHistogram", so we should provide a clear explanation of why it is not
>>> there in the new framework.
>>>
>>>
>>>      Luis
>>>
>>>
>>> -----------------------------------------
>>> On Sat, Apr 11, 2009 at 3:27 PM, Bill Lorensen <bill.lorensen at gmail.com> wrote:
>>>> If Histogram were called MeasurmentHistogram, I might be able to make
>>>> backward compatible itkHistogram. Currently, the new Histogram has
>>>> changed the template arguments from the old. Actually it matches
>>>> exactly itkVariableDimensionHistogram, but that is not such a great
>>>> name for the new Statistics.
>>>>
>>>> Bill
>>>>
>>>>
>>>
>>
>


More information about the Insight-developers mailing list