search

czaloj / bullet

Automatically exported from code.google.com/p/bullet
0 stars 0 forks source link

Wrong quantization in btAxisSweep3 #87

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
btAxisSweep3Internal::quantize does not properly implement the clamping of the 
input point. The 
point is clamped at first and then scaled and quantized. Due to numerical 
imprecision, the 
quantized result may exceed the bounds (m_handleSentinel) for large objects 
like a 
btStaticPlaneShape, and the algorithm gets mixed up. I encountered the problem 
with bScalar=float 
and bt32BitAxisSweep3.

Clamping should be done after scaling, this is implemented by the patch.

Original issue reported on code.google.com by o_knieme...@maxon.de on 5 Sep 2008 at 8:07

Attachments:

GoogleCodeExporter commented 9 years ago 
Thanks for the patch, we try to check it out and apply it before 2.72 release

Original comment by erwin.coumans on 14 Sep 2008 at 6:26

GoogleCodeExporter commented 9 years ago

Original comment by erwin.coumans on 14 Sep 2008 at 6:26

GoogleCodeExporter commented 9 years ago

Original comment by erwin.coumans on 14 Sep 2008 at 6:28

GoogleCodeExporter commented 9 years ago 

It has been fixed in SVN revision 1345
http://code.google.com/p/bullet/source/detail?r=1345

Also, this was also reported on the Bullet physics forums here:
http://bulletphysics.com/Bullet/phpBB3/viewtopic.php?f=9&t=2559&p=10674#p10674

Original comment by erwin.coumans on 30 Sep 2008 at 9:00