ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

AP_BattMonitor: Support an analog current reading on ESC telemetry battery monitors (better 4-in-1 ESC support)

Open AbelMartinez90 opened this issue 3 years ago • 5 comments

Hello all, I recently run into that issue. I wanted to get rid of my power module and use The ESC telemetry instead.

The voltage works fine. However many 4 in 1 ESC do not send the current thru ESC telemetry and they come with an analog current wire. The problem is that if BATT_MONITOR: 9 (ESC) then BATT_CURR_PIN does not appear on the parameter list and cant be configured.

It would be great if BATT_MONITOR: 9 (ESC) and also the user would be allow to configure a BATT_CURR_PIN for the cases when the ESC does not send the current info trough the ESC Telemetry.

Thank you in advance.

Regards

AbelMartinez90 avatar Jun 28 '22 15:06 AbelMartinez90

Add a second, analog battery monitor and you are done!

amilcarlucas avatar Jun 29 '22 07:06 amilcarlucas

Add a second, analog battery monitor and you are done!

hank you for your answer.

The problem in that case is that I am getting the current under Batt_2. I tried to merge both setting a 3rd one and do BATT3_SUM_MASK=0 and BATT3_SUM_MASK sum of 1 and 2. But still does not work. I cant make both the Voltage and Current to display together on the mission planer HUD under BATT, nor on Herelink when using Qgroundcontrol.

I could however do Current1 and Battery2 (User Items) to display Current and Voltage in the HUD of Mission planner. But that does not work on Qgrouncontrol Herelink if I want to see on the battery tab the percentage of remaining battery and voltage together.

AbelMartinez90 avatar Jun 30 '22 14:06 AbelMartinez90

I could however do Current1 and Battery2 (User Items) to display Current and Voltage in the HUD of Mission planner. But that does not work on Qgrouncontrol Herelink if I want to see on the battery tab the percentage of remaining battery and voltage together.

It's also worth noting that using 2 different monitors like this will result in sending 2 different packets so it's more MAVLink usage, and leads to seeing voltage/current measurements from 2 very different spots in time. You also will find that using 2 different monitors causes you to have trouble when you want to use both battery voltage correction (very important), and battery current limiting in AP_Motors, because those are required to come from the same battery instance.

I think it probably does make sense to add an optional analog current reading for the ESC telemetry path, as it would cover the 4-in-1 ESC case far better then we currently do.

WickedShell avatar Jun 30 '22 16:06 WickedShell

I could however do Current1 and Battery2 (User Items) to display Current and Voltage in the HUD of Mission planner. But that does not work on Qgrouncontrol Herelink if I want to see on the battery tab the percentage of remaining battery and voltage together.

It's also worth noting that using 2 different monitors like this will result in sending 2 different packets so it's more MAVLink usage, and leads to seeing voltage/current measurements from 2 very different spots in time. You also will find that using 2 different monitors causes you to have trouble when you want to use both battery voltage correction (very important), and battery current limiting in AP_Motors, because those are required to come from the same battery instance.

I think it probably does make sense to add an optional analog current reading for the ESC telemetry path, as it would cover the 4-in-1 ESC case far better then we currently do.

That would be GREAT, thank you for listening my suggestion.

AbelMartinez90 avatar Jun 30 '22 16:06 AbelMartinez90

I could however do Current1 and Battery2 (User Items) to display Current and Voltage in the HUD of Mission planner. But that does not work on Qgrouncontrol Herelink if I want to see on the battery tab the percentage of remaining battery and voltage together.

It's also worth noting that using 2 different monitors like this will result in sending 2 different packets so it's more MAVLink usage, and leads to seeing voltage/current measurements from 2 very different spots in time. You also will find that using 2 different monitors causes you to have trouble when you want to use both battery voltage correction (very important), and battery current limiting in AP_Motors, because those are required to come from the same battery instance.

I think it probably does make sense to add an optional analog current reading for the ESC telemetry path, as it would cover the 4-in-1 ESC case far better then we currently do.

Hello, I have updated my copters to the last firmware version but still I cant use a current analog pin when settimg the batt monitor to ESC, is this improvement going to come soon?

Regards

AbelMartinez90 avatar Aug 04 '22 17:08 AbelMartinez90