sasu-drooz / Domoticz-Rfplayer

Domoticz python Plugin for Ziblue RfPlayer Rfp1000
GNU General Public License v3.0
12 stars 16 forks source link

Mauvais subtype pour sonde Oregon #57

Closed urluba closed 6 years ago

urluba commented 6 years ago

Bonjour,

Tout d'abord merci beaucoup pour le plugin !

Je dispose de sondes Oregon THGR228N. Elles semblent parfaitement marcher dans Domoticz mais il n'est pas possible d'exporter leurs données dans Influx (ou autre base). La log affiche l'erreur "Could not determine data push value" qui provient de BasePush.ccp#L687 lorsque le subtype n'est pas connu.

Domoticz affiche d'ailleurs un 'sous-type' à 'Unknown' pour ces dispositifs et dans domoticz.db, la sonde existe effectivement avec un subtype à 0:

sqlite> select id, name, type, subtype from devicestatus;
[...]
32|Temp-Mobile|80|0
33|Hygro-Mobile|81|0
34|Temp/Hygro-Mobile|82|0

En assignant la bonne valeur au device 34 (6 dans mon cas), tout rentre dans l'ordre :)

Est-ce que le subtype == 0 est positionné par le plugin et si oui, est-il possible de positionner le bon subtype 'generique' (voir un mauvais :) ) ?

Merci

zaraki673 commented 6 years ago

Bonjour,

alors si je ne me trompe ca doit etre de l'infotype4, dc modifier ici : https://github.com/sasu-drooz/Domoticz-Rfplayer/blob/48fa58433197a9ab03710dbf655cc49a18367a2e/plugin.py#L995

(rajouter le Subtype voulu)

zzdomi commented 6 years ago

Le subtype renvoyé le le rfplayer est toujours à 0 pour les sondes oregon. Il faut tester le type pour affecter un subtype correct. Je m'en occupe.

zzdomi commented 6 years ago

Merci d'essayer la version de la branche issue#57 du fork zzdomi/Domoticz-Rfplayer avant merge avec la branche officielle.

urluba commented 6 years ago

Yop,

c'est bon pour moi !

  1. Sonde supprimée + recrée

  2. Sonde vue ok: sonde corrigée Curiosité le Lacrosse TX3 pour l'humidité.

  3. Export ok: sonde corrigée

zzdomi commented 6 years ago

J'ai mis 1 dans le subtype des devices dérivés. On peut mettre une autre valeur et avoir "unknown", c'est également possible.