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

Numark MixTrack (Pro) II mapping for Mixxx 2.1 #1540

Merged
merged 1 commit into from
Mar 28, 2018

Conversation

ninomp
Copy link
Contributor

@ninomp ninomp commented Mar 7, 2018

This PR adds mapping for Numark MixTrack (Pro) II controller to Mixxx

@ninomp ninomp force-pushed the Numark-MixTrack-II branch from 664eb30 to bb44499 Compare March 7, 2018 23:56
@Be-ing
Copy link
Contributor

Be-ing commented Mar 8, 2018

Thanks for submitting this. Please document this on the existing wiki page.

@Be-ing Be-ing added this to the 2.1.0 milestone Mar 8, 2018
@daschuer
Copy link
Member

Do you see a chance to finish this during this week?
It would be nice if this could be part of the release candidate.

@ninomp
Copy link
Contributor Author

ninomp commented Mar 19, 2018

I consider this done and ready to merge as it is currently.

Copy link
Contributor

@Be-ing Be-ing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for finally documenting and submitting a PR for this mapping that has been living on the forums for so long. I made edits to the wiki to clarify the language. There are a few things about the mapping that I have questions about, but for the most part it looks good.

var deck = NumarkMixTrackII.groupToDeck(group);

if (NumarkMixTrackII.shift_is_pressed[deck - 1]) {
engine.setValue(group, 'cue_gotoandstop', value);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to the wiki, this would use start_stop. I find start_stop more useful than cue_gotoandstop because cue_default already serves the function of going back to the cue point and stopping.

</control>
<control>
<group>[Master]</group>
<key>volume</key>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this control the hardware volume of the controller's built in audio interface? If so, it should not be mapped because the gain change will be applied twice, once in software then again in hardware.

</control>
<control>
<group>[Master]</group>
<key>headVolume</key>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like the master gain, if this controls the hardware volume of the controller's built in audio interface, it should not be mapped.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just checked. This knob does NOT change the hardware volume.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, thank you for checking.

@Be-ing Be-ing merged commit 6a42075 into mixxxdj:2.1 Mar 28, 2018
@s-fleck
Copy link

s-fleck commented May 26, 2018

Thanks for the mapping! The "Half Loop" button does nothing for me (but just midi-learning it is a workaround for me)

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

Successfully merging this pull request may close these issues.

4 participants