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

tailsitter vtol frnot/back transmission failed (no pitch set value is generated) #15037

Closed
xdwgood opened this issue Jun 5, 2020 · 4 comments · Fixed by #15352
Closed

tailsitter vtol frnot/back transmission failed (no pitch set value is generated) #15037

xdwgood opened this issue Jun 5, 2020 · 4 comments · Fixed by #15352
Assignees
Labels
bug Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing!

Comments

@xdwgood
Copy link
Contributor

xdwgood commented Jun 5, 2020

Describe the bug
When I used tailsitter to fly, I found that sometimes(Sometimes it can enter successfully, in the same flight) even if I switch, vtoll can not enter the fw mode

To Reproduce

  1. use master branch fly(tailsitter vtol)
  2. in altitude mode and try transmission(mc to fw)
  3. The drone pitch angle has not changed, there is no tilt acceleration

Expected behavior
Every time a customer uses a stick for transmission, vtol should enter the transmission state correctly

Log Files and Screenshots
https://logs.px4.io/plot_app?log=4df87c34-da2f-46be-b612-ddcc13b135c8

840347981

Additional context
I found that fw to mc sometimes fails when I try to send a transfer command using the remote control. I will do some investigations to see where the problem is. I now suspect

numerical calculations under certain extreme conditions? (eg:at the start of transmission, the actual/setpoint angle of pitch are both 0)

here: https://github.com/PX4/Firmware/blob/master/src/modules/vtol_att_control/tailsitter.cpp#L206-L218

Because I compared the two switches, the only difference I found was that the drone angle/setpoint angle was close to 0 before the failed transmission started

FYI: @sfuhrer 😃

@xdwgood xdwgood changed the title tailsitter vtol forward transmission failed (no pitch set value is generated) tailsitter vtol frnot/back transmission failed (no pitch set value is generated) Jun 9, 2020
@dagar dagar added bug Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing! labels Jun 9, 2020
@sfuhrer
Copy link
Contributor

sfuhrer commented Jun 9, 2020

@xdwgood Did I understand correct, that the issue is this spike in the transition switch log?

Edit: having had a second look it's now clear what you mean 👍 I'll have a look

@xdwgood
Copy link
Contributor Author

xdwgood commented Jul 16, 2020

@sfuhrer
https://github.com/PX4/Firmware/blob/master/src/modules/vtol_att_control/tailsitter.cpp#L216

The calculated tilt sometimes has problems. I checked it using the PX4_ISFINITE(tilt) function and found this problem.
Screenshot from 2020-07-16 20-06-54

Corresponding code

2112

@xdwgood
Copy link
Contributor Author

xdwgood commented Jul 16, 2020

No setpoint value is generated when the value check fails(8)

@xdwgood
Copy link
Contributor Author

xdwgood commented Jul 17, 2020

fix here: #15352!

@xdwgood xdwgood closed this as completed Jul 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Hybrid VTOL 🛩️🚁 Multirotor + Fixedwing!
Projects
None yet
4 participants