little-red-rover / lrr-ros

ROS packages for working with Little Red Rover
https://www.little-red-rover.com
GNU Lesser General Public License v2.1
1 stars 0 forks source link

HAL.py requires SEGKILL when not connected to the rover #3

Closed usedhondacivic closed 2 months ago

usedhondacivic commented 2 months ago

Problem

The quit condition for the HAL node is checked after calling socket.recv(). socket.recv() blocks until it receives a packet, which is forever when not connected to the rover.

Fix

Set a timeout on socket.recv(), allowing the run loop to continue even when a packet isn't recieved.

usedhondacivic commented 2 months ago

Fixed by e47310c4d3fa01ea845185841d559a5f5bf5db13