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

Piper #15

Merged
merged 13 commits into from
Jan 3, 2025
Merged

Piper #15

merged 13 commits into from
Jan 3, 2025

Conversation

eliasbe
Copy link
Contributor

@eliasbe eliasbe commented Dec 6, 2024

Add Piper TTS and misc bug fixes.

The last commit, 13c84a0, needs some work.

GPU support to be added. STDERR from Piper is suppressed for now.
Note, this is perhaps not the best way to indicate the current text
format.
@eliasbe eliasbe requested a review from sveinbjornt December 6, 2024 14:57
src/icespeak/tts.py Outdated Show resolved Hide resolved
Change class name so not to conflict with piper python bindings.
Copy link
Member

@sveinbjornt sveinbjornt left a comment

Choose a reason for hiding this comment

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

I assume issues marked resolved have been fixed but not pushed. Some sort of dependency conflict in CI, but otherwise looks fine to me. Some good bug fixes in there.

@sveinbjornt
Copy link
Member

I suggest removing

assert VOICES, """No voices available..."""

in tts.py, and making one of the Piper voices the default voice. That way, the package works out of the box without requiring an API key.

@sveinbjornt sveinbjornt requested a review from Copilot December 11, 2024 14:18

Choose a reason for hiding this comment

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

Copilot reviewed 6 out of 6 changed files in this pull request and generated no suggestions.

Remove API key warning. Small piper fixes.
@eliasbe
Copy link
Contributor Author

eliasbe commented Dec 16, 2024

I'm looking into the dependency issues. It seems python 3.12 has problems with both Piper and OpenSSL 3.0.* required by Azure services.

@eliasbe eliasbe merged commit 5dbdc80 into master Jan 3, 2025
4 checks passed
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.

2 participants