dnpwwo / Domoticz-Google-Plugin

MIT License
40 stars 10 forks source link

Performance #26

Closed rezzalex closed 3 years ago

rezzalex commented 3 years ago

Hello,

thanks again for the plugin.

My sound notifications are arriving around on my Lenovo Smart Display 30 seconds after the event that trigger the voice message. is it normal ? for an efficient day to day usage, it is way too long for me... I precise that this Lenovo smart display is connected trough a Fast wifi (5Gz very close to the Wifi access point) and that the Domoticz host (My NAS) is connected trough a gigabit lan connection

Mark-Sellwood commented 3 years ago

I'm seeing the same sort of issue but the delay is 90 seconds, log extract below:-

2020-11-11 09:16:27.117 (Dummy Switches) Lighting 1 (Test) 2020-11-11 09:16:27.295 (Google) onCommand called for Unit 5: Parameter 'SendNotification Test On Script', Level: 0 2020-11-11 09:16:27.100 Status: User: Admin initiated a switch command (348/Test/On) 2020-11-11 09:16:27.232 Status: EventSystem: Script event triggered: Speak 2020-11-11 09:18:01.520 (Google) [Office display Status] Update 9(9):'Default Media Receiver'(Screensaver): 0(0) 2020-11-11 09:18:01.548 (Google) [Office display Volume] Update 2(2):'50'(49): 0(0) 2020-11-11 09:18:02.865 (Google) [Office display Playing] Update 2(0):'1'(0): 0(0) 2020-11-11 09:18:03.693 (Google) [Office display Playing] Update 2(2):'54'(1): 0(0) 2020-11-11 09:18:03.868 (Google) [Office display Playing] Update 2(2):'62'(54): 0(0) 2020-11-11 09:18:04.639 (Google) [Office display Playing] Update 0(2):'0'(62): 0(0) 2020-11-11 09:18:05.035 (Google) Notification sent to 'Office display' completed 2020-11-11 09:18:05.054 (Google) [Office display Volume] Update 2(2):'49'(50): 0(0) 2020-11-11 09:18:05.080 (Google) [Office display Status] Update 9(9):'Screensaver'(Default Media Receiver): 0(0)

The contents of the script is:- commandArray = {} if (devicechanged['Test'] == 'On') then commandArray['Office display Status'] = "SendNotification Test On Script" end return commandArray

I can see that the plugin receives the commend as the switch is activated but nothing comes out the speaker for 90 seconds.

rezzalex commented 3 years ago

Maybe I have "only" 30 secs delay because My DZ host is on Gigabit cable and the Google speaker on 5ghz wifi.... ? still it is too long...

On Wed, 11 Nov 2020 at 10:25, Mark-Sellwood notifications@github.com wrote:

I'm seeing the same sort of issue but the delay is 90 seconds, log extract below:-

2020-11-11 09:16:27.117 (Dummy Switches) Lighting 1 (Test) 2020-11-11 09:16:27.295 (Google) onCommand called for Unit 5: Parameter 'SendNotification Test On Script', Level: 0 2020-11-11 09:16:27.100 Status: User: Admin initiated a switch command (348/Test/On) 2020-11-11 09:16:27.232 Status: EventSystem: Script event triggered: Speak 2020-11-11 09:18:01.520 (Google) [Office display Status] Update 9(9):'Default Media Receiver'(Screensaver): 0(0) 2020-11-11 09:18:01.548 (Google) [Office display Volume] Update 2(2):'50'(49): 0(0) 2020-11-11 09:18:02.865 (Google) [Office display Playing] Update 2(0):'1'(0): 0(0) 2020-11-11 09:18:03.693 (Google) [Office display Playing] Update 2(2):'54'(1): 0(0) 2020-11-11 09:18:03.868 (Google) [Office display Playing] Update 2(2):'62'(54): 0(0) 2020-11-11 09:18:04.639 (Google) [Office display Playing] Update 0(2):'0'(62): 0(0) 2020-11-11 09:18:05.035 (Google) Notification sent to 'Office display' completed 2020-11-11 09:18:05.054 (Google) [Office display Volume] Update 2(2):'49'(50): 0(0) 2020-11-11 09:18:05.080 (Google) [Office display Status] Update 9(9):'Screensaver'(Default Media Receiver): 0(0)

The contents of the script is:- commandArray = {} if (devicechanged['Test'] == 'On') then commandArray['Office display Status'] = "SendNotification Test On Script" end return commandArray

I can see that the plugin receives the commend as the switch is activated but nothing comes out the speaker for 90 seconds.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dnpwwo/Domoticz-Google-Plugin/issues/26#issuecomment-725309681, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIMCHN4PMP5NGY7A2SOHAXTSPJKARANCNFSM4TPJKJMA .

Mark-Sellwood commented 3 years ago

Like you my Domoticz running on a Raspberry Pi 3+ is connected via Gigabit cable and the Google speaker on 5ghz wifi. I wonder if the delay is the Text to speech conversion? Just done a couple of tests, looking at my DNS server logs I can see a delay in the plugin sending the text to translate.google.com Not sure if its the plugin or gtts

rezzalex commented 3 years ago

The processing time ?

Le mer. 11 nov. 2020 à 12:25, Mark-Sellwood notifications@github.com a écrit :

Like you my Domoticz running on a Raspberry Pi 3+ is connected via Gigabit cable and the Google speaker on 5ghz wifi. I wonder if the delay is the Text to speech conversion?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dnpwwo/Domoticz-Google-Plugin/issues/26#issuecomment-725368582, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIMCHN4C5366GXPWUWPE3I3SPJYCLANCNFSM4TPJKJMA .

Mark-Sellwood commented 3 years ago

Just dome some more tests from the command line The delay is with gtts, I entered :- gtts-cli 'hello' --output hello.mp3 It took 90 seconds for the command prompt to return, so to me the issue is with gtts

Mark-Sellwood commented 3 years ago

gtts 2.2.1 has just been released, some of the fixes are for:- gTTS slow on generating file gTTS takes 10-15 mins to get audio back. Slow problem I have upgraded & now the notifications are almost instantaneous.

rezzalex commented 3 years ago

gtts 2.21.1 has just been released, some of the fixes are for:- gTTS slow on generating file gTTS takes 10-15 mins to get audio back. Slow problem I have upgraded & now the notifications are almost instantaneous.

Thanks you !!!

I move to the last dz beta version, re install the plugin, " pip3 install gtts" gave me "gtts-2.2.1". Now the notification arrived within 5 / 6 seconds, Faster for you ?

best regarsd

frankdpGH commented 3 years ago

gtts 2.21.1 has just been released, some of the fixes are for:- gTTS slow on generating file gTTS takes 10-15 mins to get audio back. Slow problem I have upgraded & now the notifications are almost instantaneous.

Thanks for the hint!

Mark-Sellwood commented 3 years ago

I would say its a couple of seconds, I haven't timed it but its way better then the 90 seconds I was getting.

rezzalex commented 3 years ago

Do you have a more recent version than I Did you do " pip3 install --uograde gtts ? "

Le lun. 16 nov. 2020 à 19:12, Mark-Sellwood notifications@github.com a écrit :

I would say its a couple of seconds, I haven't timed it but its way better then the 90 seconds I was getting.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dnpwwo/Domoticz-Google-Plugin/issues/26#issuecomment-728234433, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIMCHN7K4KT5FXB5LJ3ZKH3SQFTQLANCNFSM4TPJKJMA .

Mark-Sellwood commented 3 years ago

I have 2.2.1 using pip3 install gtts -U

2.21.1 was a typo & I've corrected it now.

rezzalex commented 3 years ago

I confirm notification arrived at the exact same moment as for DZ android app.