Airframe config for px4vision devkit drone #13683
Conversation
| param set THR_MDL_FAC 0.3 | ||
| fi | ||
|
|
||
| pmw3901 start |
There was a problem hiding this comment.
Can you use SENS_EN_PMW3901 instead of starting it manually?
RomanBapst
force-pushed
the
pr-pixhawkvision-config
branch
from
3f7f274
to
1815a78
Compare
|
@dagar @Jaeyoung-Lim I fixed this branch up and squashed the commits, please have a look. |
Update high level tuning params on px4vision Update collision preventer parameter naming Add TELEM1 configurations to airframe configs px4vision airframe: fmu, oneshot, tuning, revert some to default addressed review comments Signed-off-by: RomanBapst <bapstroman@gmail.com>
Signed-off-by: RomanBapst <bapstroman@gmail.com>
| then | ||
| # System parameters | ||
| # use FMU motor outputs for less delay in the rate control loop | ||
| param set SYS_USE_IO 0 |
There was a problem hiding this comment.
I saw the parameter is using FMU PWM is instead of IO PWM here, but holybro is still using IO PWM on shipping verision now. Looks they have an old customized version. I think we should notify Holybro and early adopters.
| param set MPC_XY_VEL_P 0.079 | ||
| param set MPC_XY_TRAJ_P 0.3 | ||
| param set MPC_Z_TRAJ_P 0.3 | ||
| param set MPC_Z_VEL_I 0.15 |
There was a problem hiding this comment.
MPC_Z_VEL_I max value is 0.1
| param set MPC_ACC_DOWN_MAX 5 | ||
| param set MPC_ACC_HOR_MAX 10 | ||
| param set MPC_ACC_UP_MAX 4 | ||
| param set MPC_MANTHR_MIN 0 |
There was a problem hiding this comment.
Set manual throttle MPC_MANTHR_MIN to 0%? Why? Will this cause unsafety?
| param set MPC_VEL_MANUAL 5 | ||
| param set MPC_XY_CRUISE 5 | ||
| param set MPC_XY_VEL_MAX 5 | ||
| param set MPC_XY_VEL_P 0.079 |
There was a problem hiding this comment.
MPC_XY_VEL_P only support %.2f
| param set MPC_XY_VEL_MAX 5 | ||
| param set MPC_XY_VEL_P 0.079 | ||
| param set MPC_XY_TRAJ_P 0.3 | ||
| param set MPC_Z_TRAJ_P 0.3 |
There was a problem hiding this comment.
This MPC_Z_TRAJ_P parameter is not exist now.
Related PR: #13596
This adds a airframe config for the px4vision devkit drone produced by Holybro
Would it be possible to get this in for the 1.10 release?