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

"Gaps" in Buzzmachines cause silence in downstream machines #110

Open
dlbeswick opened this issue Jul 17, 2021 · 1 comment
Open

"Gaps" in Buzzmachines cause silence in downstream machines #110

dlbeswick opened this issue Jul 17, 2021 · 1 comment

Comments

@dlbeswick
Copy link
Contributor

image

As seen above, the symsin is set up to make a very quick pulse; a click. The PanzerDelay echoes every four ticks and the freeverb has a very long reverb.

What I see is that the freeverb only outputs when the PanzerDelay is actually outputting something; the reverb doesn't tail off as expected.

Always calling GST_BUFFER_FLAG_UNSET (outbuf, GST_BUFFER_FLAG_GAP) in gst_bml_transform_transform_ip_* fixes this issue, and the reverberation happens as it should.

Is this a BML issue or freeverb issue, or something else?

Example song here: bug-bml-gaps.txt

@ensonic
Copy link
Member

ensonic commented Aug 12, 2021

This code

gstbml_fix_data (GstElement * elem, GstMapInfo * info, gboolean has_data)

is checking results from buzzmachine effects
https://github.com/Buzztrax/buzztrax/blob/master/src/gst/bml/gstbmltransform.c#L346
and sets the gap flag is the buffer is all silence.

Luckily we do have the source for panzerdelay and it see to return the has_data flag:
https://github.com/Buzztrax/buzzmachines/blob/master/FSM/PanzerDelay/PanzerDelay.cpp#L724

Also the freeverb plugin seems to explicitly hand the gap flag:
https://github.com/GStreamer/gst-plugins-bad/blob/master/gst/freeverb/gstfreeverb.c#L919

Sounds like some printf debugging is due :/

dlbeswick added a commit to dlbeswick/buzztrax that referenced this issue Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants