Skip to content

bump ecobee version to fix issue 9190#9191

Merged
dale3h merged 1 commit into
home-assistant:devfrom
nkgilley:fix-ecobee
Aug 28, 2017
Merged

bump ecobee version to fix issue 9190#9191
dale3h merged 1 commit into
home-assistant:devfrom
nkgilley:fix-ecobee

Conversation

@nkgilley
Copy link
Copy Markdown
Contributor

Related issue (if applicable): fixes #9190

@Vasiley
Copy link
Copy Markdown

Vasiley commented Aug 28, 2017

Thanks for looking at so quick... The hold mode for the thermo set my whole house to that mode light, alarms and etc..... If there is anything I can do let me know

@n0dyjeff
Copy link
Copy Markdown

n0dyjeff commented Aug 28, 2017

After installing this, I get the following error:

2017-08-28 10:58:26 ERROR (MainThread) [homeassistant.helpers.entity] Update for climate.downstairs fails
Traceback (most recent call last):
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/helpers/entity.py", line 225, in async_update_ha_state
    yield from self.hass.async_add_job(self.update)
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/climate/ecobee.py", line 126, in update
    self.data.update(no_throttle=True)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/util/__init__.py", line 306, in wrapper
    result = method(*args, **kwargs)
TypeError: update() got an unexpected keyword argument 'no_throttle'

@dale3h
Copy link
Copy Markdown
Member

dale3h commented Aug 28, 2017

@n0dyjeff I am unable to reproduce your latest error. How did you go about installing "this?"

I opened /srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/ecobee.py and simply changed REQUIREMENTS = ['python-ecobee-api==0.0.8'] to REQUIREMENTS = ['python-ecobee-api==0.0.9'], restarted Home Assistant, and all works well again:

2017-08-28 14:07:16 INFO (MainThread) [homeassistant.loader] Loaded ecobee from homeassistant.components.ecobee
2017-08-28 14:07:27 INFO (Thread-7) [homeassistant.util.package] Attempting install of python-ecobee-api==0.0.9
2017-08-28 14:07:37 INFO (MainThread) [homeassistant.setup] Setting up ecobee
2017-08-28 14:07:37 INFO (MainThread) [homeassistant.core] Bus:Handling <Event platform_discovered[L]: discovered=, platform=ecobee, service=load_platform.binary_sensor>
2017-08-28 14:07:37 INFO (MainThread) [homeassistant.setup] Setup of domain ecobee took 0.4 seconds.
2017-08-28 14:07:37 INFO (MainThread) [homeassistant.core] Bus:Handling <Event component_loaded[L]: component=ecobee>
2017-08-28 14:07:37 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_registered[L]: service=ecobee, domain=notify>
2017-08-28 14:07:45 INFO (MainThread) [homeassistant.core] Bus:Handling <Event platform_discovered[L]: discovered=, platform=ecobee, service=load_platform.sensor>
2017-08-28 14:07:58 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_registered[L]: service=ecobee_set_fan_min_on_time, domain=climate>
2017-08-28 14:07:58 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_registered[L]: service=ecobee_resume_program, domain=climate>

@n0dyjeff
Copy link
Copy Markdown

I used the following command to install HA 0.52.1 over 0.52.0:
pip3 install --upgrade homeassistant

@n0dyjeff
Copy link
Copy Markdown

One difference I see is that I'm using python 3.4. The path to the file on my system is:
/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components.ecobee.py
and the REQUIREMENTS statement there does show 0.0.9

@nkgilley
Copy link
Copy Markdown
Contributor Author

I'm not sure about the throttle code, I don't think anything changed though.

@n0dyjeff
Copy link
Copy Markdown

I tried this again and am unable to reproduce the error. The original error was just after startup so possibly it was related to a startup timing sequence. In an case, it's working well at present. Sorry for the false alarm.

@dale3h
Copy link
Copy Markdown
Member

dale3h commented Aug 28, 2017

@n0dyjeff Thank you for reporting back. I will go and upgrade to 0.52.1 right now.

@Vasiley
Copy link
Copy Markdown

Vasiley commented Aug 28, 2017

Works great now THANKS on 52.1

@nkgilley nkgilley deleted the fix-ecobee branch August 29, 2017 12:14
@home-assistant home-assistant locked and limited conversation to collaborators Dec 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ecobee version bump to 0.0.8 cause failure is set hold mode @nkgilley

6 participants