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

Un-eject / un-replace does not respect "Loading a track when deck is playing" setting #12906

Closed
Eve00000 opened this issue Mar 3, 2024 · 19 comments
Labels
Milestone

Comments

@Eve00000
Copy link
Contributor

Eve00000 commented Mar 3, 2024

Bug Description

  • Dubbelclick on "Load/EjectSwitch" of a Deck does not clone Deck, even when the option is selected in settings
  • Switching 2 - 4 Decks and back (multiple times) and between the switching clicking on "Load/EjectSwitch" of a playing Deck causes "Ejecting" of the playing track,
  • I suppose the option "disable load a track to a playing deck" means also disable the possibility to eject a track while playing (as it is in 2.3.6)
    -> I'm testing the main from 29/2/2024

Version

2.5-alpha-255-g2d2d5f304b

OS

Win10 / Win 11

@Eve00000 Eve00000 added the bug label Mar 3, 2024
@ronso0
Copy link
Member

ronso0 commented Mar 3, 2024

"Load/EjectSwitch"

What control are you referring to?

The Clone option in the Decks preferences refers to buttons mapped to [ChannelN],LoadSelectedTrack. There is no GUI implementation of this control (in official skins), so this applies mostly to controller mappings.
How did you trigger this control?

FYI the Eject button has special behaviour, too. It does un-eject and un-replace, depending on whether a track is loaded or not. See https://manual.mixxx.org/2.4/en/chapters/appendix/mixxx_controls.html#control-[ChannelN]-eject

@Eve00000
Copy link
Contributor Author

Eve00000 commented Mar 4, 2024

  • with controller double tap on load-knob
  • with mouse in fui drag and drop playing track to other deck
  • eject button on controller / little eject button in gui, aftr a couple of times pressing on of the two the track unloads.
  • while there's a warning on exit; still playing track / iwhile it's mpossible to load a track to a playing deck, I think it should also be impossible to eject / unload a playing track. Don't you agree?

@ronso0
Copy link
Member

ronso0 commented Mar 4, 2024

I think it should also be impossible to eject / unload a playing track. Don't you agree?

Explicit eject while playing is not possible (neither in 2.4 nor main).
So your request is to also suppress cloning if over-load/eject is suppressed? (that is not quite clear to me from your initial description)

@Eve00000
Copy link
Contributor Author

Eve00000 commented Mar 4, 2024

No no, try it yourself, you need some actions... cloning, switching between 2 - 4 decks and back. Then at a certain point you can eject a playing track. (as I wrote in my original post)
Really! Try ;-)
If you can't replicate it, I will make a video of it this evening.

@ronso0
Copy link
Member

ronso0 commented Mar 4, 2024

switching between 2 - 4 decks and back

In the GUI, or in your controller mapping (1/3, 2/4 switches)?

@ronso0
Copy link
Member

ronso0 commented Mar 4, 2024

Can't reproduce with the GUI switch, this has no impact on the engine at all.

@Eve00000
Copy link
Contributor Author

Eve00000 commented Mar 4, 2024

In the Gui, I don't have it yet on my controller.... still working on it.
I tried this also without a controller connected or mapped. On 2 different pc's

@ronso0
Copy link
Member

ronso0 commented Mar 4, 2024

Okay, please make a screencast then.

@Eve00000
Copy link
Contributor Author

I tried it again with the version I downmloaded from github on friday

video: after +/- 2 minutes.
No controller, cloning a playing deck from deck 3 to deck 4 with mouse,
on deck 4 pressing with mouse on eject: button -> playing deck ejects the track.

https://youtu.be/svJfxJNutn4

@ronso0
Copy link
Member

ronso0 commented Mar 17, 2024

Thanks for the video, but I still can't reproduce with current main.

Maybe it's some weird inconsistency of the preferences option.

Please go to Preferences -> Decks
Check if 'Loading a track when deck is playing' is set to reject.
If it already is, select another option, Apply.
Re-select 'Reject', Apply.
Redo the test. No playing deck should be ejected.

@Eve00000
Copy link
Contributor Author

Hi ronso,I checked it again, with restart after change of the option. (it was my original post)
As you could see in the video loading a playing deck was not possible even the eject of a playing track was prohibited.
Only after a couple of switches and cloning it can happen.
There must be a combination of cloning and deck that does not reset a flag, Did you try it also without controller connected?
Maybe even a combination with cue-modus, track load .....

@ronso0
Copy link
Member

ronso0 commented Mar 17, 2024

Which cue mode / track load point have you selected?

@Eve00000
Copy link
Contributor Author

I've tested with all of them, I prefer Mixxx-mode without flickering; I tried with intro-start on/off, track load ....
I know whattesting is about,
I think it's something with the settings of a deck after being cloned / being a clone, maybe a variable that is copied wrongly, or that does not have a value.....

@ronso0
Copy link
Member

ronso0 commented Mar 22, 2024

Okay, so after watching the video I think I got it:
the new Un-Eject / Un-Replace (double-click Eject) is not respecting the "Loading a track when deck is playing" setting ("Reject").
The reason you see this "Only after a couple of switches and cloning it can happen." is that it requires a different being loaded previously.

I'll take a look.

(2/4 deck switching is irrelevant)

@ronso0 ronso0 changed the title Behaviour Load/Eject after cloning Un-eject / un-replace does not respect "Loading a track when deck is playing" setting Mar 22, 2024
@ronso0 ronso0 added this to the 2.4.1 milestone Mar 22, 2024
@Eve00000
Copy link
Contributor Author

Good thinking!
So you like my video after all, even when it takes more data :-)

@ronso0
Copy link
Member

ronso0 commented Mar 25, 2024

Somehow I do ; )
What made the difference to your description:

  • clicking the Eject button like mad with the first track
  • same with the second track

So now that you are an advanced bug and feature reporter you can try to list all steps you took since starting Mixxx.
Maybe we can figure it out that way next time.

@Eve00000
Copy link
Contributor Author

Clicking like mad :-) I knew I would happen, it occured while playing music in a normal way, so I had to regenerate the ejecting.
Oh I can be mad :-)
I'll do my best to describe as good as I can, (I crossed my fingers while holding two fingers up in the air)

@ronso0
Copy link
Member

ronso0 commented Mar 25, 2024

Okay, the fix was easy. It was just that the "is playing?" check was done too late, should happen right away after eject changed so that we always prohibit eject/un-replace for playing decks.

@ronso0
Copy link
Member

ronso0 commented Mar 25, 2024

PR coming soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants