Skip to content

Add simpler TTS feature.#128

Open
jtrujil43 wants to merge 2 commits intodefano:masterfrom
jtrujil43:master
Open

Add simpler TTS feature.#128
jtrujil43 wants to merge 2 commits intodefano:masterfrom
jtrujil43:master

Conversation

@jtrujil43
Copy link
Copy Markdown

CoPilot removed MaryTTS since JAM repository wasn't available. Now it will use the built in JavaTTS features built into OpenJSK 1.8

Switch from MaryTTS to JavaTTS to remove dependancies.
@defano
Copy link
Copy Markdown
Owner

defano commented Nov 25, 2025

Hi @jtrujil43, thanks for this PR. I agree that we should eliminate MaryTTS, which does not seem to be well-distributed anymore (I've had ongoing issues with its dependencies in the past). So I think you're on the right path with this.

However, there seem to be a few things missing in your change:

  1. The build.gradle file needs to have a Speech implementation library included, like implementation 'net.sf.sociaal:freetts:1.2.2', otherwise, your check Class.forName("javax.speech.synthesis.Synthesizer") will always fail.

  2. It seems you removed the code that actually calls the synthesizer. The speak() method now just logs a message and delays for a reasonable time, even when a synthesizer like freetts is available. Nothing actually invokes the speech synthesizer implementation to produce the text-to-speech audio.

Finally, your PR contains a .vscode directory/file that I don't believe should be committed. This file contains settings specific to your local machine.

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.

2 participants