ArduPilot / ArduPilot/MissionPlanner
Fix for rename of MAG_ENABLE to COMPASS_ENABLE for ArduPilot master
- 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