Skip to content
Snippets Groups Projects
Commit 0eef8bf1 authored by Nicolas Mansard's avatar Nicolas Mansard Committed by Nicolas Mansard
Browse files

[C++] Geom: remove resetDistance and reset the distanceResult in computeDistance.

parent d5050de8
No related branches found
No related tags found
No related merge requests found
...@@ -64,12 +64,12 @@ namespace se3 ...@@ -64,12 +64,12 @@ namespace se3
{ {
assert( pairId < geomModel.collisionPairs.size() ); assert( pairId < geomModel.collisionPairs.size() );
const CollisionPair & pair = geomModel.collisionPairs[pairId]; const CollisionPair & pair = geomModel.collisionPairs[pairId];
fcl::CollisionResult& collisionResult = geomData.collisionResults[pairId];
assert( pairId < geomData.distanceResults.size() ); assert( pairId < geomData.collisionResults.size() );
assert( pair.first < geomData.collisionObjects.size() ); assert( pair.first < geomData.collisionObjects.size() );
assert( pair.second < geomData.collisionObjects.size() ); assert( pair.second < geomData.collisionObjects.size() );
fcl::CollisionResult& collisionResult = geomData.collisionResults[pairId];
collisionResult.clear(); collisionResult.clear();
fcl::collide (&geomData.collisionObjects[pair.first], fcl::collide (&geomData.collisionObjects[pair.first],
&geomData.collisionObjects[pair.second], &geomData.collisionObjects[pair.second],
...@@ -127,7 +127,8 @@ namespace se3 ...@@ -127,7 +127,8 @@ namespace se3
assert( pairId < geomData.distanceResults.size() ); assert( pairId < geomData.distanceResults.size() );
assert( pair.first < geomData.collisionObjects.size() ); assert( pair.first < geomData.collisionObjects.size() );
assert( pair.second < geomData.collisionObjects.size() ); assert( pair.second < geomData.collisionObjects.size() );
geomData.distanceResults[pairId].clear();
fcl::distance ( &geomData.collisionObjects[pair.first], fcl::distance ( &geomData.collisionObjects[pair.first],
&geomData.collisionObjects[pair.second], &geomData.collisionObjects[pair.second],
geomData.distanceRequest, geomData.distanceRequest,
......
...@@ -258,11 +258,6 @@ namespace se3 ...@@ -258,11 +258,6 @@ namespace se3
/// \sa activateCollisionPair /// \sa activateCollisionPair
void deactivateCollisionPair(const PairIndex pairId); void deactivateCollisionPair(const PairIndex pairId);
/// Reset the vector distanceResults.
/// TODO: should this be called automatically before calling computeDistance?
/// TODO: should we implement the same for collisions?
void resetDistances();
#endif //WITH_HPP_FCL #endif //WITH_HPP_FCL
friend std::ostream & operator<<(std::ostream & os, const GeometryData & geomData); friend std::ostream & operator<<(std::ostream & os, const GeometryData & geomData);
......
...@@ -238,10 +238,6 @@ namespace se3 ...@@ -238,10 +238,6 @@ namespace se3
activeCollisionPairs[pairId] = false; activeCollisionPairs[pairId] = false;
} }
inline void GeometryData::resetDistances()
{
std::fill(distanceResults.begin(), distanceResults.end(), fcl::DistanceResult() );
}
#endif //WITH_HPP_FCL #endif //WITH_HPP_FCL
} // namespace se3 } // namespace se3
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment