qgroundcontrol icon indicating copy to clipboard operation
qgroundcontrol copied to clipboard

Do not REQUEST_DATA_STREAMs by default, since it's deprecated

Open TishSerg opened this issue 9 months ago • 5 comments

Proof: https://mavlink.io/en/messages/common.html#REQUEST_DATA_STREAM

Description

MavLink command REQUEST_DATA_STREAM is DEPRECATED: Replaced by MAV_CMD_SET_MESSAGE_INTERVAL. So, QGroundControl shouldn't use it by default.

Test Steps

  1. Install QGroundControl on clean machine where it wasn't installed before or wipe QGroundControl saved settings.
  2. Go to Application Settings, Telemetry, Stream Rates.
  3. See default setting of "Controlled By vehicle" checkbox slider (when it's ON it means REQUEST_DATA_STREAM is disabled).

Checklist:

Related Issue

#8304

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

TishSerg avatar May 14 '24 10:05 TishSerg