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

Add config options for TTS engine (+ MaryTTS support) #229

Merged
merged 4 commits into from
Nov 5, 2014

Conversation

Holzhaus
Copy link
Member

This makes TTS engines have config options (e.g. pitch_adjustment or words_per_minute for Espeak TTS).

It also adds support for MaryTTS. An online demo can be found here. I set en_GB/dfki-spike as default voice, because it actually makes you feel like you have a british butler. ;-)

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.45%) when pulling f8e2d2b on Holzhaus:marytts-support into 4ff5a97 on jasperproject:master.

@Holzhaus Holzhaus mentioned this pull request Oct 20, 2014
@Holzhaus
Copy link
Member Author

@crm416 @shbhrsaha Can you test this, too?

@shbhrsaha
Copy link
Member

Sure thing-- apologies we had midterms this week! Will update this and the other PR early next week

Shubhro

On Wed, Oct 22, 2014 at 12:16 PM, Jan Holthuis notifications@github.com
wrote:

@crm416 @shbhrsaha Can you test this, too?

Reply to this email directly or view it on GitHub:
#229 (comment)

@shbhrsaha
Copy link
Member

Sorry for the wait. After much re-imaging and trying different machines, it seems like something's wrong with my Pi since I cannot SSH into it :( @crm416 can test it this weekend when he's back with his Pi, will update again then!

Awesome work! Anxious to get this and #219 merged in soon after we test

@Holzhaus
Copy link
Member Author

Holzhaus commented Nov 4, 2014

No problem. Your exams have a higher priority ;)

Anyway, I'm already workung in an audio engine that should make it easier to playback/capture sound and let us have all pyaudio specific code in one place.

I stopped working on it until these PRs have been merged to avoid git rebase hell.

@shbhrsaha
Copy link
Member

Thank you for understanding. Just tested this out-- works great! Good to merge.

Holzhaus added a commit that referenced this pull request Nov 5, 2014
Add config options for TTS engine (+ MaryTTS support)
@Holzhaus Holzhaus merged commit c44b772 into jasperproject:master Nov 5, 2014
@Holzhaus Holzhaus deleted the marytts-support branch November 28, 2014 11:24
@vidhya03
Copy link

vidhya03 commented Jun 2, 2016

i am testing latest jasper and marytts, however not working. could you please tell me the steps

@ghost
Copy link

ghost commented Jun 6, 2016

Please use the forum for questions, this is not a code issue. @Vidhyadharantechdays

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants