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

Test multirotor control without mixer limits #4995

Closed
LorenzMeier opened this issue Jul 6, 2016 · 5 comments
Closed

Test multirotor control without mixer limits #4995

LorenzMeier opened this issue Jul 6, 2016 · 5 comments
Assignees

Comments

@LorenzMeier
Copy link
Member

@MaEtUgR Medium term interested in LQR / MPC, but as first step I'd like to work out if the mixer limiting is negatively impacting performance.

@LorenzMeier LorenzMeier added this to the Release v1.5.0 milestone Jul 6, 2016
@LorenzMeier LorenzMeier modified the milestones: Release v1.6.0, Release v1.5.0 Nov 19, 2016
@LorenzMeier
Copy link
Member Author

LorenzMeier commented Dec 28, 2016

@MaEtUgR Found this paper by chance today:
https://www.ifi.uzh.ch/dam/jcr:5f3668fe-1d4e-4c2b-a190-8f5608f40cf3/RAL16_Faessler.pdf

It compares a naive mixer with its implementation. Its not clear how much different their work is to what we already have, but its possible there are things left to improve in our not-completely-naive mixer.

@kd0aij
Copy link
Contributor

kd0aij commented Dec 28, 2016

@Tumbili The paper referenced above also has a discussion of motor thrust/torque modeling which may be of interest to you. They chose to use all 3 coefficients to fit a quadratic polynomial to the measured thrust.

@MaEtUgR
Copy link
Member

MaEtUgR commented Dec 28, 2016

@LorenzMeier Thank you for the reference. We talked about the mixer in a control call and we have an idea how to improve the mixer. Improving the rate controller will lead sooner or later to the mixer priorities and I'll consider the results of the paper for this.

@LorenzMeier LorenzMeier modified the milestones: Release v2.1.0, Release v1.6.0 Apr 10, 2017
@PX4BuildBot
Copy link
Collaborator

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 30 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

@MaEtUgR
Copy link
Member

MaEtUgR commented Feb 21, 2018

There's actually progress: #7920
And I just discussed with @bkueng that I could still add functionality on top like yaw priorization from the draft mixer I pushed here before: #8485

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

No branches or pull requests

6 participants