[Модуль] SSDP Finder (ssdp_finder)

Разработка дополнительных модулей, подключение различных приложений.

Модератор: immortal

Аватара пользователя
tarasfrompir
Сообщения: 3216
Зарегистрирован: Ср мар 02, 2016 8:18 pm
Откуда: Украина Пирятин
Благодарил (а): 223 раза
Поблагодарили: 816 раз

Re: Модуль - SSDPFinder

Сообщение tarasfrompir » Вт июл 31, 2018 9:32 pm

А покажи что за ошибка скрин видео или ещё что...
Спасибо нам ПОМОЖЕТ..!
T-REX-XP
Сообщения: 174
Зарегистрирован: Вт июн 16, 2015 10:54 pm
Благодарил (а): 16 раз
Поблагодарили: 32 раза

Re: Модуль - SSDPFinder

Сообщение T-REX-XP » Вт июл 31, 2018 10:00 pm

tarasfrompir1 писал(а):А покажи что за ошибка скрин видео или ещё что...
Я сча посмотрел на то как ты эту фичу прикрутил и немного в шоке((. Сча надо баги починить. И тогда начну принимать пулреквесты.

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

Аватара пользователя
tarasfrompir
Сообщения: 3216
Зарегистрирован: Ср мар 02, 2016 8:18 pm
Откуда: Украина Пирятин
Благодарил (а): 223 раза
Поблагодарили: 816 раз

Re: Модуль - SSDPFinder

Сообщение tarasfrompir » Вт июл 31, 2018 10:06 pm

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

ПС
А что не так ? :P
Спасибо нам ПОМОЖЕТ..!
T-REX-XP
Сообщения: 174
Зарегистрирован: Вт июн 16, 2015 10:54 pm
Благодарил (а): 16 раз
Поблагодарили: 32 раза

Re: Модуль - SSDPFinder

Сообщение T-REX-XP » Вт июл 31, 2018 11:29 pm

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

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

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

попробовал шаблон для лампочек - он не работает(( зато модуль от YeeLight нормально управляет через код.
Аватара пользователя
Gelezako
Сообщения: 963
Зарегистрирован: Чт июн 02, 2016 9:33 pm
Благодарил (а): 205 раз
Поблагодарили: 106 раз
Контактная информация:

Re: Модуль - SSDPFinder

Сообщение Gelezako » Вт июл 31, 2018 11:34 pm

T-REX-XP писал(а):
Вт июл 31, 2018 11:29 pm
Советую сделать пул реквест с текущим функционалом по простым устройствам и шаблоном для лампы в модуль YeeLight. а с этого модуля этот функционал удалить.
Не понимаю, зачем? В модуле YeeLight используется API YeeLight и это АПИ вот никак не связано с протоколом SSDP. Это просто ещё один способ управлять лампочками. Не стоит смешивать эти 2 разные реализации примерно по той же причине почему нельзя смешивать модули XiaomiHome Xiaomi miio.
фанат Мажордомо
тематический блог http://blog.gelezako.com
плейлист про Мажордомо на ютубе https://www.youtube.com/playlist?list=P ... EdBGtX084E
skysilver
Сообщения: 3006
Зарегистрирован: Чт авг 21, 2014 8:28 am
Откуда: Киров, Россия
Благодарил (а): 400 раз
Поблагодарили: 1754 раза
Контактная информация:

Re: Модуль - SSDPFinder

Сообщение skysilver » Вт июл 31, 2018 11:38 pm

Gelezako писал(а):
Вт июл 31, 2018 11:34 pm
В модуле YeeLight используется API YeeLight и это АПИ вот никак не связано с протоколом SSDP. Это просто ещё один способ управлять лампочками.
Давно ли стало можно рулить девайсами еелайт через SSDP?
SSDP только для поиска девайсов в локальной сети.
MajorDoMo (GitHub) на Cubietruck. ОС Debian 7 (wheezy) (kernel 3.4.105) с переносом на HDD.
Мой CONNECT | Блоги | Telegram
T-REX-XP
Сообщения: 174
Зарегистрирован: Вт июн 16, 2015 10:54 pm
Благодарил (а): 16 раз
Поблагодарили: 32 раза

Re: Модуль - SSDPFinder

Сообщение T-REX-XP » Вт июл 31, 2018 11:38 pm

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

Аватара пользователя
Gelezako
Сообщения: 963
Зарегистрирован: Чт июн 02, 2016 9:33 pm
Благодарил (а): 205 раз
Поблагодарили: 106 раз
Контактная информация:

Re: Модуль - SSDPFinder

Сообщение Gelezako » Вт июл 31, 2018 11:43 pm

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

Или я не прав?
Полностью поддерживаю идею улучшить модуль Yeelight. Там полно задач, которые я не могу решить. Например обратная связь для верного отображения состояния или например возможность привязывать объект, созданный через модуль"ПУ". Я тоже не понимаю зачем в модуль SSDPFinder добавлять код по нахождению лампочек yeelight.
фанат Мажордомо
тематический блог http://blog.gelezako.com
плейлист про Мажордомо на ютубе https://www.youtube.com/playlist?list=P ... EdBGtX084E
Аватара пользователя
Gelezako
Сообщения: 963
Зарегистрирован: Чт июн 02, 2016 9:33 pm
Благодарил (а): 205 раз
Поблагодарили: 106 раз
Контактная информация:

Re: Модуль - SSDPFinder

Сообщение Gelezako » Вт июл 31, 2018 11:45 pm

skysilver писал(а):
Вт июл 31, 2018 11:38 pm
Давно ли стало можно рулить девайсами еелайт через SSDP?
SSDP только для поиска девайсов в локальной сети.
без понятия, но судя по тому что пишет tarasfrompir1, то он вроде каким-то образом управляет ими. Ради теста попробовал у себя. Из 3-х разных устройств Yeelight нашло только ленту. Управление не работает.
фанат Мажордомо
тематический блог http://blog.gelezako.com
плейлист про Мажордомо на ютубе https://www.youtube.com/playlist?list=P ... EdBGtX084E
T-REX-XP
Сообщения: 174
Зарегистрирован: Вт июн 16, 2015 10:54 pm
Благодарил (а): 16 раз
Поблагодарили: 32 раза

Re: Модуль - SSDPFinder

Сообщение T-REX-XP » Вт июл 31, 2018 11:46 pm

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).

как то так...


Надо как то доработать поиск модулей по протоколам/ключевым словам/типам/брендам чтоб можно было поиском найти нужные модули в маркете и установить их автоматом. а потом уже добавлять простое устройство.
За это сообщение автора T-REX-XP поблагодарил:
Gelezako (Вт июл 31, 2018 11:48 pm)
Рейтинг: 1.16%
Ответить