-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
controllers: ensure required samplers are created #12769
Conversation
I'm pretty sure eslint will complain, but I'm not going to fix those issues (except those introduced by this PR, and minimal changes). |
I just so happened to accidentally delete my (pretty much unchanged apart from jog wheel LED rotation speed values) Reloop Beatmix 2/4 .js, now getting this, doh! |
Oh, so with the BeatMix mapping you also get complaints about missing controls for sampler 5++? Somehow I assumed (didn't double-check) skins were loaded prior to any mapping, and Mixxx provides skins with min. 8 samplers, so #12747 would have happened with a custom skin. |
I was going to double check so I changed back to the default controller preset and at that point the waveform changed to this!; That rings a bell though.. did someone mention it somewhere recently? Can't find an issue with a brief search Anyway, the error on start;
|
67a0e7d
to
2919461
Compare
2919461
to
602bc6f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we also need an update to traktor s3?
I'm pretty sure I checked all the Traktor mappings. But I didn't test them since I didn't set up a virtual HID device. Do you still have an S3? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes are looking reasonable. Thank you.
temp checklist
|
This PR simply ensures mappings request all required samplers being created, exactly like they set
[App],num_decks
if they need more than 2 decks.With #12657 some mapping flaws produce mapping warnings, and users may not be able to resolve the issues.
This will occur with a fresh profile or when users don't have any tracks loaded in samplers with numbers higher than the deafult (4) or skin minimum (8, Shade skin).
After having merged #12657 this is just much more likely to happen.
To reproduce:
load Shade skindoesn't matter, mappings are loaded before skins are created https://mixxx.zulipchat.com/#narrow/stream/109171-development/topic/2.2E4.20samplers.20regressionThere may be more mappings that affected in a different way, I just did a quick check where
[Sampler
is used withhttps://github.com/search?q=repo%3Amixxxdj%2Fmixxx+%22%5BSampler+path%3A%2F%5Eres%5C%2Fcontrollers%5C%2F%2F&type=code&p=2