BPM Sync for effects

[naught101]

There are quite a few effects with time-period settings. For all of these, it would be great if there was a sync option, that would change the time-setting knob to a selector for BPM multiples/divisions (1/16, 1/4, 3/4, 1/1, 2/1, etc.).

The relevant knobs are:

Flanger: Period
Echo: Delay
AutoPan: Period

There are quite a few LV2 plugins with MIDI-syncable periods as well, so it would be good if this worked with those plugins when LV2 support is introduced.

[robzilla (z-rob-e)]

This is at the top of my wishlist. Please, please prioritize this. For echo specifically.

[Be (be.ing)]

There is now a sync button for autopan, but not yet for echo or flanger.

To really make use of this will require a new type of effect parameter for discrete values. That issue is tracked separately in Bug #1656225.

[ronso0 (ronso0)]

Coming from #1656225

I'll join in to push this idea!
IMO especially the Echo effect is only usable for 'random delay' effect ATM. It's very hard to somehow adjust Echo delay on Pfl first (for a running track) then apply it to Channel and mix it in. Matching the BPM is pure chance...or hard work/time.

I imagine that it's kind of hard to implement: first press Echo's 'Sync' button then adjust a stepped Potmeter to set the delay to 1, 1/2, 1/4 of a beat.
When a new track is loaded, I'd expect both to reset to some default value, like 1/2 or 1 beat of track's BPM, or better: the stepped Potmeter's last value.

Btw: Isn't that the way way some of those fancy (Pioneer?) mixers handle it?

[ronso0 (ronso0)]

bug #1656225

[robzilla (z-rob-e)]

Just chiming in, glad to see some movement on this much needed feature.

[Be (be.ing)]

Pull request for echo effect: https://github.com/mixxxdj/mixxx/pull/1256

[Be (be.ing)]

pull request for flanger: https://github.com/mixxxdj/mixxx/pull/1257

[Be (be.ing)]

pull request for phaser: https://github.com/mixxxdj/mixxx/pull/1261

[Be (be.ing)]

Further work was done in these PRs:
https://github.com/mixxxdj/mixxx/pull/1335
https://github.com/mixxxdj/mixxx/pull/1350

[Swiftb0y (swiftb0y)]

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/8312