princeriley / ardupilot-mega

Automatically exported from code.google.com/p/ardupilot-mega
0 stars 0 forks source link

Log mode exit require slider switch which sparkfun mega board does not have #106

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Boot Arduino
2. Log pages are iterated
3. Select 3 to exit logmode
4. Code waits for slider to be switched which does not exist on the 
non-shielded mega.

What is the expected output? What do you see instead?
I would expect 3 to exit log mode. Log mode cannot be exited with out switch.

What version of the product are you using? On what operating system?
1.0.3, Win 7

Please provide any additional information below.
This can be corrected by using the following code for the switch statement in 
process_logs(void) (I don't understand the complete usage of the slide switch.)

case 3:     
  // This is the case for leaving log mode
  Serial.println("Leaving log read mode.");
  log_mode = FALSE;
  break;

Original issue reported on code.google.com by logan.gr...@gmail.com on 17 Sep 2010 at 1:07

GoogleCodeExporter commented 9 years ago
This is not an error in coding.  ArduPilot Mega is intended for use with the 
DIYDrones IMU shield, only.  The slider switch is on the shield.  We do not 
want to leave log mode with just a keyboard command.  We want the extra step of 
physical user interaction.  Unintentionally leaving log mode without the rest 
of the system properly configured can cause unintended motor starts - i.e. 
spinning propellers.

Original comment by dewei...@gmail.com on 18 Sep 2010 at 12:36

GoogleCodeExporter commented 9 years ago
I disagree. I realize the slide switch is on the shield. Log retrieval is the 
last thing that I want to be doing when I have a live plane. Flipping switches 
that are buried on boards mounted in the plane is an annoying "feature". If you 
need a fail safe then a "real" switch should be pulled out and placed on the 
side of the plane.

Original comment by logan.gr...@gmail.com on 18 Sep 2010 at 12:44