Trans effect

Bug #1693852 reported by John doe
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
New
Undecided
Unassigned

Bug Description

Mixxx should have a trans effect.

Revision history for this message
John doe (johndoemixxx) wrote :
Revision history for this message
John doe (johndoemixxx) wrote :
Revision history for this message
Be (be.ing) wrote :

Hi, thank you for the code. We coordinate development and code review on GitHub. Please make fork mixxxdj/mixxx on GitHub and make a new pull request. Our Using Git wiki page has some more tips on getting set up: https://mixxx.org/wiki/doku.php/using_git

Please add a sync button parameter to the effect and sync the period parameter to the beat length when the sync parameter is enabled. Refer to https://github.com/mixxxdj/mixxx/pull/1256 for an example.

Revision history for this message
Be (be.ing) wrote :

You will also need to modify build/depends.py and src/effects/native/nativebackend.cpp to include the new effect.

Revision history for this message
Daniel Schürmann (daschuer) wrote :

I had a short look into the code and it looks already cool, thank you.

Did you consider to beatmatch this effect? Be has recently introduced this for some other Mixxx effects like here https://github.com/mixxxdj/mixxx/pull/1256

An additional cut-off ration parameter would make this complete. I think we have now a 50 % ration.
right?

Revision history for this message
Be (be.ing) wrote :

Does a Gater or Tremolo effect satisfy this? See https://bugs.launchpad.net/bugs/1299026

Revision history for this message
Swiftb0y (swiftb0y) wrote :

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

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.