ArduPilot / ArduPilot/ArduRemoteID

ID error when not set to params

Open
#133 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
250
Forks
109
PR merge metrics
No merged PRs in 30d

Description

In `uint8_t Transport::arm_status_check()`:

` if (!g.have_basic_id_info()) {
// if there is no basic ID data stored in the parameters give warning. If basic ID data are streamed to RID device,
// it will store them in the parameters
ret += "ID ";
}`

but what if options param is set such that `OPTIONS_DONT_SAVE_BASIC_ID_TO_PARAMETERS` is enabled? Then basic ID would not be stored and there will always be an arm status check error.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in uint8_t Transport::arm_status_check() and trace g.have_basic_id_info() together with OPTIONS_DONT_SAVE_BASIC_ID_TO_PARAMETERS. Verify the arm-status result when basic ID data is streamed but intentionally not saved, and confirm that the false ID error is no longer reported.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.