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

tail sitter vtol support #9347

Merged
merged 12 commits into from
Dec 12, 2023

Conversation

shota3527
Copy link
Contributor

@shota3527 shota3527 commented Oct 6, 2023

More test needed
Still thinking about whether to use a PLATFORM_TAILSITTER or set tailsitter = on ,leaning towards tailsitter = on for less changes and more flexible setup

https://www.youtube.com/watch?v=V4Y81Pj3vs0&t=2s

@shota3527 shota3527 marked this pull request as draft October 6, 2023 02:03
@shota3527
Copy link
Contributor Author

shota3527 commented Oct 8, 2023

fc only bench test successful. next step is to confirm compass is also working as expected
Do anyone has a tail sitter to test it?

@shota3527 shota3527 mentioned this pull request Oct 8, 2023
@FlyChenPiPi
Copy link

Great, I want to give it a try!

@shota3527 shota3527 marked this pull request as ready for review October 20, 2023 00:19
@shota3527
Copy link
Contributor Author

@DzikuVx DzikuVx added this to the 7.1 milestone Oct 25, 2023
@DzikuVx DzikuVx changed the base branch from master to release_7.1.0 December 12, 2023 13:22
@DzikuVx DzikuVx merged commit 6749f7b into iNavFlight:release_7.1.0 Dec 12, 2023
15 checks passed
@bsqg64
Copy link

bsqg64 commented Dec 31, 2023

Hi,
Thanks for this feature, I would like to test it on my plane (Atom RC penguin with speedybee F405 wing).
@shota3527 would you share your inav settings for your penguin ?
Cheers

@zoran-grujic
Copy link

zoran-grujic commented Jan 4, 2024

More test needed Still thinking about whether to use a PLATFORM_TAILSITTER or set tailsitter = on ,leaning towards tailsitter = on for less changes and more flexible setup

https://www.youtube.com/watch?v=V4Y81Pj3vs0&t=2s

I think that PLATFORM_TAILSITTER is wrong as one might mix different "platforms" into a VTOL. And those "platforms" already exist. For me tailsitter is mix of two platforms, and we just switch in between.

I am also against tailsitter = on. My suggestion is to introduce additional variables that will give full flexibility, something like:

VTOL_MIXER1_PITCH = 0 - a pitch in respect to the board alignment in deg (might be forced to 0 to be used in general)
VTOL_MIXER2_PITCH = 90 - a pitch in respect to the board alignment
VTOL_TRANSITION_PITCH = 45 - a pitch in respect to the board alignment during transition phase

One can also force MIXER1 (or MIXER2 whatever seems more convinient) to have the alignment parallel to the board, thus avoiding possible confusion and over defining the problem.

I think that INAV would allow for full range of VTOLs, unlimited to tailsitters, with:

VTOL_MIXER2_PITCH = 90 - a pitch in respect to the board alignment in deg
VTOL_TRANSITION_PITCH = 45 - a pitch in respect to the board alignment during transition phase,

Nobody knows what people will invent in future. I suggest introduction of maximal flexibility already now.

I built a plank wing tailsitter and vertical flight works with 7.0. I will set transition into ACRO mode for horizontal flight in wing platform until 7.1 gets out.

@zoran-grujic
Copy link

fc only bench test successful. next step is to confirm compass is also working as expected Do anyone has a tail sitter to test it?

I would be happy to test. At the moment I do not have GPS nor compass. And weather is getting bad in my part of Europe... My board is "SpeedyBee F405 WING APP"

@erik23de
Copy link

Can anyone please post a tailsitter diff somewhere, as a part of documentation? It's cool to see videos that Inav 7.1 can do that, but how? is an open question. The docs I read so far do not cover how to do that bicopter settings...
Thank you very much!

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.

6 participants