eduardomourar / fingering-diagram

A plugin for MuseScore to display instrument Fingering Diagram below each note
https://musescore.org/en/project/fingering-diagram
GNU General Public License v3.0
20 stars 5 forks source link

Support for Tin Whistle and Low Whistle fingering #9

Closed jgadsden closed 4 years ago

jgadsden commented 4 years ago

It would be good to add a new tab for the Tin Whistle and Low Whistle. These whistles come in various sizes that are tuned to various keys:

Here are the steps we will take to achieve this:

jgadsden commented 4 years ago

Yes, looks a good plan @eduardomourar . A suggestion may be to add the D Tin Whistle to the sample MuseScore file rather than one tuned to C, as D tuning is much more common than C tuning for Tin Whistles. In the music shops typically about 80% to 90% are D whistles, the rest being a mixed bag of C, Bflat, Eflat and maybe F and G.

jgadsden commented 4 years ago

Hello @eduardomourar - nearly finished the Tin Whistle tab for the spreadsheet. The half notes have me confused though, could you supply the unicode for the halfnotes :

Many thanks, I can then put a pull request to your branch feature/add-whistles

eduardomourar commented 4 years ago

You can find the list of unicode characters and its description here: https://github.com/eduardomourar/fiati/blob/master/source/glyph_data.csv

Respectively, \uE3DF, \uE3E1, \uE3E2, \uE3E5, \uE3E6. Also, you can check the unicode for the '+' at the bottom woodwindOverblownOctave1 and '++' is woodwindOverblownOctave2.

eduardomourar commented 4 years ago

@jgadsden, I finished #12. Let me know what you think and then we can merge it.

jgadsden commented 4 years ago

Hello @eduardomourar - generally looking excellent, thankyou, but with one bug that is easily fixed. This is an edge case, but when more than one whistle staff is included in the score then the D Tin Whistle Staff will use whatever 'transpose' is currently set - which is '0' initially, but if a staff for a tin whistle has already been processed with a 'transpose' value different to 0 (say 2 for a C Tin Whistle) then the D Tin Whistle staff fingering will use this 'transpose' value instead of the correct value of '0'.

To test this, construct MuseScore score with 3 staves in order 'D Tin Whistle', 'C Tin Whistle' and 'D Tin Whistle'. Apply fingeringdiagram.qml plugin and observe first two staves are correct, but 3rd staff has C tuning not the expected D tuning. Alternatively this can be observed when using plugin for the score https://github.com/jgadsden/tin-whistle-tablature/tree/master/examples/tin_whistle_test.mscz

jgadsden commented 4 years ago

Good results from #12 - many thanks. This looks better than https://github.com/jgadsden/tin-whistle-tablature - will add to the MuseScore discussion groups.