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

added support for [[ipa-phonemes]] in text #401

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

contentnation
Copy link

Added support for inline phonemes in IPA format.
Especially useful for mixing languages or words that create wrong phonemes.
In this case "feature" an English word in a German sentence.
echo -n "Dies ist ein [[fˈiːtʃə]] des pull requests." | piper -m ...
Translation: "This is a feature of the pull request."

@chriseaton
Copy link

I just wanted to put in my 2cents here that I tested this merge request and, at least regarding the feature change subject, it worked as expected and compiled without issue (tried with a few different voices).

@synesthesiam
Copy link
Contributor

I'd prefer to use SSML for this instead. There is a dedicated tag specifically for phonetic pronunciation.

@nitinthewiz nitinthewiz mentioned this pull request Jun 18, 2024
@sassanh
Copy link

sassanh commented Jul 28, 2024

@synesthesiam Thanks for all your efforts on this open source project!
I noticed the development on this repository has slowed down a little bit, there were a single commit in last 2 months, which was a line added to the README.md to mention a vim plugin.
Having that in mind, maybe it's not a bad idea to merge this PR until you can invest on a better alternative.
At least one user has reported it is working for them and it fills a gap in the feature set.
I understand if you have other considerations regarding this issue.

I may be wrong about development pace, I just checked the insights page.

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.

4 participants