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

Fix two LP Mk3 device setup for 128 grid - to jaggednz/midigrid fork #3

Merged
merged 2 commits into from Feb 8, 2021

Conversation

@robmckinnon
Copy link

@robmckinnon robmckinnon commented Feb 6, 2021

Fix two LP Mk3 device setup for 128 grid.

Without this change log is:

  finding device: 2 with name Launchpad Mini MK3 1
  finding device: 3 with name Launchpad Mini MK3 2
  Dev3 FD launchpad_minimk3
  finding device: 4 with name Launchpad Mini MK3 1 2
  finding device: 5 with name Launchpad Mini MK3 2 2
  Dev5 FD launchpad_minimk3
  mounted_devices
  5	launchpad_minimk3
  3	launchpad_minimk3

With this change log is:

  finding device: 2 with name Launchpad Mini MK3 1
  finding device: 3 with name Launchpad Mini MK3 2
  Dev3 FD launchpad_minimk3
  finding device: 4 with name Launchpad Mini MK3 1 2
  finding device: 5 with name Launchpad Mini MK3 2 2
  Dev5 FD launchpad_minimk3_128
  mounted_devices
  5	launchpad_minimk3_128
  3	launchpad_minimk3

Also tested this change with LP Mk2 and APC mini. It did not effect the loading behaviour of these devices.

Added comments to code explaining LP MIDI device names.

robmckinnon added 2 commits Feb 6, 2021
Without this change log is:

  finding device: 2 with name Launchpad Mini MK3 1
  finding device: 3 with name Launchpad Mini MK3 2
  Dev3 FD launchpad_minimk3
  finding device: 4 with name Launchpad Mini MK3 1 2
  finding device: 5 with name Launchpad Mini MK3 2 2
  Dev5 FD launchpad_minimk3
  mounted_devices
  5	launchpad_minimk3
  3	launchpad_minimk3

With this change log is:

  finding device: 2 with name Launchpad Mini MK3 1
  finding device: 3 with name Launchpad Mini MK3 2
  Dev3 FD launchpad_minimk3
  finding device: 4 with name Launchpad Mini MK3 1 2
  finding device: 5 with name Launchpad Mini MK3 2 2
  Dev5 FD launchpad_minimk3_128
  mounted_devices
  5	launchpad_minimk3_128
  3	launchpad_minimk3

Also tested this change with LP Mk2 and APC mini. It did
not effect the loading behaviour of these devices.
@jaggednz jaggednz merged commit 226ffb0 into jaggednz:master Feb 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants