ArduPilot / ArduPilot/apm_planner

Implement a smarter MAVLink Parameter Manager (QGCUASParamManager)

Open
#7 2 comments 0 reactions 0 assignees Claimed by @malcom2073 View on GitHub
enhancement
Dominant language
C++
Stars
550
Forks
483
PR merge metrics
No merged PRs in 30d

Description

Right now, parameter management is implemented by the QGCParamWidget. Two changes that I think need to be made:

1) It should be implemented in its own class seperate of the UI.
2) It should be smarter about parameter requests. If another class requests a parameter, it should know that it has already requested this parameter and send it to the class directly, rather than re-requesting it from the MAV. It should also handle request timeouts, which are already implemented in QGCParamWidget, so can easily be brought over.

Contributor guide

Open the contributing guide

Research direction

Start by reading QGCParamWidget, where parameter management and request timeouts are currently implemented, and trace how it communicates with the MAV. Define the responsibilities of QGCUASParamManager and identify the callers that need to request parameters. Done means management is separated from the UI, repeated requests are served without another MAV request, and timeouts still work.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
desktop
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.