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

Сенсор *_kettle_ontime #141

Closed
Enzokot opened this issue Feb 4, 2023 · 11 comments
Closed

Сенсор *_kettle_ontime #141

Enzokot opened this issue Feb 4, 2023 · 11 comments

Comments

@Enzokot
Copy link

Enzokot commented Feb 4, 2023

Нome assistant версии 2023.2 после старта системы в логах предупреждение:
2023-02-04 12:40:01.464 WARNING (MainThread) [homeassistant.components.sensor] Sensor sensor.r4s01_kettle_ontime has device class None, state class total_increasing and unit None thus indicating it has a numeric value; however, it has the non-numeric value: 113:12:25 (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+mqtt%22
Видимо нужно передавать числовое значение и добавить device_class: duration

@alutov
Copy link
Owner

alutov commented Feb 6, 2023

Добавил device_class: duration. Файл в папке jpg. Вроде у меня не ругается, но нужно еще проверить. Надо сказать, в логе полно ошибок. Например, расход в m3/h и класс water несовместимы, это допустимо только для газа.

@Enzokot
Copy link
Author

Enzokot commented Feb 6, 2023

Чтобы не было в *_kettle_ontime ошибки, значение должно быть int, а сейчас судя по скриншоту - string.

Скриншот

image

@alutov
Copy link
Owner

alutov commented Feb 6, 2023

Значит, уберу все атрибуты. Если ХА такой тупой и не понимает длительность в часах минутах и секундах.

@Enzokot
Copy link
Author

Enzokot commented Feb 6, 2023

Атрибуты то правильные и они собираются потом в Long-term Statistics, просто система никак не сможет собрать статистику по строке https://developers.home-assistant.io/docs/core/entity/sensor/#long-term-statistics
Вот к примеру другой +/- аналогичный сенсор:
image

@alutov
Copy link
Owner

alutov commented Feb 6, 2023

Время в секундах. Для ХА хорошо, конечно, для других систем не знаю. Наглядности меньше стало.

@Enzokot
Copy link
Author

Enzokot commented Feb 6, 2023

В качестве предложения, может сделать отдельное значение для ХА, например working_time_int и топик discovery перевести на числовое значение?

@alutov
Copy link
Owner

alutov commented Feb 6, 2023

Пусть уже будет в секундах.)

@alutov
Copy link
Owner

alutov commented Feb 6, 2023

Посмотрел сейчас у себя. Вроде и не ругается, но статистику отображает неправильно.

@Enzokot
Copy link
Author

Enzokot commented Feb 6, 2023

У меня в логах тишина, отображается в интерфейсе корректно. В статистике предлагает сохранение (для сбора корректных значений ему нужно несколько часов). На мой взгляд идеально.
image

@alutov
Copy link
Owner

alutov commented Feb 6, 2023

Оставляем так.

@Enzokot
Copy link
Author

Enzokot commented Feb 6, 2023

Большое спасибо за исправления!

@Enzokot Enzokot closed this as completed Feb 6, 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

No branches or pull requests

2 participants