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

Pressing Hotcues buttons randomly deactivate loop #9350

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

Pressing Hotcues buttons randomly deactivate loop #9350

mixxxbot opened this issue Aug 23, 2022 · 7 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: twontu
Date: 2018-06-22T15:47:58Z
Status: Fix Released
Importance: High
Launchpad Issue: lp1778246
Tags: cue, looping


MIXXX 2.1.0 / WindowsOS

I've noticed that if
1)I set an hotcue and activate a loop (loop start = hotcue),
2) click play to start playing,
3) and then click the hotcue button to restart the loop before it ends, it randomly disables the loop.

This happens only if sync and quantize are both activated.
It happens also if you map
cue_gotoandplay
to a controller

Tested with and without controllers.

It happens randomly so you have to press/click the hotcue button several times to reproduce the problem.

Is there anything I'm doing wrong? Thanks for your support

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2019-07-06T22:11:24Z


I think #2190 is fixing this one as well.

@mixxxbot
Copy link
Collaborator Author

Commented by: goddisignz
Date: 2019-07-15T06:28:14Z


Would also be my assumption. Do I then have to backport it to version 2.1 or what is the policy in Mixxx?

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2019-07-15T08:13:18Z


Yes, in this case we may fix the bug in the 2.1 branch only. After merge the related PR, we merge the 2.1 branch into master. In the common case, the fix should not be included in the PR against master to avoid conflicts. Here, we may consider if it is worth the work.

@mixxxbot
Copy link
Collaborator Author

Commented by: goddisignz
Date: 2019-07-16T20:30:06Z


Ok, then I will basically move all the changes to the 2.1 branch and create a new PR.
I would then close #2190 and move the "is a seek queued"-stuff to a new PR as well.
This should avoid conflicts, right?

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2019-07-16T20:58:04Z


Sounds good. Thank you!

@mixxxbot
Copy link
Collaborator Author

Commented by: goddisignz
Date: 2019-07-18T20:11:25Z


New PR here: #2209

@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.2.2 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant