-
-
Notifications
You must be signed in to change notification settings - Fork 249
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
Add Roborock Core as a platform #691
Comments
It should work, but you have to set it up using map_source:
camera: image.s7_roborock_downstairs I suppose calibration points won't be added to the core, right? |
Ah that did it - Guess I should have read the docs better. I do plan to try to get calibration points added to core as an extra state attribute on the image entity, I think I'll be able to get that approved |
In case of failure I think it won't be hard to inject appropriate code using a custom integration. Please include me in a potential PR regarding this feature |
Can do. Anything else that would be helpful for me to expose? I have just started diving into your code, but one thing that is important with Roborock vacuums is that commands are map specific and room ids are not. So if I want to clean room id 12 on the downstairs map, if I click on that room to clean it, but I have the upstairs map selected as my current map, it will attempt to clean room id 12 upstairs. Is there a means on the card to set the map you are interacting with as the current map? |
It is possible to add configs for multiple maps using More info e.g. here: #248 |
Could you perhaps give more detail on how you are getting the map coordinates? I am using the new native Roborock integration. I have the image showing up but the calibration is off because I am not sure how to map the dock location to the image map. |
@dkirby-ms he modified the code of Roborock integration |
is there any ongoing development on this issue? |
I am unfortunately a bit blocked. Not much I can do at the moment and core devs have to make decisions. |
Is there a recommended workaround for generating calibration points at the moment? Docs point me towards https://github.com/PiotrMachowski/Home-Assistant-custom-components-Xiaomi-Cloud-Map-Extractor but I don't have a Xiaomi account as I've been using the Roborock app. |
You can use this integration instead: |
I did just switch to the official integration because I was having issues with that one and was hoping this would be more stable. Is the humbertogontijo version preferred for the vacuum map card? |
@jason-curtis at this moment the official integration doesn't provide data that is necessary to use the map functionality in this card |
I as well very much look forward to this card supporting the official Roborock integration |
@PiotrMachowski since our original plan failed. Would a service call work? I.e you could call vacuum: send command with a service that is like get_map_card_info and we get all of the initial info you need like calibration points, room dimensions, etc? Then we could still use the image entity to update what the map looks like? |
@Lash-L I think service call should be ok, but I also thought about implementing a dedicated WS API method. The downside of this approach is that it probably won't be possible for users to use it manually. It would also be nice to make it possible for the card to be notified that something has changed in the calibration (I think this happens quite often during map building). Can it be solved by generating an event when it happens? |
Hi! First of all, thanks for the integrations! @Lash-L Could you better explain how to get the calibration points so users can do it in the mean time? |
I think conceptually that's okay - i don't know if i really have time for it right now, whereas a service might be easier. I could also cache the latest mapdata and the service could get the latest mapdata if you think that would be better?
@carlos-48 You can't. I modified things inside the actual code base in my development environment |
Don't worry about it, at this moment I'm rewriting Map Extractor, then I'll have to adjust the card, so you have plenty of time. I think service call should be enough for my purposes |
Hey there, I have a suggestion regarding this. When you was developing the custom integration you had more freedom to do what ever you pleased and at your own saying right? Why not bring the custom integration up to date making it on par with the core integration and that way you can implement all the features at any time without having to go through the core devs themselves. This way the custom integration is always the "same" as the core, but at the same time you are able to implement whatever you want at whatever time. This is a newbie talking btw, I am just thinking out loud! Cause so far I had 3 options, in my case with a roborock:
|
i wanted to use this card so bad that i was willing to forsake the roborock app and use xiaomi home instead (and therefore lose camera view and pictures of objects detected) however the newest roborock s8 maxv thats already out for quite a while now, is not supported in the xiaomi home app, therefore i cannot use it in there. now there doesnt seem to be any option available for me to get a vacuum card working.... I am left without any working option now it seems... |
The official integration uses an |
Is there any progress here? |
Integration repository
https://www.home-assistant.io/integrations/roborock/
Supported features
Checklist
piotr.machowski.dev [at] gmail.com
(Retrieving entities info; please provide your GitHub username in the email)Vacuum entity/entities
n/a
Service calls
n/a
Other info
Creating this issue here to keep track of my work and ask questions. I plan to do the incorporation myself so I have left service calls and entities blank.
Roborock core holds the map in a image entity instead of a camera entity, so I got the calibration points from the map parser .calibration() and set up the following:
However, the image does not show up
Before I went any further I wanted to check with you that images are supported by the card.
The text was updated successfully, but these errors were encountered: