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 supported devices #20

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

Conversation

haleo9000
Copy link

This was how I got my Launchpad Pro (Mk2) working correctly. Print output:

Launchpad Pro 1	 -- Not supported
###Midi device	Launchpad Pro 2
launchpad_rgb	 -- Supported
###Midi device	Launchpad Pro 3
Launchpad Pro 3	 -- Not supported

This was how I got my Launchpad Pro (Mk2) working correctly. Print output: 
```
Launchpad Pro 1	 -- Not supported
###Midi device	Launchpad Pro 2
launchpad_rgb	 -- Supported
###Midi device	Launchpad Pro 3
Launchpad Pro 3	 -- Not supported
```
@@ -32,6 +32,10 @@ local supported_devices = {
{ midi_base_name= 'launchpad pro mk3', device_type='launchpad_rgb' },
{ midi_base_name= 'launchpad x 2', device_type='launchpad_x' },
{ midi_base_name= 'launchpad x 2 2', device_type='launchpad_x' },
{ midi_base_name= 'launchpad Pro 1', device_type='launchpad_rgb' },
{ midi_base_name= 'launchpad Pro 2', device_type='launchpad_rgb' },
{ midi_base_name= 'launchpad Pro 3', device_type='launchpad_rgb' },
Copy link
Owner

Choose a reason for hiding this comment

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

So these devices seem to create 3 USB interfaces, 1. is normally the proprietary interface, 2. is the general midi interface and 3 is something else? In short I think you only need to add 'launchpad pro 2'. Could you please try this modification and confirm if it works?

Thanks and sorry for the very late response.

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