ArduPilot / ArduPilot/ardupilot
Storage: parameter restore misses some elements of storage
- 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
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