efa2000 / node-red-contrib-alice

The module allows you to use Node-Red together with the Yandex.Alice voice assistant service (voice control of Yandex smart home devices)
MIT License
39 stars 8 forks source link

Нет возможности отключить random_access для range #49

Closed Vinndimon closed 3 years ago

Vinndimon commented 3 years ago

У умения range может быть отключён параметр _randomaccess, что крайне полезно для ИК-пультов. Например, чтобы прибавить или убавить звук на телевизоре, не устанавливая конкретное значение. Пожалуйста, добавьте возможность отключить random_access.

efa2000 commented 3 years ago

не совсем понимаю проблему при включенном random, вы говорите алисе прибавить звук и она прибавляет на шаг отключение рендом приводит только к тому что отключаются кнопки с цифрами в итерфейсе яндекса

давайте попробуем на конкретном кейсе

Vinndimon commented 3 years ago

Ок, кейс следующий: у меня есть умный ИК-пульт, который управляет телевизором. При этом нет обратной связи - я не знаю, какой сейчас на телевизоре уровень громкости. Поэтому при включённом random access я могу прибавлять громкость ограниченное количество раз, и не факт, что оно совпадает с реальными значениями на телевизоре.

Алиса считает, что громкость - 100 из 100 и больше не может посылать команду "повысить громкость", хотя реальная громкость на телевизоре может быть совсем другой. Это видно в интерфейсе Яндекса: если громкость 100 из 100, кнопка "+" становится серой. Т.е. при достижении максимального или минимального (по мнению Алисы) уровня громкости, она перестаёт повышать или понижать громкость.

Нужно иметь возможность добавить в Алису кнопки, просто повышающие или понижающие громкость. Вне зависимости от того, какой сейчас установлен уровень.

Ещё в Node Red приходится костылить некоторые вещи. К примеру, вместо того, чтобы просто получить от Алисы условную команду "volume +", я получаю цифру и вынужден сам определять, повышается громкость, или понижается. Работает довольно нестабильно.

efa2000 commented 3 years ago

Добрый день, да, спасибо за кейс. теперь проблема более понятна но это тогда не random_access, это параметр retrievable который как раз и отвечает за то знает ли Алиса текущее значение или нет и если установить его в false, то на выходе из ноде всегда будет не конкретное значение которое нужно установить, а значение на сколько нужно прибавить, или со знаком минус значение на которое нужно уменьшить

Vinndimon commented 3 years ago

Рад, что удалось разъяснить ситуацию. Вам виднее, о какой опции идёт речь. Но описание random access говорит вот что:

Возможность устанавливать произвольные значения функции. Если эта возможность выключена, пользователю будет доступно только последовательное изменение значений — в большую или меньшую сторону. Например, изменение громкости телевизора при работе через ИК пульт.

efa2000 commented 3 years ago

ок, я добавлю возможность настройки этого параметра в следущим релизе

efa2000 commented 3 years ago

Добавил для ноды Range возможность получить на выходе не финальное значение, а только дельту на которую нужно прибавить или убавить Мне не очень нравится как это реализовано из интерфейса самого яндекса. но там я уже поделать к сожалению ни чего не могу

версия 0.4.2

Vinndimon commented 3 years ago

@efa2000 обновился до 0.4.2. Не очень понимаю, как это решает задачу с повышением/уменьшением громкости без указания её точного значения. Вот так у меня выглядит настройка ноды: https://user-images.githubusercontent.com/4707428/99273760-82a50880-283a-11eb-8e57-c7793eb81089.png А вот так - панель управления телевизором: https://user-images.githubusercontent.com/4707428/99274273-33130c80-283b-11eb-9758-5a3ccd8a1827.png

А нужно, чтобы были всегда активные кнопки +/-, без цифр.

efa2000 commented 3 years ago

поставьте версию 0.4.3 там был небольшой баг с галочкой и не забудет после изменения настройки ноды сделать deploy и обновить список устройств в интерфейсе яндекса в итоге выглядит все примерно так на выходе у вас будут значения 1 / -1 (число может отличаться, к примеру если вы через ализу попросите прибавить звук на 20, основное здесь положительное или отрицательное число )

![Uploading image.png…]()

Vinndimon commented 3 years ago

@efa2000 к сожалению, скриншот не загрузился. Поэтому не могу увидеть, как должно быть. Обновился до 0.4.3 - настройки ноды у меня не изменились, количество галочек всё то же.

efa2000 commented 3 years ago

Извините, сейчас очень большая загрузка не могу оперативно отвечать в итоге у вас должна быть снята галочка как на снимке и на выходе будете иметь положительное отрицательное значение image

efa2000 commented 3 years ago

Как галочка то помогла? Удалось сделать, что планировали?