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

ALTITUDE: support reading altitude of copter #6317

Open
Ahrovan opened this issue May 24, 2017 · 9 comments
Open

ALTITUDE: support reading altitude of copter #6317

Ahrovan opened this issue May 24, 2017 · 9 comments

Comments

@Ahrovan
Copy link

Ahrovan commented May 24, 2017

Issue details

pixhawk.ethz.ch/mavlink/#ALTITUDE contain the current system altitude and useful for use in ros (and ...). please Introduce a way to solve this problem and add reading altitude of copter.

Version

ardupilot master,Copter,pixhawk

@amilcarlucas
Copy link
Contributor

This fix is easy to do. This is a good first issue for a student starting to work in the codebase. Unfortunatly we do not have a label for that.

@magicrub
Copy link
Contributor

Altitude is available in about a dozen different MAVLink packets. Adding this particular one would be good, but you can just look at others for now to get you going

@IamPete1
Copy link
Member

@aliakbarkhan
Copy link

Issue details

pixhawk.ethz.ch/mavlink/#ALTITUDE contain the current system altitude and useful for use in ros (and ...). please Introduce a way to solve this problem and add reading altitude of copter.

Version

ardupilot master,Copter,pixhawk

is this issue still open i would like to work on it

@amilcarlucas
Copy link
Contributor

Is still open, please do.

@gmazzucchi
Copy link
Contributor

This issue is solved, the message was added here

@IamPete1
Copy link
Member

@tzarjakob although it made it to the AP MAVLink repo we don't actually send it yet.

@gmazzucchi
Copy link
Contributor

That's true, I have made a patch for supporting this message.

@peterbarker
Copy link
Contributor

Oops. Didn't reference this issue on my PR: #21652

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

Successfully merging a pull request may close this issue.

10 participants