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

Update for vacuum.peyrebelle_pool_robot fails #26

Open
tmenguy opened this issue Jul 5, 2024 · 9 comments
Open

Update for vacuum.peyrebelle_pool_robot fails #26

tmenguy opened this issue Jul 5, 2024 · 9 comments

Comments

@tmenguy
Copy link

tmenguy commented Jul 5, 2024

Hi,

sorry again, another one:

Logger: homeassistant.helpers.entity
Source: helpers/entity.py:944
First occurred: July 4, 2024 at 11:46:06 PM (15 occurrences)
Last logged: 11:34:10 AM

Update for vacuum.peyrebelle_pool_robot fails
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 944, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1296, in async_device_update
await self.async_update()
File "/config/custom_components/iaqualinkRobots/vacuum.py", line 248, in async_update
self._status = data['payload']['robot']['state']['reported']['aws']['status']
~~~~~~~~~~~~~~~^^^^^^^^^
KeyError: 'robot'

@tmenguy
Copy link
Author

tmenguy commented Jul 5, 2024

Robot : RA 6900 iQ
Well connected and accessible

@galletn
Copy link
Owner

galletn commented Jul 5, 2024

would you maybe be able to change your password and send me over your credentials to [email protected] ?

I think I will be able to quick fix it if I see what is returned in your case as the other models seem to work fine so far with the code.

@tmenguy
Copy link
Author

tmenguy commented Jul 8, 2024

Hi,

I’m out for 2 weeks will see when coming back, thanks for the support !

Thomas

@galletn
Copy link
Owner

galletn commented Aug 1, 2024

should be fixed with the new code, added another try catch on that part can you confirm?

@tmenguy
Copy link
Author

tmenguy commented Aug 2, 2024

fantastic, will try asap ...

@galletn
Copy link
Owner

galletn commented Aug 24, 2024

any update on this one?

@tmenguy
Copy link
Author

tmenguy commented Aug 27, 2024

hey sorry about not coming back : I don't have the HA error above anymore .. I do have some timeout to get the status of the robot (see below) but I do thinks this particular issue can be closed ! thanks a lot!

logger: homeassistant.helpers.entity
Source: helpers/entity.py:942
First occurred: 5:51:08 PM (1 occurrences)
Last logged: 5:51:08 PM

Update for vacuum.peyrebelle_pool_robot fails
Traceback (most recent call last):
File "/usr/local/lib/python3.12/asyncio/tasks.py", line 520, in wait_for
return await fut
^^^^^^^^^
File "/config/custom_components/iaqualinkRobots/vacuum.py", line 465, in get_device_status
message = await websocket.receive()
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/client_ws.py", line 314, in receive
msg = await self._reader.read()
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/streams.py", line 681, in read
return await super().read()
^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/streams.py", line 640, in read
await self._waiter
asyncio.exceptions.CancelledError

@tmenguy tmenguy closed this as completed Aug 27, 2024
@tmenguy
Copy link
Author

tmenguy commented Aug 27, 2024

oh no it is back:

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 942, in async_update_ha_state
await self.async_device_update()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1300, in async_device_update
await self.async_update()
File "/config/custom_components/iaqualinkRobots/vacuum.py", line 312, in async_update
self._status = data['payload']['robot']['state']['reported']['aws']['status']

KeyError: 'robot'

@tmenguy tmenguy reopened this Aug 27, 2024
@tmenguy
Copy link
Author

tmenguy commented Aug 30, 2024

Forked the project and I a tried to debug…. But couldn’t reproduce for now (obviously :) ). But seeing the code it can be normal to crash here as the retry is not protected . Will see if I had time to issue a PR to protect it a bit more.

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

No branches or pull requests

2 participants