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

Catkinize pr2_kinematics #2

Closed
1 task
ahendrix opened this issue Jul 24, 2013 · 5 comments
Closed
1 task

Catkinize pr2_kinematics #2

ahendrix opened this issue Jul 24, 2013 · 5 comments

Comments

@ahendrix
Copy link
Member

Catkinize and release pr2_kinematics. Depends on catkinizing and releasing:

  • arm_navigation_msgs, collision_space, planning_environment, planning_models, kinematics_base, kinematics_msgs (from arm_navigation)

arm_navigation is no longer maintained; this stack may need to be dropped

@isucan
Copy link

isucan commented Jul 24, 2013

Austin, pr2_kinematics is actually included in moveit, so when we drop arm_navigation, we will drop that too. We can still release stacks with the old build system in Hydro, which is currently planned to be the last release to include arm_navigation

@ahendrix
Copy link
Member Author

My goal is to catkinize all of the PR2 stacks before releasing them into Hydro. It isn't required, but it allows PR2 users to use either build system for new projects.

What will it take to move the existing users of pr2_kinematics (pr2_arm_move_ik) to the use MoveIt instead? Are there tutorials for this sort of thing?

@isucan
Copy link

isucan commented Jul 24, 2013

No tutorials yet, but the functionality exists with almost the same message format, so I expect it would be easy to get that done.

@UltronDestroyer
Copy link
Contributor

@ahendrix
I've decided to go ahead and move this forward with the pr2_arm_kinematics that is used in MoveIt. (See: Releasing this with the MoveIt backend should make users who build on this package happy. I've also deprecated the pr2_arm_constraints_aware package as I didn't see what it was used for and was almost entirely dependent on arm_navigation which has been deprecated.

Any concerns/questions?

@trainman419
Copy link

That sounds reasonable. There aren't any packages listed as depending on the pr2_arm_kinematics_constraint_aware package on the wiki, so it's probably safe to deprecate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants