Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Conversation

@forslund
Copy link
Collaborator

@forslund forslund commented Oct 28, 2019

Description

Logs the exception and fallsback to Mimic as TTS backend if there is an
issue with the selected TTS backend or if the choice is invalid. Resolves #1518.

How to test

Make sure audio service starts as expected. Trigger an exception in the __init__() or validator of a TTS and verify that it falls back to mimic and logs the issue.

Contributor license agreement signed?

CLA [ Yes ]

@pep8speaks
Copy link

pep8speaks commented Oct 28, 2019

Hello @forslund! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-10-28 13:20:27 UTC

@devs-mycroft devs-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Oct 28, 2019
Logs the exception and fallsback to Mimic as TTS backend if there is an
issue with the selected TTS backend or if the choice is invalid.
@forslund forslund force-pushed the feature/tts-create-error-handling branch from 31b199f to c832852 Compare October 28, 2019 13:20
@forslund forslund merged commit 5f1cc40 into MycroftAI:dev Oct 29, 2019
@forslund forslund deleted the feature/tts-create-error-handling branch November 28, 2019 09:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Audio service crash

4 participants