Skip to content

Commit

Permalink
common.xml: remove validity requirement for COMMAND_INT_ONLY (mavlink…
Browse files Browse the repository at this point in the history
…#2059)

... and COMMAND_LONG_ONLY

It doesn't really matter why the command is being rejected, just that it is.

Adding constraints about why an autopilot might send the message reduces the usefulness of this.
  • Loading branch information
peterbarker authored and tridge committed Nov 29, 2023
1 parent b5b1429 commit 71af5c4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions message_definitions/v1.0/common.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2279,10 +2279,10 @@
<description>Command is valid and is being executed. This will be followed by further progress updates, i.e. the component may send further COMMAND_ACK messages with result MAV_RESULT_IN_PROGRESS (at a rate decided by the implementation), and must terminate by sending a COMMAND_ACK message with final result of the operation. The COMMAND_ACK.progress field can be used to indicate the progress of the operation. There is no need for the sender to retry the command, but if done during execution, the component will return MAV_RESULT_IN_PROGRESS with an updated progress.</description>
</entry>
<entry value="7" name="MAV_RESULT_COMMAND_LONG_ONLY">
<description>Command is valid, but it is only accepted when sent as a COMMAND_LONG (as it has float values for params 5 and 6).</description>
<description>Command is only accepted when sent as a COMMAND_LONG.</description>
</entry>
<entry value="8" name="MAV_RESULT_COMMAND_INT_ONLY">
<description>Command is valid, but it is only accepted when sent as a COMMAND_INT (as it encodes a location in params 5, 6 and 7, and hence requires a reference MAV_FRAME).</description>
<description>Command is only accepted when sent as a COMMAND_INT.</description>
</entry>
</enum>
<enum name="MAV_MISSION_RESULT">
Expand Down

0 comments on commit 71af5c4

Please sign in to comment.