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

[googletts] Replace custom TTS cache with common TTS cache #15208

Merged
merged 2 commits into from
Jul 10, 2023

Conversation

dalgwen
Copy link
Contributor

@dalgwen dalgwen commented Jul 8, 2023

Uses the common TTS cache instead of a custom one.
No new features

Relates to #15110

Advantages

  • use common configuration
  • cache size configuration available
  • better concurrency management
  • simplifying googletts code.

@dalgwen dalgwen requested a review from gbicskei as a code owner July 8, 2023 13:29
@dalgwen
Copy link
Contributor Author

dalgwen commented Jul 8, 2023

I didn't update the internationalization files for langage other than default.

I believe it should be done inside the dedicated tool ? If not, please tell me.

@lolodomo lolodomo mentioned this pull request Jul 8, 2023
7 tasks
Signed-off-by: Gwendal Roulleau <gwendal.roulleau@gmail.com>
@dalgwen dalgwen force-pushed the googletts_use_common_cache branch from 3c8cf1d to 877b272 Compare July 8, 2023 13:49
@lolodomo
Copy link
Contributor

lolodomo commented Jul 8, 2023

This TTS service is using a ByteArrayAudioStream. So we can merge this PR without waiting any enhancement of audio sinks.

@lolodomo lolodomo added the enhancement An enhancement or new feature for an existing add-on label Jul 8, 2023
Apply code review

Signed-off-by: Gwendal Roulleau <gwendal.roulleau@gmail.com>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM thank you

@lolodomo lolodomo merged commit 2899421 into openhab:main Jul 10, 2023
@lolodomo lolodomo added this to the 4.0 milestone Jul 10, 2023
@dalgwen dalgwen deleted the googletts_use_common_cache branch July 12, 2023 07:22
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
…5208)

* [googletts] Replace custom TTS cache with common TTS cache

--------

Signed-off-by: Gwendal Roulleau <gwendal.roulleau@gmail.com>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants