Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logger switch wrong definition #43

Closed
GoogleCodeExporter opened this issue Mar 18, 2016 · 4 comments
Closed

Logger switch wrong definition #43

GoogleCodeExporter opened this issue Mar 18, 2016 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What is the expected output? What do you see instead?

Nothing happens when logger switch is put in "dump log" position. This is 
because the logger switch should be defined as SW2. It is currently defined as 
SW1. 

Please provide any additional information below.

Changing "SW1_pin" to "SW2_pin" in the following two lines fixes the problem. 

1:  pinMode(SW1_pin,INPUT);     //Switch SW1 (pin PG0)
2:  while (digitalRead(SW1_pin)==0)

Original issue reported on code.google.com by Miles.R....@gmail.com on 29 Sep 2010 at 7:00

@GoogleCodeExporter
Copy link
Author

in my oppinion, using the push button makes more sense
so you can leave the slide switch for another option (eg using sonar/ ir 
sensors/ etc)

Original comment by sebbl.gr...@hotmail.de on 29 Sep 2010 at 7:07

@GoogleCodeExporter
Copy link
Author

Ah, ok. I assumed it was meant to function the same as ardupilot-mega. I didnt 
know it was a feature. Feel free to close this issue

Original comment by Miles.R....@gmail.com on 29 Sep 2010 at 10:11

@GoogleCodeExporter
Copy link
Author

We are changing whole switch operation, in future it switch will activate CLI 
functions from where you can navigate trough menus and do different things.

Original comment by jphelirc on 1 Oct 2010 at 11:08

@GoogleCodeExporter
Copy link
Author

CLI is now implemented with the white switch at the back.  You go into the menu 
when the switch is backwards. 

So I'm closing this issue.

Original comment by rmackay...@gmail.com on 19 Dec 2010 at 4:56

  • Changed state: Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant