Dev Call Feb 5, 2024

Issues & Pull Requests

Plane

Copter

Attendee count (max): 17

UTC1100 - MiniPix: Remove radiolink url and store url by muramura · Pull Request #5625 · ArduPilot/ardupilot_wiki · GitHub

  • Remove RadioLink URLs
  • Not complying with our license…
  • We did receive a PR from them to add support for another board
  • Peter wants to hold it off for a couple of weeks
  • Their new board PR Henry will look at
  • Revisiting this in 2 weeks

UTC1112 - development.xml: RADIO_RC_CHANNELS message by jlpoltrack · Pull Request #343 · ArduPilot/mavlink · GitHub

  • RADIO_RC_CHANNELS
  • Peter will merge once he’s happy with it
  • Hamish raised some questions

UTC1113 - Filtering: re-work harmonic notch filter freq clamping and disable by tridge · Pull Request #25442 · ArduPilot/ardupilot · GitHub

  • Rework of harmonic notch filter freq clamping
  • PH wants more logging
    • … of some sort
    • Current logging is less useful than it was
    • We’ve never logged the frequencies we’re notching at
      • We don’t log the harmonics
        • Glaring bug wasn’t visible when the harmonics got stuck at high frequencies
      • To log all the harmonic would be terrible
  • Maybe modify the filter tool to show what’s happening with v2 filtering?
  • Option bit for verbose logging?
    • 1st harmonics of first notch?
  • PH and Leonard are happy with this apart from the logging
  • Tridge will talk to Andy about his things

UTC1145 - AP_Proximity: Add support for MR72 CAN by rishabsingh3003 · Pull Request #25801 · ArduPilot/ardupilot · GitHub

  • MR72 CAN Proximity sensor
  • Sensor IDs can be a bit of an issue if you want multiple proximity sensors of hte same type
    • Would need to change other libraries…
    • Node_id is a better name?
  • Some unfortunate naming of variables, Randy has left a comment and wants a follow-up PR
  • merged!

UTC1158 - hwdef: remove bad hwdef lines by peterbarker · Pull Request #26105 · ArduPilot/ardupilot · GitHub

  • Remove bad hwdef lines
  • Merged!

UTC1158 - Extend use of AP_BATTERY_ENABLED by peterbarker · Pull Request #26106 · ArduPilot/ardupilot · GitHub

  • Remove compilation blocks dependent on AP_BATTERY_ENABLED
  • Merged!

UTC1201 - Plane: move pilot throttle output up to control modes by IamPete1 · Pull Request #26108 · ArduPilot/ardupilot · GitHub

  • Move pilot throttle output up to control modes
  • Merged!

UTC1202 - Move can_mgr.init call up to AP_Vehicle by peterbarker · Pull Request #26109 · ArduPilot/ardupilot · GitHub

  • Move can_mgr.init up to AP_Vehicle
  • Merged!

UTC1206 - Provide base-class implementation of guided_mode_request function by peterbarker · Pull Request #26112 · ArduPilot/ardupilot · GitHub

  • Provide base class implementation of guided_mode_request
  • Part of a larger cleanup
  • Merged!

UTC1208 - AP_RCProtocol: add separate define for AP_RCPROTOCOL_DSM_ENABLED by peterbarker · Pull Request #26113 · ArduPilot/ardupilot · GitHub

  • Add separate define for DSM_ENABLED
  • Merged!

UTC1209 - AP_JSON: added JSON parsing library by tridge · Pull Request #26114 · ArduPilot/ardupilot · GitHub

  • JSON parsing
  • Come back to this one

UTC1210 - Fix defines around fixed-mag-cal-yaw by peterbarker · Pull Request #26126 · ArduPilot/ardupilot · GitHub

  • Fix defines around fixed-mag-cal-yaw
  • Merged!

UTC1211 - AP_HAL: add power status bits to logger metadata by peterbarker · Pull Request #26127 · ArduPilot/ardupilot · GitHub

  • Add power status bit to logger metadata
  • Merged!

UTC1213 - AP_ExternalAHRS: Populate Microstrain7 orientation from filter by Ryanf55 · Pull Request #26131 · ArduPilot/ardupilot · GitHub

  • Populate AHRS orientation from filter
  • Bugs!
  • Can be backported
  • Merged!

UTC1215 - Parachute: relay pram conversion does not work if chute is disabled. · Issue #26132 · ArduPilot/ardupilot · GitHub

  • Relay parameter conversion doesn’t work if chute is disabled
  • You will lose your relay when you update!
  • See if the parameter is in storage

UTC1222 - AP_BattMonitor: added temperature reading to INA2xx driver by tridge · Pull Request #26134 · ArduPilot/ardupilot · GitHub

  • Add temperature reading to INA2xx driver
  • Board only
  • … but many, many amps are going through this on some vehicles
  • Merged!

UTC1126 - hwdef: add ASP5033 airspeed in minimal builds by Hwurzburg · Pull Request #26136 · ArduPilot/ardupilot · GitHub

  • Add ASP5033 in minimal builds
  • Merged!

UTC1127 - Allow using virtual environment on Ubuntu Jammy when running install-prereqs-ubuntu.sh · Issue #26137 · ArduPilot/ardupilot · GitHub

  • Allow using venv on ubuntu Jammy?
  • Peter’s OK with the concept so long as it doesn’t break existing flow

UTC1131 - Copter: improved follow at close distances by tridge · Pull Request #26138 · ArduPilot/ardupilot · GitHub

UTC1144 - board_types.txt: reserve 1410 for RadiolinkPIX6 by peterbarker · Pull Request #26140 · ArduPilot/ardupilot · GitHub

  • Reserve 1410 for RadioLink
  • merged!

UTC1145 - Dds yaw control plane by Ryanf55 · Pull Request #26143 · ArduPilot/ardupilot · GitHub

  • DDS yaw control for Plane
  • Some interactions with other parties

UTC1153 - AP_Scripting: Add example for RPM Failsafe action by rishabsingh3003 · Pull Request #26146 · ArduPilot/ardupilot · GitHub

  • Add scripting example for RPM failsafe action
  • Just switches to RTL
  • Some stylistic questions around the script
  • Need to make sure parameter table ID doesn’t conflict
  • A few renames to make

UTC1154 - Copter/Rover update

  • Willian wants another Rover 4.4 release
  • Not much in 4.5
    • Parachute conversion issue
    • Stuck-thread and logging-failed thing
    • SiyiA8 issues?
      • Prearm check?
      • Camera firmware version problem
        • 0.2.3 is latest version
        • Incorrect detection
        • Maybe just reporting?
        • Perhaps a non-enforcing pre-arm?
  • Firmware update problematic for CubePilot Here4
    • GPS-for-yaw
    • Don’t try to use uCenter for update or you will lose that feature!
    • Product differentiation thing?!
    • … if there’s a bug in the uBlox firmware a user might choose between living with the bug or gps-for-yaw
  • Important time fix on Linux
    • Long-standing bug
    • 4.5.0beta2 will get this

UTC0011 - Plane update

  • No real news on Plane Beta yet

UTC0012 - AP_Custom?

  • Maybe AP_GenericVehicle?
  • AP_CustomVehicle
  • AP_EmptyVehicle
  • Uploading scripts to custom build server

UTC0016 - AP_JSON: added JSON parsing library by tridge · Pull Request #26114 · ArduPilot/ardupilot · GitHub

  • Discussion about JSMN vs standard-library
    • Creates AP_JSON thing
    • But still uses std::map, std::vector and std::string
      • It would be nice to have the standard library things
      • We’ve shied away because of:
        • Exceptions
        • Memory allocations behind your back
      • Peter doesn’t want to do anything in SITL-on-hw which we won’t do on a real vehicle
    • Exceptions?
      • We do have “raise” in our current builds
      • …scripting calls it on an error path
        • Which tridge hopes can never happen
      • abort() can end up with a raise
      • And _abort and _raise_r
  • Since we can use the standard library already Peter’s removing his Jsmn-based thing
  • Add some blacklist stuff?
  • Might be able to use the euler library?
  • If we can get raise out of our binary then we can guarantee no exceptions are being raised
  • Would need to wrap abort()

UTC0052 - off the floor

  • Reno isn’t happening
  • Almost certainly Japan this year

UTC0054 - close