Dev Call Nov 29, 2021

Issues & Pull Requests

Plane

Copter

Rover

UTC 2255 Build Server

  • Need visualisation for broken builds
  • Vrubrain51 broken
  • Killing off Peter’s tests to get release out
  • Need to look at getting a much faster build server
    • Maybe point people at the custom build server for weird boards?

Peter adjusted the timeout againUTC2302 - https://github.com/ArduPilot/ardupilot/pull/18658

  • 2d vs 3d clarifications in InertialNav
  • Merged!

UTC2308 - AP_Rangefinder: Use default address for TFMiniPlus I2C unless specified by shiv-tyagi · Pull Request #19031 · ArduPilot/ardupilot · GitHub

  • Use default address for tfminiplus
  • Don’t need to specify default address after this
  • Non-zero means it uses the address
  • merged!

UTC2309 - AP_Compass: move param to table per instance by IamPete1 · Pull Request #19123 · ArduPilot/ardupilot · GitHub

  • Table per instance
  • MissionPlanner does support this new format now
  • New method in mission planner to make conversion easy
    • New string/old string and it works?
    • If it finds both it breaks?
  • Discussed at last week’s call
  • Longer-term is going to be good
  • Pain in the short term
  • Much more consistent naming now
  • Not going to MAG1_?
    • People are looking under C already
    • Too much confusion
      • Can’t patch people’s brains….

UTC2315 - AP_Terrain: cast result of labs to unsigned by peterbarker · Pull Request #19387 · ArduPilot/ardupilot · GitHub

  • Fix build for navio
  • Merged!

UTC2316 - Cubes Define EKx_IMU_MASK and Change INS_USE check to apply for all Cubes by hendjoshsr71 · Pull Request #19392 · ArduPilot/ardupilot · GitHub

  • Cube and IMU_MASK
    • Default doesn’t use third IMU
  • Special-case for Cubes where we use third IMU in some cases
    • Safety bulletin for when ribbon cable comes loose
      • Very, very rare cases and only when thing has been abused
  • Doesn’t actually enable it in the EKF by default on Black
  • CPU and memory on Black is tight

UTC2320 - Tradheli: fix bug for auto missions using continue after land by bnsgeyer · Pull Request #19383 · ArduPilot/ardupilot · GitHub

  • Fixes tradheli bug for continue-after-land
  • Run-up-timer / critical speed
    • Timer counts down 5 seconds then declares ground-idle on land
    • Idle-up 5 seconds isn’t necessarily enough
      • ESC smarts can make it longer
  • This PR forces full run-up timer
  • Will wait for test on hardware before merging

UTC2326 - APM_Control: make 2nd reduction of P smaller by tridge · Pull Request #19359 · ArduPilot/ardupilot · GitHub

  • Reduce 2nd reduction of P
  • Oscillation might take time to disappear
  • Double-reduction of P gain - 0.35 * 0.35
  • Second reduction only drops it by
  • Damned heuristics
  • This will fix too-low a P gain

UTC2331 - SRV_Channel: correct casting of servo function number by peterbarker · Pull Request #19393 · ArduPilot/ardupilot · GitHub

  • Correct memory overwrite issue in case of servo channel misconfiguration
  • Need to find how to get AP_Enum to work with switch statements
  • Merged!

UTC2333 - AP_Math: memcpy nanfs rather than iteratively setting them by peterbarker · Pull Request #19394 · ArduPilot/ardupilot · GitHub

  • Memcpy nanfs rather than filling
  • Saves a lot of CPU
  • Merged!
  • SITL-on-HW
    • Run Sim on hardware
    • Physics runs on flight controller
      • No latency for sensor data
      • Great for testing lots of vehicles eg. swarms

UTC2336 - Plane: stable 4.1.0 issues list · Issue #15941 · ArduPilot/ardupilot · GitHub

  • Plane update
  • Fence issue
    • Hidden fence point
    • Was being used by polygon geofence
    • Not transferred to MAVProxy
    • Easy to reproduce in SITL
    • Only an issue using old fencepoint protocol, probably
  • Airspeed affinity not correct
    • Fixed in master
  • Old 4.0 log
    • One GPS becoming lagged over the other?!
    • GMS field is 2.5 seconds
    • EKF would continue to use the bad data
      • And not indicate it was unhappy
    • Uarts being drained unevenly?
    • Need an autotest
    • 2.5 s lag and 16km off course?
      • Wasn’t actually fusing GPS position!
      • Dead-reckoning essentially for 20 minutes
        • Reporting was misleading
    • Two GPSs shows the data is lagged
    • Autotest for 4.0.7 and then forward-port it
  • Internal error on camera feedback?
    • PixRacerPro
    • Interrupt storm?
    • No long with hardfault yet
  • Potential safety switch issue?
    • 4.0 issue too
    • No throttle suppression with safety on?

UTC2352 - Copter update

UTC0000 - Rover: 4.1.0 issues list · Issue #17116 · ArduPilot/ardupilot · GitHub

  • dshot is reversed?!
    • Andy’s looking at it
  • Maybe a month before we fork for next stable release

UTC0002 - autotest: disable flightgear output by peterbarker · Pull Request #19396 · ArduPilot/ardupilot · GitHub

  • Stop outputting flightgear packets in autotest
  • Merged!

UTC0004 - scripting question

  • Methods to get firmware version?

UTC0006 - partner’s meeting tomorrow

UTC0006 - close