Skip to content

Update getting_started.rst #879

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

Merged
merged 5 commits into from
Aug 5, 2024
Merged

Update getting_started.rst #879

merged 5 commits into from
Aug 5, 2024

Conversation

timassman
Copy link
Contributor

  • Added instruction to checkout humble branch when using ROS humble
  • Added instruction to remove previously installed moveit binaries

Description

While following the tutorials myself, after waiting for the build, RViz crashed, unclear why. It turned out there is a specific "humble" branch. Also I had previously installed moveit binaries. There were conflicts. It took me quite some time, especially because the build takes so much time and the errors are unclear.

I propose to add these instructions to prevent other users from running into the same issues.

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

- Added instruction to checkout humble branch when using ROS humble
- Added instruction to remove previously installed moveit binaries
Copy link
Contributor

@sjahr sjahr left a comment

Choose a reason for hiding this comment

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

Thanks! Addition to you sentence to make it more ROS version agnostic. Main supports Humble, Iron and Rolling at the moment but the Humble support will probably be dropped in favor of jazzy soon

@sjahr sjahr enabled auto-merge (squash) August 5, 2024 17:13
@sjahr sjahr merged commit 5252035 into moveit:main Aug 5, 2024
7 of 8 checks passed
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