search

open-rmf / rmf_ros2

Internal ROS infrastructure for RMF
Apache License 2.0
74 stars 60 forks source link

Fix serialization of exit events #364

Closed mxgrey closed 5 months ago

mxgrey commented 5 months ago

As reported in #363 we were not serializing exit events correctly when converting a nav graph into a ROS message. Entry events were accidentally being serialized in place of exit events. In situations where an exit event was present but not an entry event on the same lane, this would lead to a segmentation fault.

This PR fixes the mistake.

luca-della-vedova commented 5 months ago

Rolling CI will be red until we figure out https://github.com/open-rmf/rmf_ros2/pull/362 so this should be good to go