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

Timeout since last update #30

Closed
ghost opened this issue Nov 10, 2020 · 26 comments
Closed

Timeout since last update #30

ghost opened this issue Nov 10, 2020 · 26 comments

Comments

@ghost
Copy link

ghost commented Nov 10, 2020

Hey Dennie,
Since last update I receive regulary a timeout when sending TTS. What happend?

@ghost
Copy link
Author

ghost commented Nov 18, 2020

also, even my sonos stops for halve a second when this app send to google... how is that suddenly possible?

@BWiltjer
Copy link

I updated to version 1.4.2 last week, the app does n't look stable any more, first had to adjust all volumes in my flows from 1% to the old values. Mostly a time-out in the flows, restarting the app works for a while. but the time-out's come back after a while.

In Dutch:
(De update voor Google TTS met een volume tag leek me wel handig. Eerst stonden in al mijn flows het volume op 1 en na het upgraden naar versie 1.4.2 is de app niet langer stabiel, en weigert regelmatig dienst. De app restarten geeft dan een oplossing, maar helaas niet blijvend....)

@runely
Copy link

runely commented Dec 1, 2020

Same here. Google tts times out with a red triangle on the action card..

Even tried to send a simple "Test", but this also times out.

image

@denniedegroot
Copy link
Owner

Google changed something in their API. I am not sure how to fix this correctly at the moment.

@ghost
Copy link
Author

ghost commented Dec 1, 2020 via email

@flits
Copy link

flits commented Dec 2, 2020

Same problem over here. Fingers crossed that you find the time and a solution. Using this app helped me to create the WAF (wife acceptance factor) 😂.

Meanwhile I will remove the cards using TTS.

@shaarkys
Copy link
Contributor

shaarkys commented Dec 2, 2020

Seems there is new google-api ...

https://github.com/zlargon/google-tts/blob/master/CHANGELOG.md

0.0.5 (Nov 8, 2020)

  • Upgrade the dependencies and fix the vulnerability. (#32)
  • Add retry mechanism to prevent fetching token key failed too frequently. (#33)

When running Googe TTS from console, I get :

─────────────── Logging stdout & stderr ─────────────── com.google.tts running... Error: get key failed from google at /node_modules/google-home-audio-tts/node_modules/google-tts-api/lib/key.js:30:32 at processTicksAndRejections (internal/process/task_queues.js:97:5) Error: get key failed from google at /node_modules/google-home-audio-tts/node_modules/google-tts-api/lib/key.js:30:32 at processTicksAndRejections (internal/process/task_queues.js:97:5)

Which is already discussed on Git : zlargon/google-tts#33

... with fix - freddiefujiwara/google-tts@d8ddfcf ?

@denniedegroot
Copy link
Owner

denniedegroot commented Dec 2, 2020

@shaarkys I know but that does not fix the real issue yet. People are working on it though.

@shaarkys
Copy link
Contributor

shaarkys commented Dec 2, 2020

Thank you @denniedegroot - you mean even this is not fixing it ?

zlargon/google-tts#37

@denniedegroot
Copy link
Owner

Thank you @denniedegroot - you mean even this is not fixing it ?

zlargon/google-tts#37

Still gives some timeouts for me.

@shaarkys
Copy link
Contributor

shaarkys commented Dec 2, 2020

I know nothing about development but isn't it due to the google-home-audio-tts still containing not fixed google-tts-api ?
If it's nonsense, simply ignore me ;-)

@flits
Copy link

flits commented Dec 2, 2020

@denniedegroot is it possible to rollback to the previous version? Just for the time beïng? Or would that version suffer with the same API problem? ☺️

@shaarkys
Copy link
Contributor

shaarkys commented Dec 2, 2020

@flits I guess it's Google API that changed just yesterday, it has nothing to do with Google TTS app previous version...

@shaarkys
Copy link
Contributor

shaarkys commented Dec 4, 2020

Working with latest commit - https://github.com/freddiefujiwara/google-tts/tree/feature/fix-dec-1
Via CLI method and simply replacing google-tts-api and google-home-audio-tts\node_modules\google-tts-api it works again.
(I didn't check what is used and what is obsolete)
Simply after replacing content of those two directories, install via CLI - working.
@denniedegroot will you please do it the right way ? ;-)

google-tts-api was fixed few hours ago - master can be used - https://github.com/zlargon/google-tts

@shaarkys
Copy link
Contributor

shaarkys commented Dec 6, 2020

In case someone can't wait - https://github.com/shaarkys/com.google.tts - and install via CLI. Please note it's done "dirty way" but it works. Also seems current API call method will be not permanent based on zlargon/google-tts#38

@florabtw
Copy link

florabtw commented Dec 7, 2020

If you are wiling to change dependencies, I wrote a new library for the new Translate API, which will continue to be maintained and doesn't depend on previous implementations: https://github.com/ncpierson/google-translate-tts

The big difference is that it returns a Promise<Buffer> instead of a url.

@ghost
Copy link
Author

ghost commented Dec 7, 2020 via email

@florabtw
Copy link

florabtw commented Dec 7, 2020

I know nothing about development but isn't it due to the google-home-audio-tts still containing not fixed google-tts-api ?
If it's nonsense, simply ignore me ;-)

I just realized @shaarkys is actually right here. It looks like google-home-audio-tts is actually the library that needs updated.

You could still do it the "dirty way" as part of this library, but it would be more difficult.

@denniedegroot
Copy link
Owner

Due to limited time I merged the great work of @shaarkys. The app really needs a rewrite with maybe the rewritten library of @ncpierson and a better chromecast library.

If someone can test and install the latest app via https://homey.app/a/com.google.tts/test/ and let me know if it works that would be great so I can publish it to the app store.

@bulletcather
Copy link

I have installed the test app and it works again. Thanks for the support

@flits
Copy link

flits commented Dec 7, 2020

Just installed and immediately works again 🤩🥰. Asked other users to test it and post their feedback here 👍

@paulvanderheijden90
Copy link

It works again perfect !!!!!

@BWiltjer
Copy link

BWiltjer commented Dec 8, 2020

Works perfect! Thx

@ghost
Copy link
Author

ghost commented Dec 8, 2020 via email

@ghost
Copy link
Author

ghost commented Dec 8, 2020 via email

@bakker284
Copy link

Not working here , I'm on v5

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

No branches or pull requests

9 participants