support for blackbox presets

BossHobby / QUICKSILVER

Flight Controller Firmware

MIT License

170 stars 39 forks source link

support for blackbox presets #96

Closed sakitume closed 1 year ago

sakitume commented 1 year ago

Update profile_t with new blackbox_preset_t member for persistence and exposure to Configurator
Blackbox "file" now has a "blackbox_fieldflags" member to indicate what fields were emitted
OSD menu updated to allow choosing a blackbox preset

bkleiner commented 1 year ago

ah one more thing, please bump the minor for the QUIC_PROTOCOL_VERSION in quic.h line 11, stictly speaking we are breaking protocol compatibility. We gonna need that to gate the functionality in the config.

sakitume commented 1 year ago

I went ahead and bumped up QUIC_PROTOCOL_VERSION as requested, and I agree, the protocol is not identical however I don't think it is actually incompatible in any way.

I'm curious what I might be missing

bkleiner commented 1 year ago

I went ahead and bumped up QUIC_PROTOCOL_VERSION as requested, and I agree, the protocol is not identical however I don't think it is actually incompatible in any way.

I'm curious what I might be missing

the blackbox files do not necessarily include all fields anymore. i know that you have a quite clever workaround for the BF converter, but strictly speaking we have removed data from the api surface.