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

ZynAddSubFx glitch on high sample rate #5534

Open
zynskeywolf opened this issue Jun 11, 2020 · 8 comments
Open

ZynAddSubFx glitch on high sample rate #5534

zynskeywolf opened this issue Jun 11, 2020 · 8 comments
Labels

Comments

@zynskeywolf
Copy link
Contributor

zynskeywolf commented Jun 11, 2020

A rare glitch appears with ZynAddSubFx with very specific scenarios. It Might be related to the envelopes in ZynAddSubFx, but it happens too randomly to be sure.

Steps to reproduce:

  • Set the playback sample rate to 192 kHz (currently doable in .lmmsrc.xml)
  • Open this project. (Updated)
  • Enable the oscilloscope and press play

Expected behavior:
The sound sounds, then goes to silence.

Actual behavior:
You can see a gradually appearing DC offset in the output.
Putting in a reverb in Zyn produces even more crazy things.

Tested in 1.3.0-alpha.1.606+g75d6be2ea (Linux/x86_64, Qt 5.15.13, GCC 14.1.1 20240507)

@zonkmachine
Copy link
Member

zonkmachine commented Aug 13, 2020

I can confirm the issue.

Putting in a reverb in Zyn produces even more crazy things.

That's what you'd get with a floating point error. A good candidate to trouble shoot with the WANT_DEBUG_FPE debug option.

@zonkmachine
Copy link
Member

That's what you'd get with a floating point error. A good candidate to trouble shoot with the WANT_DEBUG_FPE debug option.

Tested with -DWANT_DEBUG_FPE=ON and nothing showed up.

Change samplerate to 48000, bit depth to 16, and oversampling to 8x.

I get the same result no matter what bit depth setting I use.

@zonkmachine
Copy link
Member

Oversampling was removed in #7228

@zynskeywolf
Copy link
Contributor Author

Oversampling was removed in #7228

It's not an issue with the oversampling feature, it's a ZynAddSubFx bug when using high sample rates in general.

@zynskeywolf zynskeywolf changed the title Export glitch with oversampling and 48 kHz ZynAddSubFx glitch on high sample rate May 20, 2024
@zonkmachine zonkmachine reopened this May 20, 2024
@zynskeywolf
Copy link
Contributor Author

Updated the original post to be reproducible without the removed setting.

@zonkmachine
Copy link
Member

Set the playback sample rate to 192 kHz (currently doable in .lmmsrc.xml)

Are you sure this setting isn't just a feature in #7125 ?

OK. I tested the issue above on #7125 and can confirm the issue.

@zynskeywolf
Copy link
Contributor Author

Are you sure this setting isn't just a feature in #7125 ?

Well it is, but that pr is open, thus not in newest master.

@zonkmachine
Copy link
Member

Well it is, but that pr is open, thus not in newest master.

Ah, right! Now I found it. samplerate=""/>. I learnt something new.

Enable the oscilloscope and press play

Interesting. I do get the bug but it doesn't show up on export.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants