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] Fix #310821: Chord Symbol on drum stave sounds as percussion #6584

Merged
merged 2 commits into from
Nov 30, 2020

Conversation

Jojo-Schmitz
Copy link
Contributor

@Jojo-Schmitz Jojo-Schmitz commented Sep 26, 2020

This is just a (very) partial fix, it does playback chord symbols on drum staves as Piano, but doesn't show the correct sound in Mixer (shows "Standard" rather then "Grand Piano"), doesn't allow changes to the sound to another instrument, on trying to change it reverts the sound to Percussion and (probably) doesn't fix existing scores.

See https://musescore.org/en/node/310821

@MarcSabatella
Copy link
Contributor

Unfortunately, I don't really know enough how these things are organized - in the soundfont or in Fluid - to be confident of how to fix it better. I might guess, though, that if the Mixer isn't displaying it correctly, that's the Mixer's fault, not the chord symbol channel code's fault. That is, probably the Mixer never was designed to handle channels on different banks. And that could also be why changing it doesn't really work either.

@Jojo-Schmitz Jojo-Schmitz changed the title fix #310821: Chord Symbol on drum stave sounds as percussion [MU3] Fix #310821: Chord Symbol on drum stave sounds as percussion Oct 20, 2020
@dmitrio95
Copy link
Contributor

As far as I can tell, the Mixer issues you describe are caused by Mixer itself, not by something being wrong in the internal channels' structure. Mixer seems to assume (at least here) that a percussion instrument cannot contain non-percussion channels. This assumption is not correct anymore after harmony channels have been introduced. So this seems like something to be corrected in Mixer as it just doesn't handle this new channels configuration well.

@Jojo-Schmitz
Copy link
Contributor Author

OK, just how/where exactly?

@dmitrio95
Copy link
Contributor

Well, the easiest option I see is making harmony channels always non-percussive:
dmitrio95@1ef5315

This would disallow using percussion sounds in them but this doesn't seem to be needed anyway.

A more generic solution which would allow separate handling of percussive and non-percussive channels in one instrument seems to require more changes: useDrumset is now a property of instrument rather than individual channel.

@Jojo-Schmitz
Copy link
Contributor Author

Works perfectly, thanks!

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.

4 participants