6dd65c0d67
With this change finally one can use compound collisions (like those created by Gridmaps) without serious performance issues. The previous KinematicBody code for Bullet was practically doing a whole bunch of unnecessary calculations. Gridmaps with fairly large octant sizes (in my case 32) can get up to 10000x speedup with this change (literally!). I expect the FPS demo to get a fair speedup as well. List of fixes and improvements: - Fixed a general bug in move_and_slide that affects both GodotPhysics and Bullet, where ray shapes would be ignored unless the stop_on_slope parameter is disabled. Not sure where that came from, but looking at the 2D physics code it was obvious there's a difference. - Enabled the dynamic AABB tree that Bullet uses to allow broadphase collision tests against individual shapes of compound shapes. This is crucial to get good performance with Gridmaps and in general improves the performance whenever a KinematicBody collides with compound collision shapes. - Added code to the broadphase collision detection code used by the Bullet module for KinematicBodies to also do broadphase on the sub-shapes of compound collision shapes. This is possible thanks to the dynamic AABB tree that was previously disabled and it's the change that provides the biggest performance boost. - Now broadphase test is only done once per KinematicBody in Bullet instead of once per each of its shapes which was completely unnecessary. - Fixed the way how the ray separation results are populated in Bullet which was completely broken previously, overwriting previous results and similar non-sense. - Fixed ray shapes for good now. Previously the margin set in the editor was not respected at all, and the KinematicBody code for ray separation was complete bogus, thus all previous attempts to fix it were mislead. - Fixed an obvious bug also in GodotPhysics where an out-of-bounds index was used in the ray result array. There are a whole set of other problems with the KinematicBody code of Bullet which cost performance and may cause unexpected behavior, but those are not addressed in this change (need to keep it "simple"). Not sure whether this fixes any outstanding Github issues but I wouldn't be surprised.
108 lines
4.1 KiB
C++
108 lines
4.1 KiB
C++
/*************************************************************************/
|
|
/* btRayShape.cpp */
|
|
/*************************************************************************/
|
|
/* This file is part of: */
|
|
/* GODOT ENGINE */
|
|
/* https://godotengine.org */
|
|
/*************************************************************************/
|
|
/* Copyright (c) 2007-2019 Juan Linietsky, Ariel Manzur. */
|
|
/* Copyright (c) 2014-2019 Godot Engine contributors (cf. AUTHORS.md) */
|
|
/* */
|
|
/* Permission is hereby granted, free of charge, to any person obtaining */
|
|
/* a copy of this software and associated documentation files (the */
|
|
/* "Software"), to deal in the Software without restriction, including */
|
|
/* without limitation the rights to use, copy, modify, merge, publish, */
|
|
/* distribute, sublicense, and/or sell copies of the Software, and to */
|
|
/* permit persons to whom the Software is furnished to do so, subject to */
|
|
/* the following conditions: */
|
|
/* */
|
|
/* The above copyright notice and this permission notice shall be */
|
|
/* included in all copies or substantial portions of the Software. */
|
|
/* */
|
|
/* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, */
|
|
/* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF */
|
|
/* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.*/
|
|
/* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY */
|
|
/* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, */
|
|
/* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE */
|
|
/* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */
|
|
/*************************************************************************/
|
|
|
|
#include "btRayShape.h"
|
|
|
|
#include "core/math/math_funcs.h"
|
|
|
|
#include <LinearMath/btAabbUtil2.h>
|
|
|
|
/**
|
|
@author AndreaCatania
|
|
*/
|
|
|
|
btRayShape::btRayShape(btScalar length) :
|
|
btConvexInternalShape(),
|
|
m_shapeAxis(0, 0, 1) {
|
|
m_shapeType = CUSTOM_CONVEX_SHAPE_TYPE;
|
|
setLength(length);
|
|
}
|
|
|
|
btRayShape::~btRayShape() {
|
|
}
|
|
|
|
void btRayShape::setLength(btScalar p_length) {
|
|
|
|
m_length = p_length;
|
|
reload_cache();
|
|
}
|
|
|
|
void btRayShape::setMargin(btScalar margin) {
|
|
btConvexInternalShape::setMargin(margin);
|
|
reload_cache();
|
|
}
|
|
|
|
void btRayShape::setSlipsOnSlope(bool p_slipsOnSlope) {
|
|
|
|
slipsOnSlope = p_slipsOnSlope;
|
|
}
|
|
|
|
btVector3 btRayShape::localGetSupportingVertex(const btVector3 &vec) const {
|
|
return localGetSupportingVertexWithoutMargin(vec) + (m_shapeAxis * m_collisionMargin);
|
|
}
|
|
|
|
btVector3 btRayShape::localGetSupportingVertexWithoutMargin(const btVector3 &vec) const {
|
|
if (vec.z() > 0)
|
|
return m_shapeAxis * m_cacheScaledLength;
|
|
else
|
|
return btVector3(0, 0, 0);
|
|
}
|
|
|
|
void btRayShape::batchedUnitVectorGetSupportingVertexWithoutMargin(const btVector3 *vectors, btVector3 *supportVerticesOut, int numVectors) const {
|
|
for (int i = 0; i < numVectors; ++i) {
|
|
supportVerticesOut[i] = localGetSupportingVertexWithoutMargin(vectors[i]);
|
|
}
|
|
}
|
|
|
|
void btRayShape::getAabb(const btTransform &t, btVector3 &aabbMin, btVector3 &aabbMax) const {
|
|
btVector3 localAabbMin(0, 0, 0);
|
|
btVector3 localAabbMax(m_shapeAxis * m_cacheScaledLength);
|
|
btTransformAabb(localAabbMin, localAabbMax, m_collisionMargin, t, aabbMin, aabbMax);
|
|
}
|
|
|
|
void btRayShape::calculateLocalInertia(btScalar mass, btVector3 &inertia) const {
|
|
inertia.setZero();
|
|
}
|
|
|
|
int btRayShape::getNumPreferredPenetrationDirections() const {
|
|
return 0;
|
|
}
|
|
|
|
void btRayShape::getPreferredPenetrationDirection(int index, btVector3 &penetrationVector) const {
|
|
penetrationVector.setZero();
|
|
}
|
|
|
|
void btRayShape::reload_cache() {
|
|
|
|
m_cacheScaledLength = m_length * m_localScaling[2];
|
|
|
|
m_cacheSupportPoint.setIdentity();
|
|
m_cacheSupportPoint.setOrigin(m_shapeAxis * m_cacheScaledLength);
|
|
}
|