-
Notifications
You must be signed in to change notification settings - Fork 0
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
RST-9755 Move service calls to power cycle lidars to the uam_node #37
Conversation
urg_node/src/uam/uam_driver_ros.cpp
Outdated
@@ -229,7 +294,8 @@ void UamROS::configureTimerCallback(const ros::TimerEvent& event) | |||
if (!configured_ && !configure()) | |||
{ | |||
// Configuration failed. Restart the timer to try again. | |||
configure_timer_.start(); | |||
ROS_ERROR_STREAM("Failed to configure the lidar. Retrying in " << params_.reconfiguration_timeout.toSec() |
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 remove the timer? Timer is stopped at the top, this will never run again if it fails to connect to the lidar
urg_node/src/uam/uam_driver_ros.cpp
Outdated
if (should_reset_lidar) | ||
{ | ||
ROS_WARN_STREAM("Trying to restart the lidar."); | ||
if (lidarHardReset()) | ||
{ | ||
ROS_WARN_STREAM("Lidar reset successful. Reconfiguring the lidar."); | ||
triggerReconfigure(); | ||
} | ||
} |
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.
if (should_reset_lidar) | |
{ | |
ROS_WARN_STREAM("Trying to restart the lidar."); | |
if (lidarHardReset()) | |
{ | |
ROS_WARN_STREAM("Lidar reset successful. Reconfiguring the lidar."); | |
triggerReconfigure(); | |
} | |
} | |
if (should_reset_lidar) | |
{ | |
ROS_WARN_STREAM("Trying to restart the lidar."); | |
if (lidarHardReset()) | |
{ | |
ROS_WARN_STREAM("Lidar reset successful."); | |
} | |
} | |
triggerReconfigure(); |
urg_node/src/uam/uam_driver_ros.cpp
Outdated
else{ | ||
triggerReconfigure(); | ||
} |
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.
else{ | |
triggerReconfigure(); | |
} |
75ebe73
to
3ea6db3
Compare
This PR moves the service calls to power cycle the lidars from the lidar_check node in locus_vector to the urg_node. This is being done in order to reduce the number of unnecessary lidar restarts being seen on Vector 1.2
laser_restart.webm