Skip to content

First attempt to test moveit_cpp#4

Merged
henningkayser merged 13 commits intoPickNikRobotics:feature-moveit_cppfrom
JafarAbdi:feature-moveit_cpp-test
Sep 25, 2019
Merged

First attempt to test moveit_cpp#4
henningkayser merged 13 commits intoPickNikRobotics:feature-moveit_cppfrom
JafarAbdi:feature-moveit_cpp-test

Conversation

@JafarAbdi
Copy link

Description

This's the first attempt to create unit tests for moveit_cpp class

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

Update launch and Config files

Update launch and config files

Use raw pointer for joint_model_group_ member variable

Use MoveItErrorCodes for plan functions

Add wait_for_initial_state_timeout parameter to planning scene monitor options

Update moveit_cpp_test.cpp

Add wait_for_initial_state_timeout parameter moveit_cpp.yaml
@JafarAbdi
Copy link
Author

@henningkayser

@henningkayser henningkayser merged commit 6cd84dc into PickNikRobotics:feature-moveit_cpp Sep 25, 2019
@welcome
Copy link

welcome bot commented Sep 25, 2019

Congrats on getting your first MoveIt pull request merged and improving open source robotics!

henningkayser pushed a commit that referenced this pull request Sep 25, 2019
mlautman pushed a commit that referenced this pull request Oct 15, 2019
mlautman pushed a commit that referenced this pull request Oct 15, 2019
mlautman pushed a commit that referenced this pull request Oct 15, 2019
henningkayser pushed a commit that referenced this pull request Nov 21, 2019
henningkayser pushed a commit that referenced this pull request Nov 21, 2019
tylerjw pushed a commit that referenced this pull request May 13, 2020
tylerjw pushed a commit that referenced this pull request May 20, 2020
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