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

pop when enabling Bessel8 ISO in effect unit #13055

Closed
ronso0 opened this issue Apr 5, 2024 · 5 comments · Fixed by #13073
Closed

pop when enabling Bessel8 ISO in effect unit #13055

ronso0 opened this issue Apr 5, 2024 · 5 comments · Fixed by #13073

Comments

@ronso0
Copy link
Member

ronso0 commented Apr 5, 2024

Bug Description

  • mix mode: Dry/Wet
  • fully Wet
  • no effect before or after

Occurs only when toggling ON.
Notable especially if this exact same iso is used as EQ for the playing deck and High and Mid are fully killed (with knob) both in the mixer and the standard effect unit.

No buffer underruns.
Doesn't matter which keylock engine is used.

Loudness seems to vary. When I tried to reproduce after start, the first pop was rather quiet, on further toggling pops where significantly louder but still with some variation in loudness.

FWIW pops also occur when changing the keylock engine.

personal branch based on main 2.5-alpha, no patches related to effects.

Version

2.5-alpha

OS

No response

@ronso0 ronso0 added the bug label Apr 5, 2024
@ronso0
Copy link
Member Author

ronso0 commented Apr 7, 2024

Very strange:
with a different DAC and different PA I can't reproduce.
Same Mixxx binary, same effects setup.

Maybe it's just some frequencies in the top range that slip through, and they are somehow filtered by my second setup?

@daschuer
Copy link
Member

daschuer commented Apr 7, 2024

I can confirm a pop when you kill all and then enable the effect.
grafik
I will look into this.

@ronso0
Copy link
Member Author

ronso0 commented Apr 7, 2024

I've found this comment which I see no evidence for.
In getManifest() above, there is pManifest->setEffectRampsFromDry(true);
(but I'm not into effect processing so this might be a dead end)

if (enableState == EffectEnableState::Disabling) {

@ronso0
Copy link
Member Author

ronso0 commented Apr 7, 2024

Oh right, next time I'll record audio issues to figure if it's Mixxx or a later part of the chain.

@daschuer
Copy link
Member

daschuer commented Apr 7, 2024

A fix is here: #13073

@daschuer daschuer closed this as completed May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants