-
Notifications
You must be signed in to change notification settings - Fork 8
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
Receive multiple laserscan #52
Conversation
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.
One small change needed: print warning when using laser offset with multiple topics.
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.
remove the offset and verify if the frame_id of messages from multiple topics is set to the laser_frame
defined in a configuration file
@dom-lee Removing the offset entirely will break functionality on some robots like the cars and the CoBot that have single LIDAR. You could convert to a list of offsets, or read from the TF tree - but one or the other is needed. |
src/navigation/navigation_main.cc
Outdated
@@ -188,6 +190,31 @@ void OdometryCallback(const nav_msgs::Odometry& msg) { | |||
navigation_.UpdateOdometry(odom_); | |||
} | |||
|
|||
void RetreveTransform(const std_msgs::Header& msg, |
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.
RetrieveTransform
: Spelling
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.
Modified
fixed transformation order |
Graph Navigation can receive multiple laserscan data.
change
laser_topics
inconfig/navigation.lua
file