-
Notifications
You must be signed in to change notification settings - Fork 131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ecobee3 support #653
Merged
Merged
Add ecobee3 support #653
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sync to upstream Master
Very limited functionality at present and much more needs to be added to make this usable. Currently supports -PIN request and token authentication -Polling for changes to runtime and sensor data Todo -More error checking and appropriate recovery -Add function to update configuration when the revision number changes -Add many more user functions to get/set the data -Add remaining "user friendly" classes to access the data -Clean up and reduce the verbosity of logs when not running in debug mode
Removed most of the unused code from the original module that this was based on
Also added ability to get to the system level properties actualHumidity and actualTemperature which are aggregated from the sensors depending on how things are setup.
Also fixed a bug related to deleting alerts
Functions to populate and check $monitor_hash are not complete/working yet
…e actualTemperature property changes This still needs a lot of work.
This allows children to register for changes for any property at any depth in the data hash
This child object allows you to track the state of the actualHumidity property to be utilized within user code.
This child object allows you to track the status of the thermostat (by means of a vector) to be utilized within user code.
This child object allows you to track and change the mode of the thermostat within user code.
This child object allows you to track the current climate value (away/home/sleep) of the thermostat and use it within user code.
We were not handling the CSV correctly when more than one state was active (e.g. auxHeat1 and fan)
Reduced verbosity of logging when verbose logging is disabled in MH.
Hello @rudybrian, nice, thanks for contributing this code. Merging into the master branch so that it is available in the next stable release. Best regards, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds MisterHouse support for the ecobee3 thermostat and provides some of the ecobee API interaction framework necessary for supporting other Ecobee thermostat models like the ecobee3 lite, Smart Si, EMS, EMS Si and white label Carrier Cor and Bryant Housewise (but these models are not yet supported)
I have been working on this on and off for most for the year, and still don't consider it complete, but the majority of the functionality is in place for folks to make use and improve upon it.