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

Fix stream sync play and seek setting wrong position on certain cases #100292

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

adriano-sudario
Copy link

@adriano-sudario adriano-sudario commented Dec 11, 2024

If you have multiple streams on AudioStreamSynchronized with different lengths, start and seek methods will unsync the sound when position set is higher than one of the stream's length.

Example: if AudioStreamSynchronized has two streams, one with a beat sound with a length of 16s and the other with a base sound with a length of 32s and you call audio_player.play(20) or audio_player.seek(20), the first one will start at position 0, while the other one will start at 20.

You can reproduce this issue here with detailed informations running the scene stream_sync_play_and_seek_not_setting_position_right_sometimes

@adriano-sudario adriano-sudario requested a review from a team as a code owner December 11, 2024 20:19
@adamscott
Copy link
Member

Could you open an issue about this? I understand the fix, but I wonder if we're entering the realm of "issue/proposal" here.

cc. @reduz

@adriano-sudario
Copy link
Author

Could you open an issue about this? I understand the fix, but I wonder if we're entering the realm of "issue/proposal" here.

cc. @reduz

Sure!
Just opened an issue here

@AThousandShips AThousandShips added regression cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release regression topic:audio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stream sync play and seek is setting wrong position on certain cases
3 participants