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

TTS v2 #8

Merged
merged 4 commits into from
Jul 11, 2021
Merged

TTS v2 #8

merged 4 commits into from
Jul 11, 2021

Conversation

optimalstrategy
Copy link
Contributor

  • Implement playing TTS requests (w/ filtering and cleaning), using Rodio
    • Runs on separate thread dedicated to networking (+ Rodio's audio thread)
    • Every chat message that follows the command syntax and passes the checks is tokio::spawn'd to be downloaded
      • The download task is paused if the API quota has been exhausted (set to 5 requests per minute currently)
      • The download is re-attempted 3 times
      • If the download succeeds, the message is added to the playback queue
    • QQQ: How should we notify the user if the default audio device isn't available? Not sure how unlikely that is.
  • Send configuration changes to the networking thread after a short delay (1s)
  • Add playback control buttons
  • Remove the path text field
  • Use .config only on unix

TODO:

  • UI for bannedwords.txt
  • Bits/Redeems
  • Command cooldown
  • Drop the ws route
  • Code cleanup

…feat): add playback control buttons; (feat): send configuration changes to the networking thread after a short delay (1s)
@optimalstrategy
Copy link
Contributor Author

optimalstrategy commented Jul 10, 2021

Nvm, turns out I could do this myself.
Auto-merging breaks the codebase. You should merge this locally with --strategy ours:

$ git checkout ui-v2
$ git merge --strategy ours master
$ git checkout master
$ git merge ui-v2

src/main.rs Outdated Show resolved Hide resolved
src/tts.rs Outdated Show resolved Hide resolved
src/ui.rs Outdated Show resolved Hide resolved
src/tts.rs Outdated Show resolved Hide resolved
…l name on enter; (feat): switch to alto_logger and configure it to write to a file when compiled in release mode
Copy link
Owner

@jprochazk jprochazk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jprochazk jprochazk merged commit 4564a00 into jprochazk:master Jul 11, 2021
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

Successfully merging this pull request may close these issues.

None yet

2 participants