Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GCS_MAVLink: reject SET_MESSAGE_INTERVAL commands with p3 set #28019

Conversation

peterbarker
Copy link
Contributor

this is going to be used for changing the rate of a specific instance of a message at some stage

we have to reject it for now so that when the index is used the GCS is told that their message is invalid in this older version of the autopilot

@peterbarker peterbarker force-pushed the pr/set-message-interval-reject-param3-nonzero branch from f05b11a to 1991a5a Compare September 7, 2024 09:46
this is going to be used for changing the rate of a specific instance of a message at some stage

we have to reject it for now so that when the index is used the GCS is told that their message is invalid in this older version of the autopilot
@peterbarker peterbarker force-pushed the pr/set-message-interval-reject-param3-nonzero branch from 1991a5a to c68f020 Compare September 7, 2024 09:47
@peterbarker
Copy link
Contributor Author

STABILIZE> long SET_MESSAGE_INTERVAL 100 10 3
STABILIZE> Got COMMAND_ACK: SET_MESSAGE_INTERVAL: DENIED
long SET_MESSAGE_INTERVAL 100 10 0
STABILIZE> Got COMMAND_ACK: SET_MESSAGE_INTERVAL: ACCEPTED

@tridge tridge merged commit 78f0175 into ArduPilot:master Sep 10, 2024
94 checks passed
@peterbarker peterbarker deleted the pr/set-message-interval-reject-param3-nonzero branch September 10, 2024 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants