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

Reasonable defaults for Lawnmower SITL #23527

Merged
merged 1 commit into from
Aug 9, 2024
Merged

Conversation

slgrobotics
Copy link
Contributor

Solved Problem

When running '''make px4_sitl gz_lawnmower_lawn``` the vehicle completes the mission, but its movements aren't smooth.

To fix that, RD_* parameters must be adjusted in the airframe definition file.

Ammends #{23430}

Solution

Speed, accelerations and turn rate were adjusted.

also, it looks like with Units being rad, the upper limit for RD_TRANS_DRV_TRN and RD_TRANS_TRN_DRV parameters should be in radians too.

Test coverage

Running '''make px4_sitl gz_lawnmower_lawn``` produces smooth movements.

@dagar dagar merged commit 1211aad into PX4:main Aug 9, 2024
4 of 91 checks passed
@slgrobotics slgrobotics deleted the rd_lawnmower branch August 9, 2024 19:01
@chfriedrich98
Copy link
Contributor

Very nice, thanks for taking care of that!

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

Successfully merging this pull request may close these issues.

3 participants