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

Add dependency on moveit_resources_panda_moveit_config. #61

Merged
merged 1 commit into from
Mar 11, 2021

Conversation

DLu
Copy link
Contributor

@DLu DLu commented Mar 10, 2021

Description

Fixes #59.

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

package.xml Show resolved Hide resolved
@rhaschke
Copy link
Contributor

@ros-planning/moveit-maintainers: I suggest reverting these changes.
The tutorials should not depend on moveit_resources_panda_moveit_config (which is a minimal version of panda_moveit_config referring to a minimal version of the URDF model as well), but use the official panda_moveit_config package instead, which uses the official URDF from franka_description.

@130s
Copy link
Contributor

130s commented Jun 14, 2023

@ros-planning/moveit-maintainers: I suggest reverting these changes. The tutorials should not depend on moveit_resources_panda_moveit_config (which is a minimal version of panda_moveit_config referring to a minimal version of the URDF model as well), but use the official panda_moveit_config package instead, which uses the official URDF from franka_description.

Ticketed #704 for clarification.

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

Successfully merging this pull request may close these issues.

Migrate From panda_moveit_config to moveit_resources_panda_moveit_config
5 participants