ArduPilot / ArduPilot/ardupilot

Storage: parameter restore misses some elements of storage

Open
#19,973 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
15.9k
Forks
21.4k
Avg merge
3d 17h
Merged PRs (30d)
119

Description

We added a storage check and restore to resolve the rare issue of parameter resets but based it is likely that we overlooked that other data is also stored in storage and will also be wiped. This includes:

- Parameters
- Mission
- Fence
- Rally Points
- Mavlink2 signing keys
- Bind info for AP Radio (skyviper only)
- CAN DNA database

Some ideas on what we should do:

- ensure all these various bits of storage are restored
- reinstate the critical error to hopefully make users aware of the problem so they can take action
- add an example lua script which adds an arming check for any or all of the items above so users will be warned in case of an eeprom wipe (only really professional users would need this kind of security)

FYI @peterbarker

Contributor guide

Open the contributing guide

Research direction

Start by locating the existing storage check and restore logic, then trace how each listed item—parameters, mission, fence, rally points, signing keys, bind info, and the CAN DNA database—is handled. Confirm the intended approach before changing behavior; done means the agreed data is restored or users receive the agreed warning or arming check.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, lua
Domain
embedded-iot, robotics
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.