ArduPilot / ArduPilot/MissionPlanner

Extended Tuning page doesn't display error when parameter fails to write

Open
#2,354 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
C#
Stars
2.4k
Forks
2.9k
Avg merge
19h 16m
Merged PRs (30d)
4

Description

The Config >> Extended tuning page doesn't seem to display an error when one of the parameters on it has been marked ReadOnly in the AP firmware. I.e. ArduPilot will not allow the parameter value to be changed but the failure to write the parameter value is not reported to the user.

To recreate the issue I did this:

- Used APJTools to mark ATC_RAT_RLL_P to be readonly
- loaded the modified firmware onto a Hex Cube Black
- Modified the parameter using the Extended Tuning page and pressed the "Write Params" button
- It appeared the parameter value was modified (the field turned from green to black and the value shown was the new value) but AP had rejected the change. Pushing the "Refresh Screen" caused the parameter value to go back to it's original value

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reproducing the issue in the Config >> Extended tuning page with ATC_RAT_RLL_P marked ReadOnly in the ArduPilot firmware. Use Write Params and then Refresh Screen to compare the displayed value with the firmware value; done means a rejected write is reported to the user and the field does not falsely appear committed.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 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.