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
Add time_from_start information to feedback topic #37
Conversation
@@ -163,6 +163,8 @@ class FollowJointTrajectoryController : public Controller | |||
double goal_time_tolerance_; | |||
|
|||
control_msgs::FollowJointTrajectoryFeedback feedback_; | |||
control_msgs::FollowJointTrajectoryGoal goal_; |
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.
to avoid confusion (since you're only using the timestamp portion of this), I would suggesting using a ros::Time instance rather than a whole goal instance.
9132af9
to
406f69d
Compare
@mikeferguson |
406f69d
to
c2b68e2
Compare
c2b68e2
to
0d6703d
Compare
@mikeferguson |
I don’t have merge rights, since I’m no longer with Fetch Robotics. |
I did not know that. |
@708yamaguchi can we get an equivalent PR into the melodic branch as well? |
When I used fetch with
follow joint trajectory
, I found/*_controller/follow_joint_trajectory/feedback
topic do not havetime_from_start
information, so I addedtime_from_start
to this topic.CC @pazeshun, @k-okada