Skip to content

Commit d936160

Browse files
KarateBrotKarateBrot
committed
Removed Dynamic Notch from Features
1 parent 474ff80 commit d936160

File tree

3 files changed

+105
-106
lines changed

3 files changed

+105
-106
lines changed

‎src/js/Features.js‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,13 @@ const Features = function (config) {
8484

8585
if (semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_36)) {
8686
features.push(
87-
{bit: 28, group: 'antiGravity', name: 'ANTI_GRAVITY', haveTip: true, hideName: true},
88-
{bit: 29, group: 'other', name: 'DYNAMIC_FILTER'}
87+
{bit: 28, group: 'antiGravity', name: 'ANTI_GRAVITY', haveTip: true, hideName: true}
8988
);
89+
if (semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_44)) { // DYNAMIC_FILTER got removed from FEATURES in BF 4.3 / API 1.44
90+
features.push(
91+
{bit: 29, group: 'other', name: 'DYNAMIC_FILTER'}
92+
);
93+
}
9094
}
9195

9296
if (!semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_36)) {

‎src/js/tabs/pid_tuning.js‎

Lines changed: 57 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -377,51 +377,74 @@ TABS.pid_tuning.initialize = function (callback) {
377377
}
378378

379379
if (semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_42)) {
380-
const dynamicNotchWidthPercent_e = $('.pid_filter input[name="dynamicNotchWidthPercent"]');
381-
const dynamicNotchQ_e = $('.pid_filter input[name="dynamicNotchQ"]');
382-
const dynamicNotchCount_e = $('.pid_filter input[name="dynamicNotchCount"]');
383380

384381
$('.smartfeedforward').hide();
385382

386-
if (FC.FEATURE_CONFIG.features.isEnabled('DYNAMIC_FILTER')) {
387-
$('.dynamicNotch').show();
388-
} else {
389-
$('.dynamicNotch').hide();
383+
// Dynamic Notch Filter
384+
if (semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_44)) {
385+
$('.dynamicNotch span.inputSwitch').hide();
386+
if (!FC.FEATURE_CONFIG.features.isEnabled('DYNAMIC_FILTER')) {
387+
$('.dynamicNotch').hide();
388+
}
390389
}
391-
$('.dynamicNotchRange').toggle(semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_43));
392-
$('.pid_filter select[name="dynamicNotchRange"]').val(FC.FILTER_CONFIG.dyn_notch_range);
390+
const dynamicNotchRange_e = $('.pid_filter select[name="dynamicNotchRange"]');
391+
const dynamicNotchWidthPercent_e = $('.pid_filter input[name="dynamicNotchWidthPercent"]');
392+
const dynamicNotchCount_e = $('.pid_filter input[name="dynamicNotchCount"]');
393+
const dynamicNotchQ_e = $('.pid_filter input[name="dynamicNotchQ"]');
394+
const dynamicNotchMinHz_e = $('.pid_filter input[name="dynamicNotchMinHz"]');
395+
const dynamicNotchMaxHz_e = $('.pid_filter input[name="dynamicNotchMaxHz"]');
396+
397+
dynamicNotchRange_e.val(FC.FILTER_CONFIG.dyn_notch_range);
393398
dynamicNotchWidthPercent_e.val(FC.FILTER_CONFIG.dyn_notch_width_percent);
394-
dynamicNotchQ_e.val(FC.FILTER_CONFIG.dyn_notch_q);
395399
dynamicNotchCount_e.val(FC.FILTER_CONFIG.dyn_notch_count);
396-
$('.pid_filter input[name="dynamicNotchMinHz"]').val(FC.FILTER_CONFIG.dyn_notch_min_hz);
400+
dynamicNotchQ_e.val(FC.FILTER_CONFIG.dyn_notch_q);
401+
dynamicNotchMinHz_e.val(FC.FILTER_CONFIG.dyn_notch_min_hz);
402+
dynamicNotchMaxHz_e.val(FC.FILTER_CONFIG.dyn_notch_max_hz);
403+
397404
if (semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_43)) {
398-
$('.pid_filter input[name="dynamicNotchMinHz"]').attr("max","250");
399-
$('.pid_filter input[name="dynamicNotchMaxHz"]').val(FC.FILTER_CONFIG.dyn_notch_max_hz);
405+
dynamicNotchMinHz_e.attr("max","250");
400406
} else {
401407
$('.dynamicNotchMaxHz').hide();
402408
}
403-
if (semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_44)) {
404-
$('.dynamicNotchHelp').attr('title', i18n.getMessage('pidTuningMultiDynamicNotchFilterHelp'));
405-
$('.dynamicNotchWidthPercent').hide();
406-
} else {
407-
$('.dynamicNotchCount').hide();
408-
}
409409

410+
const dynamicNotchSwitch_e = $('.pid_filter input[id="dynamicNotchEnabled"]');
411+
412+
dynamicNotchSwitch_e.on('change', function() {
413+
414+
const checked = $(this).is(':checked');
415+
const count = parseInt(dynamicNotchCount_e.val());
416+
417+
if (checked && !count) {
418+
dynamicNotchCount_e.val(FILTER_DEFAULT.dyn_notch_count);
419+
}
420+
421+
$('.dynamicNotch span.suboption').toggle(checked);
422+
$('.dynamicNotchRange').toggle(semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_43) && checked);
423+
$('.dynamicNotchMaxHz').toggle(semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_43) && checked);
424+
$('.dynamicNotchWidthPercent').toggle(semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_44) && checked);
425+
$('.dynamicNotchCount').toggle(semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_44) && checked);
426+
427+
}).prop('checked', FC.FILTER_CONFIG.dyn_notch_count !== 0 || FC.FEATURE_CONFIG.features.isEnabled('DYNAMIC_FILTER')).trigger('change');
428+
429+
// RPM Filter
410430
$('.rpmFilter').toggle(FC.MOTOR_CONFIG.use_dshot_telemetry);
411431

412-
$('.pid_filter input[name="rpmFilterHarmonics"]').val(FC.FILTER_CONFIG.gyro_rpm_notch_harmonics);
413-
$('.pid_filter input[name="rpmFilterMinHz"]').val(FC.FILTER_CONFIG.gyro_rpm_notch_min_hz);
432+
const rpmFilterHarmonics_e = $('.pid_filter input[name="rpmFilterHarmonics"]');
433+
const rpmFilterMinHz_e = $('.pid_filter input[name="rpmFilterMinHz"]');
434+
435+
rpmFilterHarmonics_e.val(FC.FILTER_CONFIG.gyro_rpm_notch_harmonics);
436+
rpmFilterMinHz_e.val(FC.FILTER_CONFIG.gyro_rpm_notch_min_hz);
414437

415-
$('.pid_filter #rpmFilterEnabled').change(function() {
438+
$('.pid_filter #rpmFilterEnabled').on('change', function() {
416439

417-
let harmonics = $('.pid_filter input[name="rpmFilterHarmonics"]').val();
418-
let checked = $(this).is(':checked') && harmonics != 0;
440+
const harmonics = rpmFilterHarmonics_e.val();
441+
const checked = $(this).is(':checked') && harmonics !== 0;
419442

420-
$('.pid_filter input[name="rpmFilterHarmonics"]').attr('disabled', !checked);
421-
$('.pid_filter input[name="rpmFilterMinHz"]').attr('disabled', !checked);
443+
rpmFilterHarmonics_e.attr('disabled', !checked);
444+
rpmFilterMinHz_e.attr('disabled', !checked);
422445

423446
if (harmonics == 0) {
424-
$('.pid_filter input[name="rpmFilterHarmonics"]').val(FILTER_DEFAULT.gyro_rpm_notch_harmonics);
447+
rpmFilterHarmonics_e.val(FILTER_DEFAULT.gyro_rpm_notch_harmonics);
425448
}
426449

427450
if (checked !== (FC.FILTER_CONFIG.gyro_rpm_notch_harmonics !== 0)) { // if rpmFilterEnabled is not the same value as saved in the fc
@@ -442,8 +465,10 @@ TABS.pid_tuning.initialize = function (callback) {
442465
dynamicNotchQ_e.val(FC.FILTER_CONFIG.dyn_notch_q);
443466
dynamicNotchWidthPercent_e.val(FC.FILTER_CONFIG.dyn_notch_width_percent);
444467
}
468+
445469
$('.rpmFilter span.suboption').toggle(checked);
446-
}).prop('checked', FC.FILTER_CONFIG.gyro_rpm_notch_harmonics != 0).change();
470+
471+
}).prop('checked', FC.FILTER_CONFIG.gyro_rpm_notch_harmonics !== 0).trigger('change');
447472

448473
} else {
449474
$('.itermRelaxCutoff').hide();
@@ -1321,7 +1346,7 @@ TABS.pid_tuning.initialize = function (callback) {
13211346
FC.FILTER_CONFIG.dyn_notch_q = parseInt($('.pid_filter input[name="dynamicNotchQ"]').val());
13221347
FC.FILTER_CONFIG.dyn_notch_min_hz = parseInt($('.pid_filter input[name="dynamicNotchMinHz"]').val());
13231348

1324-
let rpmFilterEnabled = $('.pid_filter #rpmFilterEnabled').is(':checked');
1349+
const rpmFilterEnabled = $('.pid_filter #rpmFilterEnabled').is(':checked');
13251350
FC.FILTER_CONFIG.gyro_rpm_notch_harmonics = rpmFilterEnabled ? parseInt($('.pid_filter input[name="rpmFilterHarmonics"]').val()) : 0;
13261351
FC.FILTER_CONFIG.gyro_rpm_notch_min_hz = parseInt($('.pid_filter input[name="rpmFilterMinHz"]').val());
13271352
}
@@ -1350,7 +1375,9 @@ TABS.pid_tuning.initialize = function (callback) {
13501375
FC.ADVANCED_TUNING.vbat_sag_compensation = $('input[id="vbatSagCompensation"]').is(':checked') ? parseInt($('input[name="vbatSagValue"]').val()) : 0;
13511376
FC.ADVANCED_TUNING.thrustLinearization = $('input[id="thrustLinearization"]').is(':checked') ? parseInt($('input[name="thrustLinearValue"]').val()) : 0;
13521377
FC.FILTER_CONFIG.dyn_lpf_curve_expo = parseInt($('.pid_filter input[name="dtermLowpassExpo"]').val());
1353-
FC.FILTER_CONFIG.dyn_notch_count = parseInt($('.pid_filter input[name="dynamicNotchCount"]').val());
1378+
1379+
const dynamicNotchEnabled = $('.pid_filter input[id="dynamicNotchEnabled"]').is(':checked');
1380+
FC.FILTER_CONFIG.dyn_notch_count = dynamicNotchEnabled ? parseInt($('.pid_filter input[name="dynamicNotchCount"]').val()) : 0;
13541381

13551382
FC.TUNING_SLIDERS.slider_pids_mode = TuningSliders.sliderPidsMode;
13561383
//round slider values to nearest multiple of 5 and passes to the FW. Avoid dividing calc by (* x 100)/5 = 20

‎src/tabs/pid_tuning.html‎

Lines changed: 42 additions & 74 deletions
Original file line numberDiff line numberDiff line change
@@ -1423,7 +1423,7 @@
14231423
</td>
14241424
</tr>
14251425

1426-
<tr>
1426+
<tr class="newFilter rpmFilter">
14271427
<th class="rpmFilter" colspan="2">
14281428
<div class="pid_mode rpmFilter">
14291429
<div i18n="pidTuningRpmFilterGroup"></div>
@@ -1455,95 +1455,63 @@
14551455
</td>
14561456
</tr>
14571457

1458-
<tr>
1458+
<tr class="newFilter dynamicNotch">
14591459
<th class="dynamicNotch" colspan="2">
14601460
<div class="pid_mode dynamicNotch">
14611461
<div i18n="pidTuningDynamicNotchFilterGroup"></div>
14621462
<div class="helpicon cf_tip dynamicNotchHelp" i18n_title="pidTuningDynamicNotchFilterHelp"></div>
14631463
</div>
14641464
</th>
14651465
</tr>
1466-
<tr class="newFilter dynamicNotch dynamicNotchRange">
1466+
1467+
<tr class="newFilter dynamicNotch">
1468+
14671469
<td>
1468-
<select name="dynamicNotchRange">
1469-
<!-- Populated on execution -->
1470-
</select>
1470+
<span class="inputSwitch"><input type="checkbox" id="dynamicNotchEnabled" class="toggle" /></span>
14711471
</td>
1472-
<td>
1473-
<div>
1474-
<label>
1475-
<span i18n="pidTuningDynamicNotchRange"></span>
1476-
</label>
1472+
1473+
<td colspan="2">
1474+
<span i18n="pidTuningDynamicNotchFilterGroup"></span>
1475+
1476+
<span class="suboption dynamicNotchRange">
1477+
<select name="dynamicNotchRange">
1478+
<!-- Populated on execution -->
1479+
</select>
1480+
<label><span i18n="pidTuningDynamicNotchRange"></span></label>
14771481
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchRangeHelp"></div>
1478-
</div>
1479-
</td>
1480-
</tr>
1481-
<tr class="newFilter dynamicNotch dynamicNotchWidthPercent">
1482-
<td>
1483-
<input type="number" name="dynamicNotchWidthPercent" step="1" min="0" max="20"/>
1484-
</td>
1485-
<td>
1486-
<div>
1487-
<label>
1488-
<span i18n="pidTuningDynamicNotchWidthPercent"></span>
1489-
</label>
1482+
</span>
1483+
1484+
<span class="suboption dynamicNotchWidthPercent">
1485+
<input type="number" name="dynamicNotchWidthPercent" step="1" min="0" max="20"/>
1486+
<label><span i18n="pidTuningDynamicNotchWidthPercent"></span></label>
14901487
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchWidthPercentHelp"></div>
1491-
</div>
1492-
</td>
1493-
</tr>
1494-
<tr class="newFilter dynamicNotch dynamicNotchQ">
1495-
<td>
1496-
<input type="number" name="dynamicNotchQ" step="1" min="1" max="1000"/>
1497-
</td>
1498-
<td>
1499-
<div>
1500-
<label>
1501-
<span i18n="pidTuningDynamicNotchQ"></span>
1502-
</label>
1503-
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchQHelp"></div>
1504-
</div>
1505-
</td>
1506-
</tr>
1507-
<tr class="newFilter dynamicNotch dynamicNotchCount">
1508-
<td>
1509-
<input type="number" name="dynamicNotchCount" step="1" min="0" max="5"/>
1510-
</td>
1511-
<td>
1512-
<div>
1513-
<label>
1514-
<span i18n="pidTuningDynamicNotchCount"></span>
1515-
</label>
1488+
</span>
1489+
1490+
<span class="suboption dynamicNotchCount">
1491+
<input type="number" name="dynamicNotchCount" step="1" min="1" max="5"/>
1492+
<label><span i18n="pidTuningDynamicNotchCount"></span></label>
15161493
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchCountHelp"></div>
1517-
</div>
1518-
</td>
1519-
</tr>
1520-
<tr class="newFilter dynamicNotch">
1521-
<td>
1522-
<input type="number" name="dynamicNotchMinHz" step="1" min="60" max="1000"/>
1523-
</td>
1524-
<td>
1525-
<div>
1526-
<label>
1527-
<span i18n="pidTuningDynamicNotchMinHz"></span>
1528-
</label>
1494+
</span>
1495+
1496+
<span class="suboption dynamicNotchQ">
1497+
<input type="number" name="dynamicNotchQ" step="1" min="1" max="1000"/>
1498+
<label><span i18n="pidTuningDynamicNotchQ"></span></label>
1499+
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchQHelp"></div>
1500+
</span>
1501+
1502+
<span class="suboption dynamicNotchMinHz">
1503+
<input type="number" name="dynamicNotchMinHz" step="1" min="60" max="1000"/>
1504+
<label><span i18n="pidTuningDynamicNotchMinHz"></span></label>
15291505
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchMinHzHelp"></div>
1530-
</div>
1531-
</td>
1532-
</tr>
1533-
<tr class="newFilter dynamicNotch dynamicNotchMaxHz">
1534-
<td>
1535-
<input type="number" name="dynamicNotchMaxHz" step="1" min="200" max="1000"/>
1536-
</td>
1537-
<td>
1538-
<div>
1539-
<label>
1540-
<span i18n="pidTuningDynamicNotchMaxHz"></span>
1541-
</label>
1506+
</span>
1507+
1508+
<span class="suboption dynamicNotchMaxHz">
1509+
<input type="number" name="dynamicNotchMaxHz" step="1" min="200" max="1000"/>
1510+
<label><span i18n="pidTuningDynamicNotchMaxHz"></span></label>
15421511
<div class="helpicon cf_tip" i18n_title="pidTuningDynamicNotchMaxHzHelp"></div>
1543-
</div>
1512+
</span>
15441513
</td>
15451514
</tr>
1546-
15471515
</table>
15481516
</div>
15491517
<div class="gui_box grey topspacer pid_filter two_columns_second">

0 commit comments

Comments
 (0)