-
-
Notifications
You must be signed in to change notification settings - Fork 43
Support for Deebot 900/901 #360
Comments
Additional Info from OpenHab Addon:
So it seems like all we would have to do is to add xml handling, right? |
Feel free to add xml commands and their handling :) Are you familiar with visual code and devcontainer (including docker)? |
Will look into devcontainer :) Have used docker before :) |
Or would it be sufficient to change client library? |
Correct
Yes, this integration connects HA with the underlaying deebot-client library. So you need to add all the required commands and communication in the lib, so it's enough to fork the library first. |
My Deebot Ozmo 900 is showing as unavailable in HA, however in the Ecovacs app it is online, and I can use it. Any idea how I can get the Deebot Ozmo 900 activated in HA? My Deebot Ozmo 950 works flawlessly with the HA. |
Currently the 900 is not supported by this integration. We plan to add support but I can not give you any eta |
Would be high appreciated |
I'd just like to add my appreciation for any efforts to add support for the Deebot Ozmo 900 to the integration. |
This comment was marked as outdated.
This comment was marked as outdated.
That's 100% incorrect. This component never supported the 900 series. Thanks to @sandervankasteel, support for 900 will be added in the future. Please be patient, as the old bots use another communication protocol. I hope to ship a beta version with experimental support in September. |
That is correct. So I have a Deebot 900 (non-ozmo version) and I've been working on initial support for the Deebot 900 (and other XML based devices). So right now, the following commands are supported by the client library. Please note there is no maps support right now yet, however I do intend to implement that map support. See this PR for more information DeebotUniverse/client.py#288
A little sneak preview, how it currently shows up on the using the HACS integration. Keep in mind, some changes to the HACS integration will need to be done in order to fully take advantage of the new capabilities in the client library. I do intend to make those changes, since I locally have the modified HACS integration which loads in my proposed changes. |
I suggest waiting until the first beta with the new features out. Modifying files can break this integration or even HA if not done correctly. |
@sandervankasteel how is this coming along? Would love to test this on my Deebot Ozmo 900 (XML). |
@Arn0uDz So currently, the current status is that, I've merged my branch with the new architecture branch, so that the library can properly support both the JSON and XML data types. Instead of the "workaround" I implemented. However, I am going on vacation next week, for 2 weeks. So I'm quite preoccupied with all the preparations for that at the moment 😅. But I'd like to see if it's possible to re-implement most of the commands before my holiday. So that I can tackle any PR feedback after I am back from my holiday. I'd like to add that I do not own an Ozmo 900 but the regular 900 version. So I am incapable of testing and implementing the mopping part. |
@sandervankasteel I'd gladly test it for you if it's easy to do and if you want. Though I don't use the mopping part. I just need it to report it's state (ie: docked or cleaning) and to be able to make it clean and return to base. |
@sandervankasteel You don't need to rush 😊 Implement it when you have time. |
@edenhaus @sandervankasteel take your time and I will gladly test the beta when it's out. |
I am currently running Deebot N8 Pro+ and 900/901.
Back on OpenHab we had the 900 running with ease.
Support for 900/901 would be awesome. :)
I could also try to help, but since I am just getting started with HA, I have no experience in development for it.
Would be happy to help, if you could point me to good ressources for local development of plugin with the deebot_client library.
Example
The text was updated successfully, but these errors were encountered: