Motors_ArduCopter.initialize() has bad default values for timer initialization

herolic / aeroquad

Automatically exported from code.google.com/p/aeroquad

0 stars 0 forks source link

Motors_ArduCopter.initialize() has bad default values for timer initialization #91

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

In Motors_ArduCopter.initialize() change all OCRxy = 3000 to OCRxy = 2000, also 
add commandAllMotors(1000) at the end. This makes sure the PWM for the ESCs is 
set to 0 power. The prevision setting to half power caused the altitude hold 
problem Honk had, see 
http://aeroquad.com/showthread.php?1836-Altitude-Hold-a-new-approach&p=18674&vie
wfull=1#post18674

Original issue reported on code.google.com by al...@arcor.de on 30 Jan 2011 at 5:36

GoogleCodeExporter commented 9 years ago

Thank you for the catch.  Changes implemented and will upload to GitHub

Original comment by CaranchoEngineering@gmail.com on 31 Jan 2011 at 7:33

Changed state: Fixed
Added labels: Milestone-FlightSoftware_v2.2

GoogleCodeExporter commented 9 years ago

You mean half of the requested changes implemented. The other OCRs are still at 
3000 and there is no commandAllMotors(1000)at the end.

Original comment by al...@arcor.de on 31 Jan 2011 at 11:09

GoogleCodeExporter commented 9 years ago

OK, I caught the other OCRxx's and added commandAllMotors(1000) at the end of 
initialize().  THANKS!

Original comment by CaranchoEngineering@gmail.com on 2 Feb 2011 at 8:45