home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.38k stars 30.64k forks source link

Roborock Integration S7 VMax #92669

Closed GitMeTender closed 1 year ago

GitMeTender commented 1 year ago

The problem

Greetings! Today I switched from the Miio-Cloud-Integration to the Roborock-Integration - Setup was a breeze, but now I'm missing all the Sensors of my S7 Max in the integration. Only Mop Mode and Mop intensity are there. I can however fully control the Robot.

What version of Home Assistant Core has the issue?

2023.5.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Roborock

Link to integration documentation on our website

https://www.home-assistant.io/integrations/roborock/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 1 year ago

Hey there @humbertogontijo, @lash-l, mind taking a look at this issue as it has been labeled with an integration (roborock) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `roborock` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign roborock` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


roborock documentation roborock source (message by IssueLinks)

Lash-L commented 1 year ago

Hi @GitMeTender see #92639

When adding a new integration - there are limits to how much can be added to start off with. We went with the bare basics to begin with, but within the next few updates you should start to see more and more features show up.

LePremier commented 1 year ago

Hi @Lash-L,

just to add here: same situation with Roborock S7. I see Mop Mode and Mop Imtensity, but no further sensors.

But it reassures me when you say that this is not a fundamental problem with the S7/S7 Max series, but that the features for these devices have simply not been integrated yet.

I am therefore eagerly waiting for the upcoming versions of the integration.

Thank you so much for your work on this!!!

lukerix commented 1 year ago

Hi @Lash-L ,

I have the same for "Roborock S7 Pro Ultra". For now I see only 3 components: image

Everything is working fine but I hope to see all the additional sensors and Map shortly.

Also it would be nice if the status of vacuum component would show if it is in full/room/area/spot cleaning. Or maybe adding a separate sensor that would show this information.

ceoloide commented 1 year ago

@lukerix if you tap on "Roborock S7 Pro Ultra" you should see an additional card that lets you control your robot. From there you have additional actions at your disposal, though still no room or zone cleaning.

M1K3SD1 commented 1 year ago

Finally a native integration! Amazing job! Switched over with my S5 Max, now waiting for the statistics data to create automations. Thank you!

lukerix commented 1 year ago

@lukerix if you tap on "Roborock S7 Pro Ultra" you should see an additional card that lets you control your robot. From there you have additional actions at your disposal, though still no room or zone cleaning.

@ceoloide: I didn't talk about the controls but about the status "indicator". This integration shows in my case only docked/cleaning/returning to dock, the old xiaomi integration was showing more details: docked/cleaning/returning/collecting dust/segment cleaning/washing mop/...

This data was displayed on the vacuum card but it was coming from the "status" attribute so I'm hoping it will also come in the next release. image

Lash-L commented 1 year ago

@lukerix I have all the same information (maybe a bit more too not sure)

https://gist.github.com/Lash-L/931e7a1658f173835971666f3d1af632

I kind of already have it in the code, but it isn't finished, it will be pretty easy to add in I think and I can do that, but it will be low on my priority as things have not been approved into dev as fast as I thought they would.

Lash-L commented 1 year ago

@lukerix Status is in the latest version.

I believe everything should be good in this issue now so I am going to close it.

Lash-L commented 1 year ago

@home-assistant close