Skip to content

Resolve "Clear CMD Queue Didn't reset the timer" & Add Cancel CMD Queue Exec Command

shibohan requested to merge 149-clear-cmd-queue-didn-t-reset-the-timer into master

Changes:

  • Reduce duplicated reset timer and "until mode" code
  • Add command to cancel command queue execution
  • During command queue execution, check whether the previous command's execution is successful.
    • if not, cancel the command queue execution & set the vehicle to a complete stop.

QA Test Tips Try command like:

start cnc arm then delay 30000 then cnc takeoff 5 until alt gt 4 then cnc land end
  • There is a 30 seconds delay between arm motor and takeoff, but the motor will disarm itself after 5 seconds. So the takeoff command will fail. Observe the command queue execution should cancel and flight mode set to BRAKE.
  • There is a 30 seconds delay between arm motor and takeoff, you can cancel the queue execution by typing cancel. Observe the command queue execution should cancel.

DRONEOA Merge Request Submission Checklist

NOTE: DO NOT EDIT TEXT BELOW BEFORE CREATING YOUR MERGE REQUEST!

  • I have ensured my commit message includes explanation and description of the fix/change

  • I have updated any documentation related to any changes in this request. (e.g. wiki page, Doxygen comments)

  • If the change crosses multiple repos, the link is provided and both requests need to be approved before merge.

  • I have built and tested this change locally.

  • I have ensured that no copyright resources are used without reference. (Depends on resource policy)

  • I have ensured that the merge request is correctly labeled

  • I will squash commits on merge

  • I will change labels for both Issue(s) and merge request(s) to MERGED after merge

Closes #149 (closed)

Edited by shibohan

Merge request reports