search

AdityaSantosh / aeroquad

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

Command 'V' handles 6 channels only, Command '8' in SerialCom.pde is obsolete #126

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
'V' should send 8 values, so '8' is obsolete.

Original issue reported on code.google.com by al...@arcor.de on 25 Jul 2011 at 1:37

GoogleCodeExporter commented 8 years ago 
I thought I removed this one also, I'll fix it for next rev too.  Thanks!

Original comment by CaranchoEngineering@gmail.com on 25 Jul 2011 at 2:44

GoogleCodeExporter commented 8 years ago 
Ala42, one more fix, we can now load up all external settings via a file.  I'd 
like to get rid of most of InitializeEEPROM() code, any issues you see with 
this?

Original comment by CaranchoEngineering@gmail.com on 25 Jul 2011 at 2:47

GoogleCodeExporter commented 8 years ago 
Running code with unititialized variables is always a problem. You should at 
least initialize the values to something, e.g. all PIDs in a loop to 1,0,0.

Original comment by al...@arcor.de on 25 Jul 2011 at 3:05

GoogleCodeExporter commented 8 years ago

Original comment by CaranchoEngineering@gmail.com on 25 Jul 2011 at 6:34