Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plane: updates for 4.4.3-beta1 #25457

Merged
merged 18 commits into from
Nov 7, 2023
Merged

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Nov 4, 2023

Changes from 4.4.2:

  • fixed setup of ICM45486 IMU on CubeOrangePlus-BG edition
  • disable AFSR on IxM42xxx IMUs to prevent gyro bias for "stuck" gyros
  • fixed AK09916 compass being non-responsive
  • implement GPS_DRV_OPTION for using ellipsoid height in more GPS drivers
  • fixed SIYI AP_Mount parsing bug
  • configuration fixes for BETAFTP-F405 boards
  • fixed origin versus home relative bug in quadplane landing and guided takeoff
  • correct mavlink response for no airspeed sensor on preflight calibration
  • protect against notch filtering with uninitialised RPM source in ESC telemetry
  • allow lua scripts to populate full ESC telemetry data
  • added YJUAV_A6SE_H743 support

tridge and others added 13 commits November 1, 2023 16:00
an incorrectly configured ESC telemetry source can lead to a notch
running at very low frequencies. A simple example is a lua script like
this:

function update()
   esc_telem:update_rpm(12, 0, 0)
   return update, 10
end
return update()

where motor 12 is unused.

with that script in place we get a 1.0078 Hz filter which leads to
massive phase lag and a crashed aircraft

this is a safety protection. We should also try to find out why the
INS_HNTCH_FREQ lower limit is not working
these undocumented bits in register 0x4d control the "adaptive full
scale range" mode of the ICM42688. The feature is enabled by default
but has a bug where it gives "stuck" gyro values for short periods
(between 1ms and 2ms):, leading to a significant gyro bias at longer
time scales, enough to in some cases cause a vehicle to crash if it is
unable to switch to an alternative IMU

this fixes ArduPilot#25025
this allows a lot more drivers to use the GPS_DRV_OPTION but to use
ellisoid height. Particularly useful for DroneCAN GPS modules

using ellisoid height instead of AMSL is useful in some specialised
application
this caused lots of lost packets
when SPI goes into undefined state during reset
only in progress if we have started a task running
@tridge tridge force-pushed the pr-plane-4.4.3-beta1 branch from b90d588 to ce84b05 Compare November 4, 2023 06:53
@tridge tridge force-pushed the pr-plane-4.4.3-beta1 branch from ce84b05 to 5642b94 Compare November 7, 2023 00:03
@tridge tridge merged commit 8fd36cb into ArduPilot:Plane-4.4 Nov 7, 2023
72 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants