Skip to content
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

[MU3 Backend] ENG-71: Infer more Dolet word export quirks #8651

Merged
merged 2 commits into from Aug 5, 2021

Conversation

iveshenry18
Copy link
Contributor

Resolves: ENG-71: Staff Text export quirks can be easily replaced: NmiCmi = N.C. ; $ = segno

This PR adds two commits which make some import inferences to correct for Dolet text export quirks.

The first of these fixes a strange "NmiCmi" direction that is sometimes exported instead of a "N.C." chord symbol.

The second adds $ and Ø symbols to the map in text2syms, resulting in these being translated to segno and coda symbols on read.

  • I signed CLA
  • I made sure the code in the PR follows the coding rules
  • I made sure the code compiles on my machine
  • I made sure there are no unnecessary changes in the code
  • I made sure the title of the PR reflects the core meaning of the issue you are solving
  • I made sure the commit message(s) contain a description and answer the question "Why do those changes fix that particular issue?" or "Why are those changes really necessary as improvements?"
  • I made sure the commit message title starts with "fix #424242:" if there is a related issue
  • I created the test (mtest, vtest, script test) to verify the changes I made

Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.
@vpereverzev vpereverzev merged commit 412c137 into musescore:3.6.2_backend Aug 5, 2021
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 5, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 5, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 5, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 5, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 7, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 7, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 13, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 13, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 13, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 13, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 19, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 19, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 19, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 19, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 30, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 30, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 1, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 1, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duiplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 2, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 2, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 9, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 9, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 10, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 10, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 23, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 23, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 26, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 26, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 29, 2021
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 29, 2021
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 12, 2022
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 12, 2022
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Mar 5, 2023
Dolet exports some N.C. chord symbols as a direction with a words
element containing "NmiCmi". This adds a simple inference to handle
that.

Duplicate of musescore#8651, part 1, plus a compile fix (adding a missing
paramenter,`_isBold`)
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Mar 5, 2023
Dolet seems to occasionally export $ and Ø where segno and coda symbols
ought to be exported in lyrics and staff text. This commit adds those as
special cases to the map in text2syms.

Duplicate of musescore#8651, part 2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants