-
Notifications
You must be signed in to change notification settings - Fork 327
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
[JTC] Fill action error_strings #887
[JTC] Fill action error_strings #887
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #887 +/- ##
==========================================
+ Coverage 47.71% 47.72% +0.01%
==========================================
Files 41 41
Lines 3871 3876 +5
Branches 1833 1832 -1
==========================================
+ Hits 1847 1850 +3
Misses 751 751
- Partials 1273 1275 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
joint_trajectory_controller/src/joint_trajectory_controller.cpp
Outdated
Show resolved
Hide resolved
joint_trajectory_controller/src/joint_trajectory_controller.cpp
Outdated
Show resolved
Hide resolved
cbf28cf
to
446f8ed
Compare
446f8ed
to
c121b99
Compare
joint_trajectory_controller/src/joint_trajectory_controller.cpp
Outdated
Show resolved
Hide resolved
joint_trajectory_controller/src/joint_trajectory_controller.cpp
Outdated
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.
Just minor comments
8ab1d8a
to
dc1c494
Compare
Co-authored-by: Sai Kishor Kothakota <[email protected]>
Co-authored-by: Bence Magyar <[email protected]> Co-authored-by: Sai Kishor Kothakota <[email protected]>
dc1c494
to
8630382
Compare
(cherry picked from commit 67dbf7b)
(cherry picked from commit 67dbf7b)
This avoids warning when executing formatting action: "Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/setup-python@v2, pre-commit/[email protected]" For example: https://github.com/ros-controls/ros2_control/actions/runs/3855204458
I found out that the
control_msgs::action::FollowJointTrajectory
has an error_string field, but it was not populated in some cases.