ArduPilot / ArduPilot/MissionPlanner

Dangerous bug in parameter compare. (current params misrepresented)

Open
#2,842 0 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

#### Issue details
The parameter's current value can be displayed as "zero" instead of the real value.

This is dangerous if one is to believe something was not set, and toggles it - or somebody takes note of "current" value, and later enter it to "restore" the change.

example: check the ATC_RAT_PIT_P ?

![Screenshot from 2022-04-19 14-24-06](https://user-images.githubusercontent.com/1468842/164003984-565409c9-560b-4102-b1bb-80e488cdbd02.png)

#### Version
1.3.77

#### Platform
[ x] All
[ ] AntennaTracker
[ ] Copter
[ ] Plane
[ ] Rover
[ ] Sub

#### Airframe type
n/a

#### Hardware type
n/a

#### Logs
the screenshot says all, logs do not contain anything about this.

Contributor guide

No contributing guide indexed for this repository

Research direction

Begin by reproducing the parameter comparison for ATC_RAT_PIT_P on Mission Planner 1.3.77 and trace the UI path that renders the current value; the issue names no source file or test. Done means the displayed current value matches the actual parameter, including this example, with a regression check if the relevant test location can be found.

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
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.