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

fix #33081: acoustic bass channel info #1309

Merged
merged 1 commit into from
Nov 17, 2014

Conversation

MarcSabatella
Copy link
Contributor

No description provided.

@@ -8803,6 +8803,12 @@
<Channel>
<program value="32"/>
</Channel>
<Channel name="arco">
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't get it. 43 and 44 are tremolo and pizz strings no? Why associating them to arco and tremolo?

There are other Bass technics in MIDI
32 Acoustic Bass
33 Electric Bass (finger)
34 Electric Bass (pick)
35 Fretless Bass
36 Slap Bass 1
37 Slap Bass 2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The difference between 0-based and 1-based patch numbering might be getting in the way here. I'm just using the same numbering scheme as elsewhere in instruments.xml. 43 is the patch used for the "contrabass" instrument in its default channel, and 32 is the patch number used for the "pizz" channel. This is 0-based numbering. In the 1-based numbering you see in most published charts, that's "44 Contrabass" and "33 Acoustic Bass" respectively. We were already using 32 (=33-1) as the default channel for "Acoustic Bass", and that's correct. I just needed to add 43 (=44-1) for the arco sound - and 44 (=45-1) for tremolo, so Contrabass and Acoustic Bass have the exact same channels just with a different default.

Now I'm confusing myself, but as far as I can tell, I got it right. The three patch numbers in question (0-based) are 32 for plucked, 43 for bowed, and 44 for tremolo. For the "contrabass" instrument, 43 is the default and is unnamed; 32 is available as "pizzicato". For the "acoustic bass" instrument, 32 is the default and 43 is available as "arco". This is exactly as it was in 1.3; I'm just restoring that behavior, as for some reason the arco and tremolo options got omitted.

@MarcSabatella
Copy link
Contributor Author

Oh, and sure, it might be nice to add channels for finger / pick / slap to the electric bass instrument entry or entries. Probably other missed opportunities as well. But at least we do have instrument change now.

@chenlung
Copy link
Contributor

lasconic added a commit that referenced this pull request Nov 17, 2014
fix #33081: acoustic bass channel info
@lasconic lasconic merged commit 486ce2c into musescore:master Nov 17, 2014
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