Traktor Kontrol S2 MK3 Hotcue LED issue

[Joskagyerek3000]

Bug Description

This is in the 2.4.0 beta version!

With the built-in mapping for the Traktor Kontrol S2 MK3, the hotcue pads only light up or change colors when I press the "HOTCUES" switch button. Otherwise, they don't give any feedback. In other words, it seems like that the 'HOTCUES' button makes a refresh to the hotcue LEDs, otherwise they don't show any changes in light or color.

Version

2.4.0

OS

Windows 11

[Swiftb0y]

Hey there, thank you for filling your issue. In order to get to the bottom of this. We would like to inspect your log file. Please use our guide to find it and then upload it here and we'll see if we can spot anything in the log. Thank you.

[Joskagyerek3000]

Hello, here is the log file, I hope it helps. If I can do anything, or You need any more information, I am happy to help further!

Thanks!

Swiftb0y @.***> ezt írta (időpont: 2023. okt. 17., K, 20:22):

Hey there, thank you for filling your issue. In order to get to the bottom of this. We would like to inspect your log file. Please use our guide to find it https://github.com/mixxxdj/mixxx/wiki/Finding%20the%20Mixxx.log%20file and then upload it here and we'll see if we can spot anything in the log. Thank you.

— Reply to this email directly, view it on GitHub https://github.com/mixxxdj/mixxx/issues/12130#issuecomment-1766937039, or unsubscribe https://github.com/notifications/unsubscribe-auth/BDLF4SO47DEGQKP4IK3NHV3X73EEXAVCNFSM6AAAAAA6EHSQNKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRWHEZTOMBTHE . You are receiving this because you authored the thread.Message ID: @.***>

[ronso0]

Attachments don't show up on Github if you reply via email.

[Joskagyerek3000]

mixxx.log

sorry, here it is :)

[subtruth]

Same problem, same version version 2.4.1, different OS (Lubuntu 24.04)

mixxx.log

[Swiftb0y]

@acolombier or @ywwg any chance you can look into this?

[lc-guy]

Getting this on 2.5.0-beta (Windows 10) as well.

It seems to me that the s2 mk3 script only calls padModeHandler when a pad button is clicked, and that is the only function where the colors and lights are set for cues. Maybe it should be called when a hotcue is updated?

[ywwg]

I do not own this controller, unfortunately. I don't know who else in mixxx has an s2

[hofq]

I do not own this controller, unfortunately. I don't know who else in mixxx has an s2

i'd be happy to assist and test code snippets if needed.

[acolombier]

Sorry @Swiftb0y just seen your tag. I also don't own that controller either unfortunately, and the mapping is quite different to the S4, so even debugging in the dark might be hard sadly..

[Swiftb0y]

True. Maybe @mi01 (original author) or @olivier-mauras can help out?

[Swiftb0y]

unfortunately the issue seems to interact with the common-hid-packet-parser.js library which I'm unfamiliar with. The offending code is here: https://github.com/mixxxdj/mixxx/blob/b64251f1301c36c364c69da9348cd068d0b995dc/res/controllers/Traktor-Kontrol-S2-MK3-hid-scripts.js#L888-L894

The log file gives a pretty good indication of the cause:

Critical [Controller] HIDController.setOutput - Unknown field: [Channel2].hotcue_1_enabled
Warning [Controller] HIDController.linkOutput - Output already linked: [Channel1]
Warning [Controller] HIDController.linkOutput - Output already linked: [Channel2]

Maybe @JoergAtGithub can have a look?

[JoergAtGithub]

This seems to be a regression of #4637. Colored hotcues are a new mapping feature of 2.4.0.

The message

Warning [Controller] HIDController.linkOutput - Output already linked: [Channel1]

are easy to explain, because "[Channel1]", "pad_" + i is linked twice in the loop. But I do not see the reason for the Unknown field error logged before yet.

[Swiftb0y]

Thanks for the explanation. Well @goddisignz may be able to fix the code then.

[goddisignz]

I will try to have a look at this next week.

[Swiftb0y]

Thanks :+1: :+1: :+1: