search

hllhll / HomeAssistant-EKON-iAircon

EKON iAircon / Tadiran climate component written in Python3 for Home Assistant. On the bases of Gree Climate component for easier interfacing with HASS
GNU General Public License v3.0
14 stars 7 forks source link

Error while setting up Ekon #12

Closed liorm0505 closed 4 years ago

liorm0505 commented 4 years ago

Describe the bug component not loading after updating to the latest commit

Configuration

climate:

  - platform: ekon
    username: xxx
    password: xxx
    #base_url: https://www.airconet.xyz/
    base_url: https://www.activate-ac.com/
    # name_mapping: 
    #   - id : 2258
    #     name: Living Room AC

Platform:

Error while setting up ekon platform for climate
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 184, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/ekon/climate.py", line 189, in async_setup_platform
    await controller.async_load_init_data()
  File "/config/custom_components/ekon/climate.py", line 223, in async_load_init_data
    newdev = EkonClimate(self, dev_raw['mac'], dev_raw['id'],
  File "/config/custom_components/ekon/climate.py", line 433, in __init__
    self.cpy_current_to_set_to()
  File "/config/custom_components/ekon/climate.py", line 437, in cpy_current_to_set_to
    self._set_values._current_humidity = self._current_humidity
AttributeError: 'EkonClimate' object has no attribute '_current_humidity'
hllhll commented 4 years ago

Thank you, interesting note, Are you using tadiran connect? Airconet? or Airconet+?

For Airconet+ there's PR waiting to be merged https://github.com/hllhll/HomeAssistant-EKON-iAircon/pull/10

liorm0505 commented 4 years ago

Airconnect+

Actually i only realized i was using Airconnet+ after i opened the issue (copied the code from the PR) . So currently it is not related to you code. However, this might be relevant if you merge.

hllhll commented 4 years ago

Relevant to the fork with the support for the humidity-enriching entity. I Don't think I'll merge that any time soon... Can't it be done with other HA tooks to combine the AC and the Humidity sensor instead of combining it in code?