-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
If Quantization is enabled, Hotcues start from a offset position on a playing deck #7250
Comments
Commented by: daschuer For me, it sound like that this is a feature. |
Commented by: esbrandt Stupid me, its a feature :-) In v1.11, it works different. As described in #1 "Expected behavior" |
Commented by: rryan I'm not sure what the right behavior is here. I feel odd about quantizing a hotcue after it has been placed. Previously we only quantized it if the hotcue was placed when quantize was enabled. |
Commented by: ywwg This is bumping up against "all seeks quantize to beats". Since the enginebuffer doesn't know that a hotcue is a special kind of seek, there's no way for it to know not to quantize. And if we move the seek code back out of the engine process call, we again get off-by-one-buffer seeks. I'd like to rewrite how seeking is done because we have a need for different types of seeks with different quantization patterns... The ideal situation would be a messaging system that can express all of these things:
I could probably just change m_bSeekQueued, which is actually an int, to store an enum value that could mean one of those things. Then when the engine buffer processes the call it can take the appropriate action. The cuecontrol can ask for exact seeking, whereas the synccontrol can ask for a phase sync. (I wonder what happens if master sync is on and you push a non-aligned hotcue? ) It's really late in the game for 1.12 to be rewriting this, of course... is it worth it? |
Commented by: ywwg |
Commented by: ywwg hotcues are now exempt from Quantize. If you select a hotcue with mastersync active, it will happily unsync your tracks. |
Commented by: daschuer Hi Owen, thank you for working on that. What are the use cases for the different seek hot cue seek behaviors? For me we have two use cases:
for 1 + 2 + 3 the original behavior seems to me the desired one. Only 4 needs the patch. What is the work flow now for 1 + 2 + 3 if you don't want to get out of sync if you miss the right moment for the button press? Before the patch, it was possible to switch between "start at hotcue (possible out of sync)" and "start synced" by the quantization button. Now this "feature is gone". What is the alternative work flow? Is there a way to auto detect the use case? |
Commented by: ywwg hm, you may be right that we lost important functionality. I think the patch in general is robust, the real problem is that our hot cue system needs meta data. so maybe for now we'll keep the hot cues in regular seek mode, not seek exact mode. eventually we can have special hotcues that demand exact seeking. |
Commented by: ywwg OK I changed most of the hotcue seeks to be seekAbs, not seekExact. But for the goto-and-stop seeks, I made those exact since it would be weird for the deck to sync phase in that case. This will need testing by people who actually use hotcues! (I just use them as visual markers, I never actually seek to them). |
Issue closed with status Fix Released. |
Reported by: esbrandt
Date: 2014-01-04T08:27:30Z
Status: Fix Released
Importance: Undecided
Launchpad Issue: lp1265986
Tags: cue
Mixxx 1.12.0-alpha-pre git master r3881
Expected behavior:
Actual behavior:
Interestingly, this is only if the track is played back.
If you press Hotcue 1 on a stopped deck with quantization enabled, the hotcue's playpack position is correct.
The text was updated successfully, but these errors were encountered: