[Insight-developers] Naming Conventions with StatisticsRefactoring
Luis Ibanez
luis.ibanez at kitware.com
Sun Apr 12 13:50:06 EDT 2009
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