search

Nico0084 / domogik-plugin-ozwave

Domogik ozwave plugin
GNU General Public License v3.0
1 stars 0 forks source link

everspring-mini-detecteur-d-ouverture-z-wave-hsm02 #7

Open tikismoke opened 8 years ago

tikismoke commented 8 years ago

En cours de négociation pour l'achat.

Je pourrais le tester.

tikismoke commented 8 years ago

Lors de la création du device: HSM02 j'ai droit à une erreur 404.... Si je crée un device opening.sensor.security.battery par contre c'est bon. Le device de base openclose fontionne bien. Par contre le tamper event ne fonctionne pas si j'ouvre le couvercle.

tikismoke commented 8 years ago

De plus rest retourne la valeur true ou false au lieux de 0 1 et null' en int. Du coup domodroid ne voit pas les changement d'État

Nico0084 commented 8 years ago

Création du device depuis le product HSM02 -> corrigé (problème de char "/") Gros soucis sur le tamper-event la lib openzwave semble ne pas lié le bon champs avec la bonne valeur, elles devraient être :

Hors le 17 renvoi "Low Battery Report" !

L'instance (2) de la value Tamper event n'est pas dans le message zwave et la lib ozw n'est pas capable de le retrouvé. L'index pour retrouver la value devrait être le level (_data[2]) et non le type (_data[1]) toujours à 1 Le fichier de config ozw hms02.xml comporte également des incohérences !

Bref,a mon avis il faut corriger cette issue dans openzwave !

Log détail : 2015-11-24 22:46:37.809 Detail, Node011, Received: 0x01, 0x0a, 0x00, 0x04, 0x00, 0x0b, 0x04, 0x71, 0x05, 0x01, 0x11, 0x9a 2015-11-24 22:46:37.809 Detail, 2015-11-24 22:46:37.809 Info, Node011, Received Alarm report: type=1, level=17 2015-11-24 22:46:37.809 Detail, Node011, Refreshed Value: old value=1, new value=1, type=byte 2015-11-24 22:46:37.809 Detail, Node011, Changes to this value are not verified 2015-11-24 22:46:37.809 Detail, Node011, Refreshed Value: old value=17, new value=17, type=byte 2015-11-24 22:46:37.809 Detail, Node011, Changes to this value are not verified 2015-11-24 22:46:37.809 Detail, Node011, Notification: ValueChanged 2015-11-24 22:46:00,857 domogik-ozwave DEBUG Publishing over MMQ , data : {'NetworkID': u'homeid', 'Node count': 5, 'Node sleeping': 2, 'data': {'index': 0, 'units': '', 'type': 'Byte', 'nodeId': 11, 'value': 1, 'commandClass': 'COMMAND_CLASS_ALARM', 'instance': 1, 'readOnly': True, 'homeId': 25485663, 'label': 'Power Applied', 'genre': 'User', 'id': '72057594228523009'}, u'NodeID': 11, 'Node fail': 1, 'usermsg': 'Value has changed.', 'type': 'value-changed'} 2015-11-24 22:46:00,857 domogik-ozwave DEBUG * valueToxPLTrig : {'device': {'networkid': u'homeid', 'node': 11, 'instance': 1}, 'typexpl': 'xpl-trig', 'data': {'current': 1, 'type': 'power-applied'}, 'schema': 'alarm.basic'} 2015-11-24 22:46:37.814 Detail, Node011, Notification: ValueChanged 2015-11-24 22:46:00,859 domogik-ozwave DEBUG Publishing over MMQ , data : {'NetworkID': u'homeid', 'Node count': 5, 'Node sleeping': 2, 'data': {'index': 1, 'units': '', 'type': 'Byte', 'nodeId': 11, 'value': 17, 'commandClass': 'COMMAND_CLASS_ALARM', 'instance': 1, 'readOnly': True, 'homeId': 25485663, 'label': 'low battery', 'genre': 'User', 'id': '72057594228523025'}, u'NodeID': 11, 'Node fail': 1, 'usermsg': 'Value has changed.', 'type': 'value-changed'} 2015-11-24 22:46:00,860 domogik-ozwave DEBUG * valueToxPLTrig : {'device': {'networkid': u'homeid', 'node': 11, 'instance': 1}, 'typexpl': 'xpl-trig', 'data': {'current': 17, 'type': 'low-battery'}, 'schema': 'alarm.basic'}

tikismoke commented 8 years ago

Vue que je comprends rien du charabia zwave, je te laisse le soins de faire le ticket sur openzwave.

En même temps faut faire gaffe je crois que la pile est quasi morte dans ce détecteur à voir quand on se croise pour des tests en live.

tikismoke commented 8 years ago

Piles remplacées et toutes neuves. Capteurs en places (enfin à la patafix). Plus de soucis d'inversion mais je ne sais si je suis à jour par rapport au repo.

Nico0084 commented 8 years ago

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

fritz-smh commented 8 years ago

La liste des alarmes possible est normalisée en zwave ou c'est open bar et chaque constructeur fait les siennes ?

2016-02-25 8:02 GMT+01:00 Nico0084 notifications@github.com:

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188642531 .

Nico0084 commented 8 years ago

Version 1 - open bar Version 2 - normalisé Le 25 févr. 2016 08:49, "Fritz SMH" notifications@github.com a écrit :

La liste des alarmes possible est normalisée en zwave ou c'est open bar et chaque constructeur fait les siennes ?

2016-02-25 8:02 GMT+01:00 Nico0084 notifications@github.com:

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

— Reply to this email directly or view it on GitHub < https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188642531

.

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188657871 .

fritz-smh commented 8 years ago

v2 = zwave+ ?

2016-02-25 9:04 GMT+01:00 Nico0084 notifications@github.com:

Version 1 - open bar Version 2 - normalisé Le 25 févr. 2016 08:49, "Fritz SMH" notifications@github.com a écrit :

La liste des alarmes possible est normalisée en zwave ou c'est open bar et chaque constructeur fait les siennes ?

2016-02-25 8:02 GMT+01:00 Nico0084 notifications@github.com:

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

— Reply to this email directly or view it on GitHub <

https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188642531

.

— Reply to this email directly or view it on GitHub < https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188657871

.

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188663317 .

Nico0084 commented 8 years ago

Non, il peut y avoir plusieurs version de command class ( protocole ) Le zwave+ ou v2 c'est la version de la puce électronique Le 25 févr. 2016 09:16, "Fritz SMH" notifications@github.com a écrit :

v2 = zwave+ ?

2016-02-25 9:04 GMT+01:00 Nico0084 notifications@github.com:

Version 1 - open bar Version 2 - normalisé Le 25 févr. 2016 08:49, "Fritz SMH" notifications@github.com a écrit :

La liste des alarmes possible est normalisée en zwave ou c'est open bar et chaque constructeur fait les siennes ?

2016-02-25 8:02 GMT+01:00 Nico0084 notifications@github.com:

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

— Reply to this email directly or view it on GitHub <

https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188642531

.

— Reply to this email directly or view it on GitHub <

https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188657871

.

— Reply to this email directly or view it on GitHub < https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188663317

.

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188665646 .

fritz-smh commented 8 years ago

ah zut....

c'est quoi la proportion de v1 vs v2 à peu près ?

2016-02-25 9:20 GMT+01:00 Nico0084 notifications@github.com:

Non, il peut y avoir plusieurs version de command class ( protocole ) Le zwave+ ou v2 c'est la version de la puce électronique Le 25 févr. 2016 09:16, "Fritz SMH" notifications@github.com a écrit :

v2 = zwave+ ?

2016-02-25 9:04 GMT+01:00 Nico0084 notifications@github.com:

Version 1 - open bar Version 2 - normalisé Le 25 févr. 2016 08:49, "Fritz SMH" notifications@github.com a écrit :

La liste des alarmes possible est normalisée en zwave ou c'est open bar et chaque constructeur fait les siennes ?

2016-02-25 8:02 GMT+01:00 Nico0084 notifications@github.com:

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

— Reply to this email directly or view it on GitHub <

https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188642531

.

— Reply to this email directly or view it on GitHub <

https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188657871

.

— Reply to this email directly or view it on GitHub <

https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188663317

.

— Reply to this email directly or view it on GitHub < https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188665646

.

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188666238 .

Nico0084 commented 8 years ago

Alors la ! bonne question ! no sais au final combien de produit utilisent la version 1 Pour l'instant dans les config xml d'openzwave il ni a que ce capteur HSM002 qui utilise un paramétrage pour plusieurs valeurs d'alarm qui correspondent à plusieurs type d'alarm.

Mais ce n'est a priori pas un problème si pour la version 1 on utilise des DT_String

De plus la version 2 est normalisé sur les type d'alarm mais pas sur les valeurs des alarms, Pour l'intégrer correctement, il faut soit avoir une doc détaillé soit relever les valeurs directement envoyée par le module !

fritz-smh commented 8 years ago

sinon on aurait ou envisager un DT_ZwaveAlarms qui serait une liste des possibles

2016-02-25 9:33 GMT+01:00 Nico0084 notifications@github.com:

Alors la ! bonne question ! no sais au final combien de produit utilisent la version 1 Pour l'instant dans les config xml d'openzwave il ni a que ce capteur HSM002 qui utilise un paramétrage pour plusieurs valeurs d'alarm qui correspondent à plusieurs type d'alarm.

Mais ce n'est a priori pas un problème si pour la version 1 on utilise des DT_String

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188668850 .

Nico0084 commented 8 years ago

Possible, c'est juste une transcription à faire dans le plugin. Mais le DT_ZwaveAlarms doit pouvoir être mis à jours pour suivre de nouvelles alarmes, possible ça ?

fritz-smh commented 8 years ago

ça peut être mis à jour dans une nouvelle version de domogik seulement.

2016-02-25 9:38 GMT+01:00 Nico0084 notifications@github.com:

Possible, c'est juste une transcription à faire dans le plugin. Mais le DT_ZwaveAlarms doit pouvoir être mis à jours pour suivre de nouvelles alarmes, possible ça ?

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188670042 .

Nico0084 commented 8 years ago

Ouaifff bof,

Alors si vous êtes ok, je resterais plustôt sur : 

"COMMAND_CLASS_ALARM" : {
        "General" : {
            "0" : 0,
            "1" : 1,
            "254" : 0
        },
        "Smoke" : {
            "0" : 0,
            "1" : 1,
            "254" : 0
        },"Tamper Event" : {
            "0" : 0,
            "17" : 1
        },
        "low battery":{
            "0" : 0,
            "255" : 1
        }, 
........

https://github.com/Nico0084/domogik-plugin-ozwave/blob/develop/lib/cmd_class_conversion.json

Et y'a juste à rajouter les bons labels et bonnes valeurs renvoyées par le module, le tout sans avoir à relancer le plugin :)

fritz-smh commented 8 years ago

c'est toi qui voit, je manque de recul :)

2016-02-25 10:05 GMT+01:00 Nico0084 notifications@github.com:

Ouaifff bof,

Alors si vous êtes ok, je resterais plustôt sur :

  • v1 : un DT_String avec la valeur correspondant au label openzwave (en lower case)
  • v2 : un DT_Boolean (voir autre "DT_Alarm" mais toujours du bool) avec un fichier json qui convertie les valeurs openzwave vers le DT_Boolean: Example:

"COMMAND_CLASS_ALARM" : { "General" : { "0" : 0, "1" : 1, "254" : 0 }, "Smoke" : { "0" : 0, "1" : 1, "254" : 0 },"Tamper Event" : { "0" : 0, "17" : 1 }, "low battery":{ "0" : 0, "255" : 1 }, ........

https://github.com/Nico0084/domogik-plugin-ozwave/blob/develop/lib/cmd_class_conversion.json

Et y'a juste à rajouter les bons labels et bonnes valeurs renvoyées par le module, le tout sans avoir à relancer le plugin :)

— Reply to this email directly or view it on GitHub https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188677165 .

tikismoke commented 8 years ago

Salut Nico. Le top serait d'avoir un dt_type plus libre genre liste déroulante dans Domogik.

Sur le principe ça me parait pas mal. Juste à trouver comment dire ou montrer à l'utilisateur final l'ensemble des valeurs disponible dans le string alarme.

Après clairement ça semble ce qu'il y a de plus simple et plus facilement "genericable" et gerable à chaque device.

Le 25 février 2016 08:02:44 GMT+01:00, Nico0084 notifications@github.com a écrit :

Salut tiki Pour gérer la command class alarm correctement il faut modifier le fichier de configuration hsm02.XML. Je vais donc créer une issues pour openzwave.

Je voulais ton avis concernant le model que ça va généré pour domogik. Il y aura un seul sensor pour toute les alarms "Alarm report" celui-ci d'un type DT_String, retournerait le type de la dernière Alarm reçue (power applied ou low battery ou tamper event, voir no alarm à l'init) Pour géré cette Alarm dans un scénario il faudra donc prendre en compte une chaîne de caractères. Qu'en pense tu ? Et faut il encore que la team openzwave valide le fichier XML. @+

Reply to this email directly or view it on GitHub: https://github.com/Nico0084/domogik-plugin-ozwave/issues/7#issuecomment-188642531

Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.

tikismoke commented 8 years ago

Zwave Details :

ozwave.alarm-str-report_battery-level_binary-sensor

{"commands":[],"description":"Mini Door/Window Detector","id":"ozwave.alarm-str-report_battery-level_binary-sensor","name":" Chromagic HSM02","parameters":[{"description":"Zwave network name if refered in controller node or Openzwave Home ID","key":"networkid","type":"string","xpl":false},{"description":"Zwave node id","key":"node","max_value":255,"min_value":1,"type":"integer","xpl":false},{"description":"Zwave node instance id","key":"instance","max_value":255,"min_value":1,"type":"integer","xpl":false},{"description":"Check battery level at zwave device wakeup.","key":"batterycheck","type":"boolean","xpl":false}],"sensors":["alarm-str-report","battery-level","binary-sensor"]}