[Insight-developers] A2D2 patch to merge for this RC cycle

[Bradley Lowekamp]

Hi Matt,

Thanks for letting us know you are taking up this responsibility.

I noticed that Valgrind has a dozen defects related to the new FEM test from this patch:

http://open.cdash.org/viewDynamicAnalysis.php?buildid=2339087

Thanks,
Brad

On Jun 5, 2012, at 12:07 PM, Matt McCormick wrote:

> On Mon, Jun 4, 2012 at 10:16 PM, Bradley Lowekamp
> <blowekamp at mail.nih.gov> wrote:
>> Matt,
>> 
>> It sounds like it has been review and test about as good as it can be in gerrit ( after Bill's review of course ).
>> 
>> I would like to know that the developers of the patch are planning to working on addressing the likely dashboard issue that show up on Tuesday nights dashboard on Wednesday, before this patched is merged. Because it would be great it the dashboard is cleaned up in time for Friday's TCON, to make the tag then.
>> 
> 
> Merged so we have a few days to iron out any remaining issues.  I will
> take responsibility for any build or test failures that are not
> addressed.
> 
> Thanks,
> Matt
> 
>> On Jun 4, 2012, at 9:41 PM, Matt McCormick wrote:
>> 
>>> Hi Bill,
>>> 
>>> Here's the link:
>>> 
>>>  http://review.source.kitware.com/#/c/5869/
>>> 
>>> Thanks,
>>> Matt
>>> 
>>> On Mon, Jun 4, 2012 at 8:30 PM, Bill Lorensen <bill.lorensen at gmail.com> wrote:
>>>> Matt
>>>> 
>>>> Which gerrit topic is this?
>>>> 
>>>> Bill
>>>> 
>>>> On Mon, Jun 4, 2012 at 6:32 PM, Matt McCormick
>>>> <matt.mccormick at kitware.com> wrote:
>>>>> Hi all,
>>>>> 
>>>>> I think the following A2D2 patch should be merged for this release
>>>>> candidate cycle.  This patch has under development for a long period
>>>>> of time, and it is currently passing all tests.  I do not see related
>>>>> warnings or build errors on the platforms I have tested.  Here are the
>>>>> results from manual tests on all Linux and Windows platforms, a Mac
>>>>> build is in process:
>>>>> 
>>>>> Linux-Clang:
>>>>> 
>>>>>  http://open.cdash.org/buildSummary.php?buildid=2334099
>>>>> 
>>>>> Windows - Visual Studio 2010:
>>>>> 
>>>>>  http://open.cdash.org/buildSummary.php?buildid=2334134
>>>>> 
>>>>> 
>>>>> Without objection, I will merge tomorrow.
>>>>> 
>>>>> 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.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
>>> _______________________________________________
>>> 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
>> 

========================================================
Bradley Lowekamp  
Medical Science and Computing for
Office of High Performance Computing and Communications
National Library of Medicine 
blowekamp at mail.nih.gov



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/pipermail/insight-developers/attachments/20120606/72a80ce8/attachment.htm>