search

fmartinou / tydom2mqtt

Deltadore Tydom to MQTT Bridge
https://fmartinou.github.io/tydom2mqtt/
MIT License
90 stars 53 forks source link

"Others" Equipment not found #108

Closed MateoGreil closed 1 year ago

MateoGreil commented 1 year ago

Hi,

firstly, that's an awesome integration in HA, and thanks for that !

My query is about the "Others" Equipements. In the Tydom app, I've added equipments in "Others" / "Autres" in french, and I can't find it in the HA list of my equipments. Is it known ? Is it wanted ?

Thanks for your reply ! And again, nice job !

Neo33ASM commented 1 year ago

J'ai regardé le code et ce n'est pas pris en charge en effet, c'est juste que :

J'ai également un équipement dans "autres", je suis en train de coder une proposition de mise à jour, il n'y aura plus qu'à espérer que @fmartinou la vérifie/intègre dès qu'il le pourra.

MateoGreil commented 1 year ago

Je comptais m'en occupé mais si tu as déjà commencé alors c'est parfait ! Aussi, ca ne me concerne pas parce que j'en ai pas, mais je pense qu'il manque un paquet d'autres choses. "Je pense" parce que je ne suis pas certain d'etre allé assez en profondeur dans le code, mais apres un rapide coup d'oeil, je ne vois que 6 sensors, alors que sur l'app tydom, j'en vois 13.

Neo33ASM commented 1 year ago

@MateoGreil je viens de push une merge request : https://github.com/fmartinou/tydom2mqtt/pull/109 (assez simple, la logique existe déjà pour ces devices)

Si @fmartinou peut y jeter un coup d'œil et donner son avis dès qu'il le pourra ça serait cool :)

J'ai des modules de volets roulants que j'ai catégorisés en "stores" qui ne sont donc pas pris en compte (simple renaming), mais pour les "others", à ce jour je n'ai que des modules on/off moi.

En majorité les modules acceptés dans cette section sont de ce genre, mais j'ai vu qu'il est possible d'y mettre un Tyxia 6410 dans cette catégorie. J'en ai un mais je n'ai pas encore fait le test.

Actuellement, seul le "lastUsage" est utilisé pour déterminer la catégorie du device, pour avoir une détermination plus fine, il faudra intégrer le parsing de la signature de celui-ci, voici un morceau de code de https://github.com/mgcrea/homebridge-tydom

image

Neo33ASM commented 1 year ago

D'ailleurs je lui avait fait intégrer (à mgcrea sur homebridge) les nouveau plugs https://www.deltadore.fr/domotique/pilotage-eclairage/eclairages-connectes/easy-plug-e16em-ref-6353005. On peut le voir dans la liste du screen ci-dessus. Mais il faudra visiblement créer une nouvelle catégorie.

MateoGreil commented 1 year ago

En attendant que @fmartinou valide, on n'a pas moyen de le faire au travers de ton repo ?

fmartinou commented 1 year ago

Salut,

Désolé pour le manque de réactivité; je n'ai pas beaucoup de dispos ces temps-ci :/ La MR est mergée ; le tag latest embarque le changement.

Vous avez la possibilité de tester ? Ca me permettrait d'être rassurer avant que je release et que je rende la version dispo sur HACS.

Neo33ASM commented 1 year ago

Ok j'ai créé un addon qui pointe vers latest (https://github.com/Neo33ASM/hassio-addons/) je teste et je te dis

Neo33ASM commented 1 year ago

@fmartinou pour les volets classés en "awnings" c'est ok ça fonctionne, mais pour others j'ai visiblement oublié le 's', c'est 'others' et non 'other', du coup je viens de pull une nouvelle merge request (#110)

fmartinou commented 1 year ago

Merci @Neo33ASM ,

C'est mergé ; je vais releaser pour que tout le monde puisse profiter de ces dernières features 👍

fmartinou commented 1 year ago

Released as 3.2.0 (set as addon latest version in HASS.io)

Neo33ASM commented 1 year ago

Je confirme, le renommage des "others" a corrigé le souci, je vois bien ma pompe à chaleur en "on/off" par exemple :) J'pense qu'au niveau des modifications ultra-simples on peut pas faire beaucoup mieux, je fais faire des tests pour voir s'il y a d'autres comportements existants dans "others" que celui que j'ai rajouté (code commenté)