DShot ESC current telemetry wrong

PX4 Autopilot
1

Hello,

Let me know if I am missing something, but we recently upgraded from PX4 v1.15.4 to PX4 v1.16.0 and noticed that the battery status was being read incorrectly, lower than usual, we have 4 Dshot ESCs that we are using for current telemetry.

Looks like this line of code is the problem? Blaming PX4-Autopilot/src/modules/esc_battery/EscBattery.cpp at main · PX4/PX4-Autopilot · GitHub

I think that current should not be divided by ESC count, since it is a total, whereas voltage and temp are correct to be divided by ESC count.

We can multiply everything by 4 in post flight analysis to get the true current but I think it is in the “spirit” of the BatteryStatus message to have the total, not average current?

Thanks

2

Thanks for the report. Let me ping @dakejahl to check this.

3

Yeah that looks wrong. PR to fix here esc_battery: fix current reporting by dakejahl · Pull Request #26099 · PX4/PX4-Autopilot · GitHub

1 Like
4

This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.