-
Notifications
You must be signed in to change notification settings - Fork 187
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
Add grace and arpeggios notes to the timemap #3956
Conversation
I am presuming that grace notes are given a non-zero duration in the MIDI file? Otherwise there can be problems with MIDI playback. |
That makes me wonder if this would be a chance to add support for |
I can add it for |
Having it for |
|
Good enough 😛 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks quite good, thank you! There is one issue with the grace note duration of unaccented grace notes. I don't understand if this was changed deliberately or accidentally.
Should be good to merge |
include/vrv/vrvdef.h
Outdated
@@ -73,6 +73,7 @@ namespace vrv { | |||
#define MIDI_TEMPO 120 | |||
|
|||
#define UNACC_GRACENOTE_DUR 27 // in milliseconds | |||
#define UNACC_GRACENOTE_FRACTION Fraction(1, 2024) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is 2024
correct? 1024
or 2048
would make more sense?
This PR adds the grace notes to the timemap. Fixes #3909
A new functor has been added to do the gracenotes adjustment when preparing the timemap instead of doing it when producing the MIDI. The MIDI output is simplified and uses the timemap values.
Marked as draft for now since I will do the same for arpeggios and tablature note extension.
MEI
Timemap
Example use of the timemap with grace note highlighting (new)
Screen.Recording.2025-02-16.at.11.49.02.mov