Page 16 sur 17
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 31 juil. 2024, 19:22
par Masure
Dr No a écrit : 30 juil. 2024, 16:11
...
Reste à modifier le dépôt Github maintenant, Masurov si tu nous lis ...
Flying Domotic a écrit : 31 juil. 2024, 02:33
Tant que j'y étais, j'ai crée un pull request pour ces 2 changements.
Je devrais regarder ce week end pour le merge. Merci Flying pour ta contribution.
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 31 juil. 2024, 23:10
par Masure
J'ai fait les modifs que tu suggérais sur une branche
https://github.com/Masurov/Domoticz-Mit ... patibility
Flying tu pourrais tester pour voir si c'est ok avant que je merge ? Je peux pas tester chez moi.
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 01 août 2024, 01:53
par Flying Domotic
Je vais avoir du mal à tester, je suis en "prod" en 2020.2, donc avec l'ancienne API, et je n'utilise pas l'envoi de température.
Par contre, j'ai décortiqué le code, et je pense que le choix de l'API est à l'envers : à partir de la 2023.2, on doit appeler "?type=command¶m=getdevices", et avant "?type=device" (jettes un œil sur le pull request que je t'ai envoyé, et sur le code un peu plus bas dans ce thread).
Demandes à Dr No, il devrait avoir la bonne conf (et il a déjà testé le code proposé ici).
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 01 août 2024, 09:26
par Dr No
Je suis OK bien sûr pour tester !
Je vais attendre le retour de Masure s'il y a un petit souci.
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 01 août 2024, 16:27
par Dr No
Après avoir poussé comme prévu le timeout à 3 (ligne 213), il me reste une seule erreur à 0heure, c'est l'instant où, me semble t-il, Domoticz fait un certain nombre de choses, en plus il a fait une sauvegarde database.
Je vais donc pousser timeout à 5, là je pense que je n'aurai plus de soucis.
Réponse demain ...
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 01 août 2024, 16:53
par Masure
Dr No a écrit : 01 août 2024, 09:26
Je suis OK bien sûr pour tester !
Je vais attendre le retour de Masure s'il y a un petit souci.
Je corrige et te dis quand c'est à jour
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 01 août 2024, 19:54
par Masure
Dr No, tu peux tester la version de la branche :
https://github.com/Masurov/Domoticz-Mit ... bility.zip
Je ferai le merge quand tu auras validé.
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 01 août 2024, 20:16
par Masure
J'ai aussi introduit un paramètre pour le timeout sur la requête http de récupération de la température.
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 02 août 2024, 12:16
par Dr No
J'ai tué le répertoire et dézippé ton archive.
Il semblerait qu'il y ait un petit souci :
Code : Tout sélectionner
2024-08-02 12:11:07.378 PAC_1: Worker thread started.
2024-08-02 12:11:07.178 Status: PAC_1: Stop directive received.
2024-08-02 12:11:07.307 Status: PAC_1: Exiting work loop.
2024-08-02 12:11:07.378 Status: PAC_1: Stopping threads.
2024-08-02 12:11:07.378 Status: PAC_1: Stopped.
2024-08-02 12:11:07.378 Status: PAC_1: Entering work loop.
2024-08-02 12:11:07.378 Status: PAC_1: Started.
2024-08-02 12:11:07.763 Status: PAC_1: Initialized version 1.3, author 'Masure'
2024-08-02 12:11:07.769 Error: PAC_1: Start error: '>=' not supported between instances of 'DomoticzVersion' and 'DomoticzVersion'
J'ai 2 clims la première tourne depuis plusieurs années et semble fonctionner sans souci avec cette version. Elle n'utilise pas de senseur externe. La seconde PAC_1 utilise une sonde externe et ne fonctionne pas.
Re: [Plugin] Climatiseur Mitsubishi (interface série et non MELCloud)
Publié : 02 août 2024, 15:33
par denis_brasseur
Dr No a écrit : 02 août 2024, 12:16
Il semblerait qu'il y ait un petit souci :
Code : Tout sélectionner
2024-08-02 12:11:07.769 Error: PAC_1: Start error: '>=' not supported between instances of 'DomoticzVersion' and 'DomoticzVersion'
Flying Domotic a écrit : 31 juil. 2024, 00:50
Code : Tout sélectionner
# HTTP API changed since 2023.2
domVersion = str(Parameters["DomoticzVersion"])
if (domVersion[:2] == "20" and domVersion >= "2023.2"):
Au vu de l'erreur, essaye peut etre :
Code : Tout sélectionner
if int(Parameters["DomoticzVersion"].split('.')[0]) >= 2023 and int(Parameters["DomoticzVersion"].split('.')[1]) >= 2: