-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Refactor ArpDirection::Down and ArpDirection::DownAndUp #7007
Conversation
Fix issue with wrong pattern when going down and cycle is over 0.
325df3b
to
ab5bdc1
Compare
Merge? |
Nope. I think I just found a glitch. |
Fixed. Here is a demo project. It's just an arpeggio over four bars going down. On master the starting note of the arpeggio will change when you turn up Cycle. The start note should be the same no matter the setting of Cycle. |
Now merge? |
Found an older issue. More testing... |
I had failed testing some parts when developing the 'Repeats' function. This has now been taken care of in the last two commits. |
Fixes an issue with the arpeggiation when it is set to go downwards and the cycle is over 0.
Fix issue with wrong pattern when going down and cycle is over 0.
Working on #6478 I spotted an issue with the arpeggio pattern when using cycle. This issue was present on the initial implementation of the cycle function. In order to produce similar results, up and down they are now treated the same and later inverted if they are
ArpDirection::Down
orArpDirection::DownAndUp
.