Dev Call Aug 28, 2023

Issues & Pull Requests

Plane

Copter

Attendee count (max): 13

UTC2300 - common.xml: display GIMBAL_MANAGER_STATUS.flags as bitmask by peterbarker · Pull Request #327 · ArduPilot/mavlink · GitHub

  • Display gimbal manager status flags as bitmask
  • Merged

UTC2301 - AC_AttitudeControl: Fix yaw limit calculations by lthall · Pull Request #23326 · ArduPilot/ardupilot · GitHub

  • Fix yaw limit calculations
  • Some discussions as to whether something can’t be const
  • Get_rate_yaw_pid non-const
  • Probably need a get_rate_yaw_pid const version

UTC2308 - AP_AdvancedFailsafe: automatically set mode to auto when datalink loss by eppravitra · Pull Request #23333 · ArduPilot/ardupilot · GitHub

  • Automatically set mode to auto when datalink is lost
    • So if you’re in GUIDED you can just not do anything useful in AFS
  • LUA implementation?
  • This has to be optional or you might end up bouncing if the user is trying to force something and you’re losing the link repeatedl
  • Tridge is happy for this to go in if it is optional

UTC2314 - AP_AdvancedFailsafe: option to continue the mission even after data link is recovered by eppravitra · Pull Request #23334 · ArduPilot/ardupilot · GitHub

  • Option to continue mission after data link is recovered
  • AFS_OPTIONS required!

UTC2317 - Add AP_INVERTED_FLIGHT_ENABLED by peterbarker · Pull Request #24332 · ArduPilot/ardupilot · GitHub

  • Ability to remove inverted flight at compilation time
  • Will remove the copter changes and then merge

UTC2323 - AP_Arming: mag field check vs world magnetic model by rmackay9 · Pull Request #24501 · ArduPilot/ardupilot · GitHub

  • Prearm check to check mag field check from world magnetic model
  • 100 mgauss requirement
  • Could be a chunk of user support on this one
  • Return an enumeration from use_compass()?
    • A “maybe” return value?
  • Maximum of xy rather than length?
  • Z is double thresh because it’s often off a lot
    • Compass dance doesn’t seem to do z-axis nicely?

UTC2338 - Add PID object PD limit by lthall · Pull Request #24567 · ArduPilot/ardupilot · GitHub

  • Add PID object PD limit
  • Limit the maximum correction out of the PD term
  • It’s the sum of them
  • logging ?
    • Add a limit flag?
    • Yes, add a bit in existing logging field
  • Constructors are a bit unwieldy?
    • Would be nice to find a better way to construct PIDs?
  • Impact on Heli?
    • Different not necessarily better
  • Bill has some issues with some of the defaults changes
  • This will help the Hex frame a bit to be more consistent when it loses a propellor

UTC0000 - Correct MAVLINK_MSG_ID_SET_ATTITUDE_TARGET anglular rate input frame by lthall · Pull Request #24568 · ArduPilot/ardupilot · GitHub

  • Attitude+rate specified means angles weren’t considered to be in body frame
    • XML specifies this

UTC0014 - AP_Scripting: add argcheck to default userdata creation functions by IamPete1 · Pull Request #24601 · ArduPilot/ardupilot · GitHub

  • Add a warning if you’re doing bad things with constructors in scripting
  • Should be a lot safer
    • There’s one in our examples are wrong
    • Vector3 in particular
  • merged!

UTC0015 - AP_BattMonitor: expose CAPACITY param on periph by IamPete1 · Pull Request #24603 · ArduPilot/ardupilot · GitHub

  • Expose capacity param on periph
  • Param removed from table…
  • MergeOnCIPass

UTC0018 - AP_RPM: Allow more instances by magicrub · Pull Request #24645 · ArduPilot/ardupilot · GitHub

  • Allow more RPM instances
  • NeedsTesting but can then be merged

UTC0019 - AP_BattMonitor: extend AP_BATT_MONITOR_MAX_INSTANCES to 16 by magicrub · Pull Request #24660 · ArduPilot/ardupilot · GitHub

  • Move from 9 to 16 batteries
  • Can reset batteries in SITL, and that is now covered
  • These aren’t really batteries
    • These are subsystem voltages
  • Whole heap of stuff around these like failsafes
    • This is on periph
  • Use cells instead of batteries?
  • Probably should pull out the ADC stuff into a separate library so we can re-use it
  • Merged!

UTC0028 - Refactor microstrain to common library by Ryanf55 · Pull Request #24723 · ArduPilot/ardupilot · GitHub

  • Refactor microstrain
  • Should have done rename as a separate step
  • Merged!

UTC0032 - pre-commit: Fix current errors by Ryanf55 · Pull Request #24727 · ArduPilot/ardupilot · GitHub

  • Pre-commit on repo will pass after this
  • Need to exclude Sagetech as we’ve already gone backwards and forwards on it
  • Tridge was experimenting with pre-commit
    • Still on
    • Not on on every repo
    • Ugly on 4.3 branches
    • Does slow down some things significantly
    • Subsystem split is very slow
    • Some rebasing things
  • Probably not on by default in people’s environments as working on older branches is problematic

UTC0039 - pre-commit: Run autoupdate by Ryanf55 · Pull Request #24728 · ArduPilot/ardupilot · GitHub

  • Fix pre-commit references to fix poetry bugs
  • Merged!

UTC0040 - AP_ExternalAHRS: Fix MicroStrain nullptr crash on bootup by Ryanf55 · Pull Request #24792 · ArduPilot/ardupilot · GitHub

  • Fix nullptr crash on bootup
  • Really want the latest data?
  • Might be a packet sitting there
  • Boost priority can stuff you up
    • Higher than SPI
    • Drops priority once AHRS update is done
    • So we do need to call the update in two places

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

  • Plane update
  • 4.3.8 went out nicely
    • But parallel stables aren’t working
    • No feedback on whether people are actually flying 4.3.8
  • 4.4 is going well
    • No real problems reported
    • … airspeed use got disabled upgrading 4.1.6 to 4.4
      • Tridge will try to replicate in SITL

UTC0052 - Copter/Rover-4.4.0 issues list · Issue #23192 · ArduPilot/ardupilot · GitHub

  • Copter update
  • Couple of questions that came up
    • No ADSB DroneCAN driver at the moment
    • RemoteID bothersome stuff
      • Play games where we store parameters on H7
      • Turn on RemoteID by default on H7 2MB boards
        • But it is off by default
        • Once turned on it can’t be turned off
      • Sid’s already doing this for CubeID
    • Andy mentioned slow loops on 4.4
    • And lost logging data?

UTC0103 - Add build options for deprecated messages MOUNT_CONTROL/MOUNT_CONFIGURE by peterbarker · Pull Request #24743 · ArduPilot/ardupilot · GitHub

  • Allow some deprecated mavlink commands to be compiled out
  • Merged!

UTC0107 - Tools: Change erase sectors size by tsubashchandar · Pull Request #24750 · ArduPilot/ardupilot · GitHub

  • Erase sectors correctly on stm32l4r5
  • Merged!

UTC0111 - Tools: support new stable release of 4.3.x by tridge · Pull Request #24759 · ArduPilot/ardupilot · GitHub

  • Fix marking of things as stable in metadata
  • Merged!

UTC0112 - ArduCopter: allow MAV_CMD_SOLO_BTN_PAUSE_CLICK as COMMAND_INT by peterbarker · Pull Request #24762 · ArduPilot/ardupilot · GitHub

  • Handle cmd solo btn click as command_int
  • Will circle back around and turn these off apart from on-solo things
  • Merged!

UTC0114 - hwdef: don't include solo gimbal or oreoled on thepeach autopilots by peterbarker · Pull Request #24763 · ArduPilot/ardupilot · GitHub

  • Peach boards don’t get OreoLED or Solo Gimbal
  • Makes no sense
  • Merged!

UTC0114 - Take set-servo and set-relay commands as COMMAND_INT too by peterbarker · Pull Request #24766 · ArduPilot/ardupilot · GitHub

  • Set-servo and set-relay as command-int
  • Merged!

UTC0115 - Periph: Add BATTn_OPTION to not broadcast - keep data internal by magicrub · Pull Request #24773 · ArduPilot/ardupilot · GitHub

  • Option to not broadcast battery information in Periph

UTC0122 - SITL: support both multicast and socketcan in CAN SITL by tridge · Pull Request #24779 · ArduPilot/ardupilot · GitHub

  • Support both multicast and socketcan for SITL
    • Instead of just socketcan
  • Couple of name changes and can then be merged

UTC0133 - AP_Mount: Viewpro date fix (and other small fixes) by rmackay9 · Pull Request #24788 · ArduPilot/ardupilot · GitHub

  • Small viewpro fixes
  • Merged

UTC0134 - https://github.com/ArduPilot/ardupilot/pull/24795

  • Simplify Siyi absolute zoom

  • New A8 firmware has absolute zoom

  • Just delete the old code

  • Needs some debug removed

  • Good to merge after that

UTC0137 - autotest: remove generation of defaults files by peterbarker · Pull Request #24781 · ArduPilot/ardupilot · GitHub

  • Stop generating defaults files on autotest server
  • Merged!

UTC0140 - AP_Periph: restructure by tridge · Pull Request #24782 · ArduPilot/ardupilot · GitHub

  • Restructure AP_Periph
  • Needs rebasing and can then be reviewed….

UTC0147 - Plane: accept DO_ENGINE_CONTROL as both COMMAND_LONG and COMMAND_INT by peterbarker · Pull Request #24783 · ArduPilot/ardupilot · GitHub

  • Accept plane engine control as both command-long and command-int
  • Merged!

UTC0148 - AP_ExternalAHRS: correct AP_ExternalAHRS init by peterbarker · Pull Request #24784 · ArduPilot/ardupilot · GitHub

  • Correct external ahrs init

UTC0151 - GSoC is almost over

  • Wrap-up meetings

UTC0151 - new dev-team-member vote

UTC0151 - close