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

Holybro S500 Airframe #12322

Merged
merged 3 commits into from
Jun 24, 2019
Merged

Holybro S500 Airframe #12322

merged 3 commits into from
Jun 24, 2019

Conversation

MaEtUgR
Copy link
Member

@MaEtUgR MaEtUgR commented Jun 24, 2019

Describe problem solved by the proposed pull request
Thanks to feedback from Holybro we realized that on their S500 Kit the default parameters don't result in a user-friendly tuning and we created a profile for users using that exact kit. Furthermore we'll check the default gains on 450-500 size quadcopters to see if they need to be adjusted for best first flight experience.

Additionally the default rate constrol integral gain for roll and pitch was increased because @bkueang and me realized that on every frame we tune it's much too low. Usually it needs to be increased to 0.3 or even 0.4 to have better "locked in" flight performance and 0.2 seems like a good compromise for a safe default.

Test data / coverage
The tuning of the Holybro S500 airframe was tested on site on a brand new assembled kit (with calibrated ESCs). The team from Holybro that was present at the PX4 Dev Summit was able to test it.

MaEtUgR added 3 commits June 24, 2019 12:57
@bkueang and me realized that on every frame we tune the integral gain for
the roll and pitch rate controller is much too low. Usually it needs to be
increased to 0.3 or even 0.4 to have better "locked in" flight performance
and 0.2 seems like a good compromise for a safe default.
@bkueng bkueng requested a review from a team June 24, 2019 15:59
@LorenzMeier LorenzMeier merged commit 5002b13 into master Jun 24, 2019
@LorenzMeier LorenzMeier deleted the holybro-s500-airframe branch June 24, 2019 16:26
@dagar dagar mentioned this pull request Jun 24, 2019
2 tasks
@mrpollo
Copy link
Contributor

mrpollo commented Jun 25, 2019

👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants