From: Gamze T. <gdt...@ya...> - 2007-01-29 17:23:51
|
Hi, This is a heads up for the new files "vgl_intersection.h" and "vgl_intersection.txx" under vgl. Like vgl_distance and vgl_closest_point, we thought that gathering intersection methods into a file will help to find the available methods while doing intersection easily. The intersection methods in various vgl classes are still there but there is a deprecated warning on them. Please start to use intersection methods from vgl_intersection.h. I am planning to delete the deprecated version after a while, when vxl users are aware of the new addition. There are also intersection methods in vgl/algo/vgl_homg_operators. They may be moved into a vgl/algo/vgl_homg_intersection as a future to do. I noticed that the intersection of a vector of planes does not have an assertion on the number of planes. But it seems like it should be at least >=3 since the result is a vgl_homg_point_3d. Is it a bug or is there an explanation that I could not deduce. Gamze Tunali LEMS, Brown University ____________________________________________________________________________________ Do you Yahoo!? Everyone is raving about the all-new Yahoo! Mail beta. http://new.mail.yahoo.com |
From: Osama T. M. <ma...@cs...> - 2007-02-24 00:21:22
|
These conic coefficients have type() of real_ellipse (as they should) but when ellipse_geometry() is called, major_axis_length and minor_axis_length are calculated incorrectly (sqrt of a negative value). a_ -0.44939157 b_ 0.00000000 c_ -0.47801599 d_ 0.86451149 e_ 3.8241279 f_ -6.6496615 -om |
From: Peter V. <pet...@ya...> - 2007-02-26 16:18:07
|
> major_axis_length and minor_axis_length > are calculated incorrectly (sqrt of a negative value). You're right. It seems like this bug has been in vgl_conic since always, without showing up. I'll make the appropriate changes in the CVS repository (to find its way in a next vxl release) and add some tests to core/vgl/tests/test_conic.cxx; meanwhile, just edit core/vgl/vgl_conic.txx at around line 210 and replace if (det < 0) det = -det; by if (A<0) A = -A, B = -B, C = -C, D = -D, E = -E, F = -F; but place that line just after the declaration of A, B, C, D, E, F; i.e., just before the calculation of det. -- Peter. _________________________________________________________ Flyger tiden iväg? Fånga dagen med Yahoo! Mails inbyggda kalender. Dessutom 250 MB gratis, virusscanning och antispam. Få den på: http://se.mail.yahoo.com |