klejejs / ha-thermia-heat-pump-integration

Thermia Heat Pump Integration for Home Assistant
GNU General Public License v3.0
30 stars 7 forks source link

Thermia integration stop working after upgrade of HACS to 1.24.3 #23

Closed HenrikBurton closed 2 years ago

HenrikBurton commented 2 years ago

Updated HACS to 1.24.3 and from there on there are no readings from my heatpump.

in the system CORE logg i get this message:

_2022-04-08 13:53:04 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 137, in _handle_refresh_interval await self._async_refresh(log_failures=True, scheduled=True) File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 268, in _async_refresh update_callback() File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 328, in _handle_coordinator_update self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 532, in async_write_ha_state self._async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 570, in _async_write_ha_state state = self._stringify_state(available) File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 538, in _stringify_state if (state := self.state) is None: File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 454, in state assert native_unit_ofmeasurement AssertionError

Any body else with the same issue?

klejejs commented 2 years ago

Hello! Did this break only after the HACS update or after the Home Assistant 2022.4.0 update? I assume it could be because the latest Home Assitant release changes how errors are perceived and as you had the error about getting temperature index, this might be now breaking the integration entirely. :(

I have unfortunately been very busy for the past few weeks, but I'll definitely look into it this weekend, so it starts working for you once again.

HenrikBurton commented 2 years ago

You are probably right, I upgraded Home Assistant some hours earlier.

Appreciera your help 🙏

jwictor81 commented 2 years ago

Got the same problem. No temperatures are shown.

Running Versions core-2022.4.1 supervisor-2022.03.5 Home Assistant OS 7.6

pumpan commented 2 years ago

Same here, after updating homeassistant.

And thx for a great integration!

klejejs commented 2 years ago

@HenrikBurton @jwictor81 @pumpan I just released an update that should fix the issue. Please check if temperatures are shown now.

pumpan commented 2 years ago

@HenrikBurton @jwictor81 @pumpan I just released an update that should fix the issue. Please check if temperatures are shown now.

Yes its working again! Amazing that one change in 1 Line can do so much😆how did you find out that it was the problem?

Again thx for a great integration!

jwictor81 commented 2 years ago

@HenrikBurton @jwictor81 @pumpan I just released an update that should fix the issue. Please check if temperatures are shown now.

Great work! Got them back once again.

Other question on the topic. I miss some sensors like return line. Is it known? BR

pumpan commented 2 years ago

@HenrikBurton @jwictor81 @pumpan I just released an update that should fix the issue. Please check if temperatures are shown now.

Great work! Got them back once again.

Other question on the topic. I miss some sensors like return line. Is it known? BR

Working for me.

jwictor81 commented 2 years ago

Strange. Only got 11 entities where these 4 temperatures are available.

heat_target_temperature hot_water_temperature outdoor_temperature supply_line_temperature

HenrikBurton commented 2 years ago

The fix seams to be working with all the sensors back and the error from issue #22 also disapered.

Great work @klejejs !

klejejs commented 2 years ago

Great to hear it's working! @jwictor81 Let's discuss your sensors in another issue.