Skip to content

Traktor S3: Fix script startup#11463

Closed
ywwg wants to merge 3 commits intomixxxdj:2.4from
ywwg:2.4
Closed

Traktor S3: Fix script startup#11463
ywwg wants to merge 3 commits intomixxxdj:2.4from
ywwg:2.4

Conversation

@ywwg
Copy link
Copy Markdown
Member

@ywwg ywwg commented Apr 10, 2023

This data hack no longer works due to the the controller performance changes, and it turns out it is not needed, so remove it.

This will be eventually obviated by #11199 but this controller is broken under 2.4 currently.

This data hack no longer works due to the the controller performance changes, and it turns out it is not needed, so remove it.
@ronso0
Copy link
Copy Markdown
Member

ronso0 commented Apr 10, 2023

Seems it's the same issue as #11461 ?
If so, could please take care of all affected mappings?

Swiftb0y added a commit to Swiftb0y/mixxx that referenced this pull request Apr 11, 2023
Swiftb0y added a commit to Swiftb0y/mixxx that referenced this pull request Apr 12, 2023
Swiftb0y added a commit to Swiftb0y/mixxx that referenced this pull request Apr 12, 2023
@ywwg
Copy link
Copy Markdown
Member Author

ywwg commented Apr 22, 2023

I think those are the only two affected mappings.

@ywwg
Copy link
Copy Markdown
Member Author

ywwg commented Apr 22, 2023

it looks like an alternative fix has been pulled in, but at least on the S3 the hack was totally unnecessary. I don't have an S2 to hack but I suspect it is also unneeded there.

@Swiftb0y
Copy link
Copy Markdown
Member

@ywwg #11470 has not been merged yet but is still awaiting manual testing. Do you have affected hardware to test this fix?

@JoergAtGithub
Copy link
Copy Markdown
Member

PR #11470 is merged now.

@Swiftb0y
Copy link
Copy Markdown
Member

Swiftb0y commented May 9, 2023

Superseeded by #11470 IIUC, closing. Feel free to reopen if this is still needed.

@Swiftb0y Swiftb0y closed this May 9, 2023
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