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

Update gimbal_full_control.cpp example #2384

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

KarthiAru
Copy link
Contributor

  • Added gimbal_id parameter to all gimbal related methods.
  • Replaced the non-existing set_mode method with the set_angles method.
  • Revised set_angle methods to include the correct parameters

- Added `gimbal_id` parameter to all gimbal related methods.
- Replaced the non-existing `set_mode` method with the `set_angles` method.
- Revised `set_angle` methods to include the correct parameters
julianoes
julianoes previously approved these changes Aug 28, 2024
Copy link
Collaborator

@julianoes julianoes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, that's great, thanks!

JonasVautherin
JonasVautherin previously approved these changes Aug 29, 2024
Copy link
Collaborator

@JonasVautherin JonasVautherin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

The style check is failing, though 😕

@KarthiAru
Copy link
Contributor Author

@JonasVautherin I couldn't figure out the style issue from the logs. Could you please let me know how to fix this locally?

@JonasVautherin
Copy link
Collaborator

I think you have to run ./tools/fix_style.sh from Ubuntu 24.04. It used to work with a container if you are not on Ubuntu 24. Maybe ./tools/tools/run-docker-clang-format.sh ./tools/fix_style.sh? I haven't used it in a while 🙈

@KarthiAru
Copy link
Contributor Author

@JonasVautherin Thanks. I've fixed the style on Ubuntu 24.04

Copy link

sonarcloud bot commented Aug 29, 2024

@JonasVautherin JonasVautherin merged commit 349f97a into mavlink:main Aug 29, 2024
33 checks passed
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