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

fix last day power sensor class #10

Merged
merged 1 commit into from
Jun 7, 2023

Conversation

jsleroy
Copy link
Contributor

@jsleroy jsleroy commented Jun 7, 2023

Fix the sensor class to remove HASS warning about inconsistent default unit.

2023-06-07 22:36:38.597 WARNING (MainThread)
[homeassistant.components.sensor] Entity
sensor.linky_puissance_maximale_triphasee_atteinte_jour_n_1 (<class 'custom_components.linkytic.sensor.RegularIntSensor'>) is using native unit of measurement 'W' which is not a valid unit for the device class ('current') it is using; expected one of ['A', 'mA']; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.

Fix the sensor class to remove HASS warning about inconsistent default
unit.

2023-06-07 22:36:38.597 WARNING (MainThread)
[homeassistant.components.sensor] Entity
sensor.linky_puissance_maximale_triphasee_atteinte_jour_n_1 (<class
'custom_components.linkytic.sensor.RegularIntSensor'>) is using native
unit of measurement 'W' which is not a valid unit for the device class
('current') it is using; expected one of ['A', 'mA']; Please update your
configuration if your entity is manually configured, otherwise report it
to the custom integration author.
@hekmon
Copy link
Owner

hekmon commented Jun 7, 2023

Effectivement c'est une erreur, merci de l'avoir repérée !

Puis-je en déduire que vous êtes un utilisateur de l'intégration avec un compteur triphasé ?

Comme indiqué dans le readme je cherche des utilisateurs pour sortir cette fonctionnalité de beta. Ne possédant pas de triphasé je développe cette fonctionnalité complètement à l'aveugle... Et j'aimerai m'assurer que tout fonctionne correctement (notamment les trames courtes), si vous pouviez joindre la discussion sur #2 je vous en serai très reconnaissant 🙏

@hekmon hekmon merged commit 2aa843a into hekmon:main Jun 7, 2023
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.

2 participants