Skip to content

Update FaderFoxDJ2.midi.xml#867

Merged
daschuer merged 4 commits intomixxxdj:masterfrom
tarche:tarche-FaderFoxDJ2-update
Aug 11, 2016
Merged

Update FaderFoxDJ2.midi.xml#867
daschuer merged 4 commits intomixxxdj:masterfrom
tarche:tarche-FaderFoxDJ2-update

Conversation

@tarche
Copy link
Copy Markdown
Contributor

@tarche tarche commented Jan 21, 2016

Hi,
This is a small update to the FaderFoxDJ2 mapping.
Hope that everything is fine, this is my first commit.
You can see changes on this forum thread : http://www.mixxx.org/forums/viewtopic.php?f=7&t=7865

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Jan 21, 2016

Thank you for submitting this! Could you update http://mixxx.org/wiki/doku.php/faderfox_dj2 to explain how the mapping works? See http://mixxx.org/wiki/doku.php/contributing_mappings#documenting_the_mapping for instructions and http://mixxx.org/wiki/doku.php/hardware_compatibility for some examples (not every mapping has good documentation, which is what we'd like to improve).

@snue
Copy link
Copy Markdown
Contributor

snue commented Jan 24, 2016

@tarche @Be-ing those are different opcodes by the same control on the same (first) midi channel. The 0x90 is the button press ("Note On"), 0x80 is release ("Note off"). Make sure things like "LoadSelectedIntoFirstStopped" are not executed twice on a single button click (press+release).

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Mar 18, 2016

Considering that the author has not replied for months and this controller is quite simple, I think this should be merged as-is even if the documentation is absent.

@tarche
Copy link
Copy Markdown
Contributor Author

tarche commented Mar 18, 2016

Hi Be.
I've documented the wiki with the few points that are not clearly labeled. Is that alright ?

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Mar 18, 2016

That's a good start, but there are still some functions that aren't clear to me from looking at the picture.

  • What does the joystick do?
  • What does the lower black button by the encoders do?
  • What do the FX1/FX2 LEDs at the top indicate?
  • I'm guessing the blue buttons do temporary pitch bending. Looking at the XML, I'm guessing they do small temporary pitch bending with shift pressed.
  • The labels on the grey buttons are confusing. I see one with a play/pause icon, one with a pause icon, and one with a play icon. In the XML, I see one is mapped to play, another to cue_set, another cue_goto, and another to cue_preview, but those labels don't clearly tell me which one is which. Mappings typically only have one button for the cue point mapped to cue_default, which changes behavior depending on the cue mode the user set in Mixxx's preferences.
  • Do the grey buttons send different signals with shift held down? If so, it would be useful to map them to some additional functions like quantize, keylock, and beatjump_1_forward/backward.

@tarche
Copy link
Copy Markdown
Contributor Author

tarche commented Aug 10, 2016

Hi, sorry for the late update. I've been very busy but had time to work on the mapping during the holidays. I've modified the mapping according to your remarks and documented the wiki accordingly. Feel free to correct any mistakes, my English is far from perfect.

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Aug 11, 2016

Great, thank you! I made a few small edits to the wiki, but mostly it was good. :) I have one last small comment on the functionality. Considering that there is an encoder on each side, I think that pushing the encoder in List/Load mode should load the selected track to the corresponding deck, not the first stopped deck. Loading to the first stopped deck makes more sense when there is a single encoder on the controller for browsing the library.

@tarche
Copy link
Copy Markdown
Contributor Author

tarche commented Aug 11, 2016

Done. I actually never realized there was such a control as LoadSelectedTrack because I was looking in the Playlist group and only saw LoadSelectedIntoFirstStopped.
Thanks a lot for your help !

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Aug 11, 2016

Looks good to me! 👍

@daschuer
Copy link
Copy Markdown
Member

Thank you very much for the work!

@daschuer daschuer merged commit e645817 into mixxxdj:master Aug 11, 2016
@tarche tarche deleted the tarche-FaderFoxDJ2-update branch August 11, 2016 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants