[Insight-developers] removing GetOptimizer method

Bill Lorensen bill.lorensen at gmail.com
Fri Sep 2 16:00:38 EDT 2011


If none of our customers use this then we can probably remove it. Of course,
we don't know if any of our customers use this.

What is the impact if we keep it?

Bill

On Fri, Sep 2, 2011 at 2:14 PM, Ziv Yaniv <zyaniv at childrensnational.org>wrote:

> Hello all,
>
> I would like to remove the GetOptimizer method from the following
> optimizers: itkAmoebaOptimizer, itkConjugateGradientOptimizer,
> itkLBFGSOptimizer, itkLevenbergMarquardtOptimizer . All of these wrap vnl
> optimizers.
>
> The GetOptimizer method is problematic as it exposes the underlying vnl
> class . Such a method should never have made it into ITK as it is a back
> door, removing any sanity checks ITK may have put in place when wrapping the
> vnl optimizer.
>
> Please let me know if there is a reason not to do this. If no one has a
> reason then I will do this next week.
>
>                    thanks
>                        Ziv
>
> --
> Ziv Yaniv, PhD.,
> Principal Investigator,
> The Sheikh Zayed Institute for Pediatric Surgical Innovation
> Children's National Medical Center
> 6th Floor Main Hospital, Room M7740
> 111 Michigan Ave., N.W
> Washington, D.C 20010
>
> Phone: 202-476-1288
> email: zyaniv at childrensnational.org
> web: http://isiswiki.georgetown.**edu/zivy/<http://isiswiki.georgetown.edu/zivy/>
>
> ______________________________**_________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/**opensource/opensource.html<http://www.kitware.com/opensource/opensource.html>
>
> Kitware offers ITK Training Courses, for more information visit:
> http://kitware.com/products/**protraining.html<http://kitware.com/products/protraining.html>
>
> Please keep messages on-topic and check the ITK FAQ at:
> http://www.itk.org/Wiki/ITK_**FAQ <http://www.itk.org/Wiki/ITK_FAQ>
>
> Follow this link to subscribe/unsubscribe:
> http://www.itk.org/mailman/**listinfo/insight-developers<http://www.itk.org/mailman/listinfo/insight-developers>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110902/f5186cae/attachment.htm>


More information about the Insight-developers mailing list