romedtino / simple-wyze-vac

Home Assistant Custom Component for Wyze Vacuum
48 stars 8 forks source link

Not able to add additional wzye vacuum - configuration not loading #81

Open dwdonnelly opened 3 months ago

dwdonnelly commented 3 months ago

Describe the bug I am trying to add an additional wyze vacuum to the integration.

Reviewed documentation located here: https://github.com/romedtino/simple-wyze-vac/blob/main/README.md Could see no guidance on how to add additional wyze vacuum device.

I tried to "add device"...in the screen integration. Asked for wyze credentials - entered credentialls, stated wyze was already configured.

The following is from the log: Logger: homeassistant.config_entries Source: config_entries.py:586 First occurred: 2:38:39 PM (4 occurrences) Last logged: 2:45:17 PM

Error setting up entry Simple Wyze Vac for simple_wyze_vac Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 586, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/simple_wyze_vac/init.py", line 95, in async_setup_entry "filter": vac_info.supplies.filter.remaining, ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/wyze_sdk/models/devices/vacuums.py", line 970, in remaining return self.type.max_hours - self.usage


TypeError: unsupported operand type(s) for -: 'int' and 'NoneType'

**Simple Wyze Vac Version**
1.9.0

**Wyze Vacuum Firmware Version**
1.6.306

**Home Assistant Version**
Core   2024.7.3
Supervisor   2024.06.2
Operating System   12.4
Frontend   20240710.0

Your help is appreciated.
romedtino commented 3 months ago

1.6.306 is currently unsupported until wyze-sdk supports it. Shaun probably needs access to a vacuum with this version so he can figure out the correct endpoints for that verison

https://github.com/romedtino/simple-wyze-vac/issues/75 https://github.com/shauntarves/wyze-sdk/issues/130

dwdonnelly commented 3 months ago

@romedtino Great - so I have a new vacuum (firmware) that is not yet supported in the integration. Just my luck. OK - good info and I appreciate the feedback (very helpful)! Obviously, the next question is: any idea when it will be supported?

Also - as I was not able to add this new vacuum, I still have the problem of the integration failing to set up, the log entries provided above.

What steps should be taken to resolve this issue? Standing by.

dwdonnelly commented 3 months ago

@romedtino Before I jump and do something strange, should I remove and reload the simple wyze vac integration - or do you have a better suggestion? Thanks in advance for your help.

Ok - something strange just happened. I went to the simple wyze vac integration screen and clicked on the three dots on the far right and selected "reload" - like I have done a dozen times since this problem occurred and...wait for it.....the integration reloaded (like it always does)...but...wait for it....this time it actually reloaded without error. WOW! And ! My new third vacuum showed up (automatically!) So - yes, I am testing the setup.

I just tested all three vacs and they work as planned. Even the new vac with firmware: Wyze Vacuum Firmware Version 1.6.306

Again - I am amazed, I did nothing other than select the reload option....like I have done a dozen time. Go figure.

Any additional information you need on his issue? At this point, I am good to go.

Standing by.

Jizzay1 commented 2 months ago

I have two vacuums, one at 1.6.306 and one at 1.6.202. The integration accepts the login and keys, but produces the same error as the original poster, and neither vacuum loads. Happy to help if i can