20210804 Dev Meeting
Hamish Willee edited this page Aug 5, 2021
·
11 revisions
General:
- Update from relevant previous calls
- Tagged: issues, PR.
-
common: allow negative temperatures for ESC_INFO message #1663 - WIP message - are we OK with this?
- James: What is use case for negative ESC temperatures - low temperature can only improve performance?
- Claudio: It is inconsistent with other message temperature reporting and looks odd in GCS to clamp temp low. It is being used in certification to report temperatures.
- James - that is not a justification: the measurement is not calibrated, so can't be used for certification, even if reported.
- James - not opposed (not using it). Option is to use bitfield to indicate a value is negative.
- As WIP it can be broken in theory but this is a compatibility break for ROS. Contact ROS and original author to let them know.
- HamishW. Post meeting change of opinion from me here: https://github.com/mavlink/mavlink/pull/1663#pullrequestreview-722842830
- James: What is use case for negative ESC temperatures - low temperature can only improve performance?
-
add MAV_CMD_DO_ADSB_OUT_IDENT #1662 - thoughts?
- It's a message to trigger ADSB functionality from GCS. No particular change other than "Empty" should be "Reserved" for unused params.
- AO HamishW - merge: https://github.com/mavlink/mavlink/pull/1662#issuecomment-893079765
-
param FTP - also see https://github.com/ArduPilot/ardupilot/pull/13911
- JamesP: Heads up that ArduPilot is planning to PR a MAVFTP-based mechanism for updating parameters into MAVLink.
- HamishW: Bar for inclusion is same as anything else.
- JulianO: Awesome. Might solve a lot of problems, e.g. MISSION_CHECKSUM not particularly needed if you use this approach.
-
Discussion of stuff happening in MAVSDK re plugins that allow autopilot-side implementations and "shims"
- Servers is not a great name!
- Julian to create basic docs and cross link. Seb hopes to provide detail towards end of his project.
Follow ups
-
VTOL Safe Area - RomanB
- JamesP - Roman did not follow up with issue yet. Raised in ArduPilot - not much impact yet.
- Discuss once Roman raises issue. Hamish to ping.
-
TIMESYNC -
- Julian is going to add creating new message to his list.
- Fix up enum generation - reminder. James still considering best way forward for this, but understands its importance.
- Common calibration protocol(s)? https://github.com/mavlink/mavlink/issues/1660#issuecomment-888046086
- Generally "yes this would be good to have". Daniel started discussion at some point.
- Not sure what appetite there would be to actually make the changes though.
- AO hamish to gather more information on what different stacks can do and re-raise for further discussion.
- MAV_CMD_PAYLOAD_CONTROL_DEPLOY, MAV_CMD_PAYLOAD_PREPARE_DEPLOY, MAV_CMD_NAV_PAYLOAD_PLACE confirm whether https://github.com/mavlink/mavlink/pull/1648 can go in.
Attendees: JulianOes, JamesP, Seb, HamishW, ClaudioM