-
Notifications
You must be signed in to change notification settings - Fork 946
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
Set trajectory execution monitor params under correct namespace #72
Conversation
ae57332
to
b469135
Compare
+1 |
<!-- Allow more than the expected execution time before triggering a trajectory cancel (applied after scaling) --> | ||
<param name="trajectory_execution/allowed_goal_duration_margin" value="0.5"/> <!-- default 0.5 --> | ||
<param name="allowed_goal_duration_margin" value="0.5"/> <!-- default 0.5 --> | ||
|
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.
Why did you reverted this? IMHO it makes sense to structure the param space using the "trajectory_execution" ns.
Furthermore, the namespace is needed for dynamic reconfigure. This was the point of #68.
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.
Sorry, obviously I didn't look at the most recent code...
b469135
to
44267b7
Compare
@rhaschke Thanks for the review. Rebased, and fixed based on the comments. |
Please consider #72 (comment). Otherwise looks good to me. |
Thanks. Fixed. |
Added parameter allowed_goal_duration_margin to DynamicReconfigure fixes remaining issues in #68
Added parameter allowed_goal_duration_margin to DynamicReconfigure fixes remaining issues in #68
Thanks. |
For #61 and #68