Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Ajout d'un 2ieme compteur et remerciement #22

Closed
laurent42454 opened this issue Feb 10, 2024 · 26 comments
Closed

Ajout d'un 2ieme compteur et remerciement #22

laurent42454 opened this issue Feb 10, 2024 · 26 comments

Comments

@laurent42454
Copy link

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

This comment was marked as spam.

Sign in to view
1 similar comment
@aladin2000

This comment was marked as spam.

Sign in to view
@bokub bokub pinned this issue Feb 11, 2024
@bokub
Copy link
Owner

bokub commented Feb 12, 2024

Hello,

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

@laurent42454
Copy link
Author

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
Copy link
Owner

bokub commented Feb 12, 2024

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

  • très simple pour l'immense majorité des utilisateurs (un seul PRM)
  • flexible pour permettre aux utilisateurs comme toi de configurer plusieurs couples PRM/token en consommation et en production ?

@laurent42454
Copy link
Author

laurent42454 commented Feb 12, 2024 •
edited
Loading

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
Copy link
Owner

bokub commented Feb 12, 2024 •
edited
Loading

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
Copy link
Author

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
Copy link
Owner

bokub commented Feb 12, 2024

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
Copy link
Owner

bokub commented Feb 12, 2024

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
Copy link
Author

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
Copy link
Owner

bokub commented Feb 12, 2024

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
Copy link
Author

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
Copy link
Owner

bokub commented Feb 12, 2024 •
edited
Loading

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:

  • Trouver le bon équilibre entre configuration facile et nombre de fonctionnalités (vu les demandes d'aide que je reçois tous les jours j'ai l'impression que c'est déjà trop complexe comme ça)
  • S'efforcer de ne pas impacter les utilisateurs qui ont un add-on fonctionnel et qui vont subir une mise à jour silencieuse
  • Déterminer quelles features méritent que je prenne sur mon temps perso et lesquelles il vaut mieux ignorer

@aladin2000
Copy link

aladin2000 commented Feb 12, 2024 via email •
edited by bokub
Loading

@tirso58
Copy link

tirso58 commented Feb 14, 2024

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
Copy link

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
Copy link
Owner

bokub commented Feb 22, 2024

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
Copy link

tirso58 commented Feb 22, 2024 via email

@aladin2000
Copy link

aladin2000 commented Feb 22, 2024 •
edited
Loading

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
Copy link
Author

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
Copy link
Owner

bokub commented Feb 23, 2024

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
Copy link

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
Copy link

aladin2000 commented Feb 23, 2024 via email

@laurent42454
Copy link
Author

Super, un grand merci !

@tirso58
Copy link

tirso58 commented Feb 25, 2024 via email

@bokub bokub closed this as completed Feb 27, 2024
@bokub bokub unpinned this issue Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@aladin2000 @bokub @slapin95 @laurent42454 @tirso58 @jeandavid59