Add remaining navdata options

felixge / node-ar-drone

A node.js client for controlling Parrot AR Drone 2.0 quad-copters.

http://nodecopter.com/

MIT License

1.76k stars 428 forks source link

Closed jfsiii closed 11 years ago

jfsiii commented 11 years ago

Added the 22 missing navdata options.

All the original tests pass.
Altered the demo and rawMeasures data structures.
Added tests for the all new values.
Eliminated as many French variables as I could, but many remain. It's especially hard to translate the abbreviations. For example, is Planif short for the French planification, which would be planned in English?
Added some guesses about units for the various properties (see 4ea2a51)

closes #29

jfsiii commented 11 years ago

All done, @felixge. The only thing preventing me from merging is making sure you don't have any problems with:

felixge commented 11 years ago

Sorry for the delay.

the style of my tests (e.g. jfsiii@9c16838#L0R696)

No, those look great!

my approach to maintaining BC (support old and new keys, e.g jfsiii@5319cfc )

Well, we should figure out a normalized format going forward, as this may confuse people even more (are these the same values? or different ones?).

That being said, it's ok for now!

:heart: ! Thank you for all the work!