tesla-local-control · iainbullock · Jan 15, 2025 · Jan 5, 2025 · Jan 5, 2025 · Jan 5, 2025
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,28 @@
 # Changelog
 
+## 0.4.1
+
+- RELEASE NEW Feature: Automatic Polling is now possible for state
+
+- NEW Feature: The following new states / entities are added:
+   - Sensors: Awake (updated approx every 30 secs from body-controller-state)
+   - Binary_Sensors: Presence_BC (experimental presence detection updated approx every 30 secs from body_controller_state rather than listening for BLE mac)
+   - Switches: Polling 
+   - Numbers: Polling Interval
+   - Buttons: Force Update buttons for individual state categories 
+
+- NEW Feature: Environment variable NO_POLL_SECTIONS is provided to disable updating of various state categories during polling. This speeds up state updates, though less state entities are updated by the polling. The entities can still be manually updated by pressing the Force Update button for the relevent state category
+
+- Changes:
+   - 'Force Data Update' Button is renamed to 'Force Update All'
+   - Errors which occur whilst reading state will not automatically prevent the next state or state category being read. This will fix an issue reported in #135 below where a user doesn't have a Heated Steering Wheel. This previously prevented any states after this one from being read
+
+- Fixes:
+   - [ Dev ] Bad variable name #75 (_docker). Many thanks to aneisch and dettofatto who really helped to track down this bug
+   - parse error: Invalid numeric literal #74 (_docker). Many thanks to jipema who also helped a lot in identifying this issue
+   - All my sensor entities are "unknown" #131 (_core)
+   - Any potential to "read state" via bluetooth? #115 (_core)
+
 ## 0.3.1
 
 - NEW Feature: The following new states / entities are added:

diff --git a/env.sh b/env.sh
@@ -35,6 +35,7 @@ export PRESENCE_DETECTION_LOOP_DELAY=${PRESENCE_DETECTION_LOOP_DELAY:-120}
 export PRESENCE_DETECTION_TTL=${PRESENCE_DETECTION_TTL:-240}
 export TEMPERATURE_UNIT_FAHRENHEIT=${TEMPERATURE_UNIT_FAHRENHEIT:-false}
 export MAX_CURRENT=${MAX_CURRENT:-48}
+export NO_POLL_SECTIONS=${NO_POLL_SECTIONS:-}
 
 export BLE_LN_REGEX='S[0-9A-Fa-f]{16}C'
 export BLTCTL_COMMAND_DEVICES=false
@@ -54,7 +55,8 @@ log_info "Configuration Options are:
   PRESENCE_DETECTION_TTL=$PRESENCE_DETECTION_TTL
   TEMPERATURE_UNIT_FAHRENHEIT=$TEMPERATURE_UNIT_FAHRENHEIT
   VIN_LIST=$VIN_LIST
-  MAX_CURRENT=$MAX_CURRENT"
+  MAX_CURRENT=$MAX_CURRENT
+  NO_POLL_SECTIONS=$NO_POLL_SECTIONS"
 
 [ -n "$ENABLE_HA_FEATURES" ] && log_info "  ENABLE_HA_FEATURES=$ENABLE_HA_FEATURES"
 [ -n "$BLECTL_FILE_INPUT" ] && log_info "  BLECTL_FILE_INPUT=$BLECTL_FILE_INPUT"

diff --git a/mqtt-discovery-sensors.sh b/mqtt-discovery-sensors.sh
@@ -141,8 +141,8 @@ function setupChargeStateSensors {
    "device_class": "voltage",
    "unit_of_measurement": "V",
    "suggested_display_precision": "0",
-   "unique_id": "'${DEVICE_ID}'_charger_voltage"
-   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_charger_voltage",
+   "enabled_by_default": "false"
   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 36 10 -t homeassistant/sensor/${DEVICE_ID}/charger_voltage/config -l
 
   echo '{
@@ -310,7 +310,7 @@ function setupChargeStateSensors {
   "qos": "'${QOS_LEVEL}'",
   "device_class": "temperature",
   "unit_of_measurement": "°C",
-  "suggested_display_precision": "1",   
+  "suggested_display_precision": "1",
   "unique_id": "'${DEVICE_ID}'_inside_temp"
   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 36 10 -t homeassistant/sensor/${DEVICE_ID}/inside_temp/config -l
 
@@ -711,6 +711,48 @@ function setupChargeStateSensors {
    "unique_id": "'${DEVICE_ID}'_mirror_heater"
   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 36 10 -t homeassistant/binary_sensor/${DEVICE_ID}/mirror_heater/config -l
 
+  echo '{
+   "state_topic": "'${TOPIC_ROOT}'/binary_sensor/awake",
+   "device": {
+    "identifiers": [
+    "'${DEVICE_ID}'"
+    ],
+    "manufacturer": "tesla-local-control",
+    "model": "Tesla_BLE",
+    "name": "'${DEVICE_NAME}'",
+    "sw_version": "'${SW_VERSION}'"
+   },
+   "platform": "binary_sensor",
+   "icon": "mdi:sleep-off",
+   "name": "Awake",
+   "payload_on": "true",
+   "payload_off": "false",
+   "qos": "'${QOS_LEVEL}'",
+   "unique_id": "'${DEVICE_ID}'_awake"
+  }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 36 10 -t homeassistant/binary_sensor/${DEVICE_ID}/awake/config -l
+
+  echo '{
+   "state_topic": "'${TOPIC_ROOT}'/binary_sensor/presence_bc",
+   "device": {
+    "identifiers": [
+    "'${DEVICE_ID}'"
+    ],
+    "manufacturer": "tesla-local-control",
+    "model": "Tesla_BLE",
+    "name": "'${DEVICE_NAME}'",
+    "sw_version": "'${SW_VERSION}'"
+   },
+   "platform": "binary_sensor",
+   "name": "Presence BC",
+   "payload_on": "true",
+   "payload_off": "false",
+   "qos": "'${QOS_LEVEL}'",
+   "device_class": "presence",
+   "icon": "mdi:car-connected",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_presence_bc"
+  }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 36 10 -t homeassistant/binary_sensor/${DEVICE_ID}/presence_bc/config -l
+
   # Status is only Disengaged for short time, it gets reengaged soon after even with no charger plugged in. Not useful
   # echo '{
   #  "state_topic": "'${TOPIC_ROOT}'/binary_sensor/charge_port_latch",

diff --git a/mqtt-discovery.sh b/mqtt-discovery.sh
@@ -281,6 +281,30 @@ function setupExtendedControls() {
    "unique_id": "'${DEVICE_ID}'_steering-wheel-heater"
    }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/switch/${DEVICE_ID}/steering-wheel-heater/config -l
 
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/variables/polling",
+   "state_topic": "'${TOPIC_ROOT}'/variables/polling",  
+   "device": {
+    "identifiers": [
+    "'${DEVICE_ID}'"
+    ],
+    "manufacturer": "tesla-local-control",
+    "model": "Tesla_BLE",
+    "name": "'${DEVICE_NAME}'",
+    "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:car-connected",
+   "name": "State Polling",
+   "device_class": "switch",
+   "payload_on": "on",
+   "payload_off": "off",
+   "state_on": "on",
+   "state_off": "off",
+   "retain": "true",
+   "unique_id": "'${DEVICE_ID}'_polling",
+   "entity_category": "diagnostic"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/switch/${DEVICE_ID}/polling/config -l
+
   # Covers
 
   echo '{
@@ -471,6 +495,29 @@ function setupExtendedControls() {
     }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/number/${DEVICE_ID}/climate-temp/config -l
   fi
 
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/variables/polling_interval",
+   "device": {
+    "identifiers": [
+    "'${DEVICE_ID}'"
+    ],
+    "manufacturer": "tesla-local-control",
+    "model": "Tesla_BLE",
+    "name": "'${DEVICE_NAME}'",
+    "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:timer-sync",
+   "min": "30",
+   "max": "3600",
+   "step": "30",
+   "mode": "slider",
+   "name": "Polling Interval",
+   "retain": "true",
+   "unique_id": "'${DEVICE_ID}'_polling_interval",
+   "entity_category": "diagnostic",
+   "unit_of_measurement": "secs"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/number/${DEVICE_ID}/polling_interval/config -l
+
   # Selects
 
   echo '{
@@ -710,6 +757,7 @@ function setupDiagnostic() {
     "entity_category": "diagnostic",
     "icon": "mdi:car-info",
     "name": "Car state information",
+    "enabled_by_default": "false",    
     "payload_press": "body-controller-state",
     "qos": "'${QOS_LEVEL}'",
     "unique_id": "'${DEVICE_ID}'_body-controller-state"
@@ -766,12 +814,132 @@ function setupDiagnostic() {
      "sw_version": "'${SW_VERSION}'"
    },
    "icon": "mdi:database-sync",
-   "name": "Force Data Update",
+   "name": "Force Update All",
    "entity_category": "diagnostic",
-   "payload_press": "read-state",
+   "payload_press": "read-state-all",
    "qos": "'${QOS_LEVEL}'",
    "unique_id": "'${DEVICE_ID}'_read-state"
-   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state/config -l
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-all/config -l
+
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/config",
+   "device": {
+     "identifiers": [
+     "'${DEVICE_ID}'"
+     ],
+     "manufacturer": "tesla-local-control",
+     "model": "Tesla_BLE",
+     "name": "'${DEVICE_NAME}'",
+     "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:database-sync",
+   "name": "Force Update Env",
+   "entity_category": "diagnostic",
+   "payload_press": "read-state-envcheck",
+   "qos": "'${QOS_LEVEL}'",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_read-state-envcheck"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-envcheck/config -l
+
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/config",
+   "device": {
+     "identifiers": [
+     "'${DEVICE_ID}'"
+     ],
+     "manufacturer": "tesla-local-control",
+     "model": "Tesla_BLE",
+     "name": "'${DEVICE_NAME}'",
+     "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:database-sync",
+   "name": "Force Update Charge",
+   "entity_category": "diagnostic",
+   "payload_press": "read-state-charge",
+   "qos": "'${QOS_LEVEL}'",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_read-state-charge"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-charge/config -l
+
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/config",
+   "device": {
+     "identifiers": [
+     "'${DEVICE_ID}'"
+     ],
+     "manufacturer": "tesla-local-control",
+     "model": "Tesla_BLE",
+     "name": "'${DEVICE_NAME}'",
+     "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:database-sync",
+   "name": "Force Update Climate",
+   "entity_category": "diagnostic",
+   "payload_press": "read-state-climate",
+   "qos": "'${QOS_LEVEL}'",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_read-state-climate"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-climate/config -l
+
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/config",
+   "device": {
+     "identifiers": [
+     "'${DEVICE_ID}'"
+     ],
+     "manufacturer": "tesla-local-control",
+     "model": "Tesla_BLE",
+     "name": "'${DEVICE_NAME}'",
+     "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:database-sync",
+   "name": "Force Update Tyre",
+   "entity_category": "diagnostic",
+   "payload_press": "read-state-tyre",
+   "qos": "'${QOS_LEVEL}'",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_read-state-tyre"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-tyre/config -l
+
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/config",
+   "device": {
+     "identifiers": [
+     "'${DEVICE_ID}'"
+     ],
+     "manufacturer": "tesla-local-control",
+     "model": "Tesla_BLE",
+     "name": "'${DEVICE_NAME}'",
+     "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:database-sync",
+   "name": "Force Update Closure",
+   "entity_category": "diagnostic",
+   "payload_press": "read-state-closure",
+   "qos": "'${QOS_LEVEL}'",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_read-state-closure"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-closure/config -l
+
+  echo '{
+   "command_topic": "'${TOPIC_ROOT}'/config",
+   "device": {
+     "identifiers": [
+     "'${DEVICE_ID}'"
+     ],
+     "manufacturer": "tesla-local-control",
+     "model": "Tesla_BLE",
+     "name": "'${DEVICE_NAME}'",
+     "sw_version": "'${SW_VERSION}'"
+   },
+   "icon": "mdi:database-sync",
+   "name": "Force Update Drive",
+   "entity_category": "diagnostic",
+   "payload_press": "read-state-drive",
+   "qos": "'${QOS_LEVEL}'",
+   "enabled_by_default": "false",
+   "unique_id": "'${DEVICE_ID}'_read-state-drive"
+   }' | sed ':a;N;$!ba;s/\n//g' | retryMQTTpub 6 10 -t homeassistant/button/${DEVICE_ID}/read-state-drive/config -l
 
   log_debug "setupDiagnostic() leaving vin:$vin"
 
@@ -794,12 +962,13 @@ setupHADiscoveryAllVINsMain() {
     else
       log_info "setupHADiscoveryAllVINsMain; HA backend is disable, skipping setup for HA Auto Discovery"
     fi
+
+    # Discard /config and /command messages
+    log_notice "setupHADiscoveryAllVINsMain; Discarding unread MQTT messages for command and config topics"
+    eval $MOSQUITTO_PUB_BASE -t tesla_ble/$vin/config -n
+    eval $MOSQUITTO_PUB_BASE -t tesla_ble/$vin/command -n
   done
 
-  # Discard /config messages
-  topic=tesla_ble/$vin/config
-  log_notice "setupHADiscoveryAllVINsMain; Discarding any unread MQTT messages for topic:$topic"
-  eval $MOSQUITTO_SUB_BASE -E -i tesla_ble_mqtt -t $topic
 }
 
 ###