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.

Sign up for GitHub

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

Jump to bottom

Add an extension which can make trace files from NUClear #142

Merged
merged 47 commits into from
Aug 30, 2024
Merged

Add an extension which can make trace files from NUClear #142

merged 47 commits into from
Aug 30, 2024

Conversation

TrentHouliston
Copy link
Member

Adds an extension which can output trace files using Perfettos protobuf format.

These trace files are created by installing the TraceController extension, and then emitting a BeginTrace message, followed optionally by an EndTrace message.

@TrentHouliston
Add an extension module which outputs trace files.
b4c9931
@TrentHouliston
Swap to a protobuf based trace controller and add the extra features …
b8b9f3e 
…to make it work fast
@TrentHouliston
Add the trace control messages
e599815
@TrentHouliston
Make the trace events local emits to not slow down the main code path
e95c166
@TrentHouliston
Add a name to the timeout thread to make it show up in the trace better
eb5e680
@TrentHouliston
Add trace to every test (later fix the paths and upload it in CI as a…
ba59a58 
…n artifact)
@TrentHouliston
.
59f6512
@TrentHouliston
Fix includes
5779aab
@TrentHouliston
With stats being non direct, need to make it high priority to make th…
6dc9e85 
…e order match the old order
@TrentHouliston
Add new functionality so the trace pool can stay around until the end
9bd224e
@TrentHouliston
Make the pools be able to continue to accept tasks after shutdown
71d8640
@TrentHouliston
Tasks must run on pool broke stuff. An Indirect keyword will work better
489eb71
@TrentHouliston
Replace the Direct concept with Inline which can be influenced from b…
9d392af 
…oth sides. That way log messages can run in a pool thread.
@TrentHouliston
Fix test
393bfd6
@TrentHouliston
Merge remote-tracking branch 'origin/main' into houliston/tracer
b93d331
@TrentHouliston
Add the trace line to Inline
5077374
@TrentHouliston
Log test use idle
42f5fef
@TrentHouliston
Fix stuff
4295ba0
@TrentHouliston
Message stats are important
505f22f
@TrentHouliston
Pull out a bunch of stuff into multiple files
2bd9b8a
@TrentHouliston
fixes
935cb56
@TrentHouliston
Merge remote-tracking branch 'origin/main' into houliston/tracer
30712f6
@TrentHouliston
Fix flow
31decde
@TrentHouliston
One process is easier to read
5bc36f5
@TrentHouliston
.
f588da0
@TrentHouliston
Make all the tests trace and upload the traces in CI
a16fdcb
@TrentHouliston
.
f72f7d2
@TrentHouliston
Just use upload-artifact to upload
e6e9d50
@TrentHouliston
.
2827071
@TrentHouliston
.
5f1aabf
@TrentHouliston TrentHouliston merged commit 1f9aee6 into main Aug 30, 2024
9 of 11 checks passed
@TrentHouliston TrentHouliston deleted the houliston/tracer branch August 30, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@TrentHouliston