jeedom / plugin-openzwave

GNU General Public License v2.0
31 stars 88 forks source link

Fix support for NeoCoolcam PD01ZT #157

Closed spocky closed 4 years ago

spocky commented 5 years ago

Documentation was wrong, due to shifted parameters.

spocky commented 5 years ago

Pour être plus clair (c'est moi qui avais fait la PR pour ajouter ce device il y a quelques mois), la doc fournie était partiellement fausse. Du coup, après recherche, voilà qui devrait corriger les problèmes rencontrés (notamment la led qui ne pouvait se désactiver et le report des températures dès une variation de 0.1°, qui menait donc à un spam et un déchargement plus rapide des piles).

thierryzoller commented 5 years ago

Merci - je viens juste d'ajouter ce PIR

spocky commented 5 years ago

@sarakha63 désolé de te pinger, mais tu avais manuellement mergé ma précédente PR #139 concernant ce device. Apres quelques temps d'utilisation, j'ai constaté qu'il y avait des soucis à cause de la doc fournie qui n'était pas bonne (dont un souci de grosse conso de piles). Apres quelques recherches, tout est fixé avec cette PR. Peut-on espérer un merge (même manuel) ?

spocky commented 5 years ago

Youhouuuuuu ? Y'a quelqu'un ? Si quelque chose ne va pas, n'hésitez pas à la dire, mais là ça va faire 3 mois et toujours aucune nouvelle, je comprends que vous ayez d'autres chats à fouetter, mais malheureusement ça casse un peu l'envie de faire des PR :(

spocky commented 5 years ago

Test micro... 1, 2, 3. 1, 2... 1, 2... A tout hasard, ping @zoic21 ?

thierryzoller commented 5 years ago

+1

jim005 commented 4 years ago

+1

duylong commented 4 years ago

Je viens de voir ce commit après avoir cherché pourquoi ce modèle ne prenait pas en compte la désactivation du clignotement. C'est dommage pour le manque de considération et d'avoir un profil erroné. Il faut ré-inclure le module pour la prise en compte du nouveau profil ?

thierryzoller commented 4 years ago

Est-ce qu'il y a une raison pq ce commit n'est pas mergé ? Si oui la quelle ? On aimerai aider a le résoudre...

Horus92fr commented 4 years ago

Bonjour Spocky En cherchant à insérer des PIR V2 dans Jeedom, je suis arrivé ici.
Et dans un premier temps, je te remercie pour ton travail. A priori tu es le seul a t'être vraiment penché sur le problème de ces nouveaux PIR. J'ai donc récupéré tes fichiers corrigés et je les ai inséré dans Jeedom. La désactivation/réactivation du module Zwave n'a pas suffit pour que les paramètres soient pris correctement en compte, d'ou la première version de ce post, par contre après un reboot complet de Jeedom, tous les nouveaux paramètres sont bien remontés dans le nouveau PIR lors de sa ré-insertion. (PI le rechargement de la configuration comme on peut le lire sur d'autre forum ne suffit pas). Je vais maintenant voir comment se comporte ce nouveau PIR et faire un retour. Cordialement

Cordialement