-
Notifications
You must be signed in to change notification settings - Fork 88
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
trajectory_tracker: support PathWithVelocity #244
Conversation
…coverage report build
This reverts commit 9470be6.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
[#720] PASSED on indigoAll tests passed
[#720] PASSED on kineticAll tests passed
[#720] PASSED on lunarAll tests passed
|
@ykoga-kyutech @jojo43 PTAL |
[#733] PASSED on indigoAll tests passed
[#733] PASSED on kineticAll tests passed
[#733] PASSED on lunarAll tests passed
|
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.
LGTM 👍
@@ -122,7 +125,8 @@ class TrackerNode | |||
trajectory_tracker::Path2D path_; | |||
std_msgs::Header path_header_; | |||
|
|||
void cbPath(const nav_msgs::Path::ConstPtr&); | |||
template <typename MSG_TYPE> | |||
void cbPath(const typename MSG_TYPE::ConstPtr&); |
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.
awesome! 👏
neonavigation_launch/package.xml
Outdated
@@ -16,4 +16,5 @@ | |||
<exec_depend>safety_limiter</exec_depend> | |||
<exec_depend>tf2_ros</exec_depend> | |||
<exec_depend>trajectory_tracker</exec_depend> | |||
<!--exec_depend>trajectory_tracker_rviz_plugins</exec_depend--> |
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.
uncomment before merge
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.
trajectory_tracker_rviz_plugins must be bloom released before merge
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.
Add TODO comment and merge for now.
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.
@at-wat commented. LGTM
@@ -1,8 +1,11 @@ | |||
<?xml version="1.0"?> | |||
<launch> | |||
<arg name="use_path_with_velocity" default="true" /> |
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.
For visualizing the path_velocity
in Rviz by default, updating readme is better to clone https://github.com/at-wat/neonavigation_rviz_plugins/ .
[#734] PASSED on indigoAll tests passed
[#734] PASSED on kineticAll tests passed
[#734] PASSED on lunarAll tests passed
|
[#735] PASSED on indigoAll tests passed
[#735] FAILED on indigodocker build failed [#735] FAILED on kineticdocker build failed [#735] FAILED on lunardocker build failed |
[#736] PASSED on indigoAll tests passed
[#736] PASSED on lunarAll tests passed
[#736] PASSED on kineticAll tests passed
|
at-wat/neonavigation_msgs#9 has been in the testing repository. Merging this. |
[#737] FAILED on indigoTest failed
[#737] PASSED on kineticAll tests passed
[#737] PASSED on lunarAll tests passed
[#737] PASSED on indigoAll tests passed
|
after #239
fix #229
fix #245
require at-wat/neonavigation_msgs#9
Receive control commands from single topic
/path_velocity
, equivalent to the combination of/path
and/speed
, to make it easy to switch planner algorithm.