qgroundcontrol
qgroundcontrol copied to clipboard
Do not REQUEST_DATA_STREAMs by default, since it's deprecated
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
- Install QGroundControl on clean machine where it wasn't installed before or wipe QGroundControl saved settings.
- Go to Application Settings, Telemetry, Stream Rates.
- See default setting of "Controlled By vehicle" checkbox slider (when it's ON it means REQUEST_DATA_STREAM is disabled).
Checklist:
- [x] Review Contribution Guidelines.
- [x] Review Code of Conduct.
- [x] I have tested my changes.
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.