roopesh
commented
3 years ago I went through the guidance here (https://github.com/TD22057/insteon-mqtt/blob/master/docs/discovery.md#six-button-keypadlinc) and added a new discovery class. However I keep getting an error on restart:
mqtt:
kpl_six:
btn_on_off_payload:
- must be of dict type
btn_on_off_topic:
- must be of dict type
btn_scene_payload:
- must be of dict type
btn_scene_topic:
- must be of dict type
btn_state_payload:
- must be of dict type
btn_state_topic:
- must be of dict type
dimmer_level_payload:
- must be of dict type
dimmer_level_topic:
- must be of dict type
dimmer_state_payload:
- must be of dict type
dimmer_state_topic:
- must be of dict typeI have tried renaming all the topic/payload variables (and updating them in the associated discovery entities) and the error just changes to the new name. I've checked my syntax:
mqtt:
kpl_six:
btn_state_topic: 'insteon/{{address}}/state/{{button}}'
btn_state_payload: '{{on_str.upper()}}'
dimmer_state_topic: 'insteon/{{address}}/state/1'
dimmer_state_payload: '{ "state" : "{{on_str.upper()}}", "brightness" : {{level_255}} }'
btn_on_off_topic: 'insteon/{{address}}/set/{{button}}'
btn_on_off_payload: '{ "cmd" : "{{value.lower()}}" }'
dimmer_level_topic: 'insteon/command/{{address}}'
dimmer_level_payload: >
'{ "cmd" : "{{json.state.lower()}}"
{% if json.brightness is defined %}
, "level" : {{json.brightness}}
{% endif %}
{% if json.transition is defined %}
, "transition" : {{json.transition}}
{% endif %} }'
btn_scene_topic: 'insteon/{{address}}/scene/{{button}}'
btn_scene_payload: >
'{ "cmd" : "{{json.cmd.lower()}}"
{% if json.brightness is defined %}
, "level" : {{json.brightness}}
{% endif %}
}'
discovery_entities:
- component: 'light'
config: |-
{
"uniq_id": "{{address}}_1",
"name": "{{name_user_case}} btn 1",
"device": {{device_info_template}},
"brightness": {{is_dimmable|lower()}},
"cmd_t": "{%- if is_dimmable -%}
{{dimmer_level_topic}}
{%- else -%}
{{btn_on_off_topic_1}}
{%- endif -%}",
"schema": "json",
"stat_t": "{%- if is_dimmable -%}
{{dimmer_state_topic}}
{%- else -%}
{{btn_state_topic_1}}
{%- endif -%}"
}
- component: 'switch'
config: |-
{
"uniq_id": "{{address}}_3",
"name": "{{name_user_case}} btn 3",
"device": {{device_info_template}},
"cmd_t": "{{btn_on_off_topic_3}}",
"stat_t": "{{btn_on_off_topic_3}}"
}
- component: 'switch'
config: |-
{
"uniq_id": "{{address}}_4",
"name": "{{name_user_case}} btn 4",
"device": {{device_info_template}},
"cmd_t": "{{btn_on_off_topic_4}}",
"stat_t": "{{btn_on_off_topic_4}}"
}
- component: 'switch'
config: |-
{
"uniq_id": "{{address}}_5",
"name": "{{name_user_case}} btn 5",
"device": {{device_info_template}},
"cmd_t": "{{btn_on_off_topic_5}}",
"stat_t": "{{btn_on_off_topic_5}}"
}
- component: 'switch'
config: |-
{
"uniq_id": "{{address}}_6",
"name": "{{name_user_case}} btn 6",
"device": {{device_info_template}},
"cmd_t": "{{btn_on_off_topic_6}}",
"stat_t": "{{btn_on_off_topic_6}}"
}I've done yaml validations, too. Any ideas why I keep getting the same error?
krkeegan
When trying to control button 1 (Load) of the six button keypad from the Home Assistant UI, it's sending
insteon/<address>/set/1 ONwhich doesn't do anything. That is the expected behavior for the other buttons but not the load. For the loadinsteon/command/<address> {"cmd": "on"}needs to be sent.I think the discovery message for the six button keypad needs to be updated... at the same time, could buttons 2 and 9 be removed?