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

Autogyro takeoff mode #32

Open
roman-dvorak opened this issue Jan 12, 2021 · 4 comments
Open

Autogyro takeoff mode #32

roman-dvorak opened this issue Jan 12, 2021 · 4 comments
Assignees
Labels
enhancement New feature or request stale unmerged2upstream Issue resolved but not merged in the upstream
Milestone

Comments

@roman-dvorak
Copy link
Member

roman-dvorak commented Jan 12, 2021

After our ThunderFly devcall with @kaklik and @slimonslimon we agreed on the following implementation of takeoff mode for the autogyro.

Why:

  • FW takeoff (from runway) sequence is different from the autogyro sequence
  • We want takeoff from some moving platform (car roof or running man)

How to do takeoff with an autogyro

  1. At the beginning, it is assumed that someone pre-rotates the rotor to the minimum speed for the prerotator. These conditions will be set in some parameter
  2. Then it starts prerotator and it is waiting for obtaining flight RPM of rotor. (Prerotation can be done by some prerotator driver or with forwarding movement). Rotor (pitch) is at some maximal setpoint value.
  3. Now, autogyro is in flight parameters. Until this point, integrators are inactive. Now, the release command is transmitted (it can be mavlink command or some sound signal for running man)
  4. Stage of climbing, it is similar to FW. Controlled by TECS with limited setpoints range.

Implementation

Because VTOL autogyro is not assumed (VTOL in the sense that vertical and horizontal flight is controlled by different actuators, as VTOL plane). We will create a new controller, which will be based on the current FW runway takeoff driver.

Programming will take place in our out-of-tree repository and we will build it above the current master version of the firmware. Then I merge it with the upstream/master and I will create PR.

Testing:

Because it is better to perform the first tests in the simulator, @kaklik will prepare a modification of the TF-G2 autogyro model it will contain dummy (very heavy) weights that is possible to switch off with some keyboard command. It will simulate takeoff from running man. (more details in own issue)

Then we try to perform some real test flights from a hand or from the car roof. Here we are missing a piece of SW and HW, which will disconnect the autogyro from the roof by a command from the mavlink message.

@roman-dvorak roman-dvorak self-assigned this Jan 12, 2021
@roman-dvorak roman-dvorak added the enhancement New feature or request label Jan 12, 2021
@kaklik
Copy link
Member

kaklik commented Feb 2, 2021

The issue has been almost solved. There is needed only protection of rotor overspeed which could happen in the case where the autogyro does not been released from platform promptly and platform speed increases.

The protective algorithm should lean forward the rotor and interrupt the takeoff if applicable.

@kaklik
Copy link
Member

kaklik commented Jan 2, 2022

The flight-tested performance of the current implementation of the takeoff procedure is perfect! It could also be documented by several flight logs and video recordings.
Therefore merging that to upstream is mandatory, until there are no changes that affect the functionality. :)

@kaklik
Copy link
Member

kaklik commented Jan 2, 2022

I am closing this issue, as the main discussion already moved to PX4 upstream PR: PX4#18582

@kaklik kaklik closed this as completed Jan 2, 2022
@kaklik
Copy link
Member

kaklik commented Jan 13, 2022

Nějak se zadrho přijetí pull-requestu, takže to zřejmě bude potřebovat ještě nějakou péči.

@kaklik kaklik reopened this Jan 13, 2022
@kaklik kaklik added the unmerged2upstream Issue resolved but not merged in the upstream label Jul 10, 2023
@github-actions github-actions bot added the stale label Aug 10, 2023
@roman-dvorak roman-dvorak added this to the next_test milestone Aug 18, 2023
@github-actions github-actions bot removed the stale label Aug 18, 2023
@github-actions github-actions bot added the stale label Sep 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stale unmerged2upstream Issue resolved but not merged in the upstream
Projects
None yet
Development

No branches or pull requests

2 participants