search

ArduPilot / MissionPlanner

Mission Planner Ground Control Station for ArduPilot (c# .net)
http://ardupilot.org/planner/
GNU General Public License v3.0
1.79k stars 2.41k forks source link

InitialSetup/Compass: Remove "Enable compasses" checkbox #2264

Open rmackay9 opened 4 years ago

rmackay9 commented 4 years ago

We've removed the MAG_ENABLE parameter from recent builds and it's a real edge condition that someone would want to disable compasses. It's such an edge case that we can remove it from this screen entirely I think because it just causes confusion.

remove-enable-compasses

meee1 commented 4 years ago

i will hide it if the param doesnt exist. else it will break older users

meee1 commented 4 years ago

or i could just remove it?

rmackay9 commented 4 years ago

@meee1, I think it can just be removed completely regardless of which version is being used.

I think the only place where a user would want to disable the compasses is when doing non-GPS navigation and the wiki setup guides specify to set the MAG_ENABLE parameter to zero (like here) so we don't need any easy-to-use checkbox for the 99% of users who are not doing non-GPS navigation