Dev Call May 15, 2023

Issues & Pull Requests

Plane

Copter

Attendee count (max): 16

Pre-Agenda:

  • New HobbyWing ESCs
    • E.g.
      • 30kW
      • DroneCAN
      • 24S 200W

UTC1101 - common: ODOMETRY add optional quality field by rmackay9 · Pull Request #316 · ArduPilot/mavlink · GitHub

  • Bring in quality field for odometry from upstream
  • Good for logging ATM
  • Could use -1 or 0 to discard the data
  • Merged!

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

  • Support sending and receiving mavlink messages
  • General principle is sound but PH’s messages need addressing
    • A few new helpers should be used
    • Millisecond timestamps OK?
    • Micros?64?
    • Jitter correction might have some issues with very long times?
  • Hopefully get this in tomorrow night

UTC1107 - Add and use AP_RCPROTOCOL_ENABLED by peterbarker · Pull Request #21083 · ArduPilot/ardupilot · GitHub

  • A global AP_RCPROTOCOL_ENABLED
  • Merged!

UTC1117 - APBoardConfig: Add skip validate board by haydendonald · Pull Request #23620 · ArduPilot/ardupilot · GitHub

  • Add a factory-testing option to allow boards to boot even if they’re fault in the face of a validation bit
  • Merged!
  • Should have a mandatory hardware check in place of this!

UTC1121 - AP Scripting: Add health by haydendonald · Pull Request #23634 · ArduPilot/ardupilot · GitHub

  • Add sensor health bindings to AP_Scripting
  • Grumbling on range-checking in lua
  • A few things to fix

UTC1127 - Plane: fix rangefinder correction when terrain follow is off by jschall · Pull Request #23658 · ArduPilot/ardupilot · GitHub

  • This was last week
  • Tridge to look at

UTC1128 - Plane: treat LAND waypoint param1 as an AGL alt to climb to by jschall · Pull Request #23723 · ArduPilot/ardupilot · GitHub

  • Treat land waypoint as AGL
  • No updates
  • Tag not removed?

UTC1129 - Mode takeoff step by step by Hwurzburg · Pull Request #23728 · ArduPilot/ardupilot · GitHub

  • Break mode takeoff changes out into separate changes
  • PH has left some comments
  • Two-point thing?
    • Protect against messed-up compass
    • Head to waypoint rather than fly heading
    • Yaw vs ground-course
  • What happens if ARMING_REQUIRE 0?
  • Needs to not fly if you’re throttle-suppressed too
  • Initial goal was to climb to minimum altitude before loitering regardless of when you enter takeoff
  • Maybe add a frame for the takeoff altitude so it can be above terrain?
  • With wind the ground-course points are likely to be better
  • Ground course vs yaw questions
  • Need to avoid using ahrs yaw in this state as it is unreliable

UTC1151 - autotest: integrate features.json generation to build_binaries.py by peterbarker · Pull Request #23733 · ArduPilot/ardupilot · GitHub

  • Run extract features from the checked out branch
  • Merged!

UTC1153 - AP_HAL_ChibiOS: do not build ChibiOS with debug symbols unless debug build by peterbarker · Pull Request #23735 · ArduPilot/ardupilot · GitHub

  • Do not build ChibiOS with debug symbols unless requested
  • If you build with –debug then it should change the optimisation of
  • Merged!

UTC1155 - AP_Mount: support gimbal_manager_status and do_gimbal_manager_configure by rmackay9 · Pull Request #23737 · ArduPilot/ardupilot · GitHub

  • Support gimbal manager status and do_gimbal_manager_config
  • We’re not actual blocking additional users at the moment
  • Minor bug fix for gimbal device ID
    • PX4 use 0-based as we do in master ATM
  • Randy will merge after checking the +1 thing

UTC0004 - AP_RCProtocol: Fix the issue of ST24 receiver not working by lipper-chengdu · Pull Request #23742 · ArduPilot/ardupilot · GitHub

  • Fix ST24 RC input
  • Test harness doesn’t test ST24 at the moment
  • Could drop this to minimize and save-some-flash
  • This is apparently a well-designed protocol
    • Popularity and design quality are inversely-related in RC protocols – tridge

UTC0010 - Add support for RPLidarA1 by peterbarker · Pull Request #23749 · ArduPilot/ardupilot · GitHub

  • Add support for rplidar a1 and s1
  • Rewrites the parser
  • Merged

UTC0015 - Consolidate AP_Periph serialmanager defaults by peterbarker · Pull Request #23751 · ArduPilot/ardupilot · GitHub

  • Consolidate define initialisation of serial device defines
  • Merged!

UTC0016 - SITL: stop using mavlink channels for packing messages by peterbarker · Pull Request #23763 · ArduPilot/ardupilot · GitHub

  • Stop using communication buffers to pack simulation messages
  • Merged!
  • Should check how extensions are handled
    • And signing
    • And sequence numbers

UTC0018 - Replace HAVE_FILESYSTEM_SUPPORT with backend defines by peterbarker · Pull Request #23781 · ArduPilot/ardupilot · GitHub

  • Remove HAVE_FILESYSTEM
  • Merged

UTC0030 - Fix SkyViper scripting by peterbarker · Pull Request #23776 · ArduPilot/ardupilot · GitHub

  • Add –enable-scripting option to waf
  • And fix some bindings
  • Merged

UTC0031 - Filter: correct description of filter tracking mode by peterbarker · Pull Request #23787 · ArduPilot/ardupilot · GitHub

  • Try to fix up the description on notch filter options
    • This calls it per-esc
  • Should we really have different modes for this instead?
  • Multi-Source
  • WikiNeeded
  • We should have an arming check for bad config
    • 4-or-more-sources and bandwidth >25% of your frequency should fail by default
      • Need an option bit to allow it
    • This could cause problems on fallback!
      • Notch failsafe settings?!
        • Infer throttle for each motor instead?
  • Motor Misalignment can make notch filter work badly

UTC0036 - Tools: Web: FilterReview: add spectrograph notch tracking overlay by IamPete1 · Pull Request #23798 · ArduPilot/ardupilot · GitHub

  • Updates to filter review tool
  • Might create a new repo for web-firmware
  • Original tool allowed you to design a filter but couldn’t see what effect it was
  • ArduPilot Filter Review Tool
    • “Upload a log”
    • See what the data looks like
    • We want to be able to design a filter and apply it to this
    • For ESCs you could use this
      • But not for throttle-based filters
  • Feature requests would be great
    • PRs would be better
  • [10:44 AM]rmackay9: The graphs are very pretty, like a Van Gogh
  • Magfit javascript on the way?

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

  • RTK gate mod for EKF3 still not in
  • SMAX should be zer while twitching in EKF3
  • Resting voltage seems broken
    • 30-40% gain reduction on Randy’s vehicle?!
  • Resting voltage should not drop when you disarm!

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

  • Siyia8 not doing ROI
    • Probably a configuration issue
  • Short-range sonar with noise and bendy ruler could be endless climb
  • Baro temp cal not working?
  • Custom build server broken for one board?
    • Build product not present?
  • Mnt1_type and camera_type problems for some combined devices
  • Yaw problems?
    • Behaviour 0 not working correctly
      • Going into corners it yaw just a tiny bit
      • Leonard has a fix for it
      • Should be fixed in 4.5 and maybe backport
  • Andy’s found a few more issues but hasn’t put them on the list yet

UTC0053 - pymavlink PR to fix mavlink generation

  • merged

UTC0059 - expo stuff

  • Tridge would like someone to bounce ideas off
  • If you want to help, let him know

UTC0100 - close