Add a CUP ControlObject

Bug #1535468 reported by Daniel Schürmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Wishlist
william lemus

Bug Description

Some controllers like the Reloop Jockey 3 have a dedicated button CUP Cue-Play
* Press, pause at the cue point
* release, play

Traktor has it. Virtual DJ has a config option to replace it wit CUE. It seams to be also usual as a replacement for play.

Currently it has to be implemented in Script
https://github.com/mixxxdj/mixxx/pull/840/files#r50022616

CUP allows a new Mixing style, somehow an inverse stutter play.
If we have this as CO, it can be used from Skin and to be learned on a controller that does not has a dedicated button for it.

This is an ideal starter bug.

The implementation can be done here:
https://github.com/mixxxdj/mixxx/blob/master/src/engine/cuecontrol.cpp#L67

Tags: easy
Changed in mixxx:
importance: Undecided → Wishlist
tags: added: easy
Anirudh Mani (anirudh3)
Changed in mixxx:
assignee: nobody → Anirudh Mani (anirudh3)
Anirudh Mani (anirudh3)
Changed in mixxx:
assignee: Anirudh Mani (anirudh3) → nobody
Revision history for this message
william lemus (william-lemus) wrote :

Hey,

I'd like to take a look at this. One question I have is whether this would also be useful to also add it as a valid cue mode?

Changed in mixxx:
assignee: nobody → william lemus (william-lemus)
Revision history for this message
Daniel Schürmann (daschuer) wrote :

> One question I have is whether this would also be useful to also add it as a valid cue mode?

The Idea was to have the CUP in addition to all cue modes.

Do you wish to have a this as a replacement for play or cue or both as option?

Revision history for this message
william lemus (william-lemus) wrote :

Ok makes sense, I meant as option for cue modes. I will add it as another cue mode.

Revision history for this message
Sean M. Pappalardo (pegasus-renegadetech) wrote :

It shouldn't be another cue mode, just another ControlObject (so it can be mapped to a physical and/or GUI button) since people may want the Cue+Play ability regardless of what cue mode they've chosen.

Revision history for this message
william lemus (william-lemus) wrote :

I should've been clearer. So far I had as a separate cue mode and also a control object that can be mapped. Is there any harm in having the ability to do it both ways?

Revision history for this message
Sean M. Pappalardo (pegasus-renegadetech) wrote :

Oh, not at all. I just wanted to make sure it was at least a CO.

Changed in mixxx:
status: New → In Progress
Revision history for this message
william lemus (william-lemus) wrote :

I have made a pull request:

https://github.com/mixxxdj/mixxx/pull/1001

Please let me know if there is anything I did incorrectly or needs to be changed.

Revision history for this message
william lemus (william-lemus) wrote :

Just adding the corrected pull request:

https://github.com/mixxxdj/mixxx/pull/1010

Changed in mixxx:
status: In Progress → Fix Committed
milestone: none → 2.1.0
Revision history for this message
william lemus (william-lemus) wrote :

When do we go ahead and update the wiki to explain the new cue mode?

Revision history for this message
Daniel Schürmann (daschuer) wrote :
Revision history for this message
Sébastien BLAISOT (sblaisot) wrote :

I think there is a typo in the wiki, the associated note say "deprecated in Mixxx v2.1.0"
Shouldn't it be "Introduced in Mixxx v2.1.0" ?

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

Fixed, Thanks!

Changed in mixxx:
status: Fix Committed → Fix Released
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/8449

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.