Skip to content
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

Closed
wants to merge 6 commits into from
Closed

Tail precomp improvements #186

wants to merge 6 commits into from

Conversation

rotorflight
Copy link
Owner

@rotorflight rotorflight commented Oct 22, 2024

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.

@rotorflight rotorflight changed the title Draft PR for tail testing DRAFT: Tail testing Nov 27, 2024
@rotorflight
Copy link
Owner Author

rotorflight commented Dec 19, 2024

Parameters

set yaw_inertia_precomp_gain = <0-100>

Amount of main rotor inertia.
Start with 10. Increment by 5.

set yaw_inertia_precomp_cutoff = <1-100>

Time of inertia dissipation, unit is Hz.
Presumably 2-100 range is working.
Try 20 (~50ms) to start with.

Removed parameters

yaw_collective_dynamic_gain
yaw_collective_dynamic_decay

@rotorflight
Copy link
Owner Author

rotorflight commented Dec 21, 2024

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%.

@rotorflight rotorflight changed the title DRAFT: Tail testing Tail precomp improvements Jan 13, 2025
@rotorflight rotorflight marked this pull request as ready for review January 13, 2025 09:45
rotorflight pushed a commit that referenced this pull request Jan 21, 2025
* Add new parameters yaw_inertia_precomp_gain and yaw_inertia_precomp_cutoff.

* Remove yaw_collective_dynamic parameters.

* Use quadratic drag coefficient in yaw precomp.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants