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

QuadPlane unable to transition #6627

Closed
expntly opened this issue Jul 19, 2017 · 8 comments
Closed

QuadPlane unable to transition #6627

expntly opened this issue Jul 19, 2017 · 8 comments
Assignees
Milestone

Comments

@expntly
Copy link
Contributor

expntly commented Jul 19, 2017

Issue details

Just upgraded to a 3.8.0-beta6 firmware. I'm trying to transition but the vehicle won't gain enough speed. Worked OK before.

This is an AUTO mission (VTOL_TAKEOFF, WP, WP ...):

screen shot 2017-07-18 at 5 44 57 pm

Not using AirSpeed sensor for this test, flying GPS.

ARSPD_FBW_MIN set to 14m/s. TRIM_ARSPD_CM 22m/s.

Version

3.8.0-beta6

Platform

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

Airframe type

X8 OctaQuadPlane

Hardware type

Pixhawk1

Logs

https://www.dropbox.com/s/262i7vys5pob22h/00000025.BIN?dl=0

@expntly expntly changed the title QuadPlane unable to transition QuadPlane 3.8.0-beta6 unable to transition Jul 19, 2017
@OXINARF OXINARF added this to the Plane v3.8.0 milestone Jul 19, 2017
@expntly expntly changed the title QuadPlane 3.8.0-beta6 unable to transition QuadPlane unable to transition Jul 19, 2017
@expntly
Copy link
Contributor Author

expntly commented Jul 19, 2017

Just tested release 3.7.1 and the same thing happened, so I'm removing 3.8.0-beta6 from the title.

There seems to be another conflict somewhere though. I recently played with Q_WP* parameters but these should be related to strict VTOL navigation only?

2017-07-19 16:55:35.73: PARM {TimeUS : 1304516126, Name : Q_WP_SPEED, Value : 250.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516188, Name : Q_WP_RADIUS, Value : 200.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516247, Name : Q_WP_SPEED_UP, Value : 170.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516413, Name : Q_WP_SPEED_DN, Value : 75.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516473, Name : Q_WP_LOIT_SPEED, Value : 250.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516697, Name : Q_WP_ACCEL, Value : 100.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516761, Name : Q_WP_ACCEL_Z, Value : 100.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304516865, Name : Q_WP_LOIT_JERK, Value : 1000.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304517045, Name : Q_WP_LOIT_MAXA, Value : 100.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304517108, Name : Q_WP_LOIT_MINA, Value : 25.0}
2017-07-19 16:55:35.73: PARM {TimeUS : 1304517194, Name : Q_WP_RFND_USE, Value : 1.0}

@expntly
Copy link
Contributor Author

expntly commented Jul 19, 2017

@peterbarker recommended to look at MSG entries in the log to see if the transition starts, and I'm not seeing anything. I added a D_VTOL_TRANSITION=3 right after VTOL_TAKEOFF just to force the transition, which doesn't seem to help.

2017-07-19 16:55:35.95: MSG {TimeUS : 1304731580, Message : ArduPlane V3.7.1 (22b5c41)}
2017-07-19 16:55:35.95: MSG {TimeUS : 1304731594, Message : PX4: c4dfb9bf NuttX: 258f2257}
2017-07-19 16:55:35.95: MSG {TimeUS : 1304731639, Message : PX4v2 003B0044 34355104 38323831}
2017-07-19 16:55:35.95: MSG {TimeUS : 1304731654, Message : New mission}
2017-07-19 16:55:42.03: MSG {TimeUS : 1310819901, Message : Throttle disarmed}
2017-07-19 16:55:56.35: MSG {TimeUS : 1325138377, Message : Throttle armed}
2017-07-19 16:56:40.23: MSG {TimeUS : 1369013482, Message : Executing nav command ID #84}
2017-07-19 16:57:16.68: MSG {TimeUS : 1405466070, Message : Executing command ID #3000}
2017-07-19 16:57:16.68: MSG {TimeUS : 1405466432, Message : Executing nav command ID #16}
2017-07-19 17:01:04.79: MSG {TimeUS : 1633577263, Message : Throttle disarmed}

@tridge
Copy link
Contributor

tridge commented Jul 31, 2017

I've reproduced this issue in SITL and I'm working on a fix. Thanks for the bug report!

@tridge tridge added the BUG label Jul 31, 2017
@tridge tridge self-assigned this Jul 31, 2017
@magicrub
Copy link
Contributor

partially fixed by this merged PR: #6700

@expntly
Copy link
Contributor Author

expntly commented Aug 13, 2017

I'm taking it this made it into the 3.8.0 release? If yes I'll close.

@tridge
Copy link
Contributor

tridge commented Sep 6, 2017

I'd rather this is left open. The problem is reduced in 3.8, but not eliminated completely. It still needs a bit more work.

@tridge
Copy link
Contributor

tridge commented Oct 21, 2017

I've finally fixed this properly in #7102

@OXINARF
Copy link
Member

OXINARF commented Oct 22, 2017

#7102 is now merged.

@OXINARF OXINARF closed this as completed Oct 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants