ArduPilot / ArduPilot/MissionPlanner

getParamListAsync() should also call setAPType(), like getParamListMavftpAsync()!?

Open
#3,052 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

this piece https://github.com/ArduPilot/MissionPlanner/blob/eb09d34509214476db9afb751735c8687230526a/ExtLibs/ArduPilot/Mavlink/MAVLinkInterface.cs#L1718-L1719 should also be added here https://github.com/ArduPilot/MissionPlanner/blob/eb09d34509214476db9afb751735c8687230526a/ExtLibs/ArduPilot/Mavlink/MAVLinkInterface.cs#L1875, shouldn't it?

Both methods for param download trigger the same STATUSTEXT banner from ArduPilot.

Contributor guide

No contributing guide indexed for this repository

Research direction

Open ExtLibs/ArduPilot/Mavlink/MAVLinkInterface.cs and compare getParamListAsync() around line 1875 with getParamListMavftpAsync() around line 1718. Confirm how both parameter-download methods trigger the ArduPilot STATUSTEXT banner, then ensure the same setAPType() call is present in getParamListAsync().

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.