MajorDoMo

А покажи что за ошибка скрин видео или ещё что...

tarasfrompir1 писал(а):А покажи что за ошибка скрин видео или ещё что...

Я сча посмотрел на то как ты эту фичу прикрутил и немного в шоке((. Сча надо баги починить. И тогда начну принимать пулреквесты.

Отправлено с моего Mi MIX 2S через Tapatalk

Да с планшета клацаю вот обеда. А баги чинить надо будет потом , поскольку надо понять работает или нет

ПС
А что не так ?

починил немного картинки и IP адресс для устройств онлайн. сча надо проверять.
Мне кажется не корректным добавление в этот модуль функционала Yeelight. Так как в маркете есть уже такой модуль. Тем более на таких костылях как это сделано сейчас(это устройство понормальному не определяется даже).

Советую сделать пул реквест с текущим функционалом по простым устройствам и шаблоном для лампы в модуль YeeLight. а с этого модуля этот функционал удалить.

очень коряво добавлена поддержка дополнительных устройств, что добавляет бардака в код((
он и так не идеален(( Давайте новые девайсы в библиотеку пихать. чтоб на выходе был просто массив с однотипными девайсами. а не куча if/else в методе Scan.

попробовал шаблон для лампочек - он не работает(( зато модуль от YeeLight нормально управляет через код.

T-REX-XP писал(а): Вт июл 31, 2018 11:29 pm Советую сделать пул реквест с текущим функционалом по простым устройствам и шаблоном для лампы в модуль YeeLight. а с этого модуля этот функционал удалить.

Не понимаю, зачем? В модуле YeeLight используется API YeeLight и это АПИ вот никак не связано с протоколом SSDP. Это просто ещё один способ управлять лампочками. Не стоит смешивать эти 2 разные реализации примерно по той же причине почему нельзя смешивать модули XiaomiHome Xiaomi miio.

Gelezako писал(а): Вт июл 31, 2018 11:34 pmВ модуле YeeLight используется API YeeLight и это АПИ вот никак не связано с протоколом SSDP. Это просто ещё один способ управлять лампочками.

Давно ли стало можно рулить девайсами еелайт через SSDP?
SSDP только для поиска девайсов в локальной сети.

Gelezako писал(а):

T-REX-XP писал(а): Вт июл 31, 2018 11:29 pm Советую сделать пул реквест с текущим функционалом по простым устройствам и шаблоном для лампы в модуль YeeLight. а с этого модуля этот функционал удалить.

Не понимаю, зачем? В модуле YeeLight используется API YeeLight и это АПИ вот никак не связано с протоколом SSDP. Это просто ещё один способ управлять лампочками. Не стоит смешивать эти 2 разные реализации примерно по той же причине почему нельзя смешивать модули XiaomiHome Xiaomi miio.

Ты меня не понял. Я за то чтоб твой модуль имел шаблон отображения и другие фичи. А не лепить это костылем в SSDP Finder модуле. Насколько я знаю шаблон отображения никак не завязан на ssdp. По крайней мере это кажется логичным.

И не понимаю, зачем плодить однотипные модули. Модно же было твой улучшить. Раз он давно был и работает нормально.

Или я не прав?

Отправлено с моего Mi MIX 2S через Tapatalk

T-REX-XP писал(а): Вт июл 31, 2018 11:38 pm И не понимаю, зачем плодить однотипные модули. Модно же было твой улучшить. Раз он давно был и работает нормально.

Или я не прав?

Полностью поддерживаю идею улучшить модуль Yeelight. Там полно задач, которые я не могу решить. Например обратная связь для верного отображения состояния или например возможность привязывать объект, созданный через модуль"ПУ". Я тоже не понимаю зачем в модуль SSDPFinder добавлять код по нахождению лампочек yeelight.

skysilver писал(а): Вт июл 31, 2018 11:38 pm Давно ли стало можно рулить девайсами еелайт через SSDP?
SSDP только для поиска девайсов в локальной сети.

без понятия, но судя по тому что пишет tarasfrompir1, то он вроде каким-то образом управляет ими. Ради теста попробовал у себя. Из 3-х разных устройств Yeelight нашло только ленту. Управление не работает.

skysilver писал(а): Вт июл 31, 2018 11:38 pm

Gelezako писал(а): Вт июл 31, 2018 11:34 pmВ модуле YeeLight используется API YeeLight и это АПИ вот никак не связано с протоколом SSDP. Это просто ещё один способ управлять лампочками.

Давно ли стало можно рулить девайсами еелайт через SSDP?
SSDP только для поиска девайсов в локальной сети.

сейчас получается что в модуле  SSDP Finder есть реализация работы с DLNA, YeeLight.
Хотя по нормальному это должны быть разные модули. при этом работа этого модуля может выглядеть следующим образом.

1) модуль находит устройства посредством ssdp/UPNP
2) модуль загружает из маркета модули для управления найдеными устройствами(chromecast,Yeelight,dlna).

как то так...


Надо как то доработать поиск модулей по протоколам/ключевым словам/типам/брендам чтоб можно было поиском найти нужные модули в маркете и установить их автоматом. а потом уже добавлять простое устройство.