search

paparazzi / pprzlink

Message and communication library for the Paparazzi UAV system
Other
24 stars 55 forks source link

Camera payload messages #48

Closed podhrmic closed 7 years ago

podhrmic commented 7 years ago

Added a CAMERA_PAYLOAD message for payloads with multiple cameras and a payload computer.

Extended CAMERA_SNAPSHOT message to include additional information about each camera.

Hoping this will be a generic enough message format for other camera+computer payloads.

The extra overhead in CAMERA_SNAPSHOT hopefully won't be an issue, as the only place where this message was used in paparazzi was for ArDrone with telemetry over wifi.

gautierhattenberger commented 7 years ago

Why the two temperature field are part of SNAPSHOT and not PAYLOAD message ? Do you really to have a temperature associated to a given snapshot ? Also closing parenthesis are missing in the description.

podhrmic commented 7 years ago

The lens temperature and the array temperature can change in between the snapshots. Also the payload can contain multiple cameras, and each camera can have a different temperature (and different snapshot message).

Which parenthesis do you mean? It passed the compile checks.

gautierhattenberger commented 7 years ago

The parenthesis in some of the fields comments...

podhrmic commented 7 years ago

closing for now as I need to do some more cleanup