rdiankov / openrave

Open Robotics Automation Virtual Environment: An environment for testing, developing, and deploying robotics motion planning algorithms.
http://www.openrave.org
Other
687 stars 340 forks source link

Fix for collision report to have collided geometry name correctly #1344

Closed eisoku9618 closed 6 months ago

eisoku9618 commented 6 months ago

Because of these 3 points, now in production branch, CollisionReport::__str__() does not show us a collided geometry name when geometry group (i.e. padded geometries) is used.

In order to address this issue, I would like to store a geometry name in FCLGeometryInfo and pass the name to collision report in CheckNarrowPhaseGeomCollision().

pipelineid=426516 pipelineid=426623

cc @MiraiHattori @Puttichai

rdiankov commented 6 months ago

https://github.com/rdiankov/openrave/pull/985 fixes this, so i cannot merge this for fear of many conflicts.