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
Wario Master of Disguise soundfont doesnt work #290
Comments
It looks like Signal can't handle a soundfont with 24-bit samples. As a workaround, you might be able to use a soundfont editor to make a 16-bit version of the soundfont. For example, in the Polyphone soundfont editor, you can open the soundfont, change the "Samples" dropdown box from "24 bits" to "16 bits", then save as a new soundfont. That seems to make it into a 16-bit soundfont which works in Signal. |
I did a quick look at the sf2 format by reading this document: SoundFont2 Structure - sdta Chunk It looks like an sf2 file can store either 16-bit sounds or 24-bit sounds. When 24-bit sounds are used, it looks like they are actually stored as 16-bit sounds plus data for the additional 8 bits. The "TODO" error message comes the following place in this file:
(see source code at npm @ryohey/sf2parser Code src/Parser.ts or GitHub ryohey/sf2synth.js Parser.ts) According to my tests, it looks like you can remove If you want to try out this change, I wrote some code you can use to patch Signal using the browser console or a JavaScript bookmark: Signal Customizations |
thanks @robertnhart I will fix it. we can ignore 8bit lsb as you say i can find some real codes which handles correctly. I might do same thing that treats samples as 32bit https://github.com/FluidSynth/fluidsynth/blob/master/src/sfloader/fluid_sffile.c#L2206 |
Describe the bug
This soundfont doesnt work, when i add it and open it, theres a message saying "todo" and then it doesnt let me use it
Expected behavior
Being able to use the soundfont
Screenshots
The text was updated successfully, but these errors were encountered: