diff options
author | Jorge Bernal <jbernalmartinez@gmail.com> | 2014-07-07 19:01:49 +0400 |
---|---|---|
committer | Mitchell Stokes <mogurijin@gmail.com> | 2014-07-07 19:06:39 +0400 |
commit | 1f43b083a97ee56b8b15692ef19d0e973b6d31ac (patch) | |
tree | 358346e602ab4be4ca8b1f650955d23b60c9e742 /source/gameengine/Physics/Bullet/CcdPhysicsController.cpp | |
parent | ef22e972b1ad0bed1a79587b72d50c7da4c6b4e8 (diff) |
BGE: Fix for applyImpulse function
This is related to task T29419. Credit also goes to Goran Milovanovic
(goran) for proposing an initial fix for this issue.
The issue is the current behavior of applyImpulse doesn't match the behavior
described in the documentation as instead of a impulse point in world coordinates,
it seems to require a coordinate in a local space.
Additionally, applyImpulse function isn't consistent with similar functions (applyForce, applyTorque, etc)
as it doesn't allow to choose in which space (local or global) the impulse is applied.
Now, we have the following function:
applyImpulse(point, impulse, local=False)
being "point" the point to apply the impulse to (in world or local coordinates). When local is False will
have both point and impulse in World space and when local is True will have point and impulse in local space.
Reviewers: moguri, dfelinto, brita_
Reviewed By: moguri
Differential Revision: https://developer.blender.org/D567
Diffstat (limited to 'source/gameengine/Physics/Bullet/CcdPhysicsController.cpp')
-rw-r--r-- | source/gameengine/Physics/Bullet/CcdPhysicsController.cpp | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/source/gameengine/Physics/Bullet/CcdPhysicsController.cpp b/source/gameengine/Physics/Bullet/CcdPhysicsController.cpp index c98cf212265..72c3b13e301 100644 --- a/source/gameengine/Physics/Bullet/CcdPhysicsController.cpp +++ b/source/gameengine/Physics/Bullet/CcdPhysicsController.cpp @@ -1309,8 +1309,9 @@ void CcdPhysicsController::SetLinearVelocity(const MT_Vector3& lin_vel,bool loc } } } -void CcdPhysicsController::ApplyImpulse(const MT_Point3& attach, const MT_Vector3& impulsein) +void CcdPhysicsController::ApplyImpulse(const MT_Point3& attach, const MT_Vector3& impulsein, bool local) { + btVector3 pos; btVector3 impulse(impulsein.x(), impulsein.y(), impulsein.z()); if (m_object && impulse.length2() > (SIMD_EPSILON*SIMD_EPSILON)) @@ -1323,7 +1324,19 @@ void CcdPhysicsController::ApplyImpulse(const MT_Point3& attach, const MT_Vecto return; } - btVector3 pos(attach.x(), attach.y(), attach.z()); + btTransform xform = m_object->getWorldTransform(); + + if (local) + { + pos = btVector3(attach.x(), attach.y(), attach.z()); + impulse = xform.getBasis() * impulse; + } + else { + /* If the point of impulse application is not equal to the object position + * then an angular momentum is generated in the object*/ + pos = btVector3(attach.x()-xform.getOrigin().x(), attach.y()-xform.getOrigin().y(), attach.z()-xform.getOrigin().z()); + } + btRigidBody* body = GetRigidBody(); if (body) body->applyImpulse(impulse,pos); |