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

Plots behave glitchy. #906

Open
mcourteaux opened this issue Oct 8, 2024 · 5 comments
Open

Plots behave glitchy. #906

mcourteaux opened this issue Oct 8, 2024 · 5 comments

Comments

@mcourteaux
Copy link
Contributor

mcourteaux commented Oct 8, 2024

Zooming reveals that the data is actually completely bogus. See the video:
Wayfarer_2024-10-08_114209.webm

Also note this double-rendering in the blue and yellow plot:

image

I'm currently on e6eb989

@wolfpld
Copy link
Owner

wolfpld commented Oct 8, 2024

Zooming reveals that the data is actually completely bogus. See the video:

Yes, this looks to be completely broken. Can you share a trace with these plots?

@mcourteaux
Copy link
Contributor Author

glitchy_plots.tracy.zip

I had to zip again, to make Github accept it.

@wolfpld
Copy link
Owner

wolfpld commented Oct 8, 2024

I can reproduce the issue with your trace. It's certainly something interesting.

Zrzut ekranu_20241008_180615

Without looking at the data, this looks like an unsorted plot, as you can see values going forward and backward in the screen below, and the glitching you see when zooming is a result of a binary search that can't work properly.

Tracy handles out-of-order plot points by design, this is a very old functionality and there were no problems with it before. I am not able to reproduce your issue with the test/ application that generates millions of plot points from two threads, which results in a lot of resynchronization. The glitchy plots in your trace are in the 50-500 plot points range, and I'd expect there's no need to synchronize anything at all, with such low numbers.

The only recent thing that may affect this that comes to mind is replacement of the parallel STL with the ppqsort library.

@mcourteaux
Copy link
Contributor Author

Is this something we can test by going back to before ppqsort got introduced with the same trace file? I'm assuming the ppqsort stuff happens in the Tracy UI, but perhaps the trace file format changed since then? If it's not, I can attempt my first-ever git-bisect. 😄

@wolfpld
Copy link
Owner

wolfpld commented Oct 9, 2024

PPQSort was added in 1c1faef. The trace format has not changed. Opening the existing trace will not result in any changes, you would have to redo the trace and see what comes out.

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

No branches or pull requests

2 participants