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

Update instruments.xml to sort Articulations and Channels #8289

Merged
merged 1 commit into from
Jun 10, 2021

Conversation

shoogle
Copy link
Contributor

@shoogle shoogle commented Jun 9, 2021

Run share/instruments/update_instruments_xml.py to fetch the latest version of the online spreadsheet. In this version, the ordering of articulations and channels has been made consistent across all instruments. Sorting is done according to the order in which channels and articulations are defined in the Basics tab of the online spreadsheet.

Run share/instruments/update_instruments_xml.py to fetch the latest
version of the online spreadsheet. In this version, the ordering of
articulations and channels has been made consistent across all
instruments. Sorting is done according to the order in which channels
and articulations are defined in the Basics tab of the online
spreadsheet.
@shoogle shoogle merged commit c45e667 into musescore:master Jun 10, 2021
@shoogle shoogle deleted the instruments-xml branch June 10, 2021 00:48
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jun 29, 2021
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 1, 2021
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 5, 2021
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 6, 2021
<Channel name="pop">
<!--MIDI: Bank 0, Prog 37; MS General: Pop Bass-->
<program value="37"/> <!--Slap Bass 2-->
</Channel>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wrong, accustic bass should not default to slap but to pizzicato
Whether that should be named ' pizzicato' or just fall back to 'normal', like fot ehe other bass guitars is another issue to consider.

Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 13, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 27, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 27, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 27, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 27, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 27, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 27, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 28, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Jul 28, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 13, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 13, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 19, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 19, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Aug 30, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 1, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 2, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 10, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Sep 23, 2021
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request Mar 5, 2023
Backport of musescore#8289, additionally fixing the order of channels for
accustic bass and fixing an mtest
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.

3 participants