cienicera / Koji

The First Onchain Generative Music Library. Deployed on Starknet
9 stars 11 forks source link

Remap instruments #26

Closed caseywescott closed 1 year ago

caseywescott commented 1 year ago

Here is the implementation.

Remap_Instruments iterates through MIDI messages and increments the Control change value argument within instrument groups.

I'm uncertain about the exact pattern to use as some devices use MSB+LSB which would require finding adjacent cc msgs prior to selecting a new instrument.

Pull Request type

Please check the type of change your PR introduces:

What is the current behavior?

Issue Number: #10

What is the new behavior?

-

-

Does this introduce a breaking change?

Other information

raphaelDkhn commented 1 year ago

LGTM! For the right pattern to use I think let's start with that and in the future we refactor it

caseywescott commented 1 year ago

Sounds like a plan!