Curious, what is the goal of this?
Seablade
: D
easy find scale and chord position
We will. – See the comments on the pull-request.
We already have infrastructure prepared for this which the patch does not yet use.
So in order to make this maintainable and avoid duplicating work, the patch needs to be consolidated a bit before we accept it.
But yes, it’s about 90% there.
It’s also missing all the Messaien octatonic scales, and Locrian flat-2. Unacceptable with these limitations.
precisely. From the PR:
- I think hardcoding scales is not a good idea, Scales should be dynamically loaded and extensible. So I’d not define enums for root notes and scales. That is limiting down the road.
- The code should eventually be consolidated with
libs/surfaces/push2/scale.cc
Also ParameterDescriptor::midi_note_name()
should be used to make note names translatable. e.g. German uses “H” instead of “B” etc.
Furthermore it’d be nice to have this not per track but globally for all tracks, and allow to change the scale at given points on the timeline, since many compositions change key.
This topic was automatically closed 91 days after the last reply. New replies are no longer allowed.