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

cmt - Delay Line, wrong delay with setting 0 #6006

Closed
superpaik opened this issue May 6, 2021 · 3 comments · Fixed by #6151
Closed

cmt - Delay Line, wrong delay with setting 0 #6006

superpaik opened this issue May 6, 2021 · 3 comments · Fixed by #6151

Comments

@superpaik
Copy link
Contributor

superpaik commented May 6, 2021

When using Echo Delay Line, there is a malfunction when:

  • dry/wet balance knob set to 1 (full wet)
  • delay (ms) knob set to 0
    What I would expect in that case is no delay at all, but a weird 0.18s delay -when using the maximum delay 0.1s version- is produced.
    When the delay knob is increased (>0) then starts to work properly. When arrives at 10, there's a delay of 0.1s, which is correct. Then go back to 0 and the delay is around 0.18 again.
    Similar effect happens with other Echo Delay Line versions (0,01s, 1s, etc.) and also when channels are not linked.
    I checked it with audacity as well, but I attach a screen recording using the free vst "Wave Observer" for convenience.
    It starts with the effect deactivated, and when activated you can see how the sound is produced around 0,18s later. And see how it changes when the delay knob is adjusted.
    test-echo-delay
    Tested on lmms 1.2.2 windows 10 64bit

Use case: I use that combination to slightly delay 1 channel to create stereo effect, making a variation from left to right to create panning effect as well.
Work around: when the delay time is set to 0, dry/wet should be set to full dry.

@superpaik superpaik added the bug label May 6, 2021
@zonkmachine
Copy link
Member

@zonkmachine zonkmachine changed the title Error with "Echo Delay Line" internat effect cmt - Delay Line, wrong delay with setting 0 Sep 2, 2021
@zonkmachine
Copy link
Member

Reported upstream. Error located and fix is on it's way.

@zonkmachine
Copy link
Member

Fixed upstream in cmt. LMMS fix Depends on #6151 and LMMS/cmt#3

tresf added a commit to tresf/lmms that referenced this issue Jul 7, 2023
Closes LMMS#6006
Fixes MSVC compilation
tresf added a commit that referenced this issue Jul 11, 2023
* Bump cmt submodule
Closes #6006
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