ArduPilot / ArduPilot/MissionPlanner

Extended Tuning screen response to out-of-range input from user

Open
#1,268 2 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.