-
Notifications
You must be signed in to change notification settings - Fork 493
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
Improve Cartesian interpolation #1547
Conversation
- Fixup doc comments - Add API providing the translation vector = direction * distance - Simplify implementation
This allows performing a circular motion about a non-link origin.
Codecov ReportBase: 51.10% // Head: 51.09% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1547 +/- ##
==========================================
- Coverage 51.10% 51.09% -0.00%
==========================================
Files 380 380
Lines 31802 31806 +4
==========================================
Hits 16249 16249
- Misses 15553 15557 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
moveit_core/robot_state/include/moveit/robot_state/cartesian_interpolator.h
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me, just some documentation suggestions.
moveit_core/robot_state/include/moveit/robot_state/cartesian_interpolator.h
Outdated
Show resolved
Hide resolved
moveit_core/robot_state/include/moveit/robot_state/cartesian_interpolator.h
Outdated
Show resolved
Hide resolved
moveit_core/robot_state/include/moveit/robot_state/cartesian_interpolator.h
Show resolved
Hide resolved
moveit_core/robot_state/include/moveit/robot_state/cartesian_interpolator.h
Show resolved
Hide resolved
Co-authored-by: AndyZe <andyz@utexas.edu>
Note that when I tried to pull this branch and do a clean build, I ran into a linker error caused by moveit_ros_robot_interaction. The MR at #1617 fixed the issue. |
* Generalize computeCartesianPath() to consider a link_offset which allows performing a circular motion about a non-link origin. * Augment reference to argument global_reference_frame Co-authored-by: AndyZe <andyz@utexas.edu> (cherry picked from commit 749c5ae)
* Generalize computeCartesianPath() to consider a link_offset which allows performing a circular motion about a non-link origin. * Augment reference to argument global_reference_frame Co-authored-by: AndyZe <andyz@utexas.edu> (cherry picked from commit 749c5ae) Co-authored-by: Robert Haschke <rhaschke@users.noreply.github.com>
Port from Noetic of moveit/moveit#3197.
Prepares addressing moveit/moveit_task_constructor#359 (see moveit/moveit_task_constructor#380 as well.)