aclysma / profiling

Provides a very thin abstraction over instrumented profiling crates like puffin, optick, tracy, and superluminal-perf.
Apache License 2.0
319 stars 39 forks source link

Update tracy_client to 0.13 #36

Closed Imberflur closed 2 years ago

Imberflur commented 2 years ago

Note, tracy_client now requires that the client has been started before running instrumented code. This can be done via:

tracy_client::Client::start();

at the top of main.

Should I note this new requirement anywhere in the readme or can we expect users of tracy to understand the necessary setup?

tracy_client's new span macro currently panics when the client has not been started (instead of silently doing nothing), so I went ahead and implemented the same behavior here.

However, there is some discussion about potentially changing this behavior to avoid those panics https://github.com/nagisa/rust_tracy_client/pull/36#issue-1222267892.

aclysma commented 2 years ago

Assuming no CI problems I'll try to merge and publish today, thanks!

Imberflur commented 2 years ago

@aclysma is there a release planned?

aclysma commented 2 years ago

Sorry for the delay, just published it :)