search

Legomaniac / hass_lacrosse_alerts_mobile

A Home Assistant component for LaCrosse Alerts Mobile
1 stars 6 forks source link

Stop Working when HA switched to 3.11 Python. #3

Open yozh opened 1 year ago

yozh commented 1 year ago

Hello,

This was working perfect, until the update of HA to the latest version where support for 3.10 was dropped.

`Logger: homeassistant.helpers.entity Source: custom_components/lacrosse_alerts_mobile/sensor.py:78 Integration: lacrosse_alerts_mobile (documentation) First occurred: 12:35:44 PM (94 occurrences) Last logged: 1:25:47 PM

Update for sensor.lacrosse_probe_temp fails Traceback (most recent call last): File “/usr/src/homeassistant/homeassistant/helpers/entity.py”, line 699, in async_update_ha_state await self.async_device_update() File “/usr/src/homeassistant/homeassistant/helpers/entity.py”, line 940, in async_device_update await hass.async_add_executor_job(self.update) File “/usr/local/lib/python3.11/concurrent/futures/thread.py”, line 58, in run result = self.fn(*self.args, **self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File “/config/custom_components/lacrosse_alerts_mobile/sensor.py”, line 78, in update obs = self._lacrosse_device.getObservation(2) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File “/usr/local/lib/python3.11/site-packages/pylacrossapi/init.py”, line 72, in getObservation return json.loads(page)[“device0”][“obs”]


KeyError: ‘obs’`
yozh commented 1 year ago

Anyone can help ?

Legomaniac commented 1 year ago

Hello @yozh !

Honestly, I wasn't aware that this still had any users, ha. I haven't had it working for a while, I believe some Home Assistant update broke it on my installation, and life just got busy, so I never fixed it. I was just using it to monitor my hot tub temp.

Knowing that there's other people using this has me interested in fixing this project, but I don't know where it's going to fit in quite yet. Once I get to it though, I'll make sure to update this thread.

Thanks, @Legomaniac

yozh commented 1 year ago

Hello @Legomaniac,

Appreciate the reply and yes very much interested in this, as I was using it for a while until the update broke it. When every you can spare time on this would be appreciated it! I use this for the pool, and like having this one then others because of the screens visible outside cycling the temp.