Lots of changes and bug fixes to implement generate-harmony that creates a new Midi with notes harmonized n steps above or below at a given tonic/pitchbase and Mode. Includes both the harmony and existing melody.
I fixed a few bugs in PitchClass so that should compile fine now once this is merged. I have a ton of tests that I will load up as well.
Please check the type of change your PR introduces:
[X] Bugfix
[X] Feature
[X] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):
What is the current behavior?
Issue Number: #12
What is the new behavior?
Added mode_steps function for mapping Modes variants to respective step arrays
Fixed bugs in Pitch Class that caused compilation errors
Pull Request type
Lots of changes and bug fixes to implement generate-harmony that creates a new Midi with notes harmonized n steps above or below at a given tonic/pitchbase and Mode. Includes both the harmony and existing melody.
I fixed a few bugs in PitchClass so that should compile fine now once this is merged. I have a ton of tests that I will load up as well.
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: #12
What is the new behavior?
Does this introduce a breaking change?
Other information