tjhorner / upsy-desky

Make your standing desk smarter
https://upsy-desky.tjhorner.dev
Other
478 stars 24 forks source link

Target Height Entity Should Track Height #38

Open barndawgie opened 8 months ago

barndawgie commented 8 months ago

Troubleshooting Guide

Description

If I set the Target Height via the Target Height entity, then move the desk via some other mechanism (preset, physical controller) the target height stays at whatever value it was previously set to. I'd love to see it adjust to match the actual height of the desk (which is effectively the new target, just set via some other mechanism).

Logs

No response

Which version of the Upsy Desky firmware config are you running?

No response

How do you manage your Upsy Desky's updates?

ESPHome YAML Config

If you're using an ESPHome YAML config, which version of ESPHome are you running?

2023.9.3

ESPHome Config

---
substitutions:
  name: "upsy-desky-534044"
  friendly_name: "Upsy Desky 534044"
  device_description: "Fully Jarvis Desk Controller"

packages:
  base_config: !include config_deps/base_config.yaml
  upsy_desky:
    url: https://github.com/tjhorner/upsy-desky
    file: firmware/stock.yaml
    ref: v1.1.1

esphome:
  name: ${name}
  name_add_mac_suffix: false
  friendly_name: ${friendly_name}
  comment: ${device_description}
joshuaspence commented 6 months ago

I've done this in my configuration, which I based off this repo: https://github.com/joshuaspence/home-assistant-config/blob/master/esphome/devices/upsy_desky.yaml

I believe it was this change specifically: https://github.com/joshuaspence/home-assistant-config/commit/276733f4079255d1757fb47286da41570145fc30