ztalbot2000 / homebridge-cmd4

CMD4 Plugin for Homebridge - Supports ~All Accessory Types & now all Characteristics too
Apache License 2.0
145 stars 14 forks source link

Set ventilation off does not work #123

Closed arkadicolson closed 2 years ago

arkadicolson commented 2 years ago

Hi

I have a ventilation unit which I control with cmd4. The set from 0% to 100% does work well!. However when I shutdown (or 0%) the ventilation button in Homekit the script is not triggered. I can see "[1/19/2022, 8:55:44 AM] [Cmd4] Setting (Cached) Ventilatie On false" in the Homebridge logs. Is this something related to a cache setting?

v1.3.9 Timezone GMT+0100 OS Raspbian GNU/Linux Buster (10) Node.js Version v14.18.3 Npm Version v6.14.15

{ "platform": "Cmd4", "name": "Cmd4", "outputConstants": false, "accessories": [ { "type": "Fan", "displayName": "Ventilatie", "on": "TRUE", "rotationDirection": "CLOCKWISE", "rotationSpeed": 100, "name": "Ventilatie", "manufacturer": "VENTRO", "model": "VENTRO-275", "polling": [ { "characteristic": "rotationSpeed", "interval": 10, "timeout": 9000 } ], "serialNumber": "2894745", "outputConstants": true, "stateChangeResponseTime": 3, "state_cmd": "/opt/ventilatie.sh" } }

arkadicolson commented 2 years ago

Never mind. The on characteristic should be defined...

arkadicolson commented 2 years ago

Hmmmm Now I see [1/19/2022, 10:24:45 AM] [Cmd4] getValue: On function for: Ventilatie returned an empty string "". Perhaps your device is busy? in the logs. I indeed do not return anything on this because the only relevant char is rotationSpeed. When setting rotationSpeed to 0 it is off. Is there a way to handle this?

arkadicolson commented 2 years ago

Never mind again ;-) Fixed it, bug in my script

arkadicolson commented 2 years ago

closed

ztalbot2000 commented 2 years ago

Hi,

Sorry, I have been indisposed for awhile. Now that I know of this, I will investigate.

Ttyl, John

On Wed, Jan 19, 2022 at 3:05 AM arkadicolson @.***> wrote:

Hi

I have a ventilation unit which I control with cmd4. The set from 0% to 100% does work well!. However when I shutdown (or 0%) the ventilation button in Homekit the script is not triggered. I can see "[1/19/2022, 8:55:44 AM] [Cmd4] Setting (Cached) Ventilatie On false" in the Homebridge logs. Is this something related to a cache setting?

v1.3.9 Timezone GMT+0100 OS Raspbian GNU/Linux Buster (10) Node.js Version v14.18.3 Npm Version v6.14.15

{ "platform": "Cmd4", "name": "Cmd4", "outputConstants": false, "accessories": [ { "type": "Fan", "displayName": "Ventilatie", "on": "TRUE", "rotationDirection": "CLOCKWISE", "rotationSpeed": 100, "name": "Ventilatie", "manufacturer": "VENTRO", "model": "VENTRO-275", "polling": [ { "characteristic": "rotationSpeed", "interval": 10, "timeout": 9000 } ], "serialNumber": "2894745", "outputConstants": true, "stateChangeResponseTime": 3, "state_cmd": "/opt/ventilatie.sh" } }

— Reply to this email directly, view it on GitHub https://github.com/ztalbot2000/homebridge-cmd4/issues/123, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSBCX3ANG4XCUMHHZJ6653UWZWGJANCNFSM5MJG4M2Q . You are receiving this because you were assigned.Message ID: @.***>

ztalbot2000 commented 2 years ago

Hi,

So you had closed this and I never noticed. I think you figured out that "On" was not polled, thus always retrieved from cache.

Ttyl, John

On Wed, Feb 9, 2022 at 4:46 AM John Talbot @.***> wrote:

Hi,

Sorry, I have been indisposed for awhile. Now that I know of this, I will investigate.

Ttyl, John

On Wed, Jan 19, 2022 at 3:05 AM arkadicolson @.***> wrote:

Hi

I have a ventilation unit which I control with cmd4. The set from 0% to 100% does work well!. However when I shutdown (or 0%) the ventilation button in Homekit the script is not triggered. I can see "[1/19/2022, 8:55:44 AM] [Cmd4] Setting (Cached) Ventilatie On false" in the Homebridge logs. Is this something related to a cache setting?

v1.3.9 Timezone GMT+0100 OS Raspbian GNU/Linux Buster (10) Node.js Version v14.18.3 Npm Version v6.14.15

{ "platform": "Cmd4", "name": "Cmd4", "outputConstants": false, "accessories": [ { "type": "Fan", "displayName": "Ventilatie", "on": "TRUE", "rotationDirection": "CLOCKWISE", "rotationSpeed": 100, "name": "Ventilatie", "manufacturer": "VENTRO", "model": "VENTRO-275", "polling": [ { "characteristic": "rotationSpeed", "interval": 10, "timeout": 9000 } ], "serialNumber": "2894745", "outputConstants": true, "stateChangeResponseTime": 3, "state_cmd": "/opt/ventilatie.sh" } }

— Reply to this email directly, view it on GitHub https://github.com/ztalbot2000/homebridge-cmd4/issues/123, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSBCX3ANG4XCUMHHZJ6653UWZWGJANCNFSM5MJG4M2Q . You are receiving this because you were assigned.Message ID: @.***>

arkadicolson commented 2 years ago

Yes indeed, I figured it out myself as you say with polling ;-)

Best regards Arkadi Colson On 9/02/2022 19:39, John Talbot wrote:

WAARSCHUWING: Deze mail is afkomstig van buiten Smartbit. Klik niet op links en open geen bijlagen, tenzij je deze mail verwacht en hij afkomstig is van een betrouwbare bron.

Hi,

So you had closed this and I never noticed. I think you figured out that "On" was not polled, thus always retrieved from cache.

Ttyl, John

On Wed, Feb 9, 2022 at 4:46 AM John Talbot @.***> wrote:

Hi,

Sorry, I have been indisposed for awhile. Now that I know of this, I will investigate.

Ttyl, John

On Wed, Jan 19, 2022 at 3:05 AM arkadicolson @.***> wrote:

Hi

I have a ventilation unit which I control with cmd4. The set from 0% to 100% does work well!. However when I shutdown (or 0%) the ventilation button in Homekit the script is not triggered. I can see "[1/19/2022, 8:55:44 AM] [Cmd4] Setting (Cached) Ventilatie On false" in the Homebridge logs. Is this something related to a cache setting?

v1.3.9 Timezone GMT+0100 OS Raspbian GNU/Linux Buster (10) Node.js Version v14.18.3 Npm Version v6.14.15

{ "platform": "Cmd4", "name": "Cmd4", "outputConstants": false, "accessories": [ { "type": "Fan", "displayName": "Ventilatie", "on": "TRUE", "rotationDirection": "CLOCKWISE", "rotationSpeed": 100, "name": "Ventilatie", "manufacturer": "VENTRO", "model": "VENTRO-275", "polling": [ { "characteristic": "rotationSpeed", "interval": 10, "timeout": 9000 } ], "serialNumber": "2894745", "outputConstants": true, "stateChangeResponseTime": 3, "state_cmd": "/opt/ventilatie.sh" } }

— Reply to this email directly, view it on GitHub https://github.com/ztalbot2000/homebridge-cmd4/issues/123, or unsubscribe

https://github.com/notifications/unsubscribe-auth/ABSBCX3ANG4XCUMHHZJ6653UWZWGJANCNFSM5MJG4M2Q . You are receiving this because you were assigned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/ztalbot2000/homebridge-cmd4/issues/123#issuecomment-1034076824, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABT22SFWEFN6A4WDURT4BDTU2KYGZANCNFSM5MJG4M2Q. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you modified the open/close state.Message ID: @.***>