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

Update doc to for XDJ-RX. #53

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

fragmede
Copy link

@fragmede fragmede commented Jul 2, 2024

Document that the XDJ-RX won't crash with a player number of 0.

@brunchboy
Copy link
Member

Interesting. Have you tested with all non-DJ-Link devices and confirmed that none of them crash? And what is the value of doing this, given that they do not support Pro DJ Link?

@fragmede
Copy link
Author

fragmede commented Jul 2, 2024

Whoops, pushed the wrong commit. Only meant to update the documentation.

@fragmede
Copy link
Author

fragmede commented Jul 2, 2024

9aa1b69 is all I wanted to add.

@brunchboy
Copy link
Member

My question remains: why would someone try to use dysentery with non-DJ Link devices?

@fragmede
Copy link
Author

fragmede commented Jul 2, 2024

Just exploring what my friend's XDJ-RX is doing when I hook it into my network. It grabs an IP address and then starts spamming a bunch of UDP packets and I'm interested in seeing what they are.

@brunchboy
Copy link
Member

Interesting. However, for such exploration I think it would be better to take the path that Kris Prep did and create a simulated rekordbox instance, rather than a virtual CDJ. These controllers are supposed to work with rekordbox, not CDJs, so you will want to send packets that look like the ones rekordbox sends, and that use device numbers that rekordbox uses. Just changing your device number to zero here (not valid for any known Pro DJ Link participant) and sending CDJ-like packets is not likely to get nearly as far.

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.

2 participants