-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
manage garage as a cover instead of switch (without koleos docs) (#197) #199
base: master
Are you sure you want to change the base?
Conversation
fmartinou
commented
Jun 12, 2024
- Addition of a new sensor class to manage garage to manage garage as a cover
- Evolution to manage garage as cover and add garage_door_horizontal that was not managed before.
- Update MessageHandler.py
- Managemnent two sensors sun and thermoPack Tysense Outdoor
- Add management of automatic doors
- Sensor name used in MQTT instead of Mosquito
* Addition of a new sensor class to manage garage to manage garage as a cover * Evolution to manage garage as cover and add garage_door_horizontal that was not managed before. * Update MessageHandler.py * Managemnent two sensors sun and thermoPack Tysense Outdoor * Add management of automatic doors * Sensor name used in MQTT instead of Mosquito --------- Co-authored-by: koleos6 <[email protected]>
@fmartinou testé sur mon instance de home assistant ✅ |
Bonjour à tous, Pour ma part je viens de tester l'image docker @fmartinou, cette PR pourra-t-elle être mergée prochainement et releasée dans la version officielle 😇 ? |
Hello |
Sur l'instance que j'utilise avec cette version custom de tydom2mqtt, je n'ai que mes 2 portes de garage - car c'est uniquement cela que je voulais tester et c'était voulu. Pour info, je suis tout nouveau dans le monde HomeAssistant, et je tourne actuellement sous Homebridge avec https://github.com/mgcrea/homebridge-tydom, qui fonctionne très bien mais qui souffre du même soucis que l'objet de cette PR à savoir la non disponibilité de mes portes de garage. Pour la petite histoire, il se trouve que j'ai 2 box Tydom:
Vu le nombre de scénarios et réglages que j'ai fais dans Apple Homekit, je n'ai pas trop envie de risquer de tout casser en passant tydom2mqtt sur ma Tydom 2.0. Je ne suis pas certains que 2 clients vers la Tydom 2.0 peuvent fonctionner en parallèle (homebridge-tydom + tydom2mqtt pointant sur la même IP). Bref, je réfléchis comment je peux vous aider sans tout péter 😋 |
Thanks @aure-olivier pour la clarification et le soutien.
Je ne suis même pas nouveau, je n'utilise que Jeedom 😁
Quasi sur que non et j'ai même lu, à une autre époque, d'un expert DD sur le community Jeedom, qu'il vaut mieux ne pas trop sollicité les Tydom 2 qui gardent (priorise ?) l'exclusivité de la connexion pour l'application. Mais j'ai la même tendance à n'avoir que des devices sur l'app Tydom et tout scénariser sur Jeedom. Une petite fenêtre se profile ce we (depuis des mois, entre refonte de mon infra, travaux, vacances,...). |
Hello, J'ai quand même pensé à un truc que j'ai testé, c'est à dire associé l'un de mes volets à 2 box Tydom en même temps, et ça fonctionne bien. Je ne vois pas de soucis sur la remontée d'état ou la commande de volet depuis Apple Home que ce soit en passant par l'accessoire Tydom2MQTT ou Homebridge-Tydom. Voici ce que j'ai:
|
Hello |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello
I think you might not call light class functions with cover_id and this is why pub on lights stuk.
app/mqtt/MqttClient.py
Outdated
@@ -162,7 +187,7 @@ async def on_message(self, client, topic, payload, qos, properties): | |||
get_id = (topic.split("/"))[2] | |||
device_id = (get_id.split("_"))[0] | |||
endpoint_id = (get_id.split("_"))[1] | |||
await Light.put_level_cmd(tydom_client=self.tydom, device_id=device_id, light_id=endpoint_id, | |||
await Light.put_level_cmd(tydom_client=self.tydom, device_id=device_id, cover_id=endpoint_id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this part is causing side effect for lights and should not be change.
app/mqtt/MqttClient.py
Outdated
@@ -174,8 +199,18 @@ async def on_message(self, client, topic, payload, qos, properties): | |||
get_id = (topic.split("/"))[2] | |||
device_id = (get_id.split("_"))[0] | |||
endpoint_id = (get_id.split("_"))[1] | |||
await Light.put_level(tydom_client=self.tydom, device_id=device_id, light_id=endpoint_id, | |||
await Light.put_level(tydom_client=self.tydom, device_id=device_id, cover_id=endpoint_id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one too
Hello, @BlacKTM1190, je confirme totalement ce que tu disais plus haut.
En effet "cover_id" est utilisé à tords j'ai l'impression. Par contre la remontée du status (on/off/level) est OK dans HomeKit, suite à une commande depuis l'application Tydom. |
Hello Je ne sais pas comment on fait pour publier une image sur le hub docker, j'ai donc procédé comme ceci : Sans Jeedom ou HA cablé sur ce broker de test, j'ai passé tous les pub par MQTT Explorer (sensible à la casse 🫤) : Ce qui n'est pas très "propre", et là ou j'ai galéré, c'est qu'il y a un Au passage, il y a du y avoir une coquille sur la dernière release de la master car il n'y a plus de readme... |
Hello, Je viens de connecter mon Novoferm 423 via Tydom et je tombe sur le problème de la porte de garage pas reconnue (just un light switch). Quelqu'un sait comment je peux installer l'addon dans une version spécifique sur Home Assistant ? Ou si je peux aller modifier des fichiers quelque part a la mano ? Thanks, D. |
Hello
|
@BlacKTM1190 j'ai commit tes retours sur cette branche. |
Si vous souhaitez tester la correction, une image docker est dispo ici avec tous les fix : Pour ceux qui utilisent hassio : |
Merci @romainboucher 👏 ✌️ |
J'ai testé et ca fonctionne sur HASSIO pour la commande la porte ✔️ La commande de la lumière ne fonctionne pas par contre. D. |
Hello |