[ITK] [ITK-dev] Patches for the 4.8.2 release

Matt McCormick matt.mccormick at kitware.com
Tue Nov 10 13:31:28 EST 2015


Excellent, thank you Simon.

The patch has been merged into the master branch.  We will merge it
into the release branch tomorrow if no issues show up on the Nightly
dashboard:

  https://open.cdash.org/index.php?project=Insight

Thanks,
Matt

On Tue, Nov 10, 2015 at 12:13 PM, Simon Alexander <skalexander at gmail.com> wrote:
> sorry, issue is 20381
>
> On Tue, 10 Nov 2015 at 12:13 Simon Alexander <skalexander at gmail.com> wrote:
>>
>> I had already done that, the problem was elsewhere but I have now sorted
>> it.
>>
>> Issue is 20301.
>>
>> I added Matt as reviewer but couldn't find Dzenan in the system.
>>
>>
>>
>> On Tue, 10 Nov 2015 at 11:59 Matt McCormick <matt.mccormick at kitware.com>
>> wrote:
>>>
>>> Hi Simon,
>>>
>>> Great!
>>>
>>> You should be able to sign up for Gerrit, then add your SSH key to
>>> your account and push.  Instructions are here:
>>>
>>>   https://cmake.org/Wiki/ITK/Git/Account#Gerrit
>>>
>>> Please let us know if you have any issues.
>>>
>>> Thanks,
>>> Matt
>>>
>>> On Tue, Nov 10, 2015 at 11:35 AM, Simon Alexander <skalexander at gmail.com>
>>> wrote:
>>> > Ok, I have a patch ready off the release branch.
>>> >
>>> > I don't think I have push access for gerrit though, is there a way to
>>> > just
>>> > start a review from a patch that I am missing in the linked docs?
>>> >
>>> > On Mon, 9 Nov 2015 at 17:08 Matt McCormick <matt.mccormick at kitware.com>
>>> > wrote:
>>> >>
>>> >> Great!
>>> >>
>>> >> Instructions can be found here:
>>> >>
>>> >>   http://www.itk.org/Wiki/ITK/Git
>>> >>
>>> >> Please branch off of the "release" branch.
>>> >>
>>> >> Thanks,
>>> >> Matt
>>> >>
>>> >> On Mon, Nov 9, 2015 at 4:40 PM, Simon Alexander
>>> >> <skalexander at gmail.com>
>>> >> wrote:
>>> >> > Ok, I'll dig up my notes on how to do that and submit it shortly.
>>> >> > Should
>>> >> > this be done against trunk or a release branch ?
>>> >> >
>>> >> > On Mon, 9 Nov 2015 at 16:19 Matt McCormick
>>> >> > <matt.mccormick at kitware.com>
>>> >> > wrote:
>>> >> >>
>>> >> >> Thanks for the patch, Simon! Yes, we can wait until this patch is
>>> >> >> merged to make the release. As Dzenan recommends, please submit the
>>> >> >> patch to Gerrit with a verification for
>>> >> >> itk::TriangleCell::EvaluatePosition in itkTriangleCellTest.cxx.
>>> >> >>
>>> >> >> Thanks,
>>> >> >> Matt
>>> >> >>
>>> >> >> On Mon, Nov 9, 2015 at 3:54 PM, Dženan Zukić <dzenanz at gmail.com>
>>> >> >> wrote:
>>> >> >> > Better submit the usual way. I doubt Matt will include it without
>>> >> >> > the
>>> >> >> > usual
>>> >> >> > testing. Add me and Matt as reviewers.
>>> >> >> >
>>> >> >> > Regards,
>>> >> >> > Dženan
>>> >> >> >
>>> >> >> > On Mon, Nov 9, 2015 at 3:18 PM, Simon Alexander
>>> >> >> > <skalexander at gmail.com>
>>> >> >> > wrote:
>>> >> >> >>
>>> >> >> >> We found this today (patch attached).   If 4.8.2 is right around
>>> >> >> >> the
>>> >> >> >> corner, can you consider it for inclusion.  If time isn't so
>>> >> >> >> short,
>>> >> >> >> I'll
>>> >> >> >> submit the usual way....
>>> >> >> >>
>>> >> >> >> cheers,
>>> >> >> >> Simon
>>> >> >> >>
>>> >> >> >>
>>> >> >> >>
>>> >> >> >> On Mon, 9 Nov 2015 at 09:57 Matt McCormick
>>> >> >> >> <matt.mccormick at kitware.com>
>>> >> >> >> wrote:
>>> >> >> >>>
>>> >> >> >>> Thanks!  Added.
>>> >> >> >>>
>>> >> >> >>> On Mon, Nov 9, 2015 at 9:45 AM, Bradley Lowekamp
>>> >> >> >>> <blowekamp at mail.nih.gov>
>>> >> >> >>> wrote:
>>> >> >> >>> > We should consider this:
>>> >> >> >>> > http://review.source.kitware.com/#/c/20369/
>>> >> >> >>> >
>>> >> >> >>> > This helps with using VS2015 GUI.
>>> >> >> >>> >
>>> >> >> >>> >
>>> >> >> >>> > On Nov 9, 2015, at 9:28 AM, Matt McCormick
>>> >> >> >>> > <matt.mccormick at kitware.com>
>>> >> >> >>> > wrote:
>>> >> >> >>> >
>>> >> >> >>> >> Hi,
>>> >> >> >>> >>
>>> >> >> >>> >> It is time to make the 4.8.2 release. The following patches
>>> >> >> >>> >> are
>>> >> >> >>> >> new
>>> >> >> >>> >> on
>>> >> >> >>> >> the release branch since 4.8.1.  Is there anything missing?
>>> >> >> >>> >>
>>> >> >> >>> >> Thanks,
>>> >> >> >>> >> Matt
>>> >> >> >>> >>
>>> >> >> >>> >> Bradley Lowekamp (4):
>>> >> >> >>> >>      BUG: Avoid potential exception in
>>> >> >> >>> >> itk::Object::UnRegister
>>> >> >> >>> >>      BUG: Avoid null deference in PrintSelf with default
>>> >> >> >>> >> construction
>>> >> >> >>> >>      BUG: Use const pointer in Set method for constant
>>> >> >> >>> >> ReferenceImage
>>> >> >> >>> >> ivar
>>> >> >> >>> >>      BUG: Fix segfault when with empty IsolatedWatershed
>>> >> >> >>> >> Threshold
>>> >> >> >>> >> range
>>> >> >> >>> >>
>>> >> >> >>> >> Davis Vigneault (3):
>>> >> >> >>> >>      COMP: Warnings in itkVTKImageExport when compiling with
>>> >> >> >>> >> c++11
>>> >> >> >>> >>      DOC: Expanded Canny Edge Detection Example
>>> >> >> >>> >>      COMP: Warnings in FFTW Classes when compiling with
>>> >> >> >>> >> c++11
>>> >> >> >>> >>
>>> >> >> >>> >> Eugene Prilepin (1):
>>> >> >> >>> >>      BUG: Fix a template name formation in python wrappings
>>> >> >> >>> >>
>>> >> >> >>> >> Hans Johnson (5):
>>> >> >> >>> >>      STYLE: Minor code style issues for MGHIO
>>> >> >> >>> >>      COMP: Update to version for WikiExamples
>>> >> >> >>> >>      ENH: Update Sphinx for ITK 4.8.0
>>> >> >> >>> >>      ENH: Update MGHIO to be shared lib
>>> >> >> >>> >>      BUG: Fix MGHIO to fix write failures
>>> >> >> >>> >>
>>> >> >> >>> >> Hyun Jae Kang (4):
>>> >> >> >>> >>      BUG: Fix HDF5 CMake configuration with VS2015
>>> >> >> >>> >>      BUG: Remove std::cout calls from class
>>> >> >> >>> >>      BUG: itkVTKImageIO class does not handle blank lines in
>>> >> >> >>> >> VTK
>>> >> >> >>> >> file
>>> >> >> >>> >> header.
>>> >> >> >>> >>      BUG: Fixed older XCode linking error
>>> >> >> >>> >>
>>> >> >> >>> >> Matthew McCormick (8):
>>> >> >> >>> >>      COMP: Do not use C++11 alignas in itkAlignedTypedef.
>>> >> >> >>> >>      COMP: Remove unused argument in
>>> >> >> >>> >> ConditionVariableNoThreads.
>>> >> >> >>> >>      DOC: Correct spelling of Continuous.
>>> >> >> >>> >>      COMP: Fix KernelTransform SetFixedParameters argument
>>> >> >> >>> >> type.
>>> >> >> >>> >>      STYLE: Use prefix increment operator in ImageBase.
>>> >> >> >>> >>      COMP: Use ITK's pygccxml by correctly setting up the
>>> >> >> >>> >> sys.path
>>> >> >> >>> >>      COMP: Set CMP0054 to new.
>>> >> >> >>> >>
>>> >> >> >>> >> Michka Popoff (1):
>>> >> >> >>> >>      COMP: Disable incompatible modules when using Python 3
>>> >> >> >>> >> wrapping
>>> >> >> >>> >>
>>> >> >> >>> >> Mikhail Isakov (1):
>>> >> >> >>> >>      BUG: DiffusionTensor3DReconstruction segfault if no
>>> >> >> >>> >> baseline
>>> >> >> >>> >> image
>>> >> >> >>> >> _______________________________________________
>>> >> >> >>> >> 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://public.kitware.com/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.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://public.kitware.com/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.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://public.kitware.com/mailman/listinfo/insight-developers
>>> >> >> >>
>>> >> >> >> _______________________________________________
>>> >> >> >> Community mailing list
>>> >> >> >> Community at itk.org
>>> >> >> >> http://public.kitware.com/mailman/listinfo/community
>>> >> >> >>
>>> >> >> >
_______________________________________________
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://public.kitware.com/mailman/listinfo/insight-developers


More information about the Community mailing list