bird-sanctuary / bluejay

:bird: Digital ESC firmware for controlling brushless motors in multirotors
GNU General Public License v3.0
301 stars 32 forks source link

Flag_Ext_Tele can only be cleared in exit_run_mode when motors are not being reversed (turtle mode) #178

Closed damosvil closed 8 months ago

damosvil commented 10 months ago

Flag_Ext_Tele can only be cleared in exit_run_mode when motors are not being reversed (turtle mode) That step is done in the lowest part of exit_run_mode

hyp0dermik-code commented 10 months ago

Initial tests show this fixes the turtle bug but not the second activation in the motors tab.

I would like to do some further investigation into how the these two circumstances output motor signal, and whether the the signal is not sent when the slider is at 0, in which case EDT safety disarm would be expected to trip before proceeding.

I think .20.1 could be released without this merged as the current behaviour is safe. Worst case scenario the quad needs to be rearmed to reactivate turtle mode.

damosvil commented 10 months ago

Every time you need to arm the motors you will need to enable the "I understand the risk" switch button in order enable edt dshot command 13 is sent again, otherwise motor won't arm. That is normal If you don't send command 13 again the esc will be kept unarmed until you send it again. The only way the esc is kept armed is when motors are inverted by dshot command 21.

You can test it in the cli with the commands motor and dshotprog: dshotprog 0 13 dshotprog 0 21 motor 0 1100 motor 0 1000 motor 0 1100 motor 0 1000 motor 0 1100 motor 0 1000 motor 0 1100 motor 0 1000 dshotprog 0 20 motor 0 1100 motor 0 1000 motor 0 1100 motor 0 1000

damosvil commented 10 months ago

pz check behavior here: https://youtu.be/gdARJK0sleY

Bluejay_0.20.1_RC3.zip

When edt is enabled Betaflight sends command 13 when arming the kwad, so in that case It is like clicking "I understand the risk" button

Noctaro commented 10 months ago

Hi, i just tested RC3 on A-X-5 target, betaflight 4.5 and Configurator 10.10.0-debugf277b27. I am still not able to run motors via motors tab. They just keep beeping.

damosvil commented 10 months ago

@Noctaro Does it happen the same using 4.4.0?

Noctaro commented 10 months ago

@damosvil Did flash back to 4.4.0, now it works as expected :)

Update: I also checked 4.4.3, here it does not work either.

hyp0dermik-code commented 10 months ago

@damosvil Did flash back to 4.4.0, now it works as expected :)

Update: I also checked 4.4.3, here it does not work either.

Can you confirm your behaviour is the same as the video above (https://youtu.be/gdARJK0sleY)

Please note on the first arm, increase value through noises until motor starts spinning (should be spining by 1050 worst case).

You can hover over the motor and use the arrow keys to slowly increase it

Noctaro commented 10 months ago

@hyp0dermik-code yes i can confirm the same behaviour. Motors start running one time, then beep if i try to restart them until i reengage the "i know what i am doing" switch. Motors start running at 1010

damosvil commented 9 months ago

@Noctaro that is the normal behaviour. I have tested It in my kwad with bf4.4.0 some days ago. It works as intended after turtling. Please could you try on your side?

Noctaro commented 9 months ago

@damosvil I have tested follwing things without powering down my copter on 4.4.0:

Everything worked as it should. Also Turtlemode does not need the rearming procedure, it works as intended. Thank you for fixing :)