Add additional sensors and settings to Roborock vacuums by starkillerOG · Pull Request #1543 · rytilahti/python-miio

starkillerOG · 2022-09-30T19:12:29Z

Add additional sensors and improve the already implemented sensors for the Roborock S7 MaxV.

Change the decorator ids to qualified name instead of simply name in order to prevent unique_id collisions in HomeAssistant for example with DNDStatus.start and CleaningDetails.start

Implement choices_attribute for setting decorator such that Enums can be evaluated at runtime by retrieving it from the device.

Split vacuum enums to a separte file to prevent circular imports

Controls added:

Mop routing (enum)
Mop scrub intensity (enum)
Mulit map (which floor is active, enum)
Auto dust collection (switch)
Buttons added:
start dust collection
stop dust collection
sensors added:
floor 0-4 clean start (when the last clean was per floor)
Last clean map name

Fanspeed values for the Roborock S7 MaxV are 101, 102, 103, 104, 108 which are already coverd by the vacuum entity select. Percentage based values from 0 to 100 are not supported

Since it is already included in the vacuum entity

Change id to qualified name. Otherwise DNDStatus.start and CleaningDetails:start get the same unique ID, causing problems in HomeAssistant

crshray · 2022-11-05T04:31:37Z

This is great! Any idea if the clean & dirty water tank status is exposed?

@craigcabrey I did not see that in the status response, but I have a s7MaxV+ (auto-empty dock) not the Ultra with the Ultra dock, so I can't test the water tank status. Do you see the clean&dirty tank status in the miio home app?

I tested with my s7 MaxV ultra and it looks like clean&dirty tank status is exposed via the dock_error_status field in the get_status response, if clean water tank is empty, dock_error_status will have a value of 38, if dirty water tank is full, dock_error_status will have a value of 39, if both errors happen at the same time, dock_error_status will have a value of 38 (i.e. clean water tank error takes precedence over the dirty water tank error).

craigcabrey · 2022-11-05T05:18:35Z

This is great! Any idea if the clean & dirty water tank status is exposed?

@craigcabrey I did not see that in the status response, but I have a s7MaxV+ (auto-empty dock) not the Ultra with the Ultra dock, so I can't test the water tank status. Do you see the clean&dirty tank status in the miio home app?

Missed this, sorry! Yea as noted by @crshray it is available in the app.

starkillerOG · 2022-11-09T17:43:37Z

@crshray thanks for the info, I added a dock_error and dock_error_code sensor in the latest commit.
Unfortunately I could not find a list of "dock_error_status" codes, so I only added 0, 38 and 39 for now.

Split out from #1543 Will be used for Multi Map Enum select.

h3rbst · 2022-11-28T16:12:05Z

This looks really great! Do you know if it would be possible to start the additional Dryer Module as well? I am using the S7 Pro Ultra where the Dryer currently is not supported by the app. Would be nice if it is possible via Home Assistant.

@h3rbst Xioami Miio is based on the protocol used by the xiaomi miio app, so if it is not available in the xiaomi miio app, it will not be possible to implement in HomeAssistant. Starting a dust collection will be added as a button (but that is also possible in the xiaomi miio app).

So I found out, that the dryer setting is now available in the Xiaomi Home App. Is it possible to add this setting to Home Assistant? (see last point in attached picture)

starkillerOG · 2022-11-28T16:23:04Z

@h3rbst yes that is certainly possible, but I would need to have decoded packet captures to figure out the commands that you would like to have added to HomeAssistant (preferablly just capture as much as possible).
See the discussion here: home-assistant/core#69869

I only have the auto-empty doc, not the ultra-doc. So I cannot capture the washing related settings myself.

starkillerOG · 2022-12-05T20:09:44Z

Now that the last piece of this PR has been split out here: #1614
This PR can be closed.

h3rbst · 2022-12-09T08:53:50Z

@starkillerOG is it possible to activate this for the S7 Pro Ultra as well? The S7 Pro Ultra has the same Dock as the MaxV

starkillerOG · 2022-12-09T09:06:51Z

@h3rbst These features are added for all models that support them.
Although it is still in dev and not released yet.
We first need a new python-miio release.

h3rbst · 2022-12-09T09:12:13Z

@h3rbst These features are added for all models that support them. Although it is still in dev and not released yet. We first need a new python-miio release.

Ah okay, I see. I thought, since the PR is already merged, that it is part of the new December release.

xsasx · 2023-07-13T12:06:38Z

@starkillerOG Sorry for maybe my stupidness - as i understand you rigt - i should have with the normal miio integration in homeassistant the feature that you created with the MOP intensity etc ? Cause i have it not with my Roborock S7 or do i need to change any file under my Homeassistant server with your files (replace)

starkillerOG · 2023-07-23T20:13:02Z

@xsasx No that needs a new version of Python-miio to be released.
I have been waiting on a new version for a long time...

Maybe @rytilahti could shed some light on when a new version would be expected.

neforce · 2023-08-16T08:51:21Z

i'm experiencing sometimes trouble loading the map of the Xiaomi app. However, roborock app itself is working will. But when i use roborock app, i miss a lot of sensors which i'm using now. If i read this thread well, is it true that i can use roborock app on my phone, and use the roborock integration of Home Assistant after this commit in this issue has been merged?

JR2012thman · 2023-10-12T16:29:50Z

@xsasx No that needs a new version of Python-miio to be released. I have been waiting on a new version for a long time...

Maybe @rytilahti could shed some light on when a new version would be expected.

Is there any movement on this? I am happy to send commands as needed but this would be a great set of features for those who are less technical.

starkillerOG added 30 commits September 30, 2022 16:09

Remove Fanspeed

c808adf

Fanspeed values for the Roborock S7 MaxV are 101, 102, 103, 104, 108 which are already coverd by the vacuum entity select. Percentage based values from 0 to 100 are not supported

Fix unit

fc8c7b4

capital letters

a63270a

disable battery by default

b040a7c

Since it is already included in the vacuum entity

Add sensors

bc09260

fix typo

c766026

adjust sensors

383de8b

update icons

92dae6e

Add last_clean_details

444f8ed

Add DnD status

2c3b4b3

Change id to qualified name. Otherwise DNDStatus.start and CleaningDetails:start get the same unique ID, causing problems in HomeAssistant

use qualified_name for id

f7cbb95

add multi map id

43e7a3e

Update vacuumcontainers.py

33faf82

Add mullti_map_id to clean history

c32b24a

fix type

7488ea9

Add retrieving multi_maps_list

32d0028

add dust_collection_work_times

73fa561

add load_multi_map

60e6d70

add Multi map selector

d995d26

fix typo

a4e6dc4

make changing multi map work in homeassistant

5cc7664

Add Mop scrub intensity and mop route

b3ecd75

fix circiler import

a164846

update icons

0c0a617

Add auto_dust_collection

68a4fb4

Last clean per floor

00dfae9

fixes

4d9ca0e

add entity_categories

6c8bb0e

add device_classes

e490cd4

add state_class

298a485

Add dock_error and dock_error_code

180dd5c

Merge branch 'master' into roborock_sensors

cc7d4ea

starkillerOG mentioned this pull request Nov 9, 2022

Add additional sensors and settings to Roborock vacuums #1585

Merged

starkillerOG added 2 commits November 9, 2022 20:09

Update to latest miio

83e7e90

Update to latest miio

ac58bf3

This was referenced Nov 9, 2022

Update decorators #1586

Closed

Implement choices_attribute for setting decorator #1587

Merged

rytilahti pushed a commit that referenced this pull request Nov 9, 2022

Implement choices_attribute for setting decorator (#1587)

b5a2940

Split out from #1543 Will be used for Multi Map Enum select.

starkillerOG mentioned this pull request Nov 9, 2022

Implement introspectable actions #1588

Merged

starkillerOG added 3 commits November 9, 2022 22:27

Merge branch 'master' into roborock_sensors

f2fed51

Merge branch 'master' into roborock_sensors

1c69cc5

Merge branch 'master' into roborock_sensors

f1f3a62

starkillerOG mentioned this pull request Nov 14, 2022

Add multi map handling to roborock #1596

Merged

Merge branch 'master' into roborock_sensors

998ec16

rytilahti mentioned this pull request Nov 23, 2022

Add roborock mop features home-assistant/core#82551

Closed

19 tasks

Merge branch 'master' into roborock_sensors

a5a0e3c

starkillerOG closed this Dec 5, 2022

Uh oh!

Conversation

starkillerOG commented Sep 30, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

crshray commented Nov 5, 2022

Uh oh!

craigcabrey commented Nov 5, 2022

Uh oh!

starkillerOG commented Nov 9, 2022

Uh oh!

h3rbst commented Nov 28, 2022

Uh oh!

starkillerOG commented Nov 28, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

starkillerOG commented Dec 5, 2022

Uh oh!

h3rbst commented Dec 9, 2022

Uh oh!

starkillerOG commented Dec 9, 2022

Uh oh!

h3rbst commented Dec 9, 2022

Uh oh!

xsasx commented Jul 13, 2023

Uh oh!

starkillerOG commented Jul 23, 2023

Uh oh!

neforce commented Aug 16, 2023

Uh oh!

JR2012thman commented Oct 12, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

starkillerOG commented Sep 30, 2022 •

edited

Loading

starkillerOG commented Nov 28, 2022 •

edited

Loading