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

Update the ESpeak TTS plugin #2581

Merged
merged 5 commits into from
May 14, 2020

Conversation

forslund
Copy link
Collaborator

Description

The espeak output was done directly from the espeak executable and not through the TTS output queue, this makes the ESpeak module work more like mimic, mimic2 etc.

This should resolve #2579 allowing wait_while_speaking to work in a better fashion.

This also adds a simple test case for the espeak module and fixes a typo in a mimic-test

How to test

Assert that ESpeak output works as previously.

Contributor license agreement signed?

CLA [ Yes ]

@devs-mycroft devs-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label May 11, 2020
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

An underscore instead of a . made the subprocess check output always
pass.
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

This also normalizes the path slightly to make sure that the phonetic
spelling is loaded even if the lang code contains capitals.
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@krisgesling
Copy link
Contributor

Works great!

@krisgesling krisgesling merged commit b865e80 into MycroftAI:dev May 14, 2020
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.

wait_while_speaking() it does not work
4 participants