Reverse cuts out at full power

[BrianAdams]

Workaround is to stay in power level 4. The root issue is that the current system is trying to create a linear thrust model of the motors which requires doubling the rpm's of the motor in reverse. The system does not guard that the target servo value can be outside the response range of the ESC.

[vamfun]

Brian. Can u point to the file that needs fixing. I need to make the changes in the next few days. I will do my own testing and patches if I know where the problem code lines are. Also I think limiting to level 4 is ok except the diagnostic inputs will still go to zero on full reverse won't they?

[BrianAdams]

@vamfun, you are correct about the diagnostics. The change needs to be made in the motors.cpp file in the Arduino code. You will see that it multiplies the target thrust by Motor_Negative_Modifiers (these are the ones causing the issues since they are set to 2). It needs a limit after that calculation to that the resulting value stays above 1000. ( The range on the ESC is 1000-2000ms).

[vamfun]

Posted my quick fix here.
http://openrov.com/xn/detail/6365107:Comment:92456

[vamfun]

Txs again, u saw my fix that I posted.

I need a few hints on how to manage the capture pictures , ie delete , bulk copy or xfer...

I am delivering the ROV to Algalita tomorrow and would like to give them instructions on how to use capture function.

Thanks again , I know u have a full plate.

Sent from my iPhone

On Jun 19, 2014, at 9:03 AM, badevguru notifications@github.com wrote:

@vamfun, you are correct about the diagnostics. The change needs to be made in the motors.cpp file in the Arduino code. You will see that it multiplies the target thrust by Motor_Negative_Modifiers (these are the ones causing the issues since they are set to 2). It needs a limit after that calculation to that the resulting value stays above 1000. ( The range on the ESC is 1000-2000ms).

—
Reply to this email directly or view it on GitHub.

[BrianAdams]

http://openrov.com/xn/detail/6365107:Comment:92815

On Mon, Jun 23, 2014 at 8:43 AM, vamfun notifications@github.com wrote:

Txs again, u saw my fix that I posted.

I need a few hints on how to manage the capture pictures , ie delete ,
bulk copy or xfer...

I am delivering the ROV to Algalita tomorrow and would like to give them
instructions on how to use capture function.

Thanks again , I know u have a full plate.

Sent from my iPhone

On Jun 19, 2014, at 9:03 AM, badevguru notifications@github.com wrote:

@vamfun, you are correct about the diagnostics. The change needs to be
made in the motors.cpp file in the Arduino code. You will see that it
multiplies the target thrust by Motor_Negative_Modifiers (these are the
ones causing the issues since they are set to 2). It needs a limit after
that calculation to that the resulting value stays above 1000. ( The range
on the ESC is 1000-2000ms).

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
#239 (comment)
.