search

bruhautomation / ESP-MQTT-JSON-Multisensor

(OBSOLETE) ESP MQTT JSON Multisensor for Home Assistant. Supported sensors include the TEMT6000 light, AM312 PIR, DHT22 temperature/humidity sensors. RGB led supports flash, fade, and transition. Over-The-Air (OTA) uploading, too!
https://youtu.be/jpjfVc-9IrQ
Apache License 2.0
340 stars 152 forks source link

LED Flash not using length being set #35

Closed devinjparsons closed 6 years ago

devinjparsons commented 6 years ago

I have a weird issue where I set the led to flash for 10 seconds, but the mqtt message that is sent has the flash set to 2, no matter the value I set. Have you seen this issue?

*Edit: looks like not even brightness is being ignored too.

action:
    service: light.turn_on
      data:
        entity_id: light.master_led
        color_name: red
        brightness: 100
        flash: 10

Message arrived [(topic)] {"state": "ON", "color": {"r": 255, "g": 0, "b": 0}, "flash": 2, "brightness": 255}
devinjparsons commented 6 years ago

just found this on the mqtt_json light page, so i know where the 2 is coming from. i'll try playing with the keyword long and short, but weird that brightness still not coming through.

flash_time_long (Optional): The duration, in seconds, of a “long” flash. Default is 10. flash_time_short (Optional): The duration, in seconds, of a “short” flash. Default is 2.

devinjparsons commented 6 years ago

And closing, my indentation was off on my automations, so it was pulling default values always...