-
Notifications
You must be signed in to change notification settings - Fork 348
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
Running beep.rs example with ALSA only loads data to buffer once #142
Comments
I tried to debug this problem and I have found that Also on my machine |
Can you guys verify the above patch fixes the beep stall. I'm a little unsatisfied with the patch with where I'm calling alsa::snd_pcm_sw_params_set_avail_min(),. Its inside the bits that are setting up buffer_len/period_len values, which seems like bad form. |
@jwestfall69 The style is not a problem for me. |
snd_pcm_sw_params_set_avail_min was being hardcoded to 4096, which seems to be problematic for lower sample rates. This update sets the value to the buffer size as supplied by snd_pcm_get_params(), which is what alsa own sample code does. This should fix RustAudio/cpal#142
No modifications were made to code. When I put println! statements in the stream.for_each closure, they only are called once. Issue persists with both version 0.4.0 and 0.4.1 of crate.
No panics or anything, and the program is properly blocking on the event_loop.run() call. Really not sure what could be going wrong. I can post more specific details of my system if that would help, but I'm fairly new to rust/audio stuff so not sure what would be relevant.
The text was updated successfully, but these errors were encountered: