Added support for Message Pack

openxc / openxc-message-format

Specification for the OpenXC JSON message format

http://openxcplatform.com/

BSD 3-Clause "New" or "Revised" License

99 stars 35 forks source link

Added support for Message Pack #24

Closed dsouzahansenfrancis closed 8 years ago

dsouzahansenfrancis commented 8 years ago

Added Message pack to PayloadFormatCommand. Updated README.md

peplin commented 8 years ago

Thanks - can you expand a bit on why we should add another binary serialization format? We already support Protocol Buffers in the firmware, Python library and Android library. I did a little research on MessagePack and it doesn't seem to offer any more space savings than Protocol Buffers. The fact that it's dynamic is interesting (no schema) but I don't know if it's worth the additional complexity to support a 3rd serialization format.

emarsman commented 8 years ago

Let's not complete this PR yet. I want to take existing next with vi-firmware:next and promote to master before adding in these changes. Just working on some documentation before doing that.

@jamelseagraves, any comments on Chris's question?