-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
Tuning indication on OSD(and GCS) #28831
base: master
Are you sure you want to change the base?
Conversation
Nice idea! @Hwurzburg may have an opinion |
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.
Instead of flooding the GCS/OSD every loop with send_texts...only send the message when the value has changed a nontrivial amount, or, send it only once every 2-3 seconds
@Hwurzburg thanks! What would be a good 'nontrivial amount'? In my opinion, 0.01 could be problematic for the D coefficient. However, setting it to 0.001 might not effectively reduce spamming either. Displaying the value every 2-3 seconds could make fine adjustments difficult when using a small knob while flying. Updating it every 100ms could be the solution since OSD runs at 10Hz. How about using a 5% change from the previous value as the threshold? Also, could For minimal spam prevention we already have this in the code |
not sure what you are talking about "0.5f" preventing RC channel jitter from changing tune is different than GCS spam rate |
I meant 5 digits after the decimal. For practical tuning - I don't have much experience here, would be glad if you can share some info or point me to some direction. Will do 5% and 0.0001 - 4 digits after decimal, since autotune shows PIDs this way. Sounds good? |
display if fabs(newvalue-oldvalue) > 0.05 * oldvalue |
Done. Thanks a lot! Could you please share more practical information, such as real-life tuning procedures? We might need to add other tuning sets based on that. |
ce84f11
to
b0a7ea9
Compare
the wiki has lots of details on tuning process for each vehicle the TX tuning is really a bit dated and for special cases (like default pids results in an unstable vehicle...highly unlikely)....in Plane the autotune for fixed wing and quicktune for quadplanes is the easy path, Copter: Autotune, Rover:is pretty straightforward....using manual PID tuning is really beyond most users and can even be dangerous... most users will have a lot of difficulty using manual tuning and getting good results....sometimes even having a dangerous tune on big vehicles... |
ba3d1df
to
ff66741
Compare
ff66741
to
454afc2
Compare
This PR includes:
Displaying the current value of the parameter being tuned
When tuning knob is in the middle (note
>
instead of:
):Adding PIDFF tuning sets for FixedWing mode:
These additions are primarily designed for tuning via FPV without requiring MAVLink modems.
I'll add PR for wiki once everything is settled, currently it is here https://github.com/Yury-MonZon/ardupilot_wiki/tree/patch-1