Dev Call Oct 31, 2022

Issues & Pull Requests

Remote ID

Plane

Copter

Attendee count (max): 21

UTC2300 - Scripting: Support sending and recieving MAVLink messages by WickedShell · Pull Request #13660 · ArduPilot/ardupilot · GitHub

  • People would really love to get this!
  • This PR is massively conflicted, however
  • Need a procedure for generating the pre-compiled lua snippets required
  • Can’t load all messages as we don’t have the RAM
    • Mavgen lua binding generator works, but you have to cut stuff out
    • Need an on-line thingy
    • One file for each one and include them in your script separately?
      • Worse than having the bindings in the script
      • Also we don’t have “load”
  • There’s a lot of work in here
  • Who is going to do the work?
    • Rebase, testing, documentation?
    • Pregen all LUA bindings and put them up on firmware.ardupilot.org
    • Extensions?
      • Needs testing!
    • Any volunteers?!
      • Nope

UTC2315 - change namespace of MultiCopter and FixedWing parameter structures by peterbarker · Pull Request #21752 · ArduPilot/ardupilot · GitHub

  • Move fixed wing multicopter structures into separate files
  • If binary sizes are close then this can be merged

UTC2319 - Copter: Autoyaw updates by IamPete1 · Pull Request #21651 · ArduPilot/ardupilot · GitHub

  • Back again
  • Autoyaw updates
  • For Randy to look at

UTC2321 - Backport compass calibration fixes by peterbarker · Pull Request #21894 · ArduPilot/ardupilot · GitHub

  • Backport compass fixes
  • Leave it in the triage queue

UTC2324 - AP_Logger: scripting log format allocation fix by IamPete1 · Pull Request #21954 · ArduPilot/ardupilot · GitHub

  • Log format allocation fix
  • Needs to check the malloc return values
  • Allocating memory off main heap

UTC2330 - AP_Scripting: generation flash savings by IamPete1 · Pull Request #21980 · ArduPilot/ardupilot · GitHub

  • Flash generation savings
  • Tridge will test
  • WITH_SEMAPHORE is good
  • Needs to be re-written in Python

UTC2339 - Add support for ICM45686 IMU by bugobliterator · Pull Request #22028 · ArduPilot/ardupilot · GitHub

  • Add support for ICM45686
  • Some compass ordering issues thought through and dismissed
  • Andy’s looking at the filtering
  • merged!

UTC2339 - AP_InertialSensor: add fast reset for ICM20602 instead of full reset on bad temp sample by bugobliterator · Pull Request #22035 · ArduPilot/ardupilot · GitHub

  • Fast reset for 20602
    • Bugs in the sensor
    • We’ve known about this for a very long time
  • Reset every 2 seconds in flight is a little disturbing, however
  • Temperature data can be used to work out if the FIFO is stuffed
  • Full fifo reset has device recovering
  • This PR does a lighter-weight reset
    • Just hit one bit
    • Also adds logging so we can tell if this is happening
    • 1Hz named-value-float to help with diagnostics
  • Factory calibration registers
    • Can change mid-flight…
    • Workaround is to set them back to the original value
    • Reading these also seemed to fix the device?
  • Maybe if we never set or read these registers things won’t go wrong?
    • No, Sid tested that
  • People are becoming inured to internal errors because they’re having to force-arm because of this problem
  • We’ll make this board-specific with an AP_IMU.*RESET.*ENABLED thing
  • Will be backported
  • Bouncing back to the bad IMU is bad as we will stick with it for some time - time enough to crash the vehicle
  • Sensors have been sent to Invensense for them to look at
  • May flip this to the second IMU

UTC2358 - APM_Control: remove unused methods by peterbarker · Pull Request #22053 · ArduPilot/ardupilot · GitHub

  • Remove unused methods
  • Merged!

UTC2358 - Move defaulting of HAL_CRSF_TELEM_ENABLED to minimize_features.inc by peterbarker · Pull Request #22054 · ArduPilot/ardupilot · GitHub

  • Move more things into minimizze.inc
  • Merged

UTC2359 - AP_HAL: add a sanity check that we don't minimize features on a 2MB board by peterbarker · Pull Request #22055 · ArduPilot/ardupilot · GitHub

  • Add sanity check we’re not mimising on 2MB boards
  • Merged!

UTC2359 - AP_HAL_ChibiOS: factor out a save_some_flash.inc by peterbarker · Pull Request #22057 · ArduPilot/ardupilot · GitHub

  • Pull out common excludes into a separate file
  • Do we really need classes of boards?
    • MatekH743 vs CubrOrange
  • “Premium” boards
    • H7 or better etc
    • 2MB or better
    • 4 uarts
    • Some of these (e.g. Durandal) are down to ~115kB
      • But we can go to -Os and get vast amounts of stuff down
    • “Drone racing” board?
    • Do we really want to tie the application to the board?
  • Merged!

UTC0011 - Regularise compass backend defines by peterbarker · Pull Request #22063 · ArduPilot/ardupilot · GitHub

  • Compass define cleanups
  • Peter will write a test which builds all boards on master (base branch), then on this PR
  • Trying to get rid of ifdefs!
  • _config.h files are good

UTC0021 - AP_Baro: tidy includes into AP_Baro_config.h by peterbarker · Pull Request #22064 · ArduPilot/ardupilot · GitHub

  • Move baro defines into config.h
  • Merged

UTC0021 - AP_Baro: tidy includes into AP_Baro_config.h by peterbarker · Pull Request #22064 · ArduPilot/ardupilot · GitHub

  • Deprecate old ways of getting AUTOPILOT_VERSION
  • Randy - These were very useful for finding deprecated messages
    • Need some time when we’re going to remove support
  • How about if we made stable releases not generate the warnings?
  • Maybe don’t emit the deprecated warnings on stable releases?
  • MAVLink Interface — Dev documentation
  • QGC and QGC daily
  • Warning to info

UTC0046 - Support for SpeedyBee F405 v3 by andyp1per · Pull Request #22075 · ArduPilot/ardupilot · GitHub

  • SpeedyBeef F405
  • Henry will look at

UTC0049 - AP_OSD: fix error in stats screen introduced in #18396 by Hwurzburg · Pull Request #22088 · ArduPilot/ardupilot · GitHub

  • OSD fixes for icons
  • Merged!

UTC0050 - AP_HAL_ChibiOS: explicitly default MSP off for AP_Periph by peterbarker · Pull Request #22065 · ArduPilot/ardupilot · GitHub

  • Will move this to MAVExplorer

UTC0050 - Remove includes of AP_Vehicle_Type.h by peterbarker · Pull Request #22073 · ArduPilot/ardupilot · GitHub

  • Will bring back to DevCallEU

UTC0101 - RemoteID update

  • Testing is going well
  • Fail-over for operator location issue open
  • Ardupilot issue

UTC0101 - Plane update

  • 4.3.1 has been going well
    • Lots of acrobatics
  • Bad transition on quadplane reported
    • Tridge is looking at
  • One partner reported too-high a descent rate
    • Appears to be a sensor failure
  • Stable release is going well

UTC0103 - Rover/Copter update

  • ArduPilot issues page has lost the pins?
  • 4.3.0 Copter went out yesterday
  • Randy’s not aware of any huge issues
    • Gimbal setup screens in MissionPlanner and QGC are now not working
      • Using old parameter names
      • Need to work with MO and QGC devs to get these working again
    • No real-time gimbal control screen in QGC as far as Randy can see
    • Craig said he gets a real-time gimbal control?
      • David Worther from Phase1
      • Standard desktop QGC
      • Sliders around on a screen
      • Similar to MissionPlanner
    • DO_CHANGE_SPEED mission command not working after takeoff
    • Internal error flow of control error from position controller
    • There’s a new one in 4.3 in the crash detection
      • This isn’t the one being called
      • It’s the position controller not being called for 5 loops
        • It assumes your scheduling is good
          • Can happen at arming
          • Landed should re-init the controller each loop
        • Scheduling isn’t necessarily good
  • Thanks to MdB for doing the reported-line-number in the flow-of-control error message!
  • No Rover yet
    • Wheel encoder is very noisy
      • Blocking 4.3

UTC0125 - Plane: implement quaternion based ACRO mode by tridge · Pull Request #22086 · ArduPilot/ardupilot · GitHub

  • Quaternion based ACRO mode
    • Yaw tau?
      • Nobody’ll ever change it
  • Kff rudder mix is default
  • Yaw damper and full yaw controller could cope with a motor loss
    • Yaw rate controller is really only for acro
  • Controllers trail slightly so rates can be achieved
    • When you let go of the sticks you have to know when to stop the roll
    • There’s a 0.2s fixed constant to handle this

UTC0135 - AP_RCProtocol: move pwm_values to frontend by bugobliterator · Pull Request #22084 · ArduPilot/ardupilot · GitHub

  • Move PWM values to front end
  • Testing is the tricky part
  • Saves RAM but consumes flash
  • Protocol tester only does one at a time
  • But parallel decoding is a problem
  • This will require tridge to study it
  • Need to buy more hardware for more people

UTC0144 - Dronecan serial tunnel is looking much nicer

UTC0147 - bootloader changes

  • Pending….

UTC0147 - close - partner call tomorrow