Je possède une clim Mitsubishi et je voudrais pouvoir la controller depuis Gladys.
La petite PR qui va bien:
GladysAssistant:master
← callemand:melcloud
opened 12:32PM - 01 Aug 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to [the community](https://community.gladysassistant.com/) for testing before merging.
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
The goal of this PR is to add communication between MELCloud and Gladys to be able to control air conditioning provide by Mitsubishi
For the moment only air conditioning is available but we can add Air-To-Water and Energy-Recovery-Ventilation devices later.
<img width="363" alt="Screenshot 2023-08-01 at 14 30 51" src="https://github.com/GladysAssistant/Gladys/assets/11317212/67c95ad1-b201-4848-bb7b-34d894489fbc">
<img width="1244" alt="Screenshot 2023-08-01 at 14 31 19" src="https://github.com/GladysAssistant/Gladys/assets/11317212/0b22f2c8-6c5d-4fcc-b771-a50d6bda97ec">
<img width="404" alt="Screenshot 2023-08-01 at 14 31 59" src="https://github.com/GladysAssistant/Gladys/assets/11317212/4515e855-e328-4bce-84c5-ef12663b67fd">
<img width="1205" alt="Screenshot 2023-08-01 at 14 31 36" src="https://github.com/GladysAssistant/Gladys/assets/11317212/d07b1e8a-6618-4569-8952-f1c26d2cfc3a">
<img width="1240" alt="Screenshot 2023-08-01 at 14 31 32" src="https://github.com/GladysAssistant/Gladys/assets/11317212/998c78b5-f6fe-45f5-843e-db946bcb05ad">
1 « J'aime »
Pourrais-tu détailler de quelle clim tu parles ? Parce qu’il peut y avoir plusieurs systèmes différents (split / gainable) et des technologies de contrôle différentes.
Moi je parle de clim dans ce style là :
Lorsque tu rajoutes tes splits sur MELCloud, tu dois renseigner un tas d’info comme le numero de serie de tes splits et des unités exterieur.
Normalement si j’ai bien compris la « doc » de MELCloud, toutes leurs clims sont configurable du moment que tu peux installer le module wifi
1 « J'aime »
Bonjour.
Je possède également deux splits (unité interne) de chez Mitsubishi (clim réversible) utilisant MelCloud. Je suis donc aussi intéressé par l’intégration de MELCloud sur Gladys pour pouvoir piloter mes clims et créer des scénarios.
2 « J'aime »
@Lokkye tu as moyen de faire un build Docker de ta PR pour que @Wesley_K puisse tester chez lui en complément de chez toi ?
@pierre-gilles et @Wesley_K : Voici l’image docker qui va bien : delogzway/gladys:melcloud
1 « J'aime »
Hello @Lokkye .
Je te remercie beaucoup. Je vais tenter te tester ça dès que je peux. Je dois d’abord me trouver un mini PC pour y installer Gladys.
Petite question vu que tu possèdes les mêmes split que moi, je suppose que tu as installé un thermostat connecté style Netnano pour gérer des scénarios complet avec ta clim (du style, si la température de la pièce est >= à 28 alors tu lances à la clim à 26.
Merci encore pour le travail effectué sur l’intégration de MelCloud.
Lokkye
Septembre 4, 2023, 6:23
9
Hello @Wesley_K
Wesley_K:
Petite question vu que tu possèdes les mêmes split que moi, je suppose que tu as installé un thermostat connecté style Netnano pour gérer des scénarios complet avec ta clim (du style, si la température de la pièce est >= à 28 alors tu lances à la clim à 26.
Non je n’ai pas de thermostat connecté. C’est bien pour cela que je veux rajouter mes clims dans Gladys. Je pourrais faire des scénarios en fonction de la temperature de la piece et de la temperature de dehors (enfin c’est le projet )
N’hésite pas si tu as des retours, je n’ai pu faire les tests que sur la clim que je possède
1 « J'aime »
Avant d’avoir le retour de @Wesley_K , je t’ai fais une petite review technique @Lokkye
J’ai rien trouvé de majeur, j’ai juste remarqué qu’il y avait beaucoup de variables qui faisaient référence à l’intégration Tuya (j’imagine que tu t’es basé sur l’intégration Tuya), fais peut-être une passe sur le code avec un petit search pour remplacer toute mention de l’intégration Tuya?
Ma review:
GladysAssistant:master
← callemand:melcloud
opened 12:32PM - 01 Aug 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to [the community](https://community.gladysassistant.com/) for testing before merging.
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
The goal of this PR is to add communication between MELCloud and Gladys to be able to control air conditioning provide by Mitsubishi
For the moment only air conditioning is available but we can add Air-To-Water and Energy-Recovery-Ventilation devices later.
<img width="363" alt="Screenshot 2023-08-01 at 14 30 51" src="https://github.com/GladysAssistant/Gladys/assets/11317212/67c95ad1-b201-4848-bb7b-34d894489fbc">
<img width="1244" alt="Screenshot 2023-08-01 at 14 31 19" src="https://github.com/GladysAssistant/Gladys/assets/11317212/0b22f2c8-6c5d-4fcc-b771-a50d6bda97ec">
<img width="404" alt="Screenshot 2023-08-01 at 14 31 59" src="https://github.com/GladysAssistant/Gladys/assets/11317212/4515e855-e328-4bce-84c5-ef12663b67fd">
<img width="1205" alt="Screenshot 2023-08-01 at 14 31 36" src="https://github.com/GladysAssistant/Gladys/assets/11317212/d07b1e8a-6618-4569-8952-f1c26d2cfc3a">
<img width="1240" alt="Screenshot 2023-08-01 at 14 31 32" src="https://github.com/GladysAssistant/Gladys/assets/11317212/998c78b5-f6fe-45f5-843e-db946bcb05ad">
Lokkye
Septembre 11, 2023, 6:13
11
@pierre-gilles : Je viens de fixer les retours sur la PR (Oui j’ai basé mon code sur Tuya pour avoir la meme structure entre les integrations)
L’image docker est en cours de création
1 « J'aime »
@Wesley_K Comptes-tu tester l’intégration bientôt ou pas ? Pas de problème si non, dis le juste comme ça on peut s’organiser de notre côté !
@Lokkye J’ai fais une review technique de la PR, tout est bon sauf 2 console.log qui trainent, mais c’est vraiment du détail MELCloud integration (Mitsubishi Electric devices) by callemand · Pull Request #1839 · GladysAssistant/Gladys · GitHub
Comme j’ai pas l’impression que @Wesley_K est sur le forum, dès que la review est fixée, je mergerais la PR, on va pas attendre des siècles non plus
1 « J'aime »
Lokkye
Septembre 28, 2023, 8:59
14
@pierre-gilles : Merci pour la review.
Je viens de supprimer les console.log
1 « J'aime »
Excellent, dès que les tests passent, je merge ! Merci pour la réactivité !
1 « J'aime »
C’est mergé, et ça partira dans la prochaine version de Gladys !
2 « J'aime »
Fonctionnalité live dans Gladys Assistant 4.29 :
Salut à tous !
Nouvelle version de Gladys aujourd’hui avec Gladys Assistant 4.29, une version qui apporte pas mal de nouveautés
Je vous en parle sur le blog :
N’hésitez pas à féliciter @Lokkye pour toute ces chouettes contributions récentes