New user and im just wondering if my Automation was done correctly

Configuration
1

I created my first automation using Home Assistant, and I would just like some feed back on weather I did it correctly or not.

This is 3 separate automatons, I am using the Motion and Temp sensors from my Ecobee, and a Belkin WeMo switch to control a window AC in my converted garage where all my servers and gaming systems are.

The first Rule is supposed to Turn on the AC if the Temp is above 74 and the room is occupied. The second rule is to turn off the AC if its below 76 and the room is empty. The third rule is to turn on the AC if the temp is above 80 regardless if anyone is in the room or not.

- id: '1527634810468'
  alias: Game Room AC ON
  trigger:
  - above: '74'
entity_id: sensor.game_room_temperature
platform: numeric_state
  condition:
  - condition: state
entity_id: binary_sensor.game_room_occupancy
state: 'on'
  action:
  - data:
  entity_id: switch.gameroom_ac
service: switch.turn_on
- id: '1527637106586'
  alias: Gameroom AC OFF
  trigger:
  - below: '76'
entity_id: sensor.game_room_temperature
platform: numeric_state
  condition:
  - condition: state
entity_id: binary_sensor.game_room_occupancy
state: 'off'
  action:
  - alias: ''
data:
  entity_id: switch.gameroom_ac
service: switch.turn_off
- id: '1527639192572'
  alias: Gameroom AC Server Protect ON
  trigger:
  - above: '80'
entity_id: sensor.game_room_temperature
platform: numeric_state
  condition: []
  action:
  - data:
  entity_id: switch.gameroom_ac
service: switch.turn_on

I think I did everything correct, but it hasn’t been running long enough to really tell.

2

If you add them to a group in your setup you can click on the automation and click trigger to test

image
image750×1334 31.8 KB

3

I suppose it shouldn’t even work with the indentation in your post.

I tried to rewrite it. Hope I didn’t do mistakes. The ID is only required if you want to use the front end editor. Other vise you can use aliases in the same way. Or when you start to have massive amount of automations you would want to switch to something like in this example configuration:

- id: '1527634810468'
  alias: Game Room AC ON
  trigger:
  - platform: numeric_state
    entity_id: sensor.game_room_temperature
    above: 74
    for:          # This for is just an example. You may delete if
      minutes: 1  # not usefull. temperature would need to be 1 minute over limit to trigger.
  condition:
  - condition: state
    entity_id: binary_sensor.game_room_occupancy
    state: 'on'
  action:
  - service: switch.turn_on
    data:
      entity_id: switch.gameroom_ac

- id: '1527637106586'
  alias: Gameroom AC OFF
  trigger:
  - platform: numeric_state
    entity_id: sensor.game_room_temperature
    below: 76
  condition:
  - condition: state
    entity_id: binary_sensor.game_room_occupancy
    state: 'off'
  action:
  - service: switch.turn_off
    data:
      entity_id: switch.gameroom_ac

- id: '1527639192572'
  alias: Gameroom AC Server Protect ON
  trigger:
  - platform: numeric_state
    entity_id: sensor.game_room_temperature
    above: 80
  action:
  - service: switch.turn_on
    data:
      entity_id: switch.gameroom_ac