Skip to content

FilterReview: support up to four harmonic notches - #362

Open
j-w9 wants to merge 1 commit into
ArduPilot:mainfrom
j-w9:filter-review-4-notches
Open

j-w9 wants to merge 1 commit into
ArduPilot:mainfrom
j-w9:filter-review-4-notches

Conversation

@j-w9

@j-w9 j-w9 commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Fixes #252.

ArduPilot now supports a third harmonic notch, and a fourth on some boards (ArduPilot/ardupilot#30246). This adds INS_HNTC3_* and INS_HNTC4_* to FilterReview: their filters in the estimated post-filter spectrum, the tracking lines on the FFT plot and spectrogram, and the logged notch frequencies.

The third and fourth notch inputs and show options only appear if the log, or a loaded parameter file, has those parameters, so the page is unchanged for two-notch firmware.

Testing:

  • On five logs, all FFT, Bode and spectrogram traces are identical to main.
  • A log with its notch 2 parameters renamed to notch 3 (and to notch 4) gives identical results to the original on main.

The Filter Tool link still only carries notches 1 and 2, as the Filter Tool supports two.

ArduPilot now supports a third harmonic notch, and a fourth on some
boards (ArduPilot/ardupilot#30246). Add INS_HNTC3_* and INS_HNTC4_* to
the notch parameters, their filters and plot tracking lines, and the
logged notch frequencies for notches 3 and 4.

The third and fourth notch inputs and show options are only shown if the
log, or a loaded parameter file, has those parameters.

Fixes ArduPilot#252
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FilterReview: add support for upto 4 harmonic notches

1 participant