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

dpp::on_voice_receive_combined called only once per tick, with not enough samples #1008

Open
tt2468 opened this issue Nov 19, 2023 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@tt2468
Copy link

tt2468 commented Nov 19, 2023

Describe the bug
When subscribing to dpp::on_voice_receive_combined, the callback is only triggered once per audio tick (2 ticks per second, hardcoded). The audio data provided is generally only 960 stereo audio samples, or 1/25th of the expected amount.

dpp::on_voice_receive works as "expected", delivering 25x callbacks in sequence, per user, for every audio tick.

To Reproduce
Steps to reproduce the behavior:

  1. Load a DPP bot into a voice channel with a music bot playing sound
  2. Subscribe to combined audio data callbacks via dpp::on_voice_receive_combined
  3. Receive only one callback of 960 (usually, sometimes many more) frames twice per second

Expected behavior
While not ideal behavior, one may expect that they should also get 25x callbacks of 960 samples of combined audio data per tick, instead of just one.

System Details:

  • OS: Ubuntu 22.04

Additional Context:
Patching the library to use a tick interval of 20ms instead of 500ms produces the correct number of callbacks per second (50), however, the resulting audio data is completely garbled.

Copy link
Contributor

This issue has had no activity and is being marked as stale. If you still wish to continue with this issue please comment to reopen it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Todo
Development

No branches or pull requests

3 participants