Rachio update #7922
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,8 +8,8 @@ | |
| from homeassistant.components.switch import SwitchDevice, PLATFORM_SCHEMA | ||
| from homeassistant.const import CONF_ACCESS_TOKEN | ||
|
|
||
| REQUIREMENTS = ['https://github.com/rfverbruggen/rachiopy' | ||
| '/archive/989bd28ef5353be62e0eabb93b3339f74bc7ee44.zip' | ||
| '#rachiopy==0.1.1'] | ||
|
|
||
| _LOGGER = logging.getLogger(__name__) | ||
|
|
@@ -19,7 +19,7 @@ | |
| CONF_MANUAL_RUN_MINS = 'manual_run_mins' | ||
| DEFAULT_MANUAL_RUN_MINS = 10 | ||
|
|
||
| MIN_UPDATE_INTERVAL = timedelta(seconds=5) | ||
|
Contributor
Author
There was a problem hiding this comment.
Member
There was a problem hiding this comment. Rachio offers an API for free but hitting it every 5 seconds is going to get us banned.
Contributor
There was a problem hiding this comment. @balloob, I've been running mine with a 30 second interval and it seems to have the same desired effect, in that regardless of 5 or 30 seconds it updates the correct state on the frontend after about 30 seconds have passed. Do you think leaving timedelta at 30 seconds would be acceptable to not overload the API? |
||
| MIN_FORCED_UPDATE_INTERVAL = timedelta(seconds=1) | ||
|
|
||
| PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend({ | ||
|
|
@@ -30,6 +30,7 @@ | |
|
|
||
|
|
||
| # noinspection PyUnusedLocal | ||
| # noinspection PyUnresolvedLocal | ||
|
Member
There was a problem hiding this comment. Please remove these warnings. They are not part of linters that the project uses.
Contributor
Author
There was a problem hiding this comment.
Member
There was a problem hiding this comment. I don't understand this comment. |
||
| def setup_platform(hass, config, add_devices, discovery_info=None): | ||
| """Set up the component.""" | ||
| # Get options | ||
|
|
@@ -143,6 +144,7 @@ def list_zones(self, include_disabled=False): | |
| self.update(no_throttle=True) | ||
| return [z for z in self._zones if z.is_enabled] | ||
|
|
||
| # noinspection PyUnusedLocal | ||
| @util.Throttle(MIN_UPDATE_INTERVAL, MIN_FORCED_UPDATE_INTERVAL) | ||
| def update(self, **kwargs): | ||
| """Pull updated device info from the Rachio API.""" | ||
|
|
@@ -217,14 +219,11 @@ def update(self): | |
|
|
||
| def turn_on(self): | ||
| """Start the zone.""" | ||
| # Stop other zones first | ||
| self.turn_off() | ||
|
|
||
| _LOGGER.info("Watering %s for %d s", self.name, self._manual_run_secs) | ||
| self.rachio.zone.start(self.zone_id, self._manual_run_secs) | ||
|
|
||
| def turn_off(self): | ||
| """Stop all zones.""" | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Klikini I saw that you created this beautiful rachio component for home-assistant. If you need changes or help with the rachiopy library you can always send me a message!
I saw some messages to make the rachiopy component a pypi package, I'll look into that for you!Rachiopy is now a pypi package: https://pypi.python.org/pypi/RachioPy/0.1.1I use Home-Assistant myself for my home automation so I'm happy to see this implemented with my own API wrapper I once wrote.