goodrobots / maverick-api

API backend for maverick-web
MIT License
8 stars 5 forks source link

Altitude data? #33

Open fnoop opened 6 years ago

fnoop commented 6 years ago

vfrHudMessage.altitude gives an odd figure - when flying at reported 100m from a GCS, this message field gives about 684 value slightly lower that the navSatFix.altitude: vfrHudMessage: 684.22998046875 :: navSatFix: 703.6901112815922

fnoop commented 6 years ago

I would have thought vfrHudMessage should be altitude above home?

SamuelDudley commented 5 years ago

I'm assuming we want alt above home? If so I can send it in a separate message

fnoop commented 5 years ago

Yes please

SamuelDudley commented 5 years ago

Forgot to close this when I added rel alt into the vrf hud message https://github.com/goodrobots/maverick-api/blob/master/maverick_api/modules/api/mavros/__init__.py#L600 https://github.com/goodrobots/maverick-api/blob/master/maverick_api/modules/api/mavros/__init__.py#L151

fnoop commented 5 years ago

Just a 'meta' query here - by adding relative alt we're drifting away from the mavlink definitions: https://mavlink.io/en/messages/common.html#VFR_HUD

Does that matter?

SamuelDudley commented 5 years ago

probably best to send it in the correct mavlink message.