Merge pull request #7583 from joelucid/fix_7582

Fixing missing upper notch limit #7582
This commit is contained in:
Michael Keller 2019-02-18 02:10:33 +13:00 committed by GitHub
commit fe1a06730b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 4 deletions

View File

@ -61,6 +61,7 @@ static float pidLooptime;
static rpmNotchFilter_t filters[2];
static rpmNotchFilter_t* gyroFilter;
static rpmNotchFilter_t* dtermFilter;
static float maxFrequency;
PG_REGISTER_WITH_RESET_FN(rpmFilterConfig_t, rpmFilterConfig, PG_RPM_FILTER_CONFIG, 3);
@ -124,6 +125,7 @@ void rpmFilterInit(const rpmFilterConfig_t *config)
numberFilters = getMotorCount() * (filters[0].harmonics + filters[1].harmonics);
const float filtersPerLoopIteration = numberFilters / loopIterationsPerUpdate;
filterUpdatesPerIteration = rintf(filtersPerLoopIteration + 0.49f);
maxFrequency = 0.5f / (gyro.targetLooptime * 1e-6f);
}
static float applyFilter(rpmNotchFilter_t* filter, int axis, float value)
@ -171,10 +173,7 @@ void rpmFilterUpdate()
for (int i = 0; i < filterUpdatesPerIteration; i++) {
float frequency = (harmonic + 1) * motorFrequency[motor];
if (frequency < currentFilter->minHz) {
frequency = currentFilter->minHz;
}
float frequency = constrainf((harmonic + 1) * motorFrequency[motor], currentFilter->minHz, maxFrequency);
biquadFilter_t* template = &currentFilter->notch[0][motor][harmonic];
// uncomment below to debug filter stepping. Need to also comment out motor rpm DEBUG_SET above
/* DEBUG_SET(DEBUG_RPM_FILTER, 0, harmonic); */