Dev Call Dec 17, 2022

Issues & Pull Requests

Plane

Copter & Rover

Attendee count (max): 18

UTC2300 - mavproxy_misc.py: use MAV_CMD_REQUEST_MESSAGE for AUTOPILOT_VERSION by peterbarker · Pull Request #1115 · ArduPilot/MAVProxy · GitHub

  • MAVProxy
  • Use get-message command rather than version_request to get copy of message
  • Merged!

UTC2301 - AP_Baro: auto-set field elevation from origin by tridge · Pull Request #20867 · ArduPilot/ardupilot · GitHub

  • Set field elevation from origin
  • Merged!

UTC2304 - AP_NavEKF2/3: do not check the second range finder if RANGEFINDER_MAX_INSTANCES is 1 by tatsuy · Pull Request #20917 · ArduPilot/ardupilot · GitHub

  • Fix rangefinder second instance in EKF
  • Merged

UTC2305 - Payload Place by lthall · Pull Request #22258 · ArduPilot/ardupilot · GitHub

  • Payload place
  • Need Randy, who is not on the call

UTC2307 - AP_GPS: added support for Unicore GPS modules with AGRICA by tridge · Pull Request #22261 · ArduPilot/ardupilot · GitHub

  • Unicore GPS support
  • There’ll be a follow-up with rm3100
  • We have a corpus of logs from different GPS modules now
  • Interesting problem with zero-length-strings in NMEA parser
    • Might have problem in other NMEA parsers?
    • Convert to NaN for flag value?
  • Up to feature level with f9p but supports dual-antenna-for-yaw
    • Apparently DJI are also using these now?!
  • Seems to do RTK fixes quite well
  • Won’t get yaw without RTK fix
    • F9ps will give you moving-baseline without RTK
  • Would be nice to have a good communication path with unicore
  • AMSL model is different between f9p and unicore
    • Wsg84 ellipsoid is same
  • Shame we use amsl
  • Wgs84 is better defines
  • There are ublox settings for the model to use
  • Mean sea level mode is different
  • Merged!

UTC2324 - AP_AHRS: use active EKF type for origin and crtl limits by tridge · Pull Request #22373 · ArduPilot/ardupilot · GitHub

  • Use active EKF type for origin and control limits
  • Fix for poor configuration where arming checks are disabled
  • People are disabling arming checks when they really, really shouldn’t be
  • Merged

UTC2335 - Plane: move adjust_altitude_target functionality to per mode functions by IamPete1 · Pull Request #22380 · ArduPilot/ardupilot · GitHub

  • Adjust_altitude_target functionality to per-mode
  • Merged!

UTC2337 - hwdef: added MatekL431-BattMon support by tridge · Pull Request #22382 · ArduPilot/ardupilot · GitHub

  • MatekL431 battmon support
  • CAN-connected battery connector
  • Removing the capacity param?
    • It’s misleading for users at the moment
  • Not a smart-battery-maker, just a CAN-connected power power brick
  • Merged!

UTC2345 - GCS_MAVLink: provide and use iterator for mavlink backends (remove internal_error) by peterbarker · Pull Request #22411 · ArduPilot/ardupilot · GitHub

  • Iterator for mavlink backends
  • Custom iterator is new - do we really want a new pattern?
  • Randy doesn’t like the new pattern
  • Tridge will look at this further

UTC2409 - AP_Airspeed: Periph: remove usage of hidden params and remove tube order by IamPete1 · Pull Request #22416 · ArduPilot/ardupilot · GitHub

  • Remove usage of hidden parameters and remove tube order on AP_Periph
  • Merged!

UTC0010 - autotest: removed deprecated distutils by tridge · Pull Request #22425 · ArduPilot/ardupilot · GitHub

  • Remove old use of distutils.copytree
  • Was very annoying when using tab-completion for tests
  • Merged!

UTC0014 - Position Control Limit Handling by lthall · Pull Request #22428 · ArduPilot/ardupilot · GitHub

  • Position control limit handling
  • Fixes for overshooting landings
    • Caused by propellor stall
      • 50kg aircraft descending at 10m/s
        • So not a problem for most people
    • After this If you get limited the trajectory will continue to decelerate
  • Extended to horizontal control
    • “Prioritise stopping over elegantly flying into a tree”
  • Is the API correct here?
    • Quadplane is already calling the function now being used
  • Needs to be tested on ship-landing-abort
  • Some test suites would be nice
    • Leonard doesn’t have a lot of time to do this
  • Limits are a bit weird
    • Leonard thinks of the limits as a 3D unit vector which points in the direction of the limit
  • Currently if you hit the limit it locks in and doesn’t shift back towards zero
    • Limits would stop you increasing or correlating target velocity which could stop you braking
  • Tridge will start a little test suite

UTC0038 - Tuning for position controller angle max by lthall · Pull Request #22429 · ArduPilot/ardupilot · GitHub

  • Pass in position controller angle max parameter on a tuning knob
  • Useful for when doing stringing power lines
  • Should probably be an rc channel option instead

UTC0046 - Plane: stable 4.3.x issues list · Issue #15941 · ArduPilot/ardupilot · GitHub

  • Beta is going well
  • Final will be out in next day or two
  • Would be nice to get some feedback!

UTC0047 - Copter/Rover-4.3 issues list · Issue #21711 · ArduPilot/ardupilot · GitHub

  • No Randy for Copter update

UTC0047 - AC_WPNav: Add corner acceleration limit parameter by lthall · Pull Request #22431 · ArduPilot/ardupilot · GitHub

  • Add parameter for corner accel limit
  • Need Randy
  • Units will need to be checked on interfaces
  • People Want to go around corners faster
    • Something was put in place, but it was too much
    • So add a parameter
  • A few issues with preserving values of older parameters
  • Need to get rid if the drive-by set-and-save-ifchanged

UTC0048 - Copter: Fix Auto Takeoff when complete_alt_cm is current altitude by lthall · Pull Request #22445 · ArduPilot/ardupilot · GitHub

  • Correct auto takeoff altitude when already at altitude