ArduPilot / ArduPilot/MissionPlanner
Extended Tuning screen response to out-of-range input from user
- Dominant language
- C#
- Stars
- 2.4k
- Forks
- 2.9k
- Avg merge
- 19h 16m
- Merged PRs (30d)
- 4
Description
Not sure we should have the "Y", "X" and "Z" attached to the end of the "Rate Roll", "Rate Pitch" and "Rate Yaw".
"Rate Roll Y" -> "Rate Roll"
"Rate Pitch X" -> "Rate Pitch"
"Rate Yaw Z" -> "Rate Yaw"
If I input a number that is out of range for Rate Roll P it pops up a warning (which is good), "ATC_RAT_RLL_P Value out of range, Do you want to accept the new value?". If I push "No" it should revert the parameter to it's original value, not the maximum (or minimum) of the parameter's valid range.
Maybe it should display the valid range in the pop-up box?
By the way, if I push the "Refresh Screen" it doesn't clear the green from the boxes that I've updated.
Contributor guide
No contributing guide indexed for this repository
Research direction
The issue names the Extended Tuning screen but no files or tests. Reproduce the out-of-range edit, declining the warning, and using Refresh Screen; then trace the parameter editor and screen refresh behavior. Done means the labels and warning behavior match the requested wording, declined values revert to their original value, valid ranges are shown if adopted, and refresh clears update highlighting.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- desktop
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100