Skip to content

[#296] adding recordreplay_command from joystick

Jit Ray Chowdhury requested to merge 296-recordreplay_command-from-joystick into master

Description

As a part of #296 (closed)

For manually triggering recording, replaying and canceling record/replay, currently in test code joystick is used. I would be convenient to use the joystick for real-world test also. Currently joystick_vehicle_interface only publishes control command and state command. Blinker value from state command was reused for recordreplay triggering in the following

Notes for Reviewer

In joystick_vehicle_interface add a separate recordreplay_cmd topic and msg in joystick_vehicle_interface, instead of using VehicleStateCommand

I have not created a new msg for record replay control, and instead used std_msgs/uint8 and intend to use it's values for control. This is to avoid joystick_vehicle_interface having any dependency on recordreplay_planner. The caveat is the values for /recordreplay_cmd are not predefined in a single place. But it is better than hijacking blinker value in VehicleStateCommand msgs.

Checklist

  • Documentation was updated

/milestone %After AVP

Edited by Jit Ray Chowdhury

Merge request reports