From 942491e4e2174a7edb3bf120aaf319eecc26b35b Mon Sep 17 00:00:00 2001 From: James Mizen Date: Wed, 12 Feb 2025 14:05:39 +0000 Subject: [PATCH 1/2] Create parent percussion/drumkit instrument on xml import --- .../musicxml/import/importmusicxmlpass2.cpp | 27 +- .../musicxml/tests/data/testMS3KitAndPerc.xml | 3471 +++++++++++++++++ .../tests/data/testMS3KitAndPerc_ref.mscx | 3115 +++++++++++++++ .../musicxml/tests/data/testSticking_ref.mscx | 21 +- .../musicxml/tests/musicxml_tests.cpp | 3 + 5 files changed, 6617 insertions(+), 20 deletions(-) create mode 100644 src/importexport/musicxml/tests/data/testMS3KitAndPerc.xml create mode 100644 src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx diff --git a/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp b/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp index c805c9ba9d79c..cf1aaaae76d5f 100644 --- a/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp +++ b/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp @@ -622,6 +622,20 @@ static void updatePartWithInstrumentChange(Part* const part, const MusicXmlInstr } } +static void setPercussionInstrument(MusicXmlInstrument& mxmlInstr, const String& partName) +{ + // If there are multiple unpitched instruments we should set the part's instrument to drumkit or percussion + if (partName.contains(u"drumset", + CaseSensitivity::CaseInsensitive) || partName.contains(u"drumkit", CaseSensitivity::CaseInsensitive)) { + mxmlInstr = MusicXmlInstrument(); + mxmlInstr.name = u"Drum Kit"; + } else if (partName.contains(u"percussion", CaseSensitivity::CaseInsensitive)) { + mxmlInstr = MusicXmlInstrument(); + mxmlInstr.name = u"Percussion"; + mxmlInstr.sound = u"drum.group.set"; + } +} + //--------------------------------------------------------- // setPartInstruments //--------------------------------------------------------- @@ -638,7 +652,8 @@ static void setPartInstruments(MusicXmlLogger* logger, const XmlStreamReader* xm const Score* score, const MusicXmlInstrList& instrList, const MusicXmlIntervalList& intervList, - const MusicXmlInstruments& instruments) + const MusicXmlInstruments& instruments, + const String& partName) { if (instruments.empty()) { // no instrument details found, create a default instrument @@ -651,6 +666,9 @@ static void setPartInstruments(MusicXmlLogger* logger, const XmlStreamReader* xm // do not create multiple instruments for a drum part //LOGD("hasDrumset"); MusicXmlInstrument mxmlInstr = instruments.begin()->second; + if (instruments.size() > 1) { + setPercussionInstrument(mxmlInstr, partName); + } updatePartWithInstrument(part, mxmlInstr, {}, true); return; } @@ -2141,13 +2159,11 @@ void MusicXmlParserPass2::part() const MusicXmlInstruments& instruments = m_pass1.getInstruments(id); m_hasDrumset = hasDrumset(instruments); - // set the parts first instrument - Part* part = m_pass1.getPart(id); - setPartInstruments(m_logger, &m_e, part, id, m_score, m_pass1.getInstrList(id), m_pass1.getIntervals(id), instruments); - // set the part name + Part* part = m_pass1.getPart(id); MusicXmlPart mxmlPart = m_pass1.getMusicXmlPart(id); String partName = mxmlPart.getName(); + setPartInstruments(m_logger, &m_e, part, id, m_score, m_pass1.getInstrList(id), m_pass1.getIntervals(id), instruments, partName); partName = replacePartNameAccidentals(partName); part->setPartName(partName); if (mxmlPart.getPrintName() && !isLikelyIncorrectPartName(partName)) { @@ -2160,6 +2176,7 @@ void MusicXmlParserPass2::part() } else { m_pass1.getPart(id)->setPlainShortNameAll(u""); } + // set the parts first instrument // try to prevent an empty track name if (part->partName() == "") { String instrId = m_pass1.getInstrList(id).instrument(Fraction(0, 1)); diff --git a/src/importexport/musicxml/tests/data/testMS3KitAndPerc.xml b/src/importexport/musicxml/tests/data/testMS3KitAndPerc.xml new file mode 100644 index 0000000000000..9882df65b7717 --- /dev/null +++ b/src/importexport/musicxml/tests/data/testMS3KitAndPerc.xml @@ -0,0 +1,3471 @@ + + + + + Title + + + Composer + + MuseScore 3.6.2 + 2025-01-22 + + + + + + + + + + 6.99911 + 40 + + + 1696.94 + 1200.48 + + 85.7252 + 85.7252 + 85.7252 + 85.7252 + + + 85.7252 + 85.7252 + 85.7252 + 85.7252 + + + + + + + title + Title + + + composer + Composer + + + + bracket + + + Drumset + D. Set + + Acoustic Bass Drum + + + Bass Drum 1 + + + Side Stick + + + Acoustic Snare + + + Electric Snare + + + Low Floor Tom + + + Closed Hi-Hat + + + High Floor Tom + + + Pedal Hi-Hat + + + Low Tom + + + Open Hi-Hat + + + Low-Mid Tom + + + Hi-Mid Tom + + + Crash Cymbal 1 + + + High Tom + + + Ride Cymbal 1 + + + Chinese Cymbal + + + Ride Bell + + + Tambourine + + + Splash Cymbal + + + Cowbell + + + Crash Cymbal 2 + + + Ride Cymbal 2 + + + Open Hi Conga + + + Low Conga + + + + 10 + 1 + 36 + 78.7402 + 0 + + + 10 + 1 + 37 + 78.7402 + 0 + + + 10 + 1 + 38 + 78.7402 + 0 + + + 10 + 1 + 39 + 78.7402 + 0 + + + 10 + 1 + 41 + 78.7402 + 0 + + + 10 + 1 + 42 + 78.7402 + 0 + + + 10 + 1 + 43 + 78.7402 + 0 + + + 10 + 1 + 44 + 78.7402 + 0 + + + 10 + 1 + 45 + 78.7402 + 0 + + + 10 + 1 + 46 + 78.7402 + 0 + + + 10 + 1 + 47 + 78.7402 + 0 + + + 10 + 1 + 48 + 78.7402 + 0 + + + 10 + 1 + 49 + 78.7402 + 0 + + + 10 + 1 + 50 + 78.7402 + 0 + + + 10 + 1 + 51 + 78.7402 + 0 + + + 10 + 1 + 52 + 78.7402 + 0 + + + 10 + 1 + 53 + 78.7402 + 0 + + + 10 + 1 + 54 + 78.7402 + 0 + + + 10 + 1 + 55 + 78.7402 + 0 + + + 10 + 1 + 56 + 78.7402 + 0 + + + 10 + 1 + 57 + 78.7402 + 0 + + + 10 + 1 + 58 + 78.7402 + 0 + + + 10 + 1 + 60 + 78.7402 + 0 + + + 10 + 1 + 64 + 78.7402 + 0 + + + 10 + 1 + 65 + 78.7402 + 0 + + + + Percussion + Perc. + + Closed Hi-Hat + + + Pedal Hi-Hat + + + Open Hi-Hat + + + Ride + + + Concert Bass Drum + + + Snare Side Stick + + + Concert Snare Drum + + + Castanets + + + Tambourine + + + Splash Cymbal + + + Cowbell + + + Vibraslap + + + Concert Cymbal + + + Mute High Conga + + + Open High Conga + + + Low Conga + + + Maracas + + + Claves + + + Open Triangle + + + Shaker + + + Mark Tree + + + + 10 + 49 + 28 + 78.7402 + 0 + + + 10 + 49 + 29 + 78.7402 + 0 + + + 10 + 49 + 30 + 78.7402 + 0 + + + 10 + 49 + 31 + 78.7402 + 0 + + + 10 + 49 + 37 + 78.7402 + 0 + + + 10 + 49 + 38 + 78.7402 + 0 + + + 10 + 49 + 39 + 78.7402 + 0 + + + 10 + 49 + 40 + 78.7402 + 0 + + + 10 + 49 + 55 + 78.7402 + 0 + + + 10 + 49 + 56 + 78.7402 + 0 + + + 10 + 49 + 57 + 78.7402 + 0 + + + 10 + 49 + 59 + 78.7402 + 0 + + + 10 + 49 + 60 + 78.7402 + 0 + + + 10 + 49 + 63 + 78.7402 + 0 + + + 10 + 49 + 64 + 78.7402 + 0 + + + 10 + 49 + 65 + 78.7402 + 0 + + + 10 + 49 + 71 + 78.7402 + 0 + + + 10 + 49 + 76 + 78.7402 + 0 + + + 10 + 49 + 82 + 78.7402 + 0 + + + 10 + 49 + 83 + 78.7402 + 0 + + + 10 + 49 + 85 + 78.7402 + 0 + + + + + + + + + + 116.46 + 0.00 + + 170.00 + + + + 1 + + 0 + + + + percussion + 2 + + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + + + A + 4 + + 1 + + 1 + quarter + up + + + + A + 4 + + 1 + + 1 + quarter + up + + + + A + 4 + + 1 + + 1 + quarter + up + + + + A + 4 + + 1 + + 1 + quarter + up + + + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + + + D + 5 + + 1 + + 1 + quarter + up + + + + D + 5 + + 1 + + 1 + quarter + up + + + + D + 5 + + 1 + + 1 + quarter + up + + + + D + 5 + + 1 + + 1 + quarter + up + + + + + + + 72.86 + 0.00 + + 108.78 + + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + + + E + 5 + + 1 + + 1 + quarter + up + + + + E + 5 + + 1 + + 1 + quarter + up + + + + E + 5 + + 1 + + 1 + quarter + up + + + + E + 5 + + 1 + + 1 + quarter + up + + + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + + + + 72.86 + 0.00 + + 108.78 + + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + + + + 72.86 + 0.00 + + 108.78 + + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + + + + 72.86 + 0.00 + + 108.78 + + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + + + + 72.86 + 0.00 + + 108.78 + + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + + + + 72.86 + 0.00 + + 70.00 + + + + + A + 4 + + 1 + + 1 + quarter + up + + + + A + 4 + + 1 + + 1 + quarter + up + + + + A + 4 + + 1 + + 1 + quarter + up + + + + A + 4 + + 1 + + 1 + quarter + up + + + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + + 72.86 + 0.00 + + 229.41 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + + 72.86 + -0.00 + + 229.41 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + + 72.86 + 0.00 + + 229.41 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + light-heavy + + + + + + + + 43.51 + + + + 1 + + 0 + + + + percussion + 2 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 43.51 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 43.51 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 43.51 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 43.51 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 43.51 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 91.76 + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 91.76 + + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + + + E + 5 + + 1 + + 1 + quarter + up + + + + E + 5 + + 1 + + 1 + quarter + up + + + + E + 5 + + 1 + + 1 + quarter + up + + + + E + 5 + + 1 + + 1 + quarter + up + + + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + D + 5 + + 1 + + 1 + quarter + up + diamond + + + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + B + 5 + + 1 + + 1 + quarter + up + x + + + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + E + 5 + + 1 + + 1 + quarter + up + inverted triangle + + + + + + E + 5 + + 1 + + 1 + quarter + up + diamond + + + + E + 5 + + 1 + + 1 + quarter + up + diamond + + + + E + 5 + + 1 + + 1 + quarter + up + diamond + + + + E + 5 + + 1 + + 1 + quarter + up + diamond + + + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + A + 5 + + 1 + + 1 + quarter + up + x + + + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + + + 91.76 + + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + B + 4 + + 1 + + 1 + quarter + up + x + + + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + G + 4 + + 1 + + 1 + quarter + up + x + + + + + + D + 5 + + 1 + + 1 + quarter + up + + + + D + 5 + + 1 + + 1 + quarter + up + + + + D + 5 + + 1 + + 1 + quarter + up + + + + D + 5 + + 1 + + 1 + quarter + up + + + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + F + 5 + + 1 + + 1 + quarter + up + + + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + F + 5 + + 1 + + 1 + quarter + up + diamond + + + + + + B + 4 + + 1 + + 1 + quarter + up + + + + B + 4 + + 1 + + 1 + quarter + up + + + + B + 4 + + 1 + + 1 + quarter + up + + + + B + 4 + + 1 + + 1 + quarter + up + + + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + D + 5 + + 1 + + 1 + quarter + up + x + + + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + G + 5 + + 1 + + 1 + quarter + up + x + + + + + + 91.76 + + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + D + 4 + + 1 + + 1 + quarter + up + x + + + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + E + 5 + + 1 + + 1 + quarter + up + x + + + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + F + 5 + + 1 + + 1 + quarter + up + x + + + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + F + 4 + + 1 + + 1 + quarter + up + + + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + C + 5 + + 1 + + 1 + quarter + up + x + + + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + C + 5 + + 1 + + 1 + quarter + up + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + + + + 4 + 1 + + + light-heavy + + + + diff --git a/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx b/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx new file mode 100644 index 0000000000000..1171bd96f1bad --- /dev/null +++ b/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx @@ -0,0 +1,3115 @@ + + + + 480 + + 1 + 1 + 1 + 0 + + Composer + + + + + + + + Title + + + + perc5Line + 0 + + 3 + + 1 + + Drumset + + Drumset + D. Set + Drum Kit + drum.group.set + 1 + + normal + 7 + 0 + Acoustic Bass Drum + 1 + + + normal + 7 + 0 + Bass Drum 1 + 1 + + + cross + 3 + 0 + Side Stick + 1 + + + normal + 3 + 0 + Acoustic Snare + 1 + + + normal + 3 + 0 + Electric Snare + 1 + + + normal + 5 + 0 + Low Floor Tom + 1 + + + cross + -1 + 0 + Closed Hi-Hat + 1 + + + normal + 5 + 0 + High Floor Tom + 1 + + + cross + 9 + 0 + Pedal Hi-Hat + 1 + + + normal + 2 + 0 + Low Tom + 1 + + + cross + 1 + 0 + Open Hi-Hat + 1 + + + normal + 1 + 0 + Low-Mid Tom + 1 + + + normal + 0 + 0 + Hi-Mid Tom + 1 + + + cross + -2 + 0 + Crash Cymbal 1 + 1 + + + normal + 0 + 0 + High Tom + 1 + + + cross + 0 + 0 + Ride Cymbal 1 + 1 + + + cross + -3 + 0 + Chinese Cymbal + 1 + + + diamond + 0 + 0 + Ride Bell + 1 + + + diamond + 2 + 0 + Tambourine + 1 + + + cross + -3 + 0 + Splash Cymbal + 1 + + + triangle-down + 1 + 0 + Cowbell + 1 + + + cross + -3 + 0 + Crash Cymbal 2 + 1 + + + cross + 2 + 0 + Ride Cymbal 2 + 1 + + + cross + 4 + 0 + Open Hi Conga + 1 + + + cross + 6 + 0 + Low Conga + 1 + + PERC + 0 + + 100 + 100 + + + 100 + 33 + + + 100 + 50 + + + 100 + 67 + + + 100 + 100 + + + 120 + 67 + + + 150 + 100 + + + 150 + 50 + + + 120 + 50 + + + 120 + 100 + + + + + + + + + + + + + perc5Line + 0 + + 3 + + Percussion + + Percussion + Perc. + Percussion + drum.group.set + 1 + + cross + -1 + 0 + Closed Hi-Hat + 1 + + + cross + 9 + 0 + Pedal Hi-Hat + 1 + + + cross + 1 + 0 + Open Hi-Hat + 1 + + + cross + 0 + 0 + Ride + 1 + + + normal + 7 + 0 + Concert Bass Drum + 1 + + + cross + 3 + 0 + Snare Side Stick + 1 + + + normal + 3 + 0 + Concert Snare Drum + 1 + + + normal + 1 + 0 + Castanets + 1 + + + diamond + 2 + 0 + Tambourine + 1 + + + cross + -3 + 0 + Splash Cymbal + 1 + + + triangle-down + 1 + 0 + Cowbell + 1 + + + diamond + 1 + 0 + Vibraslap + 1 + + + cross + -2 + 0 + Concert Cymbal + 1 + + + cross + 2 + 0 + Mute High Conga + 1 + + + cross + 4 + 0 + Open High Conga + 1 + + + cross + 6 + 0 + Low Conga + 1 + + + normal + 2 + 0 + Maracas + 1 + + + normal + 0 + 0 + Claves + 1 + + + diamond + 0 + 0 + Open Triangle + 1 + + + normal + 4 + 0 + Shaker + 1 + + + cross + 2 + 0 + Mark Tree + 1 + + PERC + + 100 + 100 + + + 100 + 33 + + + 100 + 50 + + + 100 + 67 + + + 100 + 100 + + + 120 + 67 + + + 150 + 100 + + + 150 + 50 + + + 120 + 50 + + + 120 + 100 + + + + + + + + + + + + 10 + + + + Title + + + + Composer + + + + + + PERC + PERC + 1 + + + 4 + 4 + + + quarter + up + + 42 + -9 + 20 + cross + + + + quarter + up + + 42 + -9 + 20 + cross + + + + quarter + up + + 42 + -9 + 20 + cross + + + + quarter + up + + 42 + -9 + 20 + cross + + + + + + + + quarter + up + + 43 + -9 + 15 + + + + quarter + up + + 43 + -9 + 15 + + + + quarter + up + + 43 + -9 + 15 + + + + quarter + up + + 43 + -9 + 15 + + + + + + + + quarter + up + + 44 + -9 + 22 + cross + + + + quarter + up + + 44 + -9 + 22 + cross + + + + quarter + up + + 44 + -9 + 22 + cross + + + + quarter + up + + 44 + -9 + 22 + cross + + + + + + + line + + + + quarter + up + + 45 + -9 + 17 + + + + quarter + up + + 45 + -9 + 17 + + + + quarter + up + + 45 + -9 + 17 + + + + quarter + up + + 45 + -9 + 17 + + + + + + + + quarter + up + + 46 + -9 + 12 + cross + + + + quarter + up + + 46 + -9 + 12 + cross + + + + quarter + up + + 46 + -9 + 12 + cross + + + + quarter + up + + 46 + -9 + 12 + cross + + + + + + + + quarter + up + + 47 + -9 + 19 + + + + quarter + up + + 47 + -9 + 19 + + + + quarter + up + + 47 + -9 + 19 + + + + quarter + up + + 47 + -9 + 19 + + + + + + + + quarter + up + + 48 + -9 + 14 + + + + quarter + up + + 48 + -9 + 14 + + + + quarter + up + + 48 + -9 + 14 + + + + quarter + up + + 48 + -9 + 14 + + + + + + + line + + + + quarter + up + + 49 + -9 + 21 + cross + + + + quarter + up + + 49 + -9 + 21 + cross + + + + quarter + up + + 49 + -9 + 21 + cross + + + + quarter + up + + 49 + -9 + 21 + cross + + + + + + + + quarter + up + + 50 + -9 + 16 + + + + quarter + up + + 50 + -9 + 16 + + + + quarter + up + + 50 + -9 + 16 + + + + quarter + up + + 50 + -9 + 16 + + + + + + + + quarter + up + + 51 + -9 + 11 + cross + + + + quarter + up + + 51 + -9 + 11 + cross + + + + quarter + up + + 51 + -9 + 11 + cross + + + + quarter + up + + 51 + -9 + 11 + cross + + + + + + + + quarter + up + + 52 + -9 + 18 + cross + + + + quarter + up + + 52 + -9 + 18 + cross + + + + quarter + up + + 52 + -9 + 18 + cross + + + + quarter + up + + 52 + -9 + 18 + cross + + + + + + + line + + + + quarter + up + + 53 + -9 + 13 + diamond + + + + quarter + up + + 53 + -9 + 13 + diamond + + + + quarter + up + + 53 + -9 + 13 + diamond + + + + quarter + up + + 53 + -9 + 13 + diamond + + + + + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + + + + + quarter + up + + 55 + -9 + 15 + cross + + + + quarter + up + + 55 + -9 + 15 + cross + + + + quarter + up + + 55 + -9 + 15 + cross + + + + quarter + up + + 55 + -9 + 15 + cross + + + + + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + + + + line + + + + quarter + up + + 57 + -9 + 17 + cross + + + + quarter + up + + 57 + -9 + 17 + cross + + + + quarter + up + + 57 + -9 + 17 + cross + + + + quarter + up + + 57 + -9 + 17 + cross + + + + + + + + quarter + up + + 59 + -9 + 19 + cross + + + + quarter + up + + 59 + -9 + 19 + cross + + + + quarter + up + + 59 + -9 + 19 + cross + + + + quarter + up + + 59 + -9 + 19 + cross + + + + + + + + quarter + up + + 63 + -9 + 11 + cross + + + + quarter + up + + 63 + -9 + 11 + cross + + + + quarter + up + + 63 + -9 + 11 + cross + + + + quarter + up + + 63 + -9 + 11 + cross + + + + + + + + quarter + up + + 64 + -9 + 18 + cross + + + + quarter + up + + 64 + -9 + 18 + cross + + + + quarter + up + + 64 + -9 + 18 + cross + + + + quarter + up + + 64 + -9 + 18 + cross + + + + + + + line + + + + quarter + up + + 35 + -9 + 19 + + + + quarter + up + + 35 + -9 + 19 + + + + quarter + up + + 35 + -9 + 19 + + + + quarter + up + + 35 + -9 + 19 + + + + + + + + quarter + up + + 36 + -9 + 14 + + + + quarter + up + + 36 + -9 + 14 + + + + quarter + up + + 36 + -9 + 14 + + + + quarter + up + + 36 + -9 + 14 + + + + + + + + quarter + up + + 37 + -9 + 21 + cross + + + + quarter + up + + 37 + -9 + 21 + cross + + + + quarter + up + + 37 + -9 + 21 + cross + + + + quarter + up + + 37 + -9 + 21 + cross + + + + + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + + + + page + + + + quarter + up + + 40 + -9 + 18 + + + + quarter + up + + 40 + -9 + 18 + + + + quarter + up + + 40 + -9 + 18 + + + + quarter + up + + 40 + -9 + 18 + + + + + + + + quarter + up + + 41 + -9 + 13 + + + + quarter + up + + 41 + -9 + 13 + + + + quarter + up + + 41 + -9 + 13 + + + + quarter + up + + 41 + -9 + 13 + + + + + + + + quarter + up + + 42 + -9 + 20 + cross + + + + quarter + up + + 42 + -9 + 20 + cross + + + + quarter + up + + 42 + -9 + 20 + cross + + + + quarter + up + + 42 + -9 + 20 + cross + + + + + + + + measure + 1/1 + + + + + + line + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + line + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + line + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + end + + + + + + + + + PERC + PERC + 1 + + + 4 + 4 + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + + + + + quarter + up + + 39 + -9 + 11 + + + + quarter + up + + 39 + -9 + 11 + + + + quarter + up + + 39 + -9 + 11 + + + + quarter + up + + 39 + -9 + 11 + + + + + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + quarter + up + + 54 + -9 + 20 + diamond + + + + + + + + quarter + up + + 55 + -9 + 15 + cross + + + + quarter + up + + 55 + -9 + 15 + cross + + + + quarter + up + + 55 + -9 + 15 + cross + + + + quarter + up + + 55 + -9 + 15 + cross + + + + + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + quarter + up + + 56 + -9 + 22 + triangle-down + + + + + + + + quarter + up + + 58 + -9 + 12 + diamond + + + + quarter + up + + 58 + -9 + 12 + diamond + + + + quarter + up + + 58 + -9 + 12 + diamond + + + + quarter + up + + 58 + -9 + 12 + diamond + + + + + + + + quarter + up + + 59 + -9 + 19 + cross + + + + quarter + up + + 59 + -9 + 19 + cross + + + + quarter + up + + 59 + -9 + 19 + cross + + + + quarter + up + + 59 + -9 + 19 + cross + + + + + + + + quarter + up + + 62 + -9 + 16 + cross + + + + quarter + up + + 62 + -9 + 16 + cross + + + + quarter + up + + 62 + -9 + 16 + cross + + + + quarter + up + + 62 + -9 + 16 + cross + + + + + + + + quarter + up + + 63 + -9 + 11 + cross + + + + quarter + up + + 63 + -9 + 11 + cross + + + + quarter + up + + 63 + -9 + 11 + cross + + + + quarter + up + + 63 + -9 + 11 + cross + + + + + + + + quarter + up + + 64 + -9 + 18 + cross + + + + quarter + up + + 64 + -9 + 18 + cross + + + + quarter + up + + 64 + -9 + 18 + cross + + + + quarter + up + + 64 + -9 + 18 + cross + + + + + + + + quarter + up + + 70 + -9 + 12 + + + + quarter + up + + 70 + -9 + 12 + + + + quarter + up + + 70 + -9 + 12 + + + + quarter + up + + 70 + -9 + 12 + + + + + + + + quarter + up + + 75 + -9 + 11 + + + + quarter + up + + 75 + -9 + 11 + + + + quarter + up + + 75 + -9 + 11 + + + + quarter + up + + 75 + -9 + 11 + + + + + + + + quarter + up + + 81 + -9 + 17 + diamond + + + + quarter + up + + 81 + -9 + 17 + diamond + + + + quarter + up + + 81 + -9 + 17 + diamond + + + + quarter + up + + 81 + -9 + 17 + diamond + + + + + + + + quarter + up + + 82 + -9 + 12 + + + + quarter + up + + 82 + -9 + 12 + + + + quarter + up + + 82 + -9 + 12 + + + + quarter + up + + 82 + -9 + 12 + + + + + + + + quarter + up + + 84 + -9 + 14 + cross + + + + quarter + up + + 84 + -9 + 14 + cross + + + + quarter + up + + 84 + -9 + 14 + cross + + + + quarter + up + + 84 + -9 + 14 + cross + + + + + + + + quarter + up + + 27 + -9 + 11 + cross + + + + quarter + up + + 27 + -9 + 11 + cross + + + + quarter + up + + 27 + -9 + 11 + cross + + + + quarter + up + + 27 + -9 + 11 + cross + + + + + + + + quarter + up + + 28 + -9 + 18 + cross + + + + quarter + up + + 28 + -9 + 18 + cross + + + + quarter + up + + 28 + -9 + 18 + cross + + + + quarter + up + + 28 + -9 + 18 + cross + + + + + + + + quarter + up + + 29 + -9 + 13 + cross + + + + quarter + up + + 29 + -9 + 13 + cross + + + + quarter + up + + 29 + -9 + 13 + cross + + + + quarter + up + + 29 + -9 + 13 + cross + + + + + + + + quarter + up + + 30 + -9 + 20 + cross + + + + quarter + up + + 30 + -9 + 20 + cross + + + + quarter + up + + 30 + -9 + 20 + cross + + + + quarter + up + + 30 + -9 + 20 + cross + + + + + + + + quarter + up + + 36 + -9 + 14 + + + + quarter + up + + 36 + -9 + 14 + + + + quarter + up + + 36 + -9 + 14 + + + + quarter + up + + 36 + -9 + 14 + + + + + + + + quarter + up + + 37 + -9 + 21 + cross + + + + quarter + up + + 37 + -9 + 21 + cross + + + + quarter + up + + 37 + -9 + 21 + cross + + + + quarter + up + + 37 + -9 + 21 + cross + + + + + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + quarter + up + + 38 + -9 + 16 + + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + + + + + measure + 1/1 + + + end + + + + + + diff --git a/src/importexport/musicxml/tests/data/testSticking_ref.mscx b/src/importexport/musicxml/tests/data/testSticking_ref.mscx index da0c12adc8b8e..71d99d0745505 100644 --- a/src/importexport/musicxml/tests/data/testSticking_ref.mscx +++ b/src/importexport/musicxml/tests/data/testSticking_ref.mscx @@ -60,15 +60,11 @@ 3 Percussion - + Percussion Perc. - Closed Hi-Hat - 24 - 108 - 24 - 108 - strings.group + Percussion + drum.group.set 1 normal @@ -217,7 +213,7 @@ Castanets 0 - F + PERC 100 100 @@ -258,17 +254,12 @@ 120 100 - + + - - - - - - diff --git a/src/importexport/musicxml/tests/musicxml_tests.cpp b/src/importexport/musicxml/tests/musicxml_tests.cpp index 35423327c5fa4..f6d30babc3cc1 100644 --- a/src/importexport/musicxml/tests/musicxml_tests.cpp +++ b/src/importexport/musicxml/tests/musicxml_tests.cpp @@ -887,6 +887,9 @@ TEST_F(MusicXml_Tests, measureStyleSlash) { TEST_F(MusicXml_Tests, midiPortExport) { musicXmlMscxExportTestRef("testMidiPortExport"); } +TEST_F(MusicXml_Tests, ms3KitAndPerc) { + musicXmlImportTestRef("testMS3KitAndPerc"); +} TEST_F(MusicXml_Tests, multiInstrumentPart1) { musicXmlIoTest("testMultiInstrumentPart1"); } From c43d2c7b824bdf6ed4ce129a1ec397ad3486f19d Mon Sep 17 00:00:00 2001 From: James Mizen Date: Fri, 14 Feb 2025 14:18:00 +0000 Subject: [PATCH 2/2] Set tpc properly --- .../musicxml/import/importmusicxmlpass2.cpp | 3 +- .../tests/data/testMS3KitAndPerc_ref.mscx | 576 ++++++------------ .../musicxml/tests/data/testSticking_ref.mscx | 57 +- 3 files changed, 212 insertions(+), 424 deletions(-) diff --git a/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp b/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp index cf1aaaae76d5f..ada21ad727fdf 100644 --- a/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp +++ b/src/importexport/musicxml/internal/musicxml/import/importmusicxmlpass2.cpp @@ -6508,8 +6508,7 @@ static void setPitch(Note* note, const MusicXmlInstruments& instruments, const S // get pitch from instrument definition in drumset instead int unpitched = instruments.at(instrumentId).unpitched; note->setPitch(std::clamp(unpitched, 0, 127)); - // TODO - does this need to be key-aware? - note->setTpc(pitch2tpc(unpitched, Key::C, Prefer::NEAREST)); // TODO: necessary ? + note->setTpcFromPitch(); } else { //LOGD("disp step %d oct %d", displayStep, displayOctave); xmlSetPitch(note, mnp.displayStep(), 0, 0.0, mnp.displayOctave(), 0, instrument); diff --git a/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx b/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx index 1171bd96f1bad..0e201994e24b0 100644 --- a/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx +++ b/src/importexport/musicxml/tests/data/testMS3KitAndPerc_ref.mscx @@ -533,8 +533,7 @@ up 42 - -9 - 20 + 20 cross @@ -543,8 +542,7 @@ up 42 - -9 - 20 + 20 cross @@ -553,8 +551,7 @@ up 42 - -9 - 20 + 20 cross @@ -563,8 +560,7 @@ up 42 - -9 - 20 + 20 cross @@ -577,8 +573,7 @@ up 43 - -9 - 15 + 15 @@ -586,8 +581,7 @@ up 43 - -9 - 15 + 15 @@ -595,8 +589,7 @@ up 43 - -9 - 15 + 15 @@ -604,8 +597,7 @@ up 43 - -9 - 15 + 15 @@ -617,8 +609,7 @@ up 44 - -9 - 22 + 22 cross @@ -627,8 +618,7 @@ up 44 - -9 - 22 + 22 cross @@ -637,8 +627,7 @@ up 44 - -9 - 22 + 22 cross @@ -647,8 +636,7 @@ up 44 - -9 - 22 + 22 cross @@ -664,8 +652,7 @@ up 45 - -9 - 17 + 17 @@ -673,8 +660,7 @@ up 45 - -9 - 17 + 17 @@ -682,8 +668,7 @@ up 45 - -9 - 17 + 17 @@ -691,8 +676,7 @@ up 45 - -9 - 17 + 17 @@ -704,8 +688,7 @@ up 46 - -9 - 12 + 12 cross @@ -714,8 +697,7 @@ up 46 - -9 - 12 + 12 cross @@ -724,8 +706,7 @@ up 46 - -9 - 12 + 12 cross @@ -734,8 +715,7 @@ up 46 - -9 - 12 + 12 cross @@ -748,8 +728,7 @@ up 47 - -9 - 19 + 19 @@ -757,8 +736,7 @@ up 47 - -9 - 19 + 19 @@ -766,8 +744,7 @@ up 47 - -9 - 19 + 19 @@ -775,8 +752,7 @@ up 47 - -9 - 19 + 19 @@ -788,8 +764,7 @@ up 48 - -9 - 14 + 14 @@ -797,8 +772,7 @@ up 48 - -9 - 14 + 14 @@ -806,8 +780,7 @@ up 48 - -9 - 14 + 14 @@ -815,8 +788,7 @@ up 48 - -9 - 14 + 14 @@ -831,8 +803,7 @@ up 49 - -9 - 21 + 21 cross @@ -841,8 +812,7 @@ up 49 - -9 - 21 + 21 cross @@ -851,8 +821,7 @@ up 49 - -9 - 21 + 21 cross @@ -861,8 +830,7 @@ up 49 - -9 - 21 + 21 cross @@ -875,8 +843,7 @@ up 50 - -9 - 16 + 16 @@ -884,8 +851,7 @@ up 50 - -9 - 16 + 16 @@ -893,8 +859,7 @@ up 50 - -9 - 16 + 16 @@ -902,8 +867,7 @@ up 50 - -9 - 16 + 16 @@ -915,8 +879,7 @@ up 51 - -9 - 11 + 11 cross @@ -925,8 +888,7 @@ up 51 - -9 - 11 + 11 cross @@ -935,8 +897,7 @@ up 51 - -9 - 11 + 11 cross @@ -945,8 +906,7 @@ up 51 - -9 - 11 + 11 cross @@ -959,8 +919,7 @@ up 52 - -9 - 18 + 18 cross @@ -969,8 +928,7 @@ up 52 - -9 - 18 + 18 cross @@ -979,8 +937,7 @@ up 52 - -9 - 18 + 18 cross @@ -989,8 +946,7 @@ up 52 - -9 - 18 + 18 cross @@ -1006,8 +962,7 @@ up 53 - -9 - 13 + 13 diamond @@ -1016,8 +971,7 @@ up 53 - -9 - 13 + 13 diamond @@ -1026,8 +980,7 @@ up 53 - -9 - 13 + 13 diamond @@ -1036,8 +989,7 @@ up 53 - -9 - 13 + 13 diamond @@ -1050,8 +1002,7 @@ up 54 - -9 - 20 + 20 diamond @@ -1060,8 +1011,7 @@ up 54 - -9 - 20 + 20 diamond @@ -1070,8 +1020,7 @@ up 54 - -9 - 20 + 20 diamond @@ -1080,8 +1029,7 @@ up 54 - -9 - 20 + 20 diamond @@ -1094,8 +1042,7 @@ up 55 - -9 - 15 + 15 cross @@ -1104,8 +1051,7 @@ up 55 - -9 - 15 + 15 cross @@ -1114,8 +1060,7 @@ up 55 - -9 - 15 + 15 cross @@ -1124,8 +1069,7 @@ up 55 - -9 - 15 + 15 cross @@ -1138,8 +1082,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -1148,8 +1091,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -1158,8 +1100,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -1168,8 +1109,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -1185,8 +1125,7 @@ up 57 - -9 - 17 + 17 cross @@ -1195,8 +1134,7 @@ up 57 - -9 - 17 + 17 cross @@ -1205,8 +1143,7 @@ up 57 - -9 - 17 + 17 cross @@ -1215,8 +1152,7 @@ up 57 - -9 - 17 + 17 cross @@ -1229,8 +1165,7 @@ up 59 - -9 - 19 + 19 cross @@ -1239,8 +1174,7 @@ up 59 - -9 - 19 + 19 cross @@ -1249,8 +1183,7 @@ up 59 - -9 - 19 + 19 cross @@ -1259,8 +1192,7 @@ up 59 - -9 - 19 + 19 cross @@ -1273,8 +1205,7 @@ up 63 - -9 - 11 + 11 cross @@ -1283,8 +1214,7 @@ up 63 - -9 - 11 + 11 cross @@ -1293,8 +1223,7 @@ up 63 - -9 - 11 + 11 cross @@ -1303,8 +1232,7 @@ up 63 - -9 - 11 + 11 cross @@ -1317,8 +1245,7 @@ up 64 - -9 - 18 + 18 cross @@ -1327,8 +1254,7 @@ up 64 - -9 - 18 + 18 cross @@ -1337,8 +1263,7 @@ up 64 - -9 - 18 + 18 cross @@ -1347,8 +1272,7 @@ up 64 - -9 - 18 + 18 cross @@ -1364,8 +1288,7 @@ up 35 - -9 - 19 + 19 @@ -1373,8 +1296,7 @@ up 35 - -9 - 19 + 19 @@ -1382,8 +1304,7 @@ up 35 - -9 - 19 + 19 @@ -1391,8 +1312,7 @@ up 35 - -9 - 19 + 19 @@ -1404,8 +1324,7 @@ up 36 - -9 - 14 + 14 @@ -1413,8 +1332,7 @@ up 36 - -9 - 14 + 14 @@ -1422,8 +1340,7 @@ up 36 - -9 - 14 + 14 @@ -1431,8 +1348,7 @@ up 36 - -9 - 14 + 14 @@ -1444,8 +1360,7 @@ up 37 - -9 - 21 + 21 cross @@ -1454,8 +1369,7 @@ up 37 - -9 - 21 + 21 cross @@ -1464,8 +1378,7 @@ up 37 - -9 - 21 + 21 cross @@ -1474,8 +1387,7 @@ up 37 - -9 - 21 + 21 cross @@ -1488,8 +1400,7 @@ up 38 - -9 - 16 + 16 @@ -1497,8 +1408,7 @@ up 38 - -9 - 16 + 16 @@ -1506,8 +1416,7 @@ up 38 - -9 - 16 + 16 @@ -1515,8 +1424,7 @@ up 38 - -9 - 16 + 16 @@ -1531,8 +1439,7 @@ up 40 - -9 - 18 + 18 @@ -1540,8 +1447,7 @@ up 40 - -9 - 18 + 18 @@ -1549,8 +1455,7 @@ up 40 - -9 - 18 + 18 @@ -1558,8 +1463,7 @@ up 40 - -9 - 18 + 18 @@ -1571,8 +1475,7 @@ up 41 - -9 - 13 + 13 @@ -1580,8 +1483,7 @@ up 41 - -9 - 13 + 13 @@ -1589,8 +1491,7 @@ up 41 - -9 - 13 + 13 @@ -1598,8 +1499,7 @@ up 41 - -9 - 13 + 13 @@ -1611,8 +1511,7 @@ up 42 - -9 - 20 + 20 cross @@ -1621,8 +1520,7 @@ up 42 - -9 - 20 + 20 cross @@ -1631,8 +1529,7 @@ up 42 - -9 - 20 + 20 cross @@ -1641,8 +1538,7 @@ up 42 - -9 - 20 + 20 cross @@ -2134,8 +2030,7 @@ up 38 - -9 - 16 + 16 @@ -2143,8 +2038,7 @@ up 38 - -9 - 16 + 16 @@ -2152,8 +2046,7 @@ up 38 - -9 - 16 + 16 @@ -2161,8 +2054,7 @@ up 38 - -9 - 16 + 16 @@ -2174,8 +2066,7 @@ up 39 - -9 - 11 + 11 @@ -2183,8 +2074,7 @@ up 39 - -9 - 11 + 11 @@ -2192,8 +2082,7 @@ up 39 - -9 - 11 + 11 @@ -2201,8 +2090,7 @@ up 39 - -9 - 11 + 11 @@ -2214,8 +2102,7 @@ up 54 - -9 - 20 + 20 diamond @@ -2224,8 +2111,7 @@ up 54 - -9 - 20 + 20 diamond @@ -2234,8 +2120,7 @@ up 54 - -9 - 20 + 20 diamond @@ -2244,8 +2129,7 @@ up 54 - -9 - 20 + 20 diamond @@ -2258,8 +2142,7 @@ up 55 - -9 - 15 + 15 cross @@ -2268,8 +2151,7 @@ up 55 - -9 - 15 + 15 cross @@ -2278,8 +2160,7 @@ up 55 - -9 - 15 + 15 cross @@ -2288,8 +2169,7 @@ up 55 - -9 - 15 + 15 cross @@ -2302,8 +2182,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -2312,8 +2191,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -2322,8 +2200,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -2332,8 +2209,7 @@ up 56 - -9 - 22 + 22 triangle-down @@ -2346,8 +2222,7 @@ up 58 - -9 - 12 + 12 diamond @@ -2356,8 +2231,7 @@ up 58 - -9 - 12 + 12 diamond @@ -2366,8 +2240,7 @@ up 58 - -9 - 12 + 12 diamond @@ -2376,8 +2249,7 @@ up 58 - -9 - 12 + 12 diamond @@ -2390,8 +2262,7 @@ up 59 - -9 - 19 + 19 cross @@ -2400,8 +2271,7 @@ up 59 - -9 - 19 + 19 cross @@ -2410,8 +2280,7 @@ up 59 - -9 - 19 + 19 cross @@ -2420,8 +2289,7 @@ up 59 - -9 - 19 + 19 cross @@ -2434,8 +2302,7 @@ up 62 - -9 - 16 + 16 cross @@ -2444,8 +2311,7 @@ up 62 - -9 - 16 + 16 cross @@ -2454,8 +2320,7 @@ up 62 - -9 - 16 + 16 cross @@ -2464,8 +2329,7 @@ up 62 - -9 - 16 + 16 cross @@ -2478,8 +2342,7 @@ up 63 - -9 - 11 + 11 cross @@ -2488,8 +2351,7 @@ up 63 - -9 - 11 + 11 cross @@ -2498,8 +2360,7 @@ up 63 - -9 - 11 + 11 cross @@ -2508,8 +2369,7 @@ up 63 - -9 - 11 + 11 cross @@ -2522,8 +2382,7 @@ up 64 - -9 - 18 + 18 cross @@ -2532,8 +2391,7 @@ up 64 - -9 - 18 + 18 cross @@ -2542,8 +2400,7 @@ up 64 - -9 - 18 + 18 cross @@ -2552,8 +2409,7 @@ up 64 - -9 - 18 + 18 cross @@ -2566,8 +2422,7 @@ up 70 - -9 - 12 + 12 @@ -2575,8 +2430,7 @@ up 70 - -9 - 12 + 12 @@ -2584,8 +2438,7 @@ up 70 - -9 - 12 + 12 @@ -2593,8 +2446,7 @@ up 70 - -9 - 12 + 12 @@ -2606,8 +2458,7 @@ up 75 - -9 - 11 + 11 @@ -2615,8 +2466,7 @@ up 75 - -9 - 11 + 11 @@ -2624,8 +2474,7 @@ up 75 - -9 - 11 + 11 @@ -2633,8 +2482,7 @@ up 75 - -9 - 11 + 11 @@ -2646,8 +2494,7 @@ up 81 - -9 - 17 + 17 diamond @@ -2656,8 +2503,7 @@ up 81 - -9 - 17 + 17 diamond @@ -2666,8 +2512,7 @@ up 81 - -9 - 17 + 17 diamond @@ -2676,8 +2521,7 @@ up 81 - -9 - 17 + 17 diamond @@ -2690,8 +2534,7 @@ up 82 - -9 - 12 + 12 @@ -2699,8 +2542,7 @@ up 82 - -9 - 12 + 12 @@ -2708,8 +2550,7 @@ up 82 - -9 - 12 + 12 @@ -2717,8 +2558,7 @@ up 82 - -9 - 12 + 12 @@ -2730,8 +2570,7 @@ up 84 - -9 - 14 + 14 cross @@ -2740,8 +2579,7 @@ up 84 - -9 - 14 + 14 cross @@ -2750,8 +2588,7 @@ up 84 - -9 - 14 + 14 cross @@ -2760,8 +2597,7 @@ up 84 - -9 - 14 + 14 cross @@ -2774,8 +2610,7 @@ up 27 - -9 - 11 + 11 cross @@ -2784,8 +2619,7 @@ up 27 - -9 - 11 + 11 cross @@ -2794,8 +2628,7 @@ up 27 - -9 - 11 + 11 cross @@ -2804,8 +2637,7 @@ up 27 - -9 - 11 + 11 cross @@ -2818,8 +2650,7 @@ up 28 - -9 - 18 + 18 cross @@ -2828,8 +2659,7 @@ up 28 - -9 - 18 + 18 cross @@ -2838,8 +2668,7 @@ up 28 - -9 - 18 + 18 cross @@ -2848,8 +2677,7 @@ up 28 - -9 - 18 + 18 cross @@ -2862,8 +2690,7 @@ up 29 - -9 - 13 + 13 cross @@ -2872,8 +2699,7 @@ up 29 - -9 - 13 + 13 cross @@ -2882,8 +2708,7 @@ up 29 - -9 - 13 + 13 cross @@ -2892,8 +2717,7 @@ up 29 - -9 - 13 + 13 cross @@ -2906,8 +2730,7 @@ up 30 - -9 - 20 + 20 cross @@ -2916,8 +2739,7 @@ up 30 - -9 - 20 + 20 cross @@ -2926,8 +2748,7 @@ up 30 - -9 - 20 + 20 cross @@ -2936,8 +2757,7 @@ up 30 - -9 - 20 + 20 cross @@ -2950,8 +2770,7 @@ up 36 - -9 - 14 + 14 @@ -2959,8 +2778,7 @@ up 36 - -9 - 14 + 14 @@ -2968,8 +2786,7 @@ up 36 - -9 - 14 + 14 @@ -2977,8 +2794,7 @@ up 36 - -9 - 14 + 14 @@ -2990,8 +2806,7 @@ up 37 - -9 - 21 + 21 cross @@ -3000,8 +2815,7 @@ up 37 - -9 - 21 + 21 cross @@ -3010,8 +2824,7 @@ up 37 - -9 - 21 + 21 cross @@ -3020,8 +2833,7 @@ up 37 - -9 - 21 + 21 cross @@ -3034,8 +2846,7 @@ up 38 - -9 - 16 + 16 @@ -3043,8 +2854,7 @@ up 38 - -9 - 16 + 16 @@ -3052,8 +2862,7 @@ up 38 - -9 - 16 + 16 @@ -3061,8 +2870,7 @@ up 38 - -9 - 16 + 16 diff --git a/src/importexport/musicxml/tests/data/testSticking_ref.mscx b/src/importexport/musicxml/tests/data/testSticking_ref.mscx index 71d99d0745505..af1b97f61638b 100644 --- a/src/importexport/musicxml/tests/data/testSticking_ref.mscx +++ b/src/importexport/musicxml/tests/data/testSticking_ref.mscx @@ -302,8 +302,7 @@ 16th 38 - -9 - 16 + 16 @@ -314,8 +313,7 @@ 16th 38 - -9 - 16 + 16 @@ -326,8 +324,7 @@ 16th 38 - -9 - 16 + 16 @@ -337,8 +334,7 @@ 16th 38 - -9 - 16 + 16 @@ -354,8 +350,7 @@ 16th 38 - -9 - 16 + 16 @@ -366,8 +361,7 @@ 16th 38 - -9 - 16 + 16 @@ -378,8 +372,7 @@ 16th 38 - -9 - 16 + 16 @@ -389,8 +382,7 @@ 16th 38 - -9 - 16 + 16 @@ -425,8 +417,7 @@ 38 - -9 - 16 + 16 @@ -450,8 +441,7 @@ 38 - -9 - 16 + 16 @@ -461,8 +451,7 @@ eighth 38 - -9 - 16 + 16 @@ -478,8 +467,7 @@ eighth 38 - -9 - 16 + 16 @@ -507,8 +495,7 @@ 38 - -9 - 16 + 16 @@ -526,8 +513,7 @@ 38 - -9 - 16 + 16 @@ -570,8 +556,7 @@ 38 - -9 - 16 + 16 @@ -584,8 +569,7 @@ 38 - -9 - 16 + 16 @@ -598,8 +582,7 @@ 38 - -9 - 16 + 16 @@ -612,8 +595,7 @@ 38 - -9 - 16 + 16 @@ -625,8 +607,7 @@ up 38 - -9 - 16 + 16