shadow-robot/sr-ros-interface

Bug in HandCommander.set_max_force

Closed this issue · 2 comments

toliver commented

The expected and documented behaviour

hand_cmd.set_max_force('rh_FFJ3', 200)

does not work. Right now what works is:

hand_cmd.set_max_force('FFJ3', 200)

So we should either modify the documentation and add a comment in the function signature...

Or modify the function so that strips the prefix from the joint_name argument. This is probably the cleanest solution, as it keeps the consistency with the rest of Commander methods.

ugocupcic commented

Yes, let's strip the prefix.