-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tail precomp improvements #186
Conversation
1f3f2be
to
c89a3af
Compare
Parameters
|
A note on the inertia precomp.It is doing something only if the headspeed changes. If the motor/esc/governor were ideal, and the headspeed would stay constant, the estimated coll->yaw precomp would be correct. But, if the headspeed drops, the estimated coll->yaw precomp is too much, and needs to be lowered. This is what the inertia precomp does, it adjusts the yaw precomp based on the headspeed change speed. The RPM drop speed indicates how much torque is "missing" from keeping the RPM constant, and thus how much the yaw precomp must be reduced. To tune it, first tune the coll->precomp with a full battery, and avoid doing tests that drops the headspeed. Then tune the inertia precomp so that the moves that actually drops the headspeed won't affect the tail. This is escpecially visible when the battery is nearly flat, and the throttle saturates at 100%. A flat battery can be simulated by setting the Max throttle in the governor to a lower value, like 90% or 85%. |
c89a3af
to
3bc1832
Compare
6936489
to
d6ccf92
Compare
* Add new parameters yaw_inertia_precomp_gain and yaw_inertia_precomp_cutoff. * Remove yaw_collective_dynamic parameters. * Use quadratic drag coefficient in yaw precomp.
UPDATED on 8/1/2025
This PR also includes the RPM filter refactoring from #194.
There is a hack for using the old dynamic precomp MSP parameters for the new inertia precomp.
Thus the inertia settings can be changed in the Configurator with the old dynamic precomp parameters.