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.

Sign up for GitHub

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

Jump to bottom

Added altitude to telemetry plugin #1920

Merged
merged 2 commits into from
Nov 2, 2022
Merged

Added altitude to telemetry plugin #1920

merged 2 commits into from
Nov 2, 2022

Conversation

YalamareddyChaitanya
Copy link
Contributor

@YalamareddyChaitanya YalamareddyChaitanya commented Oct 27, 2022 •
edited
Loading

This pull request would close #1903. Created a pull request #300 in MAVSDK-Proto

@YalamareddyChaitanya YalamareddyChaitanya changed the title Add altitude to telemetry plugin Added altitude to telemetry plugin Oct 27, 2022
JonasVautherin
JonasVautherin reviewed Oct 28, 2022
View reviewed changes
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.

Looks good, just need to merge the MAVSDK-Proto corresponding PR and it will be good for me 👍

@julianoes julianoes force-pushed the add-altitude-to-telemetry branch from e6848cc to 29775c8 Compare November 2, 2022 16:55
@julianoes
Copy link
Collaborator

Merged the proto and updated it here, and force pushed.

@JonasVautherin
Copy link
Collaborator

I think the code needs to be re-generated 👍

@julianoes julianoes force-pushed the add-altitude-to-telemetry branch from 29775c8 to 7827c95 Compare November 2, 2022 19:47
@julianoes
Copy link
Collaborator

@JonasVautherin I think that's correct 😄

@julianoes julianoes merged commit 849cb9a into mavlink:main Nov 2, 2022
@YalamareddyChaitanya YalamareddyChaitanya deleted the add-altitude-to-telemetry branch November 16, 2022 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JonasVautherin JonasVautherin JonasVautherin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

How do you access the barometric altitude when GPS is lost?
3 participants
@YalamareddyChaitanya @julianoes @JonasVautherin