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

Echo effect is not synchronized with actual tempo #9442

Closed
mixxxbot opened this issue Aug 23, 2022 · 7 comments
Closed

Echo effect is not synchronized with actual tempo #9442

mixxxbot opened this issue Aug 23, 2022 · 7 comments
Labels
Milestone

Comments

@mixxxbot
Copy link
Collaborator

Reported by: uklotzde
Date: 2018-09-18T23:01:12Z
Status: Fix Released
Importance: High
Launchpad Issue: lp1793232
Attachments: [echo out effect.png](https://bugs.launchpad.net/bugs/1793232/+attachment/5190352/+files/echo out effect.png)


I usually adjust the time parameter of the echo effect to the 12 o'clock position to get a beat synchronized echo out effect. This doesn't seem to work correctly when pitching the playing track. The synchronization only takes the nominal tempo of the track into account, but not the actual tempo.

Currently the echo effect cannot be used for a beat synchronized echo out on tracks that are not played with their original tempo. This is a very common case when trying to match tracks with different tempos somewhere in the middle between their original tempos. For some tracks this is barely noticeable while others are producing weird double beats when activating the echo effect.

@mixxxbot mixxxbot added the bug label Aug 23, 2022
@mixxxbot
Copy link
Collaborator Author

Commented by: uklotzde
Date: 2018-09-18T23:03:50Z
Attachments: [echo out effect.png](https://bugs.launchpad.net/mixxx/+bug/1793232/+attachment/5190352/+files/echo out effect.png)

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2018-09-18T23:37:54Z


This was implemented in #1350 for 2.1. Has there been a regression?

@mixxxbot
Copy link
Collaborator Author

Commented by: uklotzde
Date: 2018-09-19T09:51:23Z


Confirmed for both 2.1.4 as well as 2.2 :(

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2018-09-19T12:06:42Z


I compiled commit c74c08f from that pull request and indeed the issue is present there too. I guess I did not test carefully enough when working on that.

@mixxxbot
Copy link
Collaborator Author

Commented by: uklotzde
Date: 2018-10-31T09:14:36Z


Probably related: https://bugs.launchpad.net/mixxx/+bug/1800771

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2018-10-31T11:16:57Z


#1878

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.1.6 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant