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 issues with "High details" enabled #13472

Closed
daschuer opened this issue Jul 14, 2024 · 21 comments · Fixed by #13474 or #13491
Closed

Waveform issues with "High details" enabled #13472

daschuer opened this issue Jul 14, 2024 · 21 comments · Fixed by #13474 or #13491
Labels
Milestone

Comments

@daschuer
Copy link
Member

daschuer commented Jul 14, 2024

Bug Description

RGB high details waveform are showing an offset:
High details, the beat marker is not on the visual beat:
image
No High details, the beat marker is at the expected position:
image

Tested with QT 6.2

Version

2.6~alpha

OS

Ubuntu Focal

@daschuer daschuer added the bug label Jul 14, 2024
@daschuer
Copy link
Member Author

With another track, I see only the upper half. The waveform disappears entirely when I disable "Stereo coloration".

image

@daschuer daschuer added this to the 2.6-beta milestone Jul 14, 2024
@Swiftb0y
Copy link
Member

To clarify, the missing bottom half only applies to the "High Details" version?

@Swiftb0y
Copy link
Member

Both of these refer to #13471 (comment) and #13471 (comment) respectively, right?

@daschuer
Copy link
Member Author

To clarify, the missing bottom half only applies to the "High Details" version?

Yes.

Non of these issues are in 2.5 with Qt5
I am just building Qt6

@daschuer
Copy link
Member Author

The 2.5 + Qt 6 build is also not affected.

@ywwg
Copy link
Member

ywwg commented Jul 14, 2024

as I suspected, the stem support is causing the problem ( @acolombier ) commit 2911c38

@acolombier
Copy link
Member

I believe I know how to fix it, I will prepare a fix for it soon.

@ywwg
Copy link
Member

ywwg commented Jul 16, 2024

it looks like the waveform is still stretched by 2x, reopening

@ywwg ywwg reopened this Jul 16, 2024
@acolombier
Copy link
Member

Could you please attach some screenshots or reproducing steps?

@ywwg
Copy link
Member

ywwg commented Jul 17, 2024

looks like it's not 2x, but some small fraction
Screencast from 2024-07-17 13-42-06.webm

@ywwg
Copy link
Member

ywwg commented Jul 17, 2024

seeking to the end of the waveform is sufficient for seeing the issue

@acolombier
Copy link
Member

Wondering if there could be some issue due to memory alignment. I should have some time this weekend, if anyone wants to give it a go in the meantime, please feel free to!

@Swiftb0y
Copy link
Member

Not trying to be "I told you so", but I think this is a convincing argument for why abusing a texture for uploading a blob to the GPU is a bad idea...

@ywwg
Copy link
Member

ywwg commented Jul 17, 2024

if we could get the same level of detail with another method I'd be all for it

@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

ah it's not an offset, there's just a point in the waveform at which suddenly it starts showing the wrong data. In this video you can see the left half of the waveform is correct, and the right half is wrong.

Screencast.from.2024-07-19.11-10-37.webm

@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

it looks like it wraps around to the beginning of the waveform -- I'm guessing there's a miscalculation of the total length of the data?

@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

Screencast.from.2024-07-19.11-20-13.webm

@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

error always starts at 2:28.60

@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

are we overflowing int32

@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

hmmmm it all goes wrong at visualindex==65535 :)

@ywwg ywwg changed the title Waveform offset with "High details" Waveform wraps around after 2:28 with "High details" enabled Jul 19, 2024
@ywwg ywwg changed the title Waveform wraps around after 2:28 with "High details" enabled Waveform issues with "High details" enabled Jul 19, 2024
@ywwg
Copy link
Member

ywwg commented Jul 19, 2024

ok well I guess this is technically a different problem than the 2x issue last time... but it seems ok to reuse the bug

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