search

Xenomes / Domoticz-TUYA-Plugin

Implements the special Tuya Home Assistant API.
22 stars 8 forks source link

Make refresh period configurable #11

Closed trekkir closed 3 years ago

trekkir commented 3 years ago

Hi @Xenomes, First of all, thank you for your work! I do have a request; Currently, the refresh period is set to 10 seconds. Is it possible to make it configurable on the Hardware settings page?

I can't seem to find the exact configuration in the python file to modify it myself (basic programming knowledge on my side must be the problem here), so if my request isn't something you want to implement, could you point me in the right direction where I can change that 10 seconds myself?

Xenomes commented 3 years ago

Hi Trekkir, What is the reason you want a refresh period? If you want to change it I think the wait on line 210 is the best to start.

trekkir commented 3 years ago

Hi, I have a plug turning itself on when the doorbell camera is pushed (using the Tuya scene in the Tuya app itself). A blocky event in Domotica triggers on the On state of that plug by triggering a script that will send me a snapshot of the doorbell camera by email and finally turning the plug back off.

But the 10s delay in your plugin means a maximum delay of 19s between pressing the doorbell and my script being executed and I would like to process that a bit faster.

I found the time.sleep at line 210, but doesn't that mean 0.5 seconds delay?

From: Xenomes @.***> Sent: Friday, June 25, 2021, 21:06 To: Xenomes/Domoticz-TUYA-Plugin Cc: trekkir; Author Subject: Re: [Xenomes/Domoticz-TUYA-Plugin] Make refresh period configurable (#11)

Hi Trekkir, What is the reason you want a refresh period? If you want to change it I think the wait on line 210 is the best to start.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/Xenomes/Domoticz-TUYA-Plugin/issues/11#issuecomment-868774602, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUS5D76RNVUSLMRD2BT7AFDTUTHSDANCNFSM47F5VMJQ.

Xenomes commented 3 years ago

Ok, I understand your problem. The 10 seconds I the shortest time I could get to pull data from the API of Tuya there is not a lot of time to win there.

trekkir commented 3 years ago

Ah ok, so it's not specified in the plugin itself?

From: Xenomes @.> Sent: Friday, June 25, 2021 10:15:07 PM To: Xenomes/Domoticz-TUYA-Plugin @.> Cc: trekkir @.>; Author @.> Subject: Re: [Xenomes/Domoticz-TUYA-Plugin] Make refresh period configurable (#11)

Ok, I understand your problem. The 10 seconds I the shortest time I could get to pull data from the API of Tuya there is not a lot of time to win there.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/Xenomes/Domoticz-TUYA-Plugin/issues/11#issuecomment-868810137, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUS5D77DQCSM7PCBVEKPS4TTUTPUXANCNFSM47F5VMJQ.

Xenomes commented 3 years ago

Nope. Maybe you can use a IFTTT or DZGA