20210512 Dev Meeting

Agenda

General:

• Update from relevant previous calls
• Tagged: issues, PR.

Notes

• Add simple FSM support to ORBIT_EXECUTION_STATUS message in common.xml #1622

  • Needs clearer info on the "use defaults" cases. Some issues with people using NaN in COMMAND_INT - breaks things.
  • The progress info seems necessary but looks like this is something generally missing, and we should fix this. Example - Takeoff - silly that people have to monitor altitude to know that the command finished.
  • Lorenz: intelligent systems may be better suited to using more "primitive" message. E.g. separate out "getting to the location" from "orbiting". Discussion with James about this and potential for synergy with the circle modes.
  • DONE AO Hamish: Remove the ORBIT_EXECUTION_STATUS changes and update MAV_COMMAND_DO_ORBIT as discussed. Merge.
  • DONE AO Hamish: Add note in COMMAND_INT (to match MISSION_ITEM_INT) on the NaN/INT32_MAX options for defaults.
  • DONE AO Hamish: Add new command progress notification message in development.xml
  • DONE AO Hamish: Remind James about Circle mode/orbit discussion so he can review offline.

• common: make PARACHUTE_ACTION enum consistent #1633 - agree reject. Can break some things like scripts silently/in-the-field. Not worth the risk.

• Common flight modes can we progress

  • Main aim is to get standard here to help work with MAVSDK (also there are ongoing questions like share my modes)
  • Some modes can already be set "in common"- e.g. MAV_CMD_DO_SET_MODE with MAV_MODE values are fairly standardised. However there are many missing useful modes, and no way to query what is available.
  • General interest in this but not right now - working on other areas of collaboration with greater urgency.

• Enum bugs

  • Generator is not properly hierarchical with respect to enums. Confirm this is a bug please - CONFIRMED.
  • AO HamishW: Have a look at whether he can fix it in a timely manner.

• Get development XML stuff into flight stacks

  • MISSION_CHECKSUM - ArduPilot implementation on the way, Julian asked to look at this.

Queries we didn't cover last time.

• MAVLink signature check in C library does not check signing_streams's link_id when creating new stream #1619 - can I get a technical review please?

Attendees: HamishW, Lorenz, JulianO, JamesP, Mattias, MaxwellInnovations - 2 members