Skip to content

Add low wind warning #771

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.

Sign up for GitHub

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

Merged
amilcarlucas merged 1 commit into from
Sep 5, 2025
Merged

Add low wind warning #771

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions TUNING_GUIDE_ArduCopter.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -782,6 +782,10 @@ Setup the lua script using:
1. Press `Upload selected params to FC, and advance to next file` button.
1. Close *ArduPilot Methodic Configurator*

WARNING: Quicktune requires moderate wind disturbances to calibrate properly.
Flying in completely calm conditions or perfectly steady wind can cause Quicktune to calculate overly aggressive PID values.
These excessive settings may cause dangerous oscillations and potential crashes when your vehicle later encounters normal wind conditions.

Comment on lines +785 to +788

Copilot AI Sep 5, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The same warning text is duplicated at lines 785-787 and 866-868. Consider creating a single, prominently placed warning section to avoid duplication and ensure consistency if the warning needs to be updated in the future.

Suggested change
WARNING: Quicktune requires moderate wind disturbances to calibrate properly.
Flying in completely calm conditions or perfectly steady wind can cause Quicktune to calculate overly aggressive PID values.
These excessive settings may cause dangerous oscillations and potential crashes when your vehicle later encounters normal wind conditions.
## Quicktune Wind Warning
**WARNING:** Quicktune requires moderate wind disturbances to calibrate properly.
Flying in completely calm conditions or perfectly steady wind can cause Quicktune to calculate overly aggressive PID values.

Copilot uses AI. Check for mistakes.
Perform the flight and afterward:

1. Connect the flight controller to the PC
Expand Down Expand Up @@ -859,6 +863,10 @@ Setup the lua script using:
1. Press `Upload selected params to FC, and advance to next file` button.
1. When asked *Update file with values from FC?* select `Close` to close the application and go perform the experiment.

WARNING: Quicktune requires moderate wind disturbances to calibrate properly.
Flying in completely calm conditions or perfectly steady wind can cause Quicktune to calculate overly aggressive PID values.
These excessive settings may cause dangerous oscillations and potential crashes when your vehicle later encounters normal wind conditions.

Perform the flight and afterward:

### 9.2.2 Store quicktune results to file
Expand Down
Loading