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

HOTFIX: Fix crossfade sequence when having a tail after transition. #3728

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

toots
Copy link
Member

@toots toots commented Feb 9, 2024

There is a problem in the new mechanism to compute crossfade over the common buffer: this code:

new Sequence.sequence ~merge:true [compound; after_tail]

Will skip over to after_tail on the first track mark from the coumpound source.

It seems good to try and keep the new crossfade computation over the common buffer so, the safe solution is to add a new, opt-in behavior to sequence that allows it to play from the same source while it is available instead of moving to the next source on the first track mark.

This is achieved by the new single_track option, sets to the default, current behavior (true).

This will require a bump of the release assets.

@toots toots force-pushed the sequence-single-track-v2.2.x branch 4 times, most recently from 0e74f8e to e2dec7c Compare February 9, 2024 22:51
@toots toots force-pushed the sequence-single-track-v2.2.x branch from e2dec7c to a11d727 Compare February 9, 2024 23:06
@toots toots merged commit 2e21088 into v2.2.4 Feb 9, 2024
19 of 21 checks passed
@toots toots deleted the sequence-single-track-v2.2.x branch February 9, 2024 23:26
@vitoyucepi
Copy link
Collaborator

Hi @toots,
Is it necessary to merge this into rolling-release-v2.2.x?

@toots
Copy link
Member Author

toots commented Feb 15, 2024

I usually re-create the rolling-release-v2.2.x branch from the latest stable release once the first bugfix happen.

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

Successfully merging this pull request may close these issues.

2 participants