vincentbesseau / gladys-netatmo Goto Github PK

Bonjour,

Je dispose chez moi d'une station météo et du thermostat Netatmo.

J'ai bien suivi la procédure d'installation mais lors du redémarrage de Gladys, après l'installation, j'ai l'erreur suivante qui fait planter Gladys et redémarrer.

Après redémarrage, le module est bien installé. Je trouve bien un device (mon thermostat uniquement) mais sans deviceType à l'intérieur. J'ai tenter les scripts proposés mais sans succès.

Peux-tu m'aider à résoudre ce plantage ?

0|gladys   | Début de la création du device Netatmo weather
0|gladys   | TypeError: Cannot read property 'temperature' of undefined
0|gladys   |     at createWeatherDevicesBase (/home/pi/gladys/api/hooks/netatmo/lib/install/createWeatherDevicesBase.js:32:51)
0|gladys   |     at /home/pi/gladys/api/hooks/netatmo/lib/install/netatmoSearch.js:31:4
0|gladys   |     at Array.forEach (<anonymous>)
0|gladys   |     at /home/pi/gladys/api/hooks/netatmo/lib/install/netatmoSearch.js:30:11
0|gladys   |     at netatmo.<anonymous> (/home/pi/gladys/api/hooks/netatmo/node_modules/netatmo/netatmo.js:227:14)
0|gladys   |     at Request.self.callback (/home/pi/gladys/api/hooks/netatmo/node_modules/request/request.js:185:22)
0|gladys   |     at emitTwo (events.js:126:13)
0|gladys   |     at Request.emit (events.js:214:7)
0|gladys   |     at Request.<anonymous> (/home/pi/gladys/api/hooks/netatmo/node_modules/request/request.js:1161:10)
0|gladys   |     at emitOne (events.js:116:13)
0|gladys   |     at Request.emit (events.js:211:7)
0|gladys   |     at IncomingMessage.<anonymous> (/home/pi/gladys/api/hooks/netatmo/node_modules/request/request.js:1083:12)
0|gladys   |     at Object.onceWrapper (events.js:313:30)
0|gladys   |     at emitNone (events.js:111:20)
0|gladys   |     at IncomingMessage.emit (events.js:208:7)
0|gladys   |     at endReadableNT (_stream_readable.js:1064:12)
0|gladys   |     at _combinedTickCallback (internal/process/next_tick.js:139:11)
0|gladys   |     at process._tickDomainCallback [as _tickCallback] (internal/process/next_tick.js:219:9)

Hello !

Je viens de regarder ton module Netatmo ! Top, joli boulot :)

Plusieurs remarques :

• Plutôt que de demander de faire un script pour que ton module fonctionne, ce qui est un peu tech, j'aimerais au possible éviter de demander de faire des scripts pour tout (bon je suis d'accord c'est pas le cas partout dans Gladys mais j'aimerais virer ça à l'avenir), pourrait tu faire un setInterval avec un défaut à 30 minute et éventuellement proposer à l'utilisateur de créer un param (facultatif) pour spécifier son temps de rafraichissement ? ( tu peux t'en affranchir dans un premier temps, juste le default peut suffire)
• La façon dont les boxs sont créées va un peu changer ( ça sera plus simple ), je viens de bosser tout le week-end dessus. Il y a aura une modification à faire dessus assez vite, je détaillerais comment faire à la sortie de la nouvelle version en bêta.
• Les phrases que tu créé je ne suis pas contre en soit, mais j'ai l'impression qu'il s'agit de features plus "natives" que lié à ton module Netatmo. Je comprends que en ayant besoin tu les ai créée pour ton module, mais je ne sais pas vraiment si ça fait sens par rapport au projet qu'elles soient ici. Genre "Je serais absent aujourd'hui", c'est 100% lié à la gestion de la présence... Je serais pour les retirer du module et réfléchir de façon globale comment on gère ça dans Gladys :) Genre passer par des events natifs Gladys. A réfléchir.

Voilà pour mon retour, dans tous les cas jolie boulot, il y a des supers possibilités avec ce module !

Ces remarques ne sont là que pour faire avancer le projet et que tout reste clean ;)