Added new scale selection highlighting in piano roll + refactored music theory utils - #2396
Open
AntoineHazebrouck wants to merge 5 commits into
Open
Conversation
Contributor
Collaborator
|
Nice first PR! Thank you for adding images, details, and the link back to the relevant issue, makes things much easier to check over. :) |
Author
Collaborator
|
First-time contributors' checks need to be manually approved, if I remember correctly. Might just be a quirk, a few of our test systems need a bit of ironing out. From here our primary maintainer StAkira will review the code , approve the workflow tests, and merge if he has no revisions. I've tagged him in for review for you. We can go a long while between stable version releases, but once merged your feature will be available in the next alpha build (which is nightly), and hopefully moved quickly into beta which more people tend to use. 😊 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


(first dev on any open source project, I'm new so I don't know the process and all)
Following this feature request : #2005
Behavior comparison
Default scale is C Ionian or C major (this did not change), users can still select the Key (aka tonic)
Users can now also select the mode out of the main ones, here we can for example have A minor (relative of C major) which was not possible before :
Before that, we could only have C major for the proper notes but the wrong degree intervals if we wanted A minor.
Also the notes highlighting (black and white keys) only followed a traditionnal piano, now the highlighting clearly follows the scale that you play in. The selected tonic is now highlighted in light blue whereas only the C note was before.
If users wish to keep the traditionnal piano highlighting, they can keep C major as the scale.
Implementation details
Uproject now has a new field : "mode" which is reflected by the dropdown menu shown above. Everything related to music theory AND required for this feature has been moved to this namespace : "OpenUtau.Core.Util.MusicTheory".