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.

Sign up for GitHub

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

Jump to bottom

Text to Speech integrated with LUIS BOT does not voice out/playback in Safari browser on iOS/iPhone devices #1829

Closed
corinagum opened this issue Mar 20, 2019 · 3 comments · Fixed by #2035
Closed

Text to Speech integrated with LUIS BOT does not voice out/playback in Safari browser on iOS/iPhone devices #1829

corinagum opened this issue Mar 20, 2019 · 3 comments · Fixed by #2035
Assignees
@compulim
Labels
area-speech front-burner p1 Painful if we don't fix, won't block releasing

Comments

@corinagum
Copy link
Contributor

Customer is facing problems using Speech API on a LUIS BOT where, Text to Speech/voice out/playback is not working in Safari browser on iOS devices, though it works on both Android and Windows clients.
Here is a sample URL for testing which I have hosted.

Related to #995
@corinagum corinagum added needs-repro Waiting for repro or investigation and removed Triage-E labels Mar 28, 2019
@compulim
Copy link
Contributor

compulim commented Apr 1, 2019

Need more investigations.

  • Go to https://compulim.github.io/web-speech-cognitive-services/ and try if text-to-speech is working on iOS or not.
    • If it is not, find out the issue in that repository
    • If yes, it could be related to "playing audio without an user interaction"
      • Prime the audio playback engine when the user click microphone button on Web Chat

@compulim compulim added the 4.5 label Apr 1, 2019
@compulim compulim removed their assignment Apr 1, 2019
@corinagum corinagum added backlog Out of scope for the current iteration but it will be evaluated in a future release. and removed 4.5 labels Apr 2, 2019
@corinagum corinagum added the bug Indicates an unexpected problem or an unintended behavior. label May 3, 2019
@cwhitten cwhitten added the 4.5 label May 13, 2019
@corinagum corinagum removed the backlog Out of scope for the current iteration but it will be evaluated in a future release. label May 13, 2019
@cwhitten cwhitten added p1 Painful if we don't fix, won't block releasing and removed needs-repro Waiting for repro or investigation labels May 13, 2019
@compulim
Copy link
Contributor

The exception thrown at this line.

image

image

@compulim compulim removed bug Indicates an unexpected problem or an unintended behavior. labels May 22, 2019
@compulim compulim mentioned this issue May 22, 2019
Merged
@compulim
Copy link
Contributor

The core bug of this issue is because we asked the speech synthesizer to cancel if it cannot start within a second. The kill-and-retry logic was used to workaround a Chrome bug and it no longer repro today.

Since we are using Cognitive Services, it could take more than a second to download and start playing the MP3. Thus, it is not very reliable today.

@compulim compulim mentioned this issue May 29, 2019
Merged
@compulim compulim mentioned this issue Jul 11, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@compulim compulim

Labels
area-speech front-burner p1 Painful if we don't fix, won't block releasing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix long text is not synthesized when using Cognitive Services compulim/BotFramework-WebChat
3 participants
@cwhitten @compulim @corinagum