sakuraclamp / arducopter

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

using knob to adjust PID's inflight may crash Your copter #428

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Arducoper NG or ACM?
What steps will reproduce the problem?
1. Set one channel assigned to a knob to adjust P value of a PID - for example 
roll rate 
2. during radio calibration my radio set min and max value for the chanel - 
let's say knob_min and knob_max. 
3. in flight I  was using the know to fine tune P parameter in rangte from 1.2 
to 1.5 

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

THe knob value somehow exceeded the range between knob_min and knob_max set 
during the calibration and suddenly it showed knob_min-1. Maybe sometimes using 
additional force You can twist the potentiometer to more extreme values . In 
that case Arducopter set P value to zero - > the copter will crash of course.

You should remove the condition that set's the parameter to zero if it's out of 
a supposed range. If the knob value is lower then min it should set the P 
parameter to Min value- in my case 1.2 , if higher than max value of the input 
- the output should by maximal P value - in my case 1.5 

What version of the product are you using? On what operating system?

Please provide any additional information below.

Original issue reported on code.google.com by dpba...@gmail.com on 14 Jun 2012 at 2:30

GoogleCodeExporter commented 8 years ago
same : http://code.google.com/p/arducopter/issues/detail?id=416 ?

Original comment by airma...@gmail.com on 15 Jun 2012 at 8:56

GoogleCodeExporter commented 8 years ago
yes the same problem - if it was in trunk on 1 Jun is this fix already included 
in current stable arducopter?

Original comment by dpba...@gmail.com on 15 Jun 2012 at 10:16

GoogleCodeExporter commented 8 years ago
This should be fixed on the trunk. I'll check.
Jason

Original comment by jasonshort on 15 Jun 2012 at 4:03

GoogleCodeExporter commented 8 years ago

Original comment by jasonshort on 26 Jun 2012 at 5:51