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

Waveform glitch when adjusting loop_out (while playing) #12506

Closed
ronso0 opened this issue Jan 3, 2024 · 11 comments
Closed

Waveform glitch when adjusting loop_out (while playing) #12506

ronso0 opened this issue Jan 3, 2024 · 11 comments

Comments

@ronso0
Copy link
Member

ronso0 commented Jan 3, 2024

Bug Description

Occurs with all renderers AFAICT

The issue is not fully visible in the video. It's like the waveform is moving in the background with ~20% opacity and is overlayed with a second stuttering view.
https://github.com/mixxxdj/mixxx/assets/5934199/cdbe9f94-4bb6-48bb-a4c2-d4c549f3db01

Doesn't occur with 2.3.6

Version

No response

OS

No response

@ronso0 ronso0 added this to the 2.4.0 milestone Jan 3, 2024
@JoergAtGithub
Copy link
Member

Just tested this: Windows seems not to be affected.
Could you please add more information about you OS, Qt version and graphics driver.

@ronso0
Copy link
Member Author

ronso0 commented Jan 4, 2024

Linux (Ubuntu 20.04.5)
Qt 5.12.8
4.6 (Compatibility Profile) Mesa 21.2.6 (Mesa Intel(R) UHD Graphics 620 (KBL GT2))

I'm currently bisecting.

@ronso0
Copy link
Member Author

ronso0 commented Jan 4, 2024

It's 38a9391
If I go back one commit it's fine as before.

@ronso0
Copy link
Member Author

ronso0 commented Jan 4, 2024

Btw, this reminds me of extending the smoke test check list.

@ywwg
Copy link
Member

ywwg commented Jan 8, 2024

This does seem like a release-blocking regression. It also seems like we can't really revert the bad commit because that is needed too.

@ywwg ywwg added the blocker label Jan 8, 2024
@JoergAtGithub
Copy link
Member

Have a look on #12522 which doesn't return the expected playpos. This might be related.

@JoergAtGithub
Copy link
Member

Could you please list step by step, what I need to click to reproduce this.
My guess is, that VisualPlayPosition::set is called with inconsistent data. Printing the arguments of this function to the log might help to understand this.

@ronso0
Copy link
Member Author

ronso0 commented Jan 9, 2024

Just enable a loop, play it, then click and hold loop_out to drag the out marker along with the playpos.
Audio is fine but it seems the visual playpos jumps between loop_in and loop_out at a very high rate, therefore the two 'shadow' images.

@JoergAtGithub
Copy link
Member

@ywwg
Copy link
Member

ywwg commented Jan 9, 2024

since this is only visual and does not affect audio output, we think this doesn't qualify as a blocker

@ronso0
Copy link
Member Author

ronso0 commented Jan 9, 2024

Well, it's very confusing, and it's a regression.
IMO that's severe enough to be fixed in 2.4 rather than wait for 2.4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants