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

Move Group C++ Interface #62

Closed
wants to merge 9 commits into from
Closed

Move Group C++ Interface #62

wants to merge 9 commits into from

Conversation

MarqRazz
Copy link
Contributor

@MarqRazz MarqRazz commented Mar 10, 2021

This is the initial port of the Move Group C++ Interface tutorial and addresses #2 . This tutorial is very similar to moveit2_demo_nodes/run_move_group which is where I grabbed the launch files.

To test this code run the following:
ros2 launch moveit2_tutorials move_group.launch.py
ros2 launch moveit2_tutorials move_group_interface_tutorial.launch.py

Items that I still need to address:

  • Clean up code and get the markers working with rviz_visual_tools until moveit_visual_tools is ported
  • Generate the webpage and see how it looks
  • Add more documentation
  • Get the collision scene examples working better (they tend to fail to plan)
  • Add to the readme and webpage that you need xterm installed

Checklist

@MarqRazz MarqRazz changed the title Move Group C++ Interface [WIP] Move Group C++ Interface Mar 31, 2021
Comment on lines +14 to +16
install(DIRECTORY config
DESTINATION share/${PROJECT_NAME}
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should use the YAML files from moveit_resources

Suggested change
install(DIRECTORY config
DESTINATION share/${PROJECT_NAME}
)

parameters=[robot_description])

# Fake joint driver
fake_joint_driver_node = Node(package='fake_joint_driver',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We replaced this node with the one from ros2_control see

@MarqRazz
Copy link
Contributor Author

Changes were added in #69

@MarqRazz MarqRazz closed this Apr 21, 2021
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.

2 participants