Skip to content

Conversation

@Demonmasterlqx
Copy link

@Demonmasterlqx Demonmasterlqx commented Apr 10, 2025

I use Ubuntu 22.04 and ros2 humble to run the tutorial. When I clone moveit_task_constructor and build, I face many problem because my moveit is humble version. Therefore I look up the tutorial and find this problem.

Description

change branch from ros2 to $ROS_DISTRO

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

I use Ubuntu 22.04 and ros2 humble to run the tutorial. When I clone moveit_task_constructor and build, I face many problem because my moveit is humble version. Therefore I look up the tutorial and find this problem.
Copy link
Contributor

@sea-bass sea-bass left a comment

Choose a reason for hiding this comment

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

ros2 and rolling ($ROSDISTRO) are not quite the same, so typing this in Rolling will fail.

Maybe leave it as is, since the main branch here tracks Rolling, but add a note saying if you're on another ROS distro to clone the appropriate branch of MTC?

@Demonmasterlqx
Copy link
Author

ros2 and rolling ($ROSDISTRO) are not quite the same, so typing this in Rolling will fail.

Maybe leave it as is, since the main branch here tracks Rolling, but add a note saying if you're on another ROS distro to clone the appropriate branch of MTC?

I think so. I will add a special claim for ros2 humble user.

@sea-bass sea-bass changed the title Update pick_and_place_with_moveit_task_constructor.rst Add note about older branches in pick_and_place_with_moveit_task_constructor.rst Apr 11, 2025
@sea-bass
Copy link
Contributor

The build errors seem unrelated.

Also changed the branch name in the humble version of the docs. #1035

@sea-bass sea-bass merged commit ef948f2 into moveit:humble Apr 11, 2025
7 of 10 checks passed
@github-project-automation github-project-automation bot moved this to ✅ Done in MoveIt Apr 11, 2025
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