2 new parameters in src/.
- Added 2 parameters:
FW_T_THR_DAMPING=0.05fandFW_T_THR_INTEG=0.02f. - Removed 2 parameters:
FW_T_I_GAIN_THRandFW_T_THR_DAMP. - Most activity in
src/(+14/−18).
Full analysis → Summary: automatic analysis
Changed values
None.
Added (2)
| Name | Value | Type | File |
|---|---|---|---|
FW_T_THR_DAMPING | 0.05f | number | src/modules/fw_pos_control/fw_path_navigation_params.c |
FW_T_THR_INTEG | 0.02f | number | src/modules/fw_pos_control/fw_path_navigation_params.c |
Removed (2)
| Name | Last value | Note |
|---|---|---|
FW_T_I_GAIN_THR | 0.05f | removed from src/modules/fw_pos_control/fw_path_navigation_params.c |
FW_T_THR_DAMP | 0.1f | removed from src/modules/fw_pos_control/fw_path_navigation_params.c |
Post text (X-ready, not posted)
PX4 update (Jan 12): FW_T_I_GAIN_THR gain removed (was 0.05f)
- new FW_T_THR_DAMPING = 0.05f
- new FW_T_THR_INTEG = 0.02f
Details: https://fitzyracing1.github.io/knoblog/demo/px4/entries/a38abdb.html
Commit: https://github.com/PX4/PX4-Autopilot/commit/a38abdbf9d5590a8572651ab1d0baecccd2cb7bb
Values are PX4 firmware defaults in the repo.
233/280 weighted chars
Modules touched
| Module | Added | Modified | Removed | Lines + | Lines − |
|---|---|---|---|---|---|
src | 0 | 1 | 0 | +14 | −18 |
Watched files (1)
Show file list
| Path | Lines | Note | |
|---|---|---|---|
| M | src/modules/fw_pos_control/fw_path_navigation_params.c | +14/−18 |
Commit message
TECS throttle gains default reduction and transition values from previous PX4 version (#22548) * TECS: reduce default of FW_T_I_GAIN_THR Signed-off-by: Silvan Fuhrer <silvan@auterion.com> * TECS: reduce default of FW_T_THR_DAMP Signed-off-by: Silvan Fuhrer <silvan@auterion.com> * TECS: improve param descriptions and meta data of some params Signed-off-by: Silvan Fuhrer <silvan@auterion.com> * FW Pos C: rename FW_T_THR_DAMP to FW_T_THR_DAMPING Rename wihtout param translation as the interpretation of this param has changed recently (a bug was fixed that caused the damping to have a much lower effect than it should). We want to avoid that users keep the previously tuned value (with the bug) and do not update the tuning when using the fixed version (v1.14.0 and later). Signed-off-by: Silvan Fuhrer <silvan@auterion.com> * FW Pos C: rename FW_T_I_GAIN_THR to FW_T_THR_INTEG Rename wihtout param translation as the interpretation of this param has changed recently (a bug was fixed that caused the damping to have a much lower effect than it should). We want to avoid that users keep the previously tuned value (with the bug) and do not update the tuning when using the fixed version (v1.14.0 and later). Signed-off-by: Silvan Fuhrer <silvan@auterion.com> --------- Signed-off-by: Silvan Fuhrer <silvan@auterion.com>