polpo / picogus

Emulation of ISA sound cards on Raspberry Pi Pico (GUS, Adlib, MPU-401, Tandy, CMS)
GNU General Public License v2.0
518 stars 34 forks source link

Proper fix for emu8950 missing note issue #52

Closed TerryFi closed 1 week ago

TerryFi commented 3 weeks ago

Replaced workaround for https://github.com/polpo/picogus/issues/46 with proper fix from https://github.com/digital-sound-antiques/emu8950. Link to original emu8950 commit https://github.com/digital-sound-antiques/emu8950/commit/570dc998dd83cee87b021e15e1d86da736934393.

Compared both the new fix and previous workaround sample audio to hardware opl audio output. Workaround shows inaccurate note attack timing whereas the new fix attack timing shows good accuracy. The inaccuracies were verified to be caused by changes in previous workaround.

polpo commented 1 week ago

Thank you @TerryFi!