Gathiyo / arducopter

Automatically exported from code.google.com/p/arducopter
0 stars 0 forks source link

CH7 function should be do-nothing by default instead of save waypoint #333

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. leave channel 7 (input) unconnected
2. note via APMissionPlanner that default channel 7 function is save wp
3. fly around
4. try load waypoint in apmission planner.  note mission is full of dozens of 
LAND commands

What is the expected output? What do you see instead?
channel 7's function shoudl be to do nothing by default so user needs to 
specificially specify that they want to save waypoints.

Original issue reported on code.google.com by rmackay...@gmail.com on 9 Feb 2012 at 4:04

GoogleCodeExporter commented 8 years ago
in config.h
line 196
#ifndef CH7_OPTION
# define CH7_OPTION     CH7_SAVE_WP
#endif

Original comment by Meee...@gmail.com on 9 Feb 2012 at 9:58

GoogleCodeExporter commented 8 years ago
Thanks Michael O.  Checked in your change here:
http://code.google.com/p/ardupilot-mega/source/detail?r=861a3d314a6edcef9599dfcc
f81bece237d73527

Original comment by rmackay...@gmail.com on 9 Feb 2012 at 11:27

GoogleCodeExporter commented 8 years ago
channel 7's function shoudl be to do nothing by default so user needs to 
specificially specify that they want to save waypoints.

Maybe we should fix the bug that causes the issue rather than disable the 
feature!
Jason

Original comment by jasonshort on 9 Feb 2012 at 4:54

GoogleCodeExporter commented 8 years ago
It's such a cool feature, I would rather have it as the default, and make sure 
a bad CH7 doesn't cause unwanted behavior.

Original comment by jasonshort on 9 Feb 2012 at 11:32