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

Port common_planning_interface_objects #159

Merged

Conversation

JafarAbdi
Copy link
Contributor

@JafarAbdi JafarAbdi commented Jan 22, 2020

Description

This's essential for moveit_ros_visualization port, but it's blocked by planning_scene_monitor

TODO:

  • Port package.xml

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • Extend the tutorials / documentation reference
  • Document API changes relevant to the user in the MIGRATION.md notes
  • Create tests, which fail without this PR reference
  • Include a screenshot if changing a GUI
  • While waiting for someone to review your request, please help review another open pull request to support the maintainers

@JafarAbdi JafarAbdi mentioned this pull request Jan 22, 2020
7 tasks
@JafarAbdi JafarAbdi force-pushed the pr-common_planning_interface_objects branch 2 times, most recently from f2a6178 to b13974d Compare January 28, 2020 12:11
@JafarAbdi JafarAbdi force-pushed the pr-common_planning_interface_objects branch from b13974d to d29d477 Compare January 28, 2020 15:08
@JafarAbdi JafarAbdi changed the title WIP: Port common_planning_interface_objects Port common_planning_interface_objects Jan 28, 2020
@JafarAbdi JafarAbdi force-pushed the pr-common_planning_interface_objects branch 2 times, most recently from b07cd55 to 14f8bed Compare January 30, 2020 21:03
@henningkayser henningkayser force-pushed the pr-common_planning_interface_objects branch from 14f8bed to 019854c Compare January 31, 2020 11:44
@henningkayser henningkayser merged commit 0d50f50 into moveit:master Jan 31, 2020
MikeWrock pushed a commit to MikeWrock/moveit2 that referenced this pull request Aug 15, 2022
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.

None yet

3 participants