Refactored UART output

Disabled presence and target messages from sensor as they were unused and to reduce load on the software UART on the D1 Mini

Author: igiannakas

header/leapmmw_sensor.h

@@ -69,14 +69,8 @@ class leapmmw : public Component, public UARTDevice {
       if(readline(read(), buffer, max_line_length) >= 4) {
         std::string line = buffer;
 
-        if (line.substr(0, 6) == "$JYRPO" && id(mmwave_sensor).state == 0) {
-          publishSwitch("mmwave_sensor", 1);
-        }
-
         // compare last line
         if (line.substr(0, 8) == "Response") {
-          // ESP_LOGD("custom", "Found Response - line is: %s", line.c_str());
-          // ESP_LOGD("custom", "Found Response - lastline is: %s", getline.c_str());
 
           // leapMMW:/>getSensitivity
           if (getline.substr(0, 24) == "leapMMW:/>getSensitivity" || getline.substr(0, 14) == "getSensitivity") {
@@ -130,8 +124,6 @@ class leapmmw : public Component, public UARTDevice {
           }
         }
         if (line.substr(0, 4) == "Done") {
-          // ESP_LOGD("custom", "Found Done - line is: %s", line.c_str());
-          // ESP_LOGD("custom", "Found Done - lastline is: %s", getline.c_str());
           // leapMMW:/>sensorStop
           if (getline.substr(0, 20) == "leapMMW:/>sensorStop") {
             // ESP_LOGD("custom", "sensorStop completed successfully");

packages/leapmmw_sensor.yml

@@ -1,11 +1,6 @@
 substitutions:
-  device_name: mmwave
-  ## FIXME: To be removed later. Would cause a breaking change.
-  uart_tx_pin: GPIO19
-  uart_rx_pin: GPIO18
-  ##
-  gpio_pin: GPIO23
-  header_file: leapmmw_sensor.h
+  # include the header file with the UART parsing logic 
+  header_file: header/leapmmw_sensor.h
 
 esphome:
   name: ${device_name}
@@ -16,14 +11,14 @@ logger:
   logs:
     sensor: INFO # reduced logging to minimize web_server target overload..
 
-## FIXME: To be removed later. Would cause a breaking change
 uart:
   id: uart_bus
   tx_pin: ${uart_tx_pin}
   rx_pin: ${uart_rx_pin}
   baud_rate: 115200
 
-##
+
+## sensor definitions
 
 binary_sensor:
   - platform: gpio
@@ -33,6 +28,13 @@ binary_sensor:
     pin:
       number: ${gpio_pin}
       mode: INPUT_PULLDOWN
+    # when motion is detected, the radar sensor is on so publish radar state to HA
+    on_press:
+      then:
+        lambda: !lambda |-
+          if (!id(mmwave_sensor).state) {
+            id(mmwave_sensor).publish_state(true);
+          }
     
 sensor:      
   - platform: custom
@@ -53,11 +55,9 @@ switch:
     optimistic: true
     turn_on_action:
       - uart.write: "setUartOutput 1 0"
-      - delay: 2s
-      - uart.write: "setUartOutput 2 1 1 2"
-      - delay: 2s
+      - delay: 1s
       - uart.write: "saveConfig"
-      - delay: 5s 
+      - delay: 4s 
       - uart.write: "sensorStart"
     turn_off_action:
       - uart.write: "sensorStop"

sensor.yaml

@@ -13,8 +13,6 @@ substitutions:
   uart_rx_pin: GPIO5
   # RX Pin configured for a D1 Mini stacked setup
   gpio_pin: GPIO16
-  # include the header file with the UART parsing logic 
-  header_file: header/leapmmw_sensor.h
 
 # include the yml file with the sensor configuration
 packages:
@@ -26,9 +24,6 @@ esphome:
 esp8266:
   board: d1_mini
 
-# Enable logging
-logger:
-
 # Enable Home Assistant API
 api:
   password: ""