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

Add colored hotcues for Traktor Kontrol S2 MK3 #4637

Merged
merged 6 commits into from
Jan 26, 2023

Conversation

goddisignz
Copy link
Contributor

Title already says it. I implemented the Mixxx color palette for the hotcue pads of the Traktor Kontrol S2 MK3.

Co-authored-by: ronso0 <ronso0@mixxx.org>

};

// Mixxx hotcue colors
Copy link
Member

Choose a reason for hiding this comment

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

These are Traktor's hotcue colors, not Mixxx', right? These RGB values should be the equivalent of the color you actually see on the controller when you send the midi value.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nono, this should be the RGB-Colors, the mixxx palette uses. I tried to find a very close color on the pads for each of the palette colors. It is hard to determine the exact RGB-colors the pads have but it should be close.

Copy link
Member

Choose a reason for hiding this comment

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

Nono, this should be the RGB-Colors, the mixxx palette uses. I tried to find a very close color on the pads for each of the palette colors.

This is not how the color mapper works. The RGB colors should resemble the pads as close as possible. The Mixxx palette is unrelated, and can be configured with arbitrary colors anyway.

It is hard to determine the exact RGB-colors the pads have but it should be close.

I suppose for most of the colors you can take a screenshot of the Traktor GUI and then use a color picker. I'm pretty sure these colors match the LEDs on the controller. Then you know the exact RGB values of green (saved loops), blue (hot cues), white (grid markers), yellow (load in) and orange (fade in/out). The RGB colors (purple/pink/celeste/red) need to be guessed.

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 have no windows PC so I cannot run traktor. However, I took photos of every possible pad color and analyzed them in gimp using the color picker with an average filter. The result that I see now for the hotcue colors I use looks pretty nice.

Comment on lines 439 to 441
engine.setValue(field.group, "LoadSelectedTrack", field.value);
engine.setValue(field.group, "pfl", 1);
}
Copy link
Member

Choose a reason for hiding this comment

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

this is unrelated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, sorry, force of habit. I am always thinking of an option in the settings for enabling PFL on track loading.

@goddisignz
Copy link
Contributor Author

BTW: Good to see you guys are still here. I didn't contribute for a while but I will try to do little things again in the future.

@github-actions
Copy link

This PR is marked as stale because it has been open 90 days with no activity.

@github-actions github-actions bot added the stale Stale issues that haven't been updated for a long time. label Apr 29, 2022
Copy link
Member

@JoergAtGithub JoergAtGithub left a comment

Choose a reason for hiding this comment

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

I read through the changes, and from my point of view everything looks fine. If nobody has objections, I will merge this.

@JoergAtGithub
Copy link
Member

  • Code looks fine to me
  • Changes require ES6, therefore Main is the correct target

@JoergAtGithub JoergAtGithub merged commit 80fed1d into mixxxdj:main Jan 26, 2023
@JoergAtGithub
Copy link
Member

@goddisignz Thank you, for your contribution!

@JoergAtGithub JoergAtGithub added changelog This PR should be included in the changelog and removed stale Stale issues that haven't been updated for a long time. labels Jan 26, 2023
@daschuer
Copy link
Member

Is this a candidate for back-porting to the stable 2.3 branch es well?

@JoergAtGithub
Copy link
Member

This will not work on 2.3 as it is, due to ES6 syntax. But it should go into 2.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog This PR should be included in the changelog controller mappings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants