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

double 'ip' in config.ini #9

Closed
lanket opened this issue Dec 13, 2018 · 8 comments
Closed

double 'ip' in config.ini #9

lanket opened this issue Dec 13, 2018 · 8 comments

Comments

@lanket
Copy link

lanket commented Dec 13, 2018

[settings]
.....
# IP терминала, определяется автоматически при первом запуске.
# Используется только для проговаривания.
ip = 
[mpd]
......
# ip и порт сервера
ip = 127.0.0.1

Так как все настройки модуля в мажоре храняться в одной таблице немного заморочно отслеживать одноименные опции.
может сделаем так: ip_terminal и ip_mpd_server

@lanket
Copy link
Author

lanket commented Dec 13, 2018

Анологично:

[majordomo]
.....
# IP сервера majordomo, нужен для отправки команд.
# Пока он не задан терминал будет проговаривать свой IP при старте.
ip = 192.168.1.197

Переименовать в ip_majordomo

@lanket
Copy link
Author

lanket commented Dec 13, 2018

Опция "speaker"
тески в Yandex и aws
Аналогично speaker_aws и speaker_yandex

@lanket
Copy link
Author

lanket commented Dec 13, 2018

Раздел rhvoice-rest
speaker => speaker_rhvoice_rest
server => server_rhvoice_rest

Раздел rhvoice
speaker => speaker_rhvoice

Раздел pocketsphinx-rest
server => server_pocketsphinx_rest

@lanket
Copy link
Author

lanket commented Dec 13, 2018

Теперь я понял что ты имел ввиду :

Параметры для других секций лучше всего присылать во вложенных json, например:

settings:{"proxy": {"enable": "1"}}

Они нормально парсятся. Терминал принимает любые параметры, но не принимает несуществующие секции.
Установку server для локальных провайдеров можно через ключи сделать.
@Aculeasis Aculeasis referenced this issue 27 days ago
Closed
Список команд требуемых от модуля #2
@Aculeasis
Aculeasis commented 24 days ago

Еще можно отправлять {"majordomo": {"terminal": "sysname"}}, как я понял если в запросе задано системное имя терминала то и ответ придет на тот же терминал.

Если конечно тебе заморочно переделывать такое кол-во переменных, то в принципе могу я тогда разбить на несколько таблиц и форм в модуле.

@lanket
Copy link
Author

lanket commented Dec 13, 2018

Дай знать, как поступить.

@Aculeasis
Copy link
Owner

Конечно, лучше если настройки будут приходить в своих подсекциях.

А вообще можешь присылать с префиксами. Я при разборе замаплю их куда надо, как сейчас переношу ip_server в [majordomo] ip:

    # Автоматически переносим ключи в подсекции из settings.
    # Ключ: (новая секция, новое имя ключа)
    KEY_MOVE = {
        'ip_server': ('majordomo', 'ip'),
        'linkedroom': ('majordomo', ''),
    }

Добавлю туда server_rhvoice_rest, server_pocketsphinx_rest и т.д. Только список всех ключей надо.

Учитывая что маппинг уже есть и делается одной строчкой, зачем переименовывать параметры в конфиге?

@lanket
Copy link
Author

lanket commented Dec 14, 2018

Тогда предлагаю прийти к шаблону мапинга для всех параметров как выход:
(ключ)-(секция)
Обращаю внимание на разделитель тире а не подчёркивание чтобы не было путаницы в таких параметрах:

secret_access_key_aws
server_rhvoice_rest

@Aculeasis
Copy link
Owner

В именах секций вместо - идет 0, вроде все ок.

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