-
-
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
feat(controllers): add controller settings for the Numark Scratch #13404
Conversation
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.
LGTM. I don't have much experience with the settings though. Do you spot anything wrong @acolombier?
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.
Glad to see this feature being used. Just added some very minor comments, otherwise LGTM
@acolombier this is ready. @Swiftb0y can someone please merge the 2.4 manual into 2.5, so I can update the controller manual. |
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.
Unfortunately, the beatloop value half
and double
cannot be used without some special handling, since beatlooproll_half_activate
and beatlooproll_double_activate
don't exist (and wouldn't make sense)
The handling should be fairly easy to add here
Done ;) |
@acolombier I reverted my last commit. Thinking about this more the mixer has only 4 pads instead of 8 and using up 2 pads might not be ideal. Let's merge as is and ill investigate more if it's worth it when I'm back from holidays. |
Oops clicked the dreaded comment and close by mistake. |
Great. Can you do me a favor and rebase out the reverted commit so it doesn't unnecessarily end up in the commit history? |
341bc38
to
6d5cd6a
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.
Thank you. LGTM.
add controller settings for the Numark Scratch , inspired by the s4mk3