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

[Matt McCormick]

Hi Brad,

Thanks  for the note.  I'll start working on a patch.

Matt

On Wed, Jun 6, 2012 at 10:04 AM, Bradley Lowekamp
<blowekamp at mail.nih.gov> wrote:
> 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
>
>
>
>