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 TextToSpeechManager to support dynamic speech #26

Closed
wants to merge 2 commits into from
Closed

Added TextToSpeechManager to support dynamic speech #26

wants to merge 2 commits into from

Conversation

jbienzms
Copy link
Contributor

@jbienzms jbienzms commented May 3, 2016

Please note that the code for TextToSpeechManager is 100% new code by
me. However, portions of the Wav class were based on this sample:
http://answers.unity3d.com/questions/737002/wav-byte-to-audioclip.html

The author does release that code for others to use and I have
attributed the article in the class comments, but if this source
is not acceptable we will need to rewrite the WAV class.

Here are the steps for adding Text to Speech to your own apps:

  1. Create an AudioSource somewhere in the scene. I recommend creating it on an empty GameObject that is a child of the Main Camera and is positioned about 0.6 units above the camera.
  2. Add the TextToSpeechManager component to the scene.
  3. Assoicate the AudioSource with the manager.
  4. In code call TextToSpeechManager.Instance.Speak("Hello World");

@msftclas
Copy link

msftclas commented May 3, 2016

Hi @jbienzms, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Jared Bienz). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

/// </remarks>
public class TextToSpeechManager : Singleton<TextToSpeechManager>
{
#region Region Name
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like the default region name was accidentally left in! Also, since each region really only contains one variable/method, it might be okay to leave them all out.
We haven't really been using regions in the Toolkit so far, so style-matching-wise it might be best to remove them. Others can weigh in with their opinions though!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not tied to the regions. I'm willing to let them go for the sake of matching style.

…our instead of Singleton, fixed a spelling error, and regions were removed. Also added the ability to speak SSML which was omitted from the original submission.
@jbienzms
Copy link
Contributor Author

About to submit a new pull request with all of the above change requests implemented, so I'll close this pull request.

@jbienzms jbienzms closed this May 13, 2016
keveleigh pushed a commit that referenced this pull request Aug 13, 2018
…interUpdates

Updated angle math and removed the need for the 90 degree offset
drusk-unity added a commit to drusk-unity/MixedRealityToolkit-Unity that referenced this pull request Jul 6, 2023
…ple-Feedback

MRH3-62 Eye Tracking Examples Feedback
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.

5 participants