Logging! #1
Open
Logging! #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
jlmcmchl
commented
Feb 13, 2021
jlmcmchl
commented
- updated to 2021
- implemented device logging
- configuration load-in from JSON files (separate config of LO/CF/GH, etc)
- we get subsystem logging for "free"
- ramsete follower - trajectory generation, parameterization + verified results empirically
- flatbuffers set to v1.12 instead of master - setup using UDP to send flatbuffer status frames to a server - created a simple python UDP server to consume that data
- Now based on a StatusFrameHolder of a vec of StatusFrames - con: no python support - plus: shouldn't be trouble for the log sink since it just needs to append to a file - future work - Allow the sender to provide file naming information - future work - more device support - future work - validate deserialization
also fixed encoder ticks and added a status frame to declare contextual information about the log. Concept: this should include match information _if_ possible.
Logging got a major refactor - separate thread and I think the architecture is in a good spot memcpy is gross
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.