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

[airframes] use standard motor mixing types for some quads #1231

Merged
merged 2 commits into from
Jul 21, 2015

Conversation

flixr
Copy link
Member

@flixr flixr commented Jul 8, 2015

This will change the effect of the gains (depending on the type), but should be finally correct.
Especially the yaw commands have about half the effect now...

Needs testing on ardrone and bebop, and probably some adjustment of the gains.

@flixr flixr added this to the v5.6 milestone Jul 14, 2015
@flixr
Copy link
Member Author

flixr commented Jul 15, 2015

@gautierhattenberger @kevindehecker @fvantienen @EwoudSmeur
Would be great if you guys could test the updated ardrone and bebop airframe files.
Btw, I was wondering anyhow that none of those have an integral term set...

Also the ardrone2_optitrack.xml (which I didn't change here) has different gains, which I don't quite understand as it is the same controller and airframe.

This will change the effect of the gains (depending on the type), but should be finally correct.
Especially the yaw commands have about half the effect now...
After changing the motor mixing to the predefined QUAD_X type, multiply the roll/pitch gains by sqrt(2) and yaw by 2.
This should result in the same effectiveness of the gains as before.
@gautierhattenberger
Copy link
Member

Tested and it seems to be working as expected

@flixr
Copy link
Member Author

flixr commented Jul 21, 2015

You tested both ardrone2 and bebop?
The behaviour (with these updated gains and mixing) seems to be the same as before?

@kevindehecker
Copy link
Contributor

I tested ARDrone2 in ATT and NAV mode with the indi airframe file, which seems to work.

@flixr
Copy link
Member Author

flixr commented Jul 21, 2015

The question is if the updated gains match the new motor mixing scaling.
Since INDI does not use these gains, it needs to be tested with the "classic" controllers....

Also I'm still curious on why neither ardrone2 nor bebop have any igains set...
Neither why the optitrack airframe uses different gains...

@gautierhattenberger
Copy link
Member

I tested both, it looks the same. I just don't like some of the parameters for the ardrone2 (max yaw rate, gains a bit higher than what I'm used to). Otherwise it is flyable like this. Only did ATT so far.

@flixr flixr merged commit 6ada818 into master Jul 21, 2015
@flixr flixr deleted the default_quad_x_mixing branch July 21, 2015 20:11
@EwoudSmeur
Copy link
Member

It does also matter for the INDI controller, as the actuator effectiveness is sqrt(2) lower... Though Kevin was not flying with INDI but with quat_int

@flixr
Copy link
Member Author

flixr commented Jul 22, 2015

Yes, but the indi airframe was not changed in this PR... And the question
was specifically if the updated gains match the mixing...
Am 22.07.2015 11:06 schrieb "Ewoud Smeur" notifications@github.com:

It does also matter for the INDI controller, as the actuator effectiveness
is sqrt(2) lower... Though Kevin was not flying with INDI but with quat_int


Reply to this email directly or view it on GitHub
#1231 (comment).

@flixr
Copy link
Member Author

flixr commented Jul 22, 2015

Ups, sorry... I did change the ardrone indi airframe... Wanted to omit that one first...

@kevindehecker
Copy link
Contributor

But now I actually have flown with adaptive indi, and it works great!

And as @EwoudSmeur mentioned, apparently I flied with standard stuff op to now, and that also seems to work. (in both cases for ARDrone2 in optitrack enviroment that is)

@flixr
Copy link
Member Author

flixr commented Jul 22, 2015

And what is up with the optitrack airframe and it's totally different gains?

@kevindehecker
Copy link
Contributor

That I don't know. I use the standard indi gains with the optitrack systen, except for the h&v P and D guidance gains, which I have set much higher since there is perfect gps. Using those I get almost no position error, and pretty stable flight otherwise.

@flixr
Copy link
Member Author

flixr commented Jul 22, 2015

Could you please discuss this with @fvantienen, you guys are the only ones using optitrack, if the ardrone2_optitrack.xml airframe file is outdated, then plz update it to what you are using...
If it is not used at all, plz remove it...

@kevindehecker
Copy link
Contributor

@fvantienen I think my setup for optitrack runs pretty smooth by now. So, how about taking this one #1257 ?

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.

None yet

4 participants