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.

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

Ajout UNMONITORED_DEVICES pour utilitisation dans hilo/sensor.py #170

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

maxyvon
Copy link
Contributor

@maxyvon maxyvon commented Jan 26, 2024

Désolé, j'aurais du l'ajouté hier en même temps que l'autre commit. Au début je pensais l'ajouter dans hilo/const.py, mais finalement puisque c'est une liste de devices je trouvais que ça faisait peut-être plus de sens de le mettre ici pour les regroupés. Je ne l'utiliserai pas dans python-hilo. Donc dites moi si vous trouvez que ça serait mieux dans hilo.
Merci!

@ic-dev21
Copy link
Collaborator

Pour pas qu'on essaye d'en faire des energysensors? J'ai un peu de mal à comprendre :)

@maxyvon
Copy link
Contributor Author

maxyvon commented Jan 26, 2024

Oui c'est ça. En fait, Hilo ne fait pas la différence avec les devices qui n'ont pas l'option de monitoring. Même dans l'app ils ne font pas la différence, ses devices affichent 0W au lieu de ne rien afficher. L'attribut existe quand même sur ses devices donc des sensors sont créé inutilement dans HA et ça crée des erreurs dans les logs. Ainsi, pour ses devices on ne créerait pas de sensor de power (donc ceux d'energy ne seraient aussi pas créés).

@ic-dev21
Copy link
Collaborator

Oui c'est ça. En fait, Hilo ne fait pas la différence avec les devices qui n'ont pas l'option de monitoring. Même dans l'app ils ne font pas la différence, ses devices affichent 0W au lieu de ne rien afficher. L'attribut existe quand même sur ses devices donc des sensors sont créé inutilement dans HA et ça crée des erreurs dans les logs. Ainsi, pour ses devices on ne créerait pas de sensor de power (donc ceux d'energy ne seraient aussi pas créés).

Je déteste pas l’optimisation. Es-tu sur le discord? Je gosse après les energy sensors depuis quelques jours à tâtons mais quelqu’un de plus expérimenté que moi serait pas de trop

@maxyvon
Copy link
Contributor Author

maxyvon commented Jan 26, 2024

Je t'ai écrit sur Discord!

@ic-dev21
Copy link
Collaborator

Donc ça prend un petit bout dans sensor.py pour aller avec ça, right? As-tu une PR de prête pour ça aussi? @valleedelisle des objections?

@maxyvon
Copy link
Contributor Author

maxyvon commented Jan 27, 2024

Je viens de la créé PR#364

@valleedelisle valleedelisle merged commit d7f938f into dvd-dev:main Jan 31, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants