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

Text to speech for readback #39

Closed
smplman opened this issue Dec 16, 2014 · 10 comments
Closed

Text to speech for readback #39

smplman opened this issue Dec 16, 2014 · 10 comments

Comments

@smplman
Copy link

smplman commented Dec 16, 2014

I have found a nice HTML5 JS text to speech in chrome. I'm interested in jumping in on this project and this is where I may start.

http://updates.html5rocks.com/2014/01/Web-apps-that-talk---Introduction-to-the-Speech-Synthesis-API

@smplman
Copy link
Author

smplman commented Dec 16, 2014

@zlsa what icon pack are you using for the controls? I have been looking into implementing ionicons, but I didn't want to change them over if you were already using an icon pack.

https://github.com/driftyco/ionicons

@zlsa
Copy link
Owner

zlsa commented Dec 16, 2014

The icons were all handmade in Inkscape. If you want any additional icon, I can make it.

@daviessm
Copy link
Contributor

I had a play and added speech synthesis for readbacks using the W3C web speech API. There's no on/off switch and on my machine (Chromium 39 on Linux) the speech gets stuck after every readout so I have to call speech.cancel() manually each time but it does read things out. Also I needed to start Chromium with the --enable-speech-dispatcher parameter for it to work.

https://github.com/daviessm/atc/commit/116836f7fa70c09352b563808e1b9c3e0c2791ff

@Diastro
Copy link
Collaborator

Diastro commented Dec 20, 2014

@daviessm hahahaha! I just cloned your repo to test and I gotta say it's pretty cool! I'd be amazing if the voice could be a little more "natural".

Still pretty cool.

I don't think read-back should be on by default but if we could add a button to enable/toggle this and maybe work on the voice it'd be a great addition!

@daviessm
Copy link
Contributor

@Diastro The speech is done through the browser which calls OS libraries to do the actual synthesis. How it sounds will depend entirely on that. I've just tried it on my Android phone (5.0.1) and it sounds pretty natural.

Agree about the on-by-default but I suck at UI design so I'll leave that up to someone else. It would be easy enough to add another variable speechOn or something to speech.js and toggle it.

I would also like to have it pick random voices for each plane. So some are male and some are female. And have the tower controller as yet another voice.

@Diastro
Copy link
Collaborator

Diastro commented Dec 20, 2014

@daviessm Gotcha. Yeah regarding UI @zlsa is probably the best person to help for that last part.

@Diastro
Copy link
Collaborator

Diastro commented Dec 23, 2014

Can we close this?

@daviessm
Copy link
Contributor

@smp4488 Sorry to have kind of jumped past you on this - does it do what you were thinking of?

@smplman
Copy link
Author

smplman commented Dec 23, 2014

@daviessm it's all good. Holidays have me all tied up. I haven't had a chance to look at your changes yet, but you can see what I have added here https://github.com/smp4488/atc. I created a voice / voices model wanting to offer the user a choice of the available system voices. I have most of it hashed out, but the ui still needs a little love.

@smplman smplman closed this as completed Dec 23, 2014
@daviessm
Copy link
Contributor

@smp4488 The additions I made weren't configurable at all - it just adds a call to the speech synthesis API. @zlsa added an on/off toggle to that. I think it'd be worthwhile to have a UI for changing other settings too.

erikquinn pushed a commit that referenced this issue Nov 12, 2016
* feature/ATC-11 - Adds link to tools/readme updates working in tools/readme.
* feature/ATC-11 - Adds info on adding a new airport
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants