[Insight-users] Code integration and code design - Question

[Alexandre GOUAILLARD]

Dear steve,

That's a very good point.

Yes, we saw the warning. We have implemented a very simple test for the
robustness, which actually test both normal ITK, and the new code with 4
points lying on circle, and adding an arbitrary small number to the 4th
point coordinate, and we could not find a platform where this was a
problem. We tried linux with gcc 4.2, mac with both gcc and clang, and
windows (xp, 7) with MSVC 2005, 2008 and 2010.

Now, that does not mean there is no problem. If you had a good test to
exhibit this problem, we would be happy to implement it in the module, and
modify the design accordingly if needed. We just haven't found the need yet.

Again, this is a very good point, and you might want t omake a short, quick
review of the paper on Insight Journal to point this out.

thanks again,

alex.



On Tue, Nov 13, 2012 at 7:14 AM, Steve M. Robbins <steve at sumost.ca> wrote:

> On Mon, Nov 12, 2012 at 04:56:13PM +0800, Steffan wrote:
>
> > * To test the orientation of vectors, we are using a library in C
>     that we have wrap into some ITK function. This library is a single
>     file, done by Jonathan Shewchuk
>     (http://www.cs.cmu.edu/~quake/robust.html) under public domain.
>
> I presume you've read the Warning on that page about extended
> precision.
>
> In my past experience with this code, this has always been the
> trickiest part of using it -- especially in a portable manner.  I'm
> curious about your plan to handle this, especially in a
> general-purpose library like ITK.
>
> Regards,
> -Steve
>
>
> _____________________________________
> 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://www.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-users
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/pipermail/insight-users/attachments/20121114/9187b6a8/attachment.htm>