20230913 Dev Meeting

General:

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

Notes

• #2037 Generate .deb file in CI - anything we need to know? Versioning/publishing?

  • No objections in principle.
  • Concern is that we don't want to implicitly define a versioning process that we will be forced to follow. Propose version file by git hash.
  • If we do end up doing proper releases there will need to be some mechanism to encapsulate the pymavlink toolchain version and message versions.

• #2024 common: Add camera instance field to MAV_CMD_IMAGE_START/STOP_CAPTURE (and APM #328 common: Add camera instance field to MAV_CMD_IMAGE_START/STOP_CAPTURE)

  • JamesP and Julian OK with this, and OK to merge now.
  • HamishW happy to wait a little longer.

• #2027 {Sponsored by Intelliterra} Added MAV_CMD_DO_SET_EMERGENCY for manual emergency setting

  • JamesP It's not mandatory: ArduPilot set emergency for automated stuff, but not directly via pilot - and already have approved integrations.
  • Julian not opposed.
  • General feeling is that IF you wanted to comply this approach is reasonable.
  • We'll follow normal rules - merge if there is actual implementation/interest from flight stacks.

• Move enums in description to real enums 1997

  • Yes, it is always a good idea, but some concerns that retrofitting it might break some cases.
  • Suggest that we update all cases with 3 or more options to enums, then revisit the remainer.
  • AO hamish.

• generate WIP warnings on the use of WIP messages in C code #240 - get it in as "default off WIP test". That is useful for Beat. And once in is easier to flip this if we can show it is flawed.

  • JamesP to follow up.

• #2029 common.xml: MISSION_CURRENT progress notification

  • This was modified as discussed then modified a bit more. Assume we are all OK with it - holding for a prototype?

Updates:

• #2015 Add VELOCITY_LIMITS msg - no progress - users on holiday
• Fix Wireshark dissector and add snapshot tests #817 - fix went in thank you.
• #2031 common: extend MANUAL_CONTROL with auxiliary continuous inputs - fix went in.

Attendees: ?Hamish, JulianO, JamesP, MarcinZ, MatthiasG