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

Add generic vtol tailsitter airframe and modifiy 4001_quad_x and 13001_caipirinha_vtol to simplify PR 9849 #12506

Merged
merged 1 commit into from
Sep 1, 2019

Conversation

mcsauder
Copy link
Contributor

Describe problem solved by the proposed pull request
This PR is designed to advance PR #9849. A generic VTOL tailsitter airframe and mixer is added and the modifications to 4001_quad_x and 13001_caipirinha_vtol from PR #9849 are made here.

Additional context
This PR is a step toward accomplishing the work in #9489 in a piecewise process. The full diff for the completed work can be seen here.

Please let me know if you have any questions on this PR! Thanks!

-Mark

@mcsauder mcsauder requested review from dagar and RomanBapst July 17, 2019 16:44
@mcsauder mcsauder force-pushed the airframe_cleanup_1 branch from bc6b40e to 15f8209 Compare July 19, 2019 19:18
@mcsauder mcsauder force-pushed the airframe_cleanup_1 branch 2 times, most recently from 2fc5ce4 to 029db09 Compare July 25, 2019 22:51
@mcsauder mcsauder force-pushed the airframe_cleanup_1 branch 3 times, most recently from 3e57f5b to b8a3aa4 Compare August 2, 2019 17:58
@mcsauder mcsauder force-pushed the airframe_cleanup_1 branch from b8a3aa4 to 527e9ce Compare August 7, 2019 16:53
@mcsauder
Copy link
Contributor Author

mcsauder commented Aug 7, 2019

@RomanBapst, I rebased on current master, let me know if you need anything from me on this PR. Thanks!

@mcsauder
Copy link
Contributor Author

@RomanBapst and @dagar , rebased against current master. Do you need anything else from me on this PR?

RomanBapst
RomanBapst previously approved these changes Aug 18, 2019
Copy link
Contributor

@RomanBapst RomanBapst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mcsauder Sorry for holding you up so long. Good job!

@mcsauder
Copy link
Contributor Author

Thanks for your review @RomanBapst!

@mcsauder
Copy link
Contributor Author

Hi @dagar and @RomanBapst , anything else you'd like to see in this PR from me?

…ons from PR 9849 in 4001_quad_x and 13001_caipirinha_vtol.
@mcsauder
Copy link
Contributor Author

Rebased on current master to see if we can make CI happy again.

@dagar dagar merged commit 0817ee4 into PX4:master Sep 1, 2019
@mcsauder
Copy link
Contributor Author

mcsauder commented Sep 1, 2019

Thanks @dagar !

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.

3 participants