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

Failure to emulate echo buffer volume-related overflow effect from FIR coefficents part 2 #61

Closed
KungFuFurby opened this issue Aug 24, 2023 · 2 comments
Assignees
Labels
bug Something isn't working fixed This issue was fixed, waiting for stable release

Comments

@KungFuFurby
Copy link

Here I am again, forwarding more playback differences. This is the sequel to #37.

This particular song is the next test case (it uses a different set of FIR coefficients, as I had refined the distortion effect from the past two songs): https://battleofthebits.com/arena/Entry/Stop_+Duck_+Rock+and+Roll/60690/

The overflow effect appears to have been emulated incorrectly, as this song's distortion effect came out more as static noise instead of the intended distortion effect that was being done (the feedback parameter was also being fooled around in the middle of the song). To make matters worse, the static lasts throughout the entire song instead of actually stopping as intended.

The .spc file is mirrored here in a .zip file.

@dgrfactory dgrfactory self-assigned this Aug 26, 2023
@dgrfactory dgrfactory added W.I.P Working in progress bug Something isn't working fixed This issue was fixed, waiting for stable release and removed W.I.P Working in progress labels Aug 26, 2023
@dgrfactory
Copy link
Owner

Hi, @KungFuFurby
Thank you for reporting the issue, and very nice sample song!

I found a mistake in the processing of the FIR filter, and fixed it in the following version.
https://github.com/dgrfactory/spcplay/releases/tag/2.19.6.8179

There seems to be a slight difference in sound yet when listening in detail.
Probably, because SNESAPU is computed in 32-bits unlike the actual device.
Still, it should have gotten a lot closer to the actual sound.
I will continue to explore ways to achieve both high quality sound and output close to the actual device.

dgrfactory added a commit that referenced this issue Oct 15, 2023
* FIR フィルタの処理順序が反対になっているバグ修正
* FIR フィルタのオーバーフロー処理を修正
* エコーを使用しているチャンネルをすべてミュートしたとき、FIR 処理を停止するように修正
@dgrfactory
Copy link
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed This issue was fixed, waiting for stable release
Projects
None yet
Development

No branches or pull requests

2 participants