ratgdo / mqtt-ratgdo

ratgdo via mqtt
GNU General Public License v2.0
76 stars 16 forks source link

Periodic status changes when door not opened/closed #12

Closed lukescott closed 7 months ago

lukescott commented 7 months ago

I have two Security 1.0 doors integrated into HomeAssistant. I'm getting periodic status changes where it will say the door opened, but then immediately closed. Here is some logs from HomeAssistant:

Screenshot 2023-11-26 at 10 21 31 AM Screenshot 2023-11-26 at 10 20 39 AM

During this time I have not opened the garage door, and as you can see, the time in-between status changes are very short.

PaulWieland commented 7 months ago

A few people have seen this. Not much can be done about the door reporting the wrong status. I could have it ignore short status changes, but then it would add a delay in updating whenever the door state really changes.

lukescott commented 7 months ago

@PaulWieland Is there anything we can do to look into this a little deeper? This is a fairly big problem. I can't have this reporting the door is open when it's not. If the door actually opens, then I'm bound to ignore it as a false positive.

The only other thing I can think of it the dummy button has some capacitors in it. I'm going to try and disconnect one and monitor it from there. But if this issue persists, I need a way to debug this and get to the root cause.

Snuffy2 commented 6 months ago

I know this is already closed, but I'd agree that it would be helpful if something could be done to filter these out.

SNAG-2670

Snuffy2 commented 6 months ago

For now, I've put a Template Cover in HA that is now what Automations and HomeKit are using to reduce false alerts.

cover:
  - platform: template
    covers:
      garage_door_bikes:
        device_class: garage
        friendly_name: "Garage Door - Bikes"
        unique_id: garage_door_bikes_safe
        value_template: >-
          {% if is_state('cover.garage_door_bikes_mqtt',states('cover.garage_door_bikes')) %}
          {{ states('cover.garage_door_bikes') }}
          {% elif (is_state('cover.garage_door_bikes','opening') and is_state('cover.garage_door_bikes_mqtt','open') and (as_timestamp(now()) - as_timestamp(states.cover.garage_door_bikes_mqtt.last_changed)) <= 5) %}
          opening
          {% elif (is_state('cover.garage_door_bikes_mqtt','open') and (as_timestamp(now()) - as_timestamp(states.cover.garage_door_bikes_mqtt.last_changed)) > 5) or is_state('cover.garage_door_bikes_mqtt',['opening','closing','closed']) %}
          {{ states('cover.garage_door_bikes_mqtt') }}
          {% else %}
          closed
          {% endif %}
        open_cover:
          - service: cover.open_cover
            target:
              entity_id: cover.garage_door_bikes_mqtt
        close_cover:
          - service: cover.close_cover
            target:
              entity_id: cover.garage_door_bikes_mqtt
        stop_cover:
          - service: cover.stop_cover
            target:
              entity_id: cover.garage_door_bikes_mqtt      
lukescott commented 6 months ago

@Snuffy2 HA should take care of this on it's own very soon. See https://github.com/ratgdo/mqtt-ratgdo/issues/14 and https://github.com/home-assistant/core/pull/104726.

In fact it looks like that commit made it into 2023.12.1. Perhaps updating HA will address the issue. I'm updating mine now.

Snuffy2 commented 6 months ago

@lukescott thanks! 2023.12.2+ has indeed improved the status of the MQTT Garage Door.