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

WEBOTS_SITL: adjust_params model #19250

Merged
merged 1 commit into from
Nov 16, 2021

Conversation

HefnySco
Copy link
Member

This PR adjusts dimensions and parm files of quadcopter models for more stability.
Also PWM data of all servos are sent to the simulator by default to enable adding more propellers or servos.

No core changes. Only enhancements and better stability for drones.

@HefnySco
Copy link
Member Author

@peterbarker it passed :)
Thanks

@peterbarker peterbarker merged commit 5aa3714 into ArduPilot:master Nov 16, 2021
@halil93ibrahim
Copy link

halil93ibrahim commented Jun 20, 2022

Hi @HefnySco, I have a question regarding the parameters of the Ardupilot SITL. I am trying to run a geometric controller implementation but the rate controllers do not seem to be well-tuned. See discussion here

I tried to tune it using qgroundcontrol, but what I notice is there are some max/min limits in qgroundcontrol some are already not fit in the interval. Such as, the ATC_RAT_PIT_P parameter is supposed to be at max 0.35 but it is set to 3.5.

Can you give any hint for the tuning process of the parameters?

Also, I noticed that ATC_RAT_PIT_D and ATC_RAT_RLL_D params are defined twice to 0.03 and 0.003.

@shubham-shahh
Copy link
Contributor

Hi @HefnySco, I have a question regarding the parameters of the Ardupilot SITL. I am trying to run a geometric controller implementation but the rate controllers do not seem to be well-tuned. See discussion here

I tried to tune it using qgroundcontrol, but what I notice is there are some max/min limits in qgroundcontrol some are already not fit in the interval. Such as, the ATC_RAT_PIT_P parameter is supposed to be at max 0.35 but it is set to 3.5.

Can you give any hint for the tuning process of the parameters?

Also, I noticed that ATC_RAT_PIT_D and ATC_RAT_RLL_D params are defined twice to 0.03 and 0.003.

Try using autotune mode

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.

4 participants