[Insight-developers] ITK 4.0 release clean up

M Stauffer -V- mstauff at verizon.net
Tue Dec 13 16:28:47 EST 2011


I'm available, but only from about 10pm-12.30.

-M

>-----Original Message-----
>From: insight-developers-bounces at itk.org 
>[mailto:insight-developers-bounces at itk.org] On Behalf Of Matt McCormick
>Sent: Tuesday, December 13, 2011 4:20 PM
>To: Bill Lorensen
>Cc: Hans J. Johnson; ITK
>Subject: Re: [Insight-developers] ITK 4.0 release clean up
>
>If there is interest, I'd be up for a renaming hackathon tonight.  I
>think this is *very* important.  We could hack on it, and see where we
>are tomorrow.  If it does no go well -- abandon.
>
>On Tue, Dec 13, 2011 at 4:02 PM, Bill Lorensen 
><bill.lorensen at gmail.com> wrote:
>> I'm OK with the renaming if:
>>
>> 1) You can do it fast, like now!
>> 2) It won't delay the release
>> 3) It won't fixing defects before the release.
>>
>> Bill
>>
>> On Tue, Dec 13, 2011 at 3:48 PM, Matt McCormick
>> <matt.mccormick at kitware.com> wrote:
>>> Hi Michael, Brian,
>>>
>>> I also think it would be better to clean up the names now, 
>but we have
>>> run out of time.  We should definitely make the change, though.
>>>
>>> Thank you for coming to consensus on v4.  There was not enthusiasm
>>> about v4 at the TCon, but better ideas were not put forth.  
>v4 sounds
>>> fine to me, and as Arnaud recommended, we can use the same for the
>>> doxygen version tag.  We could talk to Arnaud about a 
>possible change
>>> to V4, too -- it would not be a huge issue.
>>>
>>> It will be an inconvenience to the users to change their 
>code, but we
>>> will not have many users if they cannot figure out what is going on.
>>> It would be foolish to not make a very important change for the sake
>>> of a month's users to the detriment of the next ten year's 
>of use and
>>> development that follows.  There are many improvements that users
>>> should access with ITK v4.0.0, but the state of level sets and
>>> registration is, in reality, only RC or beta.  They have come a long
>>> way, but to freeze API here would just make a mess permanent.
>>>
>>> I will put together a patch that adds a CMake warning if the new
>>> registration or level sets modules are enabled.  Similar to
>>> ITK_USE_REVIEW=ON in ITKv3, it will warn that the API is not stable
>>> yet.  There was agreement to hold off backwards 
>incompatibility until
>>> June at the Bethesda meeting, and to use BIG BOLD LETTERS that it is
>>> not to be expected in the release notes, but this should be helpful,
>>> too.
>>>
>>> Thanks,
>>> Matt
>>>
>>> On Tue, Dec 13, 2011 at 3:11 PM, brian avants 
><stnava at gmail.com> wrote:
>>>> personally, i think it would be better to clean up the names
>>>>
>>>> it's kind of ugly now and is likely to be confusing
>>>>
>>>> however, there was strong resistance to this on t-con , mainly from
>>>> bill, i think
>>>>
>>>> v4 was also not met with enthusiasm
>>>>
>>>> brian
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> On Tue, Dec 13, 2011 at 12:07 PM, M Stauffer -V- 
><mstauff at verizon.net> wrote:
>>>>> Hi,
>>>>>
>>>>> Should we do the renaming of the new registartion 
>framework before the
>>>>> release? Brian and I have been thinking about using the 
>same 'v4' suffix
>>>>> that the new LevelSets framework uses. This would be nice 
>and short and
>>>>> have some continuity. Did you talk about that in tcon last week?
>>>>>
>>>>> So, it'd be Metricsv4 and RegistrationMethodsv4 for the 
>modules, and
>>>>> itkImageToImageMetricv4, etc, for the classes.
>>>>>
>>>>> I think 'V4' would look better than 'v4', but since 
>LevelSetsv4 already
>>>>> uses 'v4', we should use 'v4'.
>>>>>
>>>>> -M
>>>>>
>>>>>>-----Original Message-----
>>>>>>From: insight-developers-bounces at itk.org
>>>>>>[mailto:insight-developers-bounces at itk.org] On Behalf Of 
>Matt McCormick
>>>>>>Sent: Tuesday, December 13, 2011 1:59 PM
>>>>>>To: ITK; insight-users at itk.org
>>>>>>Subject: [Insight-developers] ITK 4.0 release clean up
>>>>>>
>>>>>>Hi,
>>>>>>
>>>>>>cc19bacb has been tagged as v4.0rc05.  As discussed 
>during the weekly
>>>>>>TCon, please refrain from merging any new features until after the
>>>>>>4.0.0 release, which
>>>>>>is scheduled for December 20th:
>>>>>>
>>>>>>  http://www.itk.org/Wiki/ITK_Release_4/ReleaseSchedules
>>>>>>
>>>>>>Please have all fixes in by Sunday the 18th for a nightly
>>>>>>dashboard build,
>>>>>>tagging the 19th, and tarball release on the 20th.
>>>>>>
>>>>>>
>>>>>>Dashboard pimples to pop:
>>>>>>
>>>>>>- TemporalProcessObjectTest failure:
>>>>>>https://issues.itk.org/jira/browse/ITK-2862
>>>>>>
>>>>>>- VideoSourceTest failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2863
>>>>>>
>>>>>>- itkKdTreeTest11 failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2864
>>>>>>
>>>>>>- itkSimpleImageRegistrationTest on gcc-4.4.4 Shared Release:
>>>>>> https://issues.itk.org/jira/browse/ITK-2865
>>>>>>
>>>>>>- RGBColormapTest_* failures:
>>>>>> https://issues.itk.org/jira/browse/ITK-2866
>>>>>>
>>>>>>- vnl_test_sparse_lst_sqr_function failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2867
>>>>>>
>>>>>>- itkMRIBiasFieldCorrectionFilterTest failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2868
>>>>>>
>>>>>>- itkLSMImageIOTest failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2869
>>>>>>
>>>>>>- itkMultiThreaderEnvTest{123,88} failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2870
>>>>>>
>>>>>>- itkTimeVaryingVelocityFieldImageRegistrationTest failure:
>>>>>> https://issues.itk.org/jira/browse/ITK-2871
>>>>>>
>>>>>>- SunOS-g++ intmax_t build errors:
>>>>>> https://issues.itk.org/jira/browse/ITK-2872
>>>>>>
>>>>>>- SunOS-g++ type conversion warnings:
>>>>>> https://issues.itk.org/jira/browse/ITK-2873
>>>>>>
>>>>>>- Video shadow warnings:
>>>>>> https://issues.itk.org/jira/browse/ITK-2874
>>>>>>
>>>>>>- JointHistogramMutualInformation registration tests memory leaks:
>>>>>> https://issues.itk.org/jira/browse/ITK-2875
>>>>>>
>>>>>>- itkQuasiNewtonObjectOptimizerRegistrationTest3 
>uninitialized memory
>>>>>>conditional: https://issues.itk.org/jira/browse/ITK-2876
>>>>>>
>>>>>>- itkTransformTest uninitialized memory conditional:
>>>>>>https://issues.itk.org/jira/browse/ITK-2877
>>>>>>
>>>>>>- itkFloatingPointExceptionsTest5 potential memory leak, freeing
>>>>>>invalid memory: https://issues.itk.org/jira/browse/ITK-2878
>>>>>>
>>>>>>Cheers to Bill Lorensen for fixing the FFTConvolution Sobel test
>>>>>>failure before I could create a ticket.
>>>>>>
>>>>>>Thanks,
>>>>>>Matt
>>>>>>_______________________________________________
>>>>>>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.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
>>>>>
>>> _______________________________________________
>>> 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.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
>>
>>
>>
>> --
>> Unpaid intern in BillsBasement at noware dot com
>_______________________________________________
>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.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



More information about the Insight-developers mailing list