personalrobotics / or_urdf

OpenRAVE plugin for loading URDF and SRDF files.
BSD 3-Clause "New" or "Revised" License
19 stars 32 forks source link

Manipulator name wrong #28

Closed beatrizleon closed 7 years ago

beatrizleon commented 7 years ago

The name of the manipulator is assigned the name of the parent group, in here: https://github.com/personalrobotics/or_urdf/blob/master/src/urdf_loader.cpp#L600

Instead of the end_effector name. In our robots, we have several end_effectors with same parent name which results in several manipulators with same name in openrave which is very confusing.

it should be replaced by: manip_info->_name = end_effector.name_;