Skip to content

Update Ecobee definitions#548

Open
sirwolfgang wants to merge 1 commit intoJc2k:mainfrom
sirwolfgang:patch/ecobee
Open

Update Ecobee definitions#548
sirwolfgang wants to merge 1 commit intoJc2k:mainfrom
sirwolfgang:patch/ecobee

Conversation

@sirwolfgang
Copy link
Copy Markdown

@sirwolfgang sirwolfgang commented Apr 5, 2026

This pull request updates and expands the Ecobee-specific characteristic definitions in characteristic_types.py. The main changes include correcting UUID assignments for several thermostat characteristics, improving documentation for scheduled change time handling, and adding new characteristics for equipment status and SmartSensor data.

Updates to Ecobee characteristic UUIDs and documentation:

  • Corrected UUID assignments for sleep/away heat and cool targets to match the Ecobee specification.
  • Improved and clarified documentation for VENDOR_ECOBEE_NEXT_SCHEDULED_CHANGE_TIME, including details on value formats, meaning, and correct write behavior for timed holds.

New Ecobee characteristic definitions:

  • Added new read-only characteristics for equipment running state, thermostat status code, and auxiliary heat activity.
  • Introduced SmartSensor-specific characteristics for reporting seconds since last motion and occupancy activations.

@sirwolfgang sirwolfgang changed the title Add new vendor characteristics and improve scheduled event documentation Update Ecobee definitions Apr 5, 2026
@bdraco
Copy link
Copy Markdown
Collaborator

bdraco commented Apr 19, 2026

Please explain the history and how the the existing ones were incorrect

@sirwolfgang
Copy link
Copy Markdown
Author

@bdraco I don't understand how I can explain the history, I didn't create the first version. The diff shows how they were swapped.

@bdraco
Copy link
Copy Markdown
Collaborator

bdraco commented Apr 28, 2026

@bdraco I don't understand how I can explain the history, I didn't create the first version. The diff shows how they were swapped.

Unfortunately without understanding why they were set to the UUIDs currently in use, I can't merge this PR because I don't know if its going to break older/different models.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants