Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing/ empty/ extra ESC telemetry log data #28807

Open
MarbleJames opened this issue Dec 5, 2024 · 26 comments
Open

Missing/ empty/ extra ESC telemetry log data #28807

MarbleJames opened this issue Dec 5, 2024 · 26 comments

Comments

@MarbleJames
Copy link

MarbleJames commented Dec 5, 2024

Bug report

Issue details
I have a Quadplane with 5 esc’s connected. Since updating from 4.4.4, extra empty ESC’s (up to 8 in total) have been appearing in logs and the cruise ESC data is often empty.

On occasion data for 5 esc's has been observed in mission planner but only data for 4 has been logged.

Just verified to be a firmware issue as follows;

Flashed back to 4.4.4 - 5 esc’s correct data
Flashed 4.6.0 - 8 esc’s 4 with data 4 without
Flashed to 4.4.4 - 5 esc’s correct again.
I haven’t tried all the firmware releases in-between but 4.5.4 and 4.5.0 have also been shown to not work.

(https://discuss.ardupilot.org/t/esc-telemetry-phantom-escs/127106)

Version
Post 4.4.4

Platform
[ ] All
[ ] AntennaTracker
[ ] Copter
[ X ] Plane
[ ] Rover
[ ] Submarine

Airframe type
4+1 quadplane

Hardware type
Cube orange plus bdshot, 4x apd80, 1x apd120

Logs
Two logs included, both same params one 4.4.4 and one 4.6.0
ESC telem logs.zip
https://drive.google.com/drive/folders/17_GhgOKQxd0-tNSXUfivTVYeqpDNg5mY?usp=sharing

@andyp1per
Copy link
Collaborator

This is a change in 4.5 to support dshot on IOMCU. ESC numbers are now aligned with servo numbers.

@MarbleJames
Copy link
Author

This is a change in 4.5 to support dshot on IOMCU. ESC numbers are now aligned with servo numbers.

Thanks for the reply, I don’t understand how that affects this though, as far as I understand IOMCU is only the first 8 channels of cube orange? I’m running esc’s on 9-13. Anywhere i can read more on this.

I can deal with empty ESC’s in logs but surely this shouldn’t stop recording the telem stream for the cruise ESC? (especially when i'm getting live telemetry for the 5th in mission planner)

v4.4.4
RC9 - ESC[0]
RC10 - ESC[1]
RC11 - ESC[2]
RC12 - ESC[3]
RC13 - ESC[4]

v4.6.0
RC9 - ESC[1]
RC10 - ESC[2]
RC11 - ESC[3]
RC12 - ESC[4]
RC13 - No telem
? ESC[8]
? ESC[9]
? ESC[10]
? ESC[11]

@andyp1per
Copy link
Collaborator

It should be:

v4.6.0
RC9 - ESC[8]
RC10 - ESC[9]
RC11 - ESC[10]
RC12 - ESC[11]

You shouldn't be getting logging from ESC[0]-ESC[3] if you are not using RC1-4

@andyp1per andyp1per reopened this Dec 9, 2024
@andyp1per
Copy link
Collaborator

There may be a logging issue here with S1-S8 if they are not being used

@MarbleJames
Copy link
Author

There may be a logging issue here with S1-S8 if they are not being used

Just to be clear, RC1-8 are being used for normal PWM servo outputs - so yes shouldn't be producing ESC telem data

@rmackay9
Copy link
Contributor

rmackay9 commented Dec 9, 2024

I've added this to the 4.6 issues list although it sounds like the issue is in 4.5 as well which means it's not a blocker for the release (but it would probably still be good to resolve if we can)

@andyp1per
Copy link
Collaborator

I actually can't reproduce this with your setting. I found a bug - #28833 but I think EDTv2 is only is 4.6 so don't think that can be your issue. Also CubeOrange doesn't support bdshot, so there can by no telemetry coming from IOMCU. So I am rather at a loss to say what is going on here. I correctly get a log with telemetry on 9-12 only.

@andyp1per
Copy link
Collaborator

If you are able to reproduce this on a cut down config that would help me narrow down.

@MarbleJames
Copy link
Author

Ok made a simple setup, to test. 4x apd f80 connected to 9-12 and one f120 to rc13, telemetry connected to serial 3 -nothing else connected. Using a cube orange plus on airbot mini carrier - i thought bshot was supported if running bdshot fw (which i am)?

Getting the same behaviour (logs in the drive link above).
4.4.4 - ESC[0],[1],[2],[3],[4] populated
4.6.0 - only logging 4 ESC's again as shown in photo but seeing 5 in mission planner
image
image

@andyp1per
Copy link
Collaborator

i thought bshot was supported if running bdshot fw

Sorry, yes - just not supported on the main channels - only aux

@andyp1per
Copy link
Collaborator

Ok made a simple setup, to test. 4x apd f80 connected to 9-12 and one f120 to rc13, telemetry connected to serial 3 -nothing else connected. Using a cube orange plus on airbot mini carrier - i thought bshot was supported if running bdshot fw (which i am)?

Getting the same behaviour (logs in the drive link above). 4.4.4 - ESC[0],[1],[2],[3],[4] populated 4.6.0 - only logging 4 ESC's again as shown in photo but seeing 5 in mission planne

Can you post your parameters for the test? How did you test?

@MarbleJames
Copy link
Author

Params for both FW's in the drive link, couple of small differences when comparing (nothing to do with dshot/serial etc) presumably just fw version related.
Testing I'm just checking I'm getting telem from all the esc's in mission planner then enabling log disarmed, for a while, disable and then download logs. I Get the same behaviour in flight etc this just keeps the files a bit smaller.

@andyp1per
Copy link
Collaborator

Params for both FW's in the drive link, couple of small differences when comparing (nothing to do with dshot/serial etc) presumably just fw version related. Testing I'm just checking I'm getting telem from all the esc's in mission planner then enabling log disarmed, for a while, disable and then download logs. I Get the same behaviour in flight etc this just keeps the files a bit smaller.

Are you arming? Running motor test?

@MarbleJames
Copy link
Author

MarbleJames commented Dec 10, 2024

Params for both FW's in the drive link, couple of small differences when comparing (nothing to do with dshot/serial etc) presumably just fw version related. Testing I'm just checking I'm getting telem from all the esc's in mission planner then enabling log disarmed, for a while, disable and then download logs. I Get the same behaviour in flight etc this just keeps the files a bit smaller.

Are you arming? Running motor test?

No have just been looking at temp/ voltage fields which are available all the time so no need to arm. Getting the same behaviour with these log disarmed tests as was in flight.

@andyp1per
Copy link
Collaborator

No have just been looking at temp/ voltage fields which are available all the time so no need to arm. Getting the same behaviour with these log disarmed tests as was in flight.

Oh wait, when you say "serial" you mean that's how you are getting the ESC telemetry?

@MarbleJames
Copy link
Author

No have just been looking at temp/ voltage fields which are available all the time so no need to arm. Getting the same behaviour with these log disarmed tests as was in flight.

Oh wait, when you say "serial" you mean that's how you are getting the ESC telemetry?

Yes getting esc telemetry over serial. (protocol 16)

@andyp1per
Copy link
Collaborator

Still can't reproduce. Can you try 4.6beta1 I think it's likely it has been fixed, just don't know by what.

@andyp1per
Copy link
Collaborator

sorry, try 4.7

@andyp1per
Copy link
Collaborator

It's also possible this is an issue with APD I suppose

@MarbleJames
Copy link
Author

Flashed to the latest 4.7.0 build for CubeOrangePlus-bdshot and still seeing the same issue (log on the drive)

I have spoken to APD and they think its an AP issue, seems the most likely to be based on the fact its a AP firmware upgrade that breaks the telemetry stream.

image

@andyp1per
Copy link
Collaborator

What are you graphing with?

@MarbleJames
Copy link
Author

UAV log viewer

@andyp1per
Copy link
Collaborator

This is what I get in my test with 4.7
log4.zip

@andyp1per
Copy link
Collaborator

Please can you try with the firmware I am using:
arduplane.zip

@MarbleJames
Copy link
Author

What are you graphing with?

Just tried viewing the same log as plotted above in the log browser built into mission planner and i'm getting logs for all 5 esc's there, so there might be a separate issue here as well.

This doesn't solve the empty ESC issue though.
image

@MarbleJames
Copy link
Author

Please can you try with the firmware I am using: arduplane.zip

No change with your FW
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants