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

Use service call for collision-object-publisher #324

Merged
merged 4 commits into from
Oct 26, 2017

Conversation

knorth55
Copy link
Member

Improve version of #317 (thanks to @eisoku9618)

Use ApplyPlanningScene service to operate collision_objects and attached_collision_objects.
Currently, publish and sleep method is implemented, but the old-style method is not recommended. (moveit/moveit_msgs#40 (comment))
We use service-call method instead.

@eisoku9618
Copy link

Thank you! You can squash my commit and your first commit if you like.

(unix::usleep (* 500 1000)))
(send req :scene :robot_state :attached_collision_objects
(mapcar #'(lambda (msg)
(progn
Copy link
Member

Choose a reason for hiding this comment

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

progn is not required inside lamda

Copy link
Member Author

Choose a reason for hiding this comment

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

thank you so much! I updated.

Copy link
Member

@YoheiKakiuchi YoheiKakiuchi Oct 25, 2017

Choose a reason for hiding this comment

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

👍

@knorth55
Copy link
Member Author

I checked with baxter and it works with no problem.

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.

4 participants