Log task node restarts to /infoLog to make errors more visible

BCLab-UNM / SwarmathonIV-Cabrillo-Public

Swarmathon code repository for Cabrillo College

MIT License

6 stars 1 forks source link

Log task node restarts to /infoLog to make errors more visible #198

Closed darrenchurchill closed 5 years ago

darrenchurchill commented 5 years ago

To make errors more obvious to a user monitoring behavior through the GUI's /infoLog topic, we can publish a warning whenever task starts in an initialized state.

A user should know whether the restart message is concerning, depending on whether the rover was put in manual mode temporarily.

Logging a restart message is more reliable than trying to publish an error message on exit because rospy doesn't guarantee your messages will be published if your node exits immediately afterward.

Carter90 commented 5 years ago

Looks good I manually ran rosnode kill /achilles/task a few times to make sure the task stated back up with the current behavior