search

skiwithpete / alarmpi

This is the latest version of the alarm clock for Raspberry Pi project.
116 stars 49 forks source link

Added support for Ivona text to speech #2

Closed victorbjorklund closed 8 years ago

victorbjorklund commented 9 years ago

I added support for Ivona text to speech (https://www.ivona.com).

I used pygame for audio since I couldn't get the subprocess audio to work (altough someone with more experience with that can probably do it) and thought it might not be a problem since the raspberry pi python installation comes with pygame package installed. But if for some reason pygame shouldn't be used it should possible to substitute it in a later update.

skiwithpete commented 8 years ago

Hi,

I hadn't seen this, and merged a newer pull request.

Can you integrate your code again? I'm sorry that I didn't attend to this earlier.

Thanks,

P

cpenning commented 8 years ago

Gentlemen,

I was able to merge this seemlessly with the master, but the install instructions weren't sufficient for me to get pynova installed on raspbian/jessie.

If nobody objects I would like to make a few modifications to have it be optional so it doesn't error when it's not installed as long as long as it's not enabled. This is also how I want to add my support for the pico2wave/aplay TTS option since it is not yet trivial to install either.

(OK, I also want to fix an annoyance that I introduced where if you call sound_the_alarm.py without an explicit path it throws an exception.)

I will defer to either of you since it is your code. Otherwise I can have another pull request with these changes by tomorrow a bit after work.

Cheers, Craig

skiwithpete commented 8 years ago

Yes.

Please continue. I'm happy to have your help.

All the best,

P

On Tue, Mar 8, 2016 at 8:07 PM, Craig Pennnington notifications@github.com wrote:

Gentlemen,

I was able to merge this seemlessly with the master, but the install instructions weren't sufficient for me to get pynova installed on raspbian/jessie.

If nobody objects I would like to make a few modifications to have it be optional so it doesn't error when it's not installed as long as long as it's not enabled. This is also how I want to add my support for the pico2wave/aplay TTS option since it is not yet trivial to install either.

(OK, I also want to fix an annoyance that I introduced where if you call sound_the_alarm.py without an explicit path it throws an exception.)

I will defer to either of you since it is your code. Otherwise I can have another pull request with these changes by tomorrow a bit after work.

Cheers, Craig

— Reply to this email directly or view it on GitHub https://github.com/skiwithpete/alarmpi/pull/2#issuecomment-194051687.