search

bokub / ha-linky

📊 Home Assistant add-on for Linky smart meters
MIT License
145 stars 8 forks source link

Ajout d'un 2ieme compteur et remerciement #22

Closed laurent42454 closed 4 months ago

laurent42454 commented 4 months ago

Bonjour,

tout d'abord, un grand merci pour ce travail ! ca fait longtemps que je cherche une méthode fiable pour récupérer les datas ..

une question, est il possible / envisageable de gérer 2 compteurs différents (même Token / 2 PRM) ?

peut-être en éditant à la main le YAML

Bien à vous

Laurent

slapin95 commented 4 months ago

+1

aladin2000 commented 4 months ago

+1

bokub commented 4 months ago

Hello,

Pour mieux comprendre votre besoin, ce serait forcément un seul token pour plusieurs compteurs ?

laurent42454 commented 4 months ago

Bonjour Boris,

pas forcement, je pense que le mieux serait de pouvoir configurer des paires Token / PRM .. comme ça ca règle toutes les situations .. ceux qui ont un seul compte Enedis et ceux qui en ont plusieurs (un compte perso et un compte pro par exemple)

En tout cas, super boulot, bravo :)

Laurent

bokub commented 4 months ago

Comment tu verrais le format de la configuration pour que celle-ci soit à la fois:

laurent42454 commented 4 months ago

Pour la configuration YAML, je verrais un truc du genre

consumption:
    - PRM: "123123"
      Token: "sqjkshkjsqjkjksjkhsdfhjk"
      name: "Compteur 1"
      action: "sync"

    - PRM: "123124"
      Token: "sqjkshkjsqjkjksjkhsdfhjk"
      name: "Compteur 2"
      action: "sync"

  production:
    - PRM: "123123"
      Token: "sqjkshkjsqjkjksjkhsdfhjk"
      name: "station solaire 1"
      action: "sync"

    - PRM: "123124"
      Token: "sqjkshkjsqjkjksjkhsdfhjk"
      name: "station solaire 2"
      action: "sync"

Pour l'interface graphique, juste ajouter un autre bloc consommation et un autre bloc production.. Ca remplira de mon point de vue plus de 99% des usages .. pour les 1% restants, une petite édition du YAML fera l'affaire :)

bokub commented 4 months ago

Alors je vois 2 problèmes potentiels:

  1. L'interface de configuration est générée automatiquement par Home Assistant à partir de ce fichier. Si on lui dit que la config c'est plus juste des chaines de caractères mais un deux tableaux d'objets, à mon avis l'interface elle va sauter et il faudra se taper une config en YAML. Y'a beaucoup d'utilisateurs inexpérimentés donc pas ouf. N.B: Ça c'est une supposition de ma part, je me trompe peut-être, le mieux ce serait de tester pour voir ce que HA génère comme interface
  2. Si je fais une MAJ avec ce nouveau format, l'ancien format devient invalide, donc ça fait sauter la config de tous les utilisateurs qui avaient un add-on fonctionnel jusqu'à maintenant et qui n'ont rien demandé. T'es à l'aise avec l'idée de demander à 99% des gens de reconfigurer leur add-on pour faire plaisir aux 1% qui ont 2 compteurs ?
laurent42454 commented 4 months ago

Je t'avoue que je ne connais pas comment l'architecture de HA est faite donc je propose des trucs pas forcement pertinents. Notamment lorsque tu fais un upgrade, je sais pas s'il est facile de modifier la config YAML des utilisateurs existants.

Pour garder une compatibilité ascendante, peut être juste modifier le config comme ça ?

homeassistant_api: true
options:
  consumption PRM: ''
  consumption token: ''
  consumption name: 'Linky consumption'
  consumption action: sync
  production PRM: ''
  production token: ''
  production name: 'Linky production'
  production action: sync

  consumption PRM 2: ''
  consumption token 2: ''
  consumption name 2: 'Linky consumption'
  consumption action 2 : sync
  production PRM: ''
  production token: ''
  production name: 'Linky production'
  production action: sync
  production PRM 2: ''
  production token 2: ''
  production name 2: 'Linky production'
  production action 2 : sync

----------------------

Ca permet juste de rajouter un compteur et un producteur mais encore une fois, le besoin d'avoir plus de 2 compteurs doit être encore plus marginal

bokub commented 4 months ago

Pas possible de modifier la configuration des utilisateurs, c'est de la lecture seule malheureusement. La doc est ici si tu veux en savoir plus: https://developers.home-assistant.io/docs/add-ons/configuration

Du coup oui, avec ta proposition, ça pourrait marcher. C'est un peu dommage parce que ça va faire une interface de configuration un peu complexe pour gérer des cas à la marge mais c'est pas dramatique non plus.

Par contre le jour où je reçois une issue "ajout 3ième compteur" je fais quoi, j'en rajoute un ? Ou je decide que y'a une limite max de 2 ?

bokub commented 4 months ago

Ah je viens de voir cette section dans la doc qui pourrait être intéressante:

https://developers.home-assistant.io/docs/add-ons/configuration#add-on-advanced-options

J'ai pas beaucoup de temps en ce moment mais ça pourrait valoir le coup de creuser un peu un de ces jours

laurent42454 commented 4 months ago

Hello

merci pour les liens, je vais aller regarder ça.

Alors si j'étais toi, afin de répondre au max des besoins tout en ayant une interface de config pas trop lourde, je limiterais à 2 compteurs et une seule production. Je pense qu'avoir 2 sites de production est encore beaucoup plus rare que d'avoir 2 compteurs et ca peut être définit comme une limite dans la doc...

bokub commented 4 months ago

je limiterais à 2 compteurs et une seule production.

Oui forcément, tu es biaisé par ta propre situation ! Tu veux juste que ton cas soit supporté et après stop, on dit que ça sert à rien d'aller plus loin ça sera trop encombré

Moi je n'ai qu'un compteur en consommation donc au départ cet add-on ne gérait que la consommation, et ça m'allait très bien

Puis plein de gens m'ont dit "j'ai un compteur en consommation et un en production, rajoute la production s'il te plait !" Donc j'ai bossé gratos pour ajouter un compteur en production, alors que ça ne me sert strictement à rien ! Je ne gagne absolument rien à améliorer et à partager mon add-on, ça me coûte plus d'argent et beaucoup plus de temps que si je gardais tout pour moi !

Et comme tu me le suggères, j'ai défini une limite, cette limite c'est 1 compteur en production et 1 compteur en consommation.

Forcément, si y'a une limite, y'aura toujours des gens pour se plaindre que ça ne suffit pas. Si demain je change la limite pour accepter 2 compteurs en consommation (comme demandé ici), y'aura un autre utilisateur qui va demander de modifier cette limite. Mais pourquoi je céderai à ta demande et pas à la sienne ? En quoi elle est plus légitime ? Parce que ça concerne plus de cas et que c'est moins rare ? Peux-tu me sortir des chiffres et des sources fiables qui me permette de décider ce qui est assez courant ou trop rare ? Ou tu te bases juste sur les 2 lourdingues qui ont commenté "+1" pour faire tes statistiques ?

laurent42454 commented 4 months ago

Bien sur, je suis biaisé par ma propre situation et par les "+1" .. au niveau des stats 10% des résidences sont des résidences secondaires donc on va dire que 10% ont 2 compteurs ou plus. Pour la production, j'ai pas de stats

Ton plug-in est déjà super comme ça donc si tu as pas le temps / l'envie de le changer, pas de souci ... c'était juste une proposition

bokub commented 4 months ago

Oui, je te remercie pour la proposition, je la mets dans un coin de ma tête, mais toutes c'est questions c'était surtout pour te faire réaliser que ajouter la feature que tu demandes c'est pas juste quelques lignes de code qui font faire plaisir à quelques dizaines de personnes qui réclament la fonctionnalité, mais toute une réflexion à pousser autour de:

aladin2000 commented 4 months ago

On te remercie  déjà pour tout tout ton travail

Oui le cas où des personnes ont une résidence secondaire est intéressant car touchent sûrement  plus de cas que de producteurs Enfin j'en suis pas sùr ....

tirso58 commented 4 months ago

Bonjour, et merci pour ce qui a été fait jusquà présent, c'est très aprécié. Je confirme qu'il y a pas mal de gens qui ont 2 compteurs, tous n'utilisent pas Home Assistant mais il y en a de plus en plus. J'ai compris que celà n'est pas simple, neanmoins, le besoin persiste. Merci

jeandavid59 commented 4 months ago

Bonjour, Je viens d'installer l'application avec succès et j'en suis très satisfait. Par contre j'aimerais suivre la consommation de ma résidence secondaire, et là je bloque. J'espère que cette possibilité sera intégrée dans une nouvelle version. Merci

bokub commented 4 months ago

Hello, Tu tombes bien, je viens de publier une nouvelle version de HA Linky (la 1.3.0) avec un format de config complètement différent qui permet désormais d'ajouter autant de compteurs que voulu 🎉

La migration de la config sera faite automatiquement au premier démarrage de l'add-on

tirso58 commented 4 months ago

En voilà  une bonne nouvelle. J'essayerai  dès  que possible.  Merci

⁣Télécharger BlueMail pour Android ​

Le 22 févr. 2024, 21:19, à 21:19, Boris K @.***> a écrit:

Hello, Tu tombes bien, je viens de publier une nouvelle version de HA Linky (la 1.3.0) avec un format de config complètement différent qui permet désormais d'ajouter autant de compteurs que voulu 🎉

La migration de la config sera faite automatiquement au premier démarrage de l'add-on

-- Reply to this email directly or view it on GitHub: https://github.com/bokub/ha-linky/issues/22#issuecomment-1960237348 You are receiving this because you commented.

Message ID: @.***>

aladin2000 commented 4 months ago

J'ai configuré les deux prm Tout est ok merci Question : j'ai voulu changer le nom des 2 compteurs via l'extension (name:) mais c'est tjs les anciens noms qui subsistent sous le dashboard energie

Ya t il une solution ?

laurent42454 commented 4 months ago

Hello, Tu tombes bien, je viens de publier une nouvelle version de HA Linky (la 1.3.0) avec un format de config complètement différent qui permet désormais d'ajouter autant de compteurs que voulu 🎉

La migration de la config sera faite automatiquement au premier démarrage de l'add-on

Super !! un grand merci pour cet ajout !

bokub commented 4 months ago

Question : j'ai voulu changer le nom des 2 compteurs via l'extension (name:) mais c'est tjs les anciens noms qui subsistent sous le dashboard energie

@aladin2000 Il faut attendre qu'au moins une nouvelle statistique soit poussée, c'est à dire que si toutes tes données sont déjà à jour tu dois attendre la synchronisation du lendemain matin.

Peux-tu me dire si ce matin les noms ont bien changé ?

jeandavid59 commented 4 months ago

Hello, Tu tombes bien, je viens de publier une nouvelle version de HA Linky (la 1.3.0) avec un format de config complètement différent qui permet désormais d'ajouter autant de compteurs que voulu 🎉

La migration de la config sera faite automatiquement au premier démarrage de l'add-on

Bonjour, Merci beaucoup je viens d'installer la nouvelle version. Tout est OK . Bonne journée

aladin2000 commented 4 months ago

Parfait ce matin  et sur l'accès a energy il faut faire un refresh du browser C'était peut être ok hier... En tout cas merci tout fonctionne parfaitement

23 févr. 2024 09:49:18 Boris K @.***>:

Question : j'ai voulu changer le nom des 2 compteurs via l'extension (name:) mais c'est tjs les anciens noms qui subsistent sous le dashboard energie

@aladin2000[https://github.com/aladin2000] Il faut attendre qu'au moins une nouvelle statistique soit poussée, c'est à dire que si toutes tes données sont déjà à jour tu dois attendre la synchronisation du lendemain matin.

Peux-tu me dire si ce matin les noms ont bien changé ?

— Reply to this email directly, view it on GitHub[https://github.com/bokub/ha-linky/issues/22#issuecomment-1960938015], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD7K4I24RT32V72L6L7JZPLYVBJYZAVCNFSM6AAAAABDCUA5FSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRQHEZTQMBRGU]. You are receiving this because you were mentioned. [Image de pistage][https://github.com/notifications/beacon/AD7K4I5GOV6IU3CPIVHXPMDYVBJYZA5CNFSM6AAAAABDCUA5FSWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTU4GFB6.gif]

laurent42454 commented 4 months ago

Super, un grand merci !

tirso58 commented 4 months ago

Re grand merci.

⁣Télécharger BlueMail pour Android ​

Le 23 févr. 2024, 17:14, à 17:14, laurent42454 @.***> a écrit:

Super, un grand merci !

-- Reply to this email directly or view it on GitHub: https://github.com/bokub/ha-linky/issues/22#issuecomment-1961609580 You are receiving this because you commented.

Message ID: @.***>