Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Errors in HA system log #198

Closed
SanderM2 opened this issue Sep 25, 2022 · 2 comments · Fixed by #229
Closed

Errors in HA system log #198

SanderM2 opened this issue Sep 25, 2022 · 2 comments · Fixed by #229
Milestone

Comments

@SanderM2
Copy link

SanderM2 commented Sep 25, 2022

After installing thermal comfort using HACS and configuring it using the temperature + humidity sensor of my ComfoAir ventilation system (which measures outside temperature + humidity) I keep getting errors like this in my HA system log:

2022-09-25 11:40:18.172 ERROR (MainThread) [homeassistant.helpers.entity] Update for sensor.thermal_comfort_outside_using_comfoair_perception fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 515, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 710, in async_device_update
raise exc
File "/config/custom_components/thermal_comfort/sensor.py", line 446, in async_update
value = await getattr(self._device, self._sensor_type)()
File "/config/custom_components/thermal_comfort/sensor.py", line 261, in wrapped
setattr(self, f"_{sensor_type}", await func(self, *args, **kwargs))
File "/config/custom_components/thermal_comfort/sensor.py", line 631, in thermal_perception
dewpoint = await self.dew_point()
File "/config/custom_components/thermal_comfort/sensor.py", line 261, in wrapped
setattr(self, f"_{sensor_type}", await func(self, *args, **kwargs))
File "/config/custom_components/thermal_comfort/sensor.py", line 597, in dew_point
Td = math.log(VP / 0.61078)
ValueError: math domain error

Note: the sensor does seem to work. At least, the entities show values that look correct...

@SanderM2
Copy link
Author

I think this can be closed already. It turns out my comforair sensors are sometimes dropping to 0 for an unknown reason and the timestamps match the timestamps of the errors I see here.

I'll have to dig into the 'comfoconnect' integration to fix the issue causing the values to drop to 0 first ...

@rautesamtr
Copy link
Collaborator

Let's keep this open till we provide a nicer error message in this case. Thanks for the report.

Repository owner deleted a comment from bbr111 Dec 10, 2022
@rautesamtr rautesamtr added this to the 2.0 milestone Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants