search

webaccess007 / aeroquad

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

BatteryMonitor incorrectly sets holdAltitude = 0 #100

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The offending statement has already been commented out in the latest github 
revisions so this is just to track the issue to make sure it gets updated in 
the next release.

in batteryMonitor.h  lowBatteryEvent(): 

if (level == OK) {
      digitalWrite(BUZZERPIN, LOW);
      autoDescent = 0;
      holdAltitude = 0;      <---- This should be commented out
    }

If not commented or removed this has the effect of incorrectly zeroing out  
holdAltitude causing the quad to fall out of the sky.

Original issue reported on code.google.com by wila...@gmail.com on 6 Feb 2011 at 7:10

GoogleCodeExporter commented 9 years ago 
Just commented out since working on altitude hold.  It was commented because of 
your bug reporting.  Thanks!

Original comment by CaranchoEngineering@gmail.com on 7 Feb 2011 at 2:06