-
Notifications
You must be signed in to change notification settings - Fork 938
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
jog_arm: add a C++ API #1763
jog_arm: add a C++ API #1763
Conversation
@AndyZe Thanks for preparing this! I'll give this a test but so far the implementation seems very reasonable to me. |
@AndyZe thanks for fixing this up, CI fails unrelated to this PR |
moveit_experimental/moveit_jog_arm/include/moveit_jog_arm/jog_interface_base.h
Outdated
Show resolved
Hide resolved
Looks great! lgtm |
Start a new folder, src/cpp_interface_example
Codecov Report
@@ Coverage Diff @@
## master #1763 +/- ##
==========================================
- Coverage 48.3% 48.26% -0.04%
==========================================
Files 294 297 +3
Lines 23232 23261 +29
==========================================
+ Hits 11223 11228 +5
- Misses 12009 12033 +24
Continue to review full report at Codecov.
|
moveit_experimental/moveit_jog_arm/include/moveit_jog_arm/jog_interface_base.h
Outdated
Show resolved
Hide resolved
* Example to launch C++ interface in a new thread * A better multithread example Start a new folder, src/cpp_interface_example * Update license formatting in new files only * JogROSInterface inherits from a base class * Base interface class is don * The new API half-works :P * Sharing data between threads workks * It works! The arm moves * Add C++ interface launch file * Implement joint commands * Debug the demo transition from cartesian to joint commands * Add an example of retrieving the current joint state * Clang format * Satisfy linter re. function names * Ensure the library is available to other packages * Update license formatting again * Clang format * Switch pthread to std::thread, change C++14 compiler options * Clang format
* Example to launch C++ interface in a new thread * A better multithread example Start a new folder, src/cpp_interface_example * Update license formatting in new files only * JogROSInterface inherits from a base class * Base interface class is don * The new API half-works :P * Sharing data between threads workks * It works! The arm moves * Add C++ interface launch file * Implement joint commands * Debug the demo transition from cartesian to joint commands * Add an example of retrieving the current joint state * Clang format * Satisfy linter re. function names * Ensure the library is available to other packages * Update license formatting again * Clang format * Switch pthread to std::thread, change C++14 compiler options * Clang format
* Example to launch C++ interface in a new thread * A better multithread example Start a new folder, src/cpp_interface_example * Update license formatting in new files only * JogROSInterface inherits from a base class * Base interface class is don * The new API half-works :P * Sharing data between threads workks * It works! The arm moves * Add C++ interface launch file * Implement joint commands * Debug the demo transition from cartesian to joint commands * Add an example of retrieving the current joint state * Clang format * Satisfy linter re. function names * Ensure the library is available to other packages * Update license formatting again * Clang format * Switch pthread to std::thread, change C++14 compiler options * Clang format
Adds a C++ API. There's an example in src/cpp_interface_example