home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.91k stars 30.98k forks source link

Incomfort climate entity not created #24478

Closed Michelli closed 5 years ago

Michelli commented 5 years ago

Upgraded to 0.94.1, which is supposed to add a climate entity for the Incomfort/Intergas LAN2RF gateway. While the water_heater.boiler entity is added, the climate component is not.

The log doesn't show any information relating to the climate entities and/or the incomfort component.

My configuration.yaml only contains the host ip.

ghost commented 5 years ago

Hey there @zxdavb, mind taking a look at this issue as its been labeled with a integration (incomfort) you are listed as a codeowner for? Thanks!

This is a automatic comment generated by codeowners-mention to help ensure issues and pull requests are seen by the right people.

zxdavb commented 5 years ago

@Michelli I wrote the integration, and the corresponding client library. I will have a look for you.

zxdavb commented 5 years ago

@Michelli

I am sorry, it still works fro me (but I don't have a thermostat).

Could you provide the response from the following command:

cat ~/.homeassistant/home-assistant.log | grep incomfort | grep homeassistant

It may be easier to do this immediately after a restart.

In addition, could you add the following to the logger: section of you configuration.yaml file and provide me with the output:

logger:
...
  logs:
    incomfortclient: debug

The output I need can be obtained via:

cat ~/.homeassistant/home-assistant.log | grep incomfortclient | head -n 15

In particular, I am interested in a debug line that looks a lot like this:

2019-06-15 12:46:03 DEBUG (MainThread) [incomfortclient] _get(url=http://cr-router.bonnes.me/data.json?heater=0): response = {'nodenr': 200, 'ch_temp_lsb': 160, 'ch_temp_msb': 11, 'tap_temp_lsb': 84, 'tap_temp_msb': 10, 'ch_pressure_lsb': 114, 'ch_pressure_msb': 0, 'room_temp_1_lsb': 255, 'room_temp_1_msb': 127, 'room_temp_set_1_lsb': 70, 'room_temp_set_1_msb': 5, 'room_temp_2_lsb': 255, 'room_temp_2_msb': 127, 'room_temp_set_2_lsb': 255, 'room_temp_set_2_msb': 127, 'displ_code': 126, 'IO': 0, 'serial_year': 17, 'serial_month': 9, 'serial_line': 29, 'serial_sn1': 2, 'serial_sn2': 30, 'serial_sn3': 82, 'room_set_ovr_1_msb': 5, 'room_set_ovr_1_lsb': 70, 'room_set_ovr_2_msb': 2, 'room_set_ovr_2_lsb': 88, 'rf_message_rssi': 31, 'rfstatus_cntr': 0}
zxdavb commented 5 years ago

@NdS-Research-Facilities, @royduin

Do either of you have a room thermostat? If so, are you getting the above issue? If so, could you provide me with the debugging info requested above?

royduin commented 5 years ago

I've and I'm using the room thermostat but I'm still running on 0.91.4 with my own integration. Haven't had the time yet to upgrade and test the new integration.

zxdavb commented 5 years ago

OK, would love your feedback, anyway - I've since found the bug & pushed a fix.

zxdavb commented 5 years ago

BTW, this is in the pipeline too: #23812 - I'll get around to it again this weekend.

Michelli commented 5 years ago

Could you provide the response from the following command:

cat ~/.homeassistant/home-assistant.log | grep incomfort | grep homeassistant

It may be easier to do this immediately after a restart.

2019-06-15 06:10:43 WARNING (MainThread) [homeassistant.components.incomfort.water_heater] Update failed, message: File "/usr/src/homeassistant/homeassistant/components/incomfort/water_heater.py", line 91, in async_update File "/usr/src/homeassistant/homeassistant/components/incomfort/water_heater.py", line 91, in async_update

The output from the logger is: 2019-06-16 00:33:06 DEBUG (MainThread) [incomfortclient] _get(url=http://192.168.0.8/data.json?heater=0): response = {'nodenr': 146, 'ch_temp_lsb': 136, 'ch_temp_msb': 20, 'tap_temp_lsb': 235, 'tap_temp_msb': 15, 'ch_pressure_lsb': 100, 'ch_pressure_msb': 0, 'room_temp_1_lsb': 92, 'room_temp_1_msb': 9, 'room_temp_set_1_lsb': 8, 'room_temp_set_1_msb': 7, 'room_temp_2_lsb': 255, 'room_temp_2_msb': 127, 'room_temp_set_2_lsb': 255, 'room_temp_set_2_msb': 127, 'displ_code': 126, 'IO': 0, 'serial_year': 17, 'serial_month': 3, 'serial_line': 16, 'serial_sn1': 0, 'serial_sn2': 45, 'serial_sn3': 11, 'room_set_ovr_1_msb': 7, 'room_set_ovr_1_lsb': 8, 'room_set_ovr_2_msb': 7, 'room_set_ovr_2_lsb': 208, 'rf_message_rssi': 32, 'rfstatus_cntr': 0}