ArduPilot / ArduPilot/MissionPlanner

Fix for rename of MAG_ENABLE to COMPASS_ENABLE for ArduPilot master

Open
#2,214 1 comment 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

The MAG_ENABLE parameter is renamed in master to COMPASS_ENABLE. Some users are confused by the greyed out checkbox on the Compass setup page "Enable compasses".
Can we get that updated to cope with both names?

PS: I initially submitted this issue thinking MAG_ENABLE was gone, then realised it was just renamed

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at the Compass setup page and search the codebase for MAG_ENABLE and COMPASS_ENABLE. Update the "Enable compasses" checkbox handling so it works with both parameter names, then verify that the checkbox is no longer incorrectly greyed out for ArduPilot master.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.