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

Update pick_and_place_with_moveit_task_constructor.rst #532

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

lost-robot
Copy link

@lost-robot lost-robot commented Oct 14, 2022

The property 'hand' was not set in the example code. The demo runs as expected once the proposed changes are made to the code.

The property 'hand' was not set in the example code.  The demo runs as expected once the proposed changes are made to the code.
@MikeWrock
Copy link
Contributor

Could you please share the error you encountered? I would like to check if it is happening in my docker container for this tutorial as well

@lost-robot
Copy link
Author

lost-robot commented Oct 20, 2022 via email

@sea-bass
Copy link
Contributor

This was fixed on main with this PR: #420

@MikeWrock you can check whether your container has that change in.

@MikeWrock
Copy link
Contributor

@sea-bass Thanks for pointing out the PR for me, if the example cpp file was fixed then it made its way in to my container. Since this is just updating the tutorial to match the code, I'm going to merge the PR.

@MikeWrock MikeWrock merged commit 2bf3e23 into moveit:humble Oct 20, 2022
@MikeWrock
Copy link
Contributor

Check out the description in #485 by @JohnTGZ if anyone is looking for added detail.

@tylerjw tylerjw mentioned this pull request Feb 21, 2023
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.

3 participants