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

feature/289: Температура для кондиционера с датчика из конфигурации #340

Closed
wants to merge 1 commit into from

Conversation

lmorningstar116
Copy link

@lmorningstar116 lmorningstar116 commented Feb 13, 2023

Fix #289
Изменение в конфигурации для указания дополнительных параметров у объектов прокидываемых в HA.

Пример конфигурации

yandex_station:
  include:
    - name: Кондиционер
      temp_sensor: sensor.temperatura_spalnia_temperature

@Aligatorru
Copy link

  - name: Кондиционер  
    temp_sensor: sensor.xiaomi_bedroom
  - name: Пылесос

Оно работает так если написать или нет? Или ещё надо в интеграцию внести изменения.. чёт у меня не выходит, крашатся устройства если строчку добавляю с градусником

@lmorningstar116
Copy link
Author

  - name: Кондиционер  
    temp_sensor: sensor.xiaomi_bedroom
  - name: Пылесос

Оно работает так если написать или нет? Или ещё надо в интеграцию внести изменения.. чёт у меня не выходит, крашатся устройства если строчку добавляю с градусником

Если изменения из этого PR внести, то работает

@sashayohan
Copy link

А есть возможность добавить бинарный датчик (умную розетку)? что бы видеть реальное состояние.
В статусе всегда отображается что кондиционер (или например телевизор) всегда включён.

@lmorningstar116
Copy link
Author

А есть возможность добавить бинарный датчик (умную розетку)? что бы видеть реальное состояние.
В статусе всегда отображается что кондиционер (или например телевизор) всегда включён.

Наверное есть. Но данный PR не об этом. Он про использование кондиционером пробрасываемым из Я интеграции датчика температуры из HA

@lmorningstar116
Copy link
Author

Добавлена обратная совместимость конфига

Пример конфигурации

yandex_station:
  include:
    - name: Кондиционер
      temp_sensor: sensor.temperatura_spalnia_temperature
    - Телевизор

@AlexxIT вроде ПР не ломает ничего, может влить? )

@AlexxIT
Copy link
Owner

AlexxIT commented Jun 27, 2023

Не люблю вливать того, что не понимаю. Нужно вникнуть, проверить и потом уже одобрять.

@AlexxIT
Copy link
Owner

AlexxIT commented Jan 14, 2024

Реализовано другим способом. Но возможно будет проблема с оперативностью обновлений показания датчика
https://github.com/AlexxIT/YandexStation/releases/tag/v3.13.0

@AlexxIT AlexxIT closed this Jan 14, 2024
@AlexxIT AlexxIT added this to the v3.13.0 milestone Jan 14, 2024
@lmorningstar116 lmorningstar116 deleted the feature/289 branch January 14, 2024 15:37
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.

При интеграции кондиционера температура с другого датчика
5 participants