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

Bug: Full throttle despite not being armed in bidirectional dshot300 mode. #734

Open
maxikrie opened this issue Apr 18, 2024 · 2 comments
Open

Comments

@maxikrie
Copy link

Hi,

Today we had a fairly scary incident where one motor went full throttle despite not being armed. We were testing bidirectional dshot300 mode and forgot to invert the checksum being sent to the ESC (Tekko32 F4 45A ESC). Hence the ESC wasn't armed, which we confirmed as the motor wouldn't make the 'armed' sound. When we set the throttle for testing to 150, one motor out of six suddenly went full throttle.

We couldn't reproduce this, hence we are suspecting a weird coincidence. However, we are quite worried that the motor could spin without being armed.

This issue from some time ago seems very similar - #353.

Thanks!

@sskaug
Copy link
Collaborator

sskaug commented Apr 19, 2024

Sorry to hear about your scary experience. And glad that there was no injury. We did quite a few changes to Rev32.10 to address unintentional spinups, and have not heard many reports since. But unfortunately fails do occur, it is impossible to prevent completely.

@maxikrie
Copy link
Author

maxikrie commented Apr 19, 2024

Thank you for this quick and honest answer.

However, respectfully, I must disagree. I cannot think of a reason why this would be impossible to prevent completely other than a bug in either the state machine or bad memory access. Clearly, the function spinning the motor should turn off the motor safely if the ESC is not armed and nothing but the arming command should be able to arm the ESC.

We have decided to move away from BLHeli for our application due the fact that the code is closed source and you are confirming that the issue, which even was reported several years ago, may occasionally still occur.

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

2 participants