ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
11.02k stars 17.58k forks source link

AP_Camera, AP_Mount: Camera Protocol messages don't have component IDs #24955

Open nexton-winjeel opened 1 year ago

nexton-winjeel commented 1 year ago

The AP_Camera and AP_Mount libraries have support for some of the messages in the MAVLink Camera Protocol.

However, the handlers for some of these messages (e.g. CAMERA_SETTINGS and CAMERA_INFORMATION) are problematic if there are multiple camera backends:

_Originally posted by @nexton-winjeel in https://github.com/ArduPilot/ardupilot/pull/24953#discussion_r1322303167_

rmackay9 commented 1 year ago

I haven't looked at the details of this particular issue but we hit a similar issue for MAV_CMD_IMAGE_START_CAPTURE and MAV_CMD_IMAGE_STOP_CAPTURE which led to us adding an extension (see PR https://github.com/ArduPilot/mavlink/pull/328)