Dear steve,<div><br></div><div>That's a very good point.</div><div><br></div><div>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.</div>
<div><br></div><div>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.</div>
<div><br></div><div>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.</div><div><br></div><div>thanks again,</div><div><br></div><div>alex.</div>
<div> </div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 13, 2012 at 7:14 AM, Steve M. Robbins <span dir="ltr"><<a href="mailto:steve@sumost.ca" target="_blank">steve@sumost.ca</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, Nov 12, 2012 at 04:56:13PM +0800, Steffan wrote:<br>
<br>
> * To test the orientation of vectors, we are using a library in C<br>
that we have wrap into some ITK function. This library is a single<br>
file, done by Jonathan Shewchuk<br>
(<a href="http://www.cs.cmu.edu/~quake/robust.html" target="_blank">http://www.cs.cmu.edu/~quake/robust.html</a>) under public domain.<br>
<br>
</div>I presume you've read the Warning on that page about extended<br>
precision.<br>
<br>
In my past experience with this code, this has always been the<br>
trickiest part of using it -- especially in a portable manner. I'm<br>
curious about your plan to handle this, especially in a<br>
general-purpose library like ITK.<br>
<br>
Regards,<br>
-Steve<br>
<br>
<br>_____________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at<br>
<a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Kitware offers ITK Training Courses, for more information visit:<br>
<a href="http://www.kitware.com/products/protraining.php" target="_blank">http://www.kitware.com/products/protraining.php</a><br>
<br>
Please keep messages on-topic and check the ITK FAQ at:<br>
<a href="http://www.itk.org/Wiki/ITK_FAQ" target="_blank">http://www.itk.org/Wiki/ITK_FAQ</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.itk.org/mailman/listinfo/insight-users" target="_blank">http://www.itk.org/mailman/listinfo/insight-users</a><br>
<br></blockquote></div><br></div>