Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migration of sr_hand_kinematics from arm_navigation to moveit #4

Merged
merged 8 commits into from
May 27, 2014

Conversation

guihomework
Copy link
Contributor

Both service and plugin mode were migrated to moveit. Plugin needed a complete rework due to many changes in kinematics_base package
Both service and plug mode have now a rostest to validate the FK/IK for every finger
No migration to catkin yet, but patches ready for the code at least (kinematics_base in hydro changed again)

…n from pr2_moveit_plugins

Added a test but not working yet
Added new launch file for plugin test as rostest and added a local srdf description
Removed getToolFrame and getBaseFrame (now in parent class)
Fixed library dependencies (moveit_core),
Switched base plugin name to moveit_core name
Added checking and warning for missing parameters in plugin
Removed unnecessary param loading (passed as argument from parent)
…ers)

Added launch file for it
Moves test launch files to test directory
Fixed hand kinematics coupling to handle 1:1 coupling in seed generation
Fixed hand kinematics coupling to handle prefixed tip names
Updated comments in makefiles and manifest regarding plugin building
Fixed tf deprecated functions
Added plugin test for the other 4 fingers
Fixed plugin test using map instead of vector in joint_state_group->getVariableValues
because vector is for the whole group (FFJ4 always first even for other fingers)
Trivial warning fix
Cleaned indentation
@ugocupcic
Copy link
Contributor

Added #5 - merging

ugocupcic added a commit that referenced this pull request May 27, 2014
Migration of sr_hand_kinematics from arm_navigation to moveit
@ugocupcic ugocupcic merged commit 93848ca into shadow-robot:groovy-devel May 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants