Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/1096 protobuffer update #353

Closed
wants to merge 19 commits into from

Conversation

thibaudteil
Copy link
Collaborator

  • Tickets addressed: bsk-1096
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

The first two commits clean up some recently added modules and messages.
The third commit cleans out the cielimInterface module from all the unused messages
The other commits add the new message data and test the reading and writing of the protobufs

Verification

Unit test

Documentation

None

Future work

Many improvements to come

@thibaudteil thibaudteil force-pushed the feature/1096-protobuffer-update branch 10 times, most recently from 854a4d4 to b32fcf9 Compare October 15, 2024 02:40
@thibaudteil thibaudteil force-pushed the feature/1096-protobuffer-update branch 3 times, most recently from 61f1a58 to f3122e5 Compare October 24, 2024 18:16
@thibaudteil thibaudteil force-pushed the feature/1096-protobuffer-update branch 2 times, most recently from eb9860e to ee0d72f Compare November 6, 2024 00:42
@thibaudteil thibaudteil force-pushed the feature/1096-protobuffer-update branch from ee0d72f to aef5943 Compare November 6, 2024 00:44
@thibaudteil thibaudteil force-pushed the feature/1096-protobuffer-update branch from 4bbabf8 to f17cdd0 Compare November 10, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants