feat!: add support for solar systems and powerwall (#341) #353
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
- New module `car.py` that contains a `TeslaCar` class. - New module `energy.py` that contains energy site classes. - New method `Controller.generate_car_objects` that generates `TeslaCar` objects and stores them into `self.cars` by vin. - New method `Controller.generate_energysite_objects` that generates `SolarSite`, `PowerwallSite` or `SolarPowerwallSite` objects and stores them in the `self.energysites` dictionary by `energysite_id`. - Modified `Controller.update` to now just send a single request to the `PRODUCT_LIST` endpoint (instead of both `PRODUCT_LIST` and `VEHICLE_LIST` to get all products on a Tesla account. From there, create a list of cars `self._vehicle_list` and energy sites `self._energysite_list`. - Storing JSON responses as-is into their own dictionary by VIN for cars and energysite_id for energysites. These are then passed to `TeslaCar` and `EnergySite` when instantiated. - Added `include_vehicles` and `include_energysites` arguments to `Controller.connect` which default to True. This provides the option to completely ignore vehicles or energysites. - Removed `Controller.command`, `Controller.get` and `Controller.post` methods. Only `Controller._wake_up` was using `post` which now uses `Controller.api`. - Removed all Home Assistant specific modules. - Removed some unused code specific to energy sites. - Changes to some naming to try to better align with what the Tesla API uses. closes #348 closes #334 closes #24 BREAKING CHANGE: HomeAssistant specific code has been moved out. The API is now just a communication layer Co-authored-by: Alan D. Tse <[email protected]>
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.
car.pythat contains aTeslaCarclass.energy.pythat contains energy site classes.Controller.generate_car_objectsthat generatesTeslaCarobjects and stores them intoself.carsby vin.Controller.generate_energysite_objectsthat generatesSolarSite,PowerwallSiteorSolarPowerwallSiteobjects and stores them in theself.energysitesdictionary byenergysite_id.Controller.updateto now just send a single request to thePRODUCT_LISTendpoint (instead of bothPRODUCT_LISTandVEHICLE_LISTto get all products on a Tesla account. From there, create a list of carsself._vehicle_listand energy sitesself._energysite_list.TeslaCarandEnergySitewhen instantiated.include_vehiclesandinclude_energysitesarguments toController.connectwhich default to True. This provides the option to completely ignore vehicles or energysites.Controller.command,Controller.getandController.postmethods. OnlyController._wake_upwas usingpostwhich now usesController.api.closes #348
closes #334
closes #24
BREAKING CHANGE: HomeAssistant specific code has been moved out. The API is now just a communication layer
Co-authored-by: Alan D. Tse [email protected]