You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The way that tracy support is implemented, as least from what I can see, you can't have multiple devices on one profiler. But this is (currently) required to support multiple queues on wgpu side.
Maybe I'm missing something, thus, and for the case I'm right, I wanted to check in with you.
I don't really require this to work, it'd just be a nice to have.
The text was updated successfully, but these errors were encountered:
Having multiple devices associated with a singe GpuProfiler won't work because one can't use buffers across devices easily.
But that still shouldn't preclude from sharing the same tracy client between multiple profilers, which might be nice regardelss. That bit should be fairly easy to fix though with some changes in the interface, allowing to take in an existing tracy client plus
Note that create_tracy_gpu_client itself only needs device & queue to establish a rough timestamp mapping, see https://docs.rs/tracy-client/latest/tracy_client/struct.Client.html#method.new_gpu_context
That would be useful regardless and should be split out I think 🤔
But that still shouldn't preclude from sharing the same tracy client between multiple profilers, which might be nice regardelss.
Ah, so that's not an issue. Good to know.
That bit should be fairly easy to fix though with some changes in the interface, allowing to take in an existing tracy client plus
TBH I don't know what you mean with that. ATM., as far as I can tell, the interface should probably just work, I just have to invoke it multiple times for each device and queue I want to look at?
Well, the issue is that right now that each GpuProfiler creates a tracy gpu context. You'd want to share a single tracy context across several GpuProfiler instances (one for each device/queue)
The way that tracy support is implemented, as least from what I can see, you can't have multiple devices on one profiler. But this is (currently) required to support multiple queues on wgpu side.
Maybe I'm missing something, thus, and for the case I'm right, I wanted to check in with you.
I don't really require this to work, it'd just be a nice to have.
The text was updated successfully, but these errors were encountered: