TauLabs / TauLabs

taulabs.org
Other
456 stars 392 forks source link

Add Flight Mode support to MavLink protocol #1110

Closed cGiesen closed 9 years ago

cGiesen commented 10 years ago

Actual we don't have the flight mode in MavLink HeartBeat.

I like to add this. The Problem; our IDs didn't fit to the flight mode IDs from MavLink.

Only this are in Mavlink Stabilization1 = Mavlink = Stab AltitudeHold = Mavlink = alth PositionHold = Mavlink = posi ReturnToHome = Mavlink = retl (Return to launch)

I ask the MinimOSD developer to add our 'missing' mods, but he didn't like to do it, because he like to stay compatible to APM. APM is using IDs up to 13 in the moment.

Any idea how to deal with that? Ok, we can fork the actual code and patch. But thats additional work, to keep the code up to date

cu Carsten

peabody124 commented 10 years ago

I would make all the stab modes say stab honestly. What other modes do you want? Pathplanner? When you send a value greater than 13 what does it day?

cGiesen commented 10 years ago

I like to see RTH, PH and the other Flight Modes. In GCS we show them too ;) Values > 13 are ignored today.

No problem to add this. Its arduino ;) and easy source code, not like TauLabs ;)

On the other site, MinimOSD Extra do all I (we?) need. Fork and make our own. There is no need for updates I think.

peabody124 commented 10 years ago

When you say "There is no need for updates I think" do you mean this issue should be closed?

cGiesen commented 10 years ago

No, I mean, MinimOSD have all features we need. MinimOSD need no more updates.

Flightmode Support in MavLink is still needed. I have done it in my fork, but I have to test it and patch MinimOSD firmware

mlyle commented 9 years ago

Fixed by #1795