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

Enable flaps for plane model in SITL #14788

Merged
merged 1 commit into from
May 2, 2020
Merged

Enable flaps for plane model in SITL #14788

merged 1 commit into from
May 2, 2020

Conversation

Jaeyoung-Lim
Copy link
Member

@Jaeyoung-Lim Jaeyoung-Lim commented Apr 29, 2020

Describe problem solved by this pull request
Previously, the plane model didn't have a flap although flaps were defined in the SITL mixers. This PR adds the ability to simulate flaps in the plane model. This makes the plane model more or less complete, since all actuator controls are actually consumed by the simulated model

Describe your solution
To enable this, the lift drag elements on the wings are split into two, where one simulates the aileron and one simulates the flaps. The center of pressure is placed separately along the wing near the center of the control surface.

Since it is not possible to control two joints through a single channel in sitl_gazebo, a channel was added to the mixer file.

The meshfile comes from https://github.com/osrf/gazebo_models

Test data / coverage
Log: https://review.px4.io/plot_app?log=f5655069-3579-447b-8e9d-e158d1e7e0f8

ezgif com-video-to-gif(12)

Additional context
Flaps added on PX4/PX4-SITL_gazebo-classic#476

Since sitl gazebo is unable to control to joints from a single channel, an additional mixer is defined
@Jaeyoung-Lim Jaeyoung-Lim changed the title Add additional flap channel in mixer Enable flaps for plane model in SITL Apr 29, 2020
@dagar dagar merged commit cd59f95 into master May 2, 2020
@dagar dagar deleted the pr-plane-flaps branch May 2, 2020 01:01
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

Successfully merging this pull request may close these issues.

2 participants