-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
AP_Scripting: raise max FLTE for yaw to 8 #24933
Conversation
Perhaps you could also update the .md file. it currently says "the YAW_FLTE filter will be set to a maximum of 2Hz". It seems like it might be actually tuning YAW_FLTE rather than simply setting the value, which is what it seems to say right now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this actually works.....it will always be left at the default of 2.5 ....before it was always set to 2.0 (old FLTE_MAX) because the default exceeded it....now it does nothing
this patch is just to stop quicktune from overriding and reducing FLTE so low. It allows the user to set a FLTE before tuning |
better yaw for many vehicles
d9bbf77
to
40b41aa
Compare
thanks! fixed |
QuikTune does no tuning on Yaw FLTE at this point....it just resets it to FLTE_MAX if the tune starts with it either <=0 or greater than FLTE_MAX....I don't think you want to prevent someone from setting it > FLTE_MAX in some cases..... having QuikTune either leave it alone totally or set it like you proposed at one time to GYRO_FLT/8 might be better? |
I was wondering if QuickTune should not tune yaw on a tilt rotor? From what I think I understand from what others have said (and have seen from my own tuning), it doesn't seem to work on a tilt rotor. Not sure about other frame types. |
- if no SMAX is set for a rate controller than the SMAX will be set to 50Hz | ||
- the YAW_FLTE filter will be set to a maximum of 8Hz | ||
|
||
Additionally, if no SMAX is set for a rate controller than the SMAX will be set to 50Hz. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
then
Andy Piper and I have had it tune yaw fine in two different vectored yaw QPlanes....just the FLTE was too low after....for me setting it to 10Hz fixed the issue after QuickTune |
I actually can no longer repeat this using such a high value, I had to use 2.5Hz. @lthall explained the basic problem to me - with vectored yaw and tri-tilts in particular there is a momentum effect from the movement of the motors backwards and forwards and the momentum effect is a physical D term working in the wrong direction. This makes it very hard to tune good values of D - at least for QAutotune - it's possible that QuickTune is doing better here since it is just looking for oscillation, but its also a very complex mechanical system so hard to see what the right approach might be. Using a low FLTE linearizes the effect, but then has other issues. |
As discussed in the dev call 2024-02-12, I flew this yesterday with Q_A_RAT_Y_FLTE = 10 and Q_A_ACCEL_Y_MAX = 4000 (reduced from 6000) and still getting a slow Yaw oscillation, although much improved from what I had been seeing. Also want to re-up this video which shows how this seems to be happening, although this was with a lower value for FLTE. You can actually watch the front motors tilting backwards and forwards as they try to stabilize the aircraft. https://youtu.be/pezygLLC45Q |
better yaw for many vehicles