[Beendet] Ecovacs Deebot Adapter v1.0.13 Beta/Latest

[mrbungle64]

@webzimm said in Test Adapter ecovacs-deebot v1.0.12 Latest:

Vielen Dank @mrbungle64, das wusste ich nicht. Jetzt funktioniert es bestens. Vielen Dank für Deine Hilfe. Viele Grüsse Axel

Hallo Axel,

wie sind denn bisher Deine Erfahrungen mit der aktuellen Version?

[mrbungle64]

@mrbungle64

Irgendwie hält sich das Feedback hier bisher doch sehr in Grenzen

Bevor ich eine Stable veröffentliche (und dann wieder die Weiterentwicklung starte) müsste ich schon ein bisschen mehr Feedback bekommen.

Vor allem interessieren mich aktuell die folgenden Punkte.
Bestehen die bei anderen Benutzern ebenfalls oder sind es Einzelfälle?

Zum OZMO T8 gibt es einen Issue, wo seit einer der letzten Versionen der Fehler "Error: The id is empty! Please provide a valid id" kommt (ggf. nur mit dem JS-Controller 3.2.x):
https://github.com/mrbungle64/ioBroker.ecovacs-deebot/issues/122
Der Fehler sollte eigentlich mit der Version 1.0.12 behoben sein. Leider bekomme ich dort gerade keine Rückmeldung dazu.
Edit: Das scheint auch so zu sein. Der Issue wurde inzwischen geschlossen.

Zum Deebot 900 gibt es einen Issue, wo seit Version 1.0.7 (ggf. auch früher) der Datenpunkt "map.deebotPositionCurrentSpotAreaID" nicht mehr aktualisiert wird:
https://github.com/mrbungle64/ioBroker.ecovacs-deebot/issues/112

Zum OZMO 950 gibt es einen Fall, wo von den Ecovacs Servern der Error code 1 gemeldet wird ("Error message received: unknown errorCode: 1" im Log). Es scheint aber keine negativen Auswirkungen zu geben.
https://forum.iobroker.net/topic/38584/mrbungle64-neuer-deebot-adapter

Ich selbst habe sowohl einen Deebot 901 (quasi identisch mit dem 900) als auch einen OZMO 920 (ein 950 mit kleinerem Akku) - habe beide Probleme aber nicht und bekomme die auch nicht nachgestellt.

Vielen Dank im Voraus für Euer Feedback

benep created this issue in mrbungle64/ioBroker.ecovacs-deebot

closed Error: The id is empty #122

helfi9999 created this issue in mrbungle64/ioBroker.ecovacs-deebot

closed Current Spot Area Name und diverse andere Sachen #112

[Marc Berg]

@mrbungle64 Ich habe zwei Instanzen, einen Ozmo 950 sowie einen Slim2. Das einzige kleine Problem, was ich bisher feststellen konnte: In der "Objekte" Übersicht werden beide Instanzen mit dem gleichen Symbol (Ozmo 950) dargestellt. Es wäre natürlich schöner, wenn man auf dem ersten Blick beide unterscheiden könnte.

[mrbungle64]

@marc-berg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

@mrbungle64 Ich habe zwei Instanzen, einen Ozmo 950 sowie einen Slim2.

Hallo @Marc-Berg

vielen Dank für das Feedback

Das einzige kleine Problem, was ich bisher feststellen konnte: In der "Objekte" Übersicht werden beide Instanzen mit dem gleichen Symbol (Ozmo 950) dargestellt. Es wäre natürlich schöner, wenn man auf dem ersten Blick beide unterscheiden könnte.

Ich wüsste jetzt nicht, wie man das Icon der einzelnen Instanzen ändern könnte. Der Admin Adapter zeigt da meines Wissens nach immer das Icon des Adapters an.
Ich habe selbst mehrere Geräte und würde mich ebenfalls dafür interessieren, ob das möglich ist.
Vielleicht kann mir ja jemand einen Tipp geben

[SBorg]

@mrbungle64 Gibt halt nix zu berichten, funktioniert einfach zu 99.99% perfekt

0.01% = beim starten + Ozmo 950 "charge status" (aber das ist eher ein Joke )

...und meines Wissens auch ja, ist nicht die Instanz, sondern der Adapter. Ergibt ja auch Sinn, denn es ist derselbe Adapter, nur eben mit mehreren Instanzen. Aber kein Problem, man könnte den Namen ja auch selbst eintragen, wenn der Creator des Adapters so nett wäre und es zulässt <-- feature request

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

@mrbungle64 Gibt halt nix zu berichten, funktioniert einfach zu 99.99% perfekt

Das freut mich
Vielen Dank für Dein regelmäßiges Feedback

0.01% = beim starten + Ozmo 950 "charge status" (aber das ist eher ein Joke )

Meinst Du die Warnung "Unhandled chargestatus: null" im Log?
Den "Fehler" (hat eigentlich keine negativen Auswirkungen) habe ich eben behoben - sollte mit der nächsten Version dann weg sein

...und meines Wissens auch ja, ist nicht die Instanz, sondern der Adapter. Ergibt ja auch Sinn, denn es ist derselbe Adapter, nur eben mit mehreren Instanzen. Aber kein Problem, man könnte den Namen ja auch selbst eintragen, wenn der Creator des Adapters so nett wäre und es zulässt <-- feature request

Wie kann ich das "zulassen"?
Würde mich schon interessieren, welche Möglichkeiten ich da hätte in Bezug auf die Instanzen.

[SBorg]

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

Meinst Du die Warnung "Unhandled chargestatus: null" im Log?
Den "Fehler" (hat eigentlich keine negativen Auswirkungen) habe ich eben behoben - sollte mit der nächsten Version dann weg sein

Jepp, und ich weiß doch, deswegen auch "Joke", aber dann kriegste die 0.01 % auch noch

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

Wie kann ich das "zulassen"?

Mein JS ist ganz leidlich, für einen Adapter hat es bisher aber nie gereicht, ich bin in anderem einfach schneller ^^
Aber hier mal ein JS-Beispiel unter "0_userdata.0.Statistik"...

Wie man sieht lässt sich "Statistik" nicht editieren (per JS createState erzeugt), "Wetter" dagegen schon. Ich weiß nicht ob das gewollt ist oder ein Bug, zumindest wenn man zB.

createState("0_userdata.0.Statistik", ...);
createState("0_userdata.0.Statistik.Wetter", ...);

ausführt, kann man "Statistik" editieren, wobei ein

createState("0_userdata.0.Statistik.Wetter", ...);

dieselbe Struktur anlegt, "Statistik" aber dann nicht editierbar ist.

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

Wie kann ich das "zulassen"?

Mein JS ist ganz leidlich, für einen Adapter hat es bisher aber nie gereicht, ich bin in anderem einfach schneller ^^
Aber hier mal ein JS-Beispiel unter "0_userdata.0.Statistik"...

Wie man sieht lässt sich "Statistik" nicht editieren (per JS createState erzeugt), "Wetter" dagegen schon. Ich weiß nicht ob das gewollt ist oder ein Bug, zumindest wenn man zB.

createState("0_userdata.0.Statistik", ...);
createState("0_userdata.0.Statistik.Wetter", ...);

ausführt, kann man "Statistik" editieren, wobei ein

createState("0_userdata.0.Statistik.Wetter", ...);

dieselbe Struktur anlegt, "Statistik" aber dann nicht editierbar ist.

Das sind aber jetzt nur Ordner unterhalb von "0_userdata" - aber nicht die root-Ordner der Instanzen.
Ich sehe jetzt auch, dass der Benutzer z.B. beim admin, web, node-red und lovelace Adapter die Rechte hat was zu ändern.
Ich weiß nur noch nicht wie man das "zulassen" kann und ob ich das überhaupt will
Da können Benutzer dann auch Dinge treiben die dann u.a. Probleme verursachen.

[SBorg]

@mrbungle64 Das war auch nur als Beispiel, selbiges gilt aber auch im root, nur das man da eigentlich halt nicht so ganz direkt Ordner anlegen kann

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

Ich weiß nur noch nicht wie man das "zulassen" kann und ob ich das überhaupt will
Da können Benutzer dann auch Dinge treiben die dann u.a. Probleme verursachen.

Ich brauche es nicht wirklich, ich habe nur einen + könnte es auch notfalls selbst editieren. Aber sämtliche ioBroker-eigen-Adapter erlauben es zum Beispiel
Aber dein Adapter, deine Entscheidung und es hat ja auch keinerlei Auswirkung auf die Funktionalität. Streich es einfach

...und nach dem ich getippt habe, könntest du nicht einfach "deviceModel oder -Name" in die Instanz-Beschreibung hineinschreiben lassen? "Dann wär der Käse gegessen..."

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

...und nach dem ich getippt habe, könntest du nicht einfach "deviceModel oder -Name" in die Instanz-Beschreibung hineinschreiben lassen? "Dann wär der Käse gegessen..."

Das Wörtchen "einfach" würde ich jetzt "einfach" mal streichen, aber ich bin gerade dabei herauszufinden wie das geht.
Das gehört meiner Meinung nach nicht zum Standardumfang eines Adapters

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

@mrbungle64 Das war auch nur als Beispiel, selbiges gilt aber auch im root, nur das man da eigentlich halt nicht so ganz direkt Ordner anlegen kann

Ich verstehe noch nicht so ganz, was Dein Beispiel dann damit zu tun hat

Grundsätzlich sind das zwar alles Objekte (also "objects" im Sinne des Core Concepts vom ioBroker) - aber der root-Ordner der Adapter Instanz (z.B. "ecovacs-deebot.0") wird initial vom ioBroker generiert und das Objekt beim Start der Instanz geladen.
Man arbeitet also zur Laufzeit mit einer Instanz dieses Objekts. Dieses dann zur Laufzeit zu modifizieren (also z.B. in dem Moment wo der Adapter den Namen des Devices per Event gemeldet bekommt) ist meiner Meinung nach mit Vorsicht zu genießen.

Wenn mir das ein Adapter Entwickler erklären kann, wie man das quasi gefahrlos zur Laufzeit machen kann und man auch das gewünschte Ergebnis dabei bekommt, dann würde ich mich freuen.

Edit: Siehe nächster Post von mir

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

@mrbungle64 Das war auch nur als Beispiel, selbiges gilt aber auch im root, nur das man da eigentlich halt nicht so ganz direkt Ordner anlegen kann

Inzwischen habe ich herausgefunden, warum es Adapter gibt, wo man den Namen der Instanz ändern kann - diese haben den "type" meta.

Beispiel aus der "io-package.json" vom Web Adapter:

"instanceObjects": [
        {
            "_id": "",
            "type": "meta",
            "common": {
                "name": "user files and images for background image",
                "type": "meta.user"
            },
            "native": {}
        }, ...

Warum das so ist und ob das "richtig" ist kann ich nicht sagen. Müsste mir mal jmd erklären

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

Ich weiß nur noch nicht wie man das "zulassen" kann und ob ich das überhaupt will
Da können Benutzer dann auch Dinge treiben die dann u.a. Probleme verursachen.

Ich brauche es nicht wirklich, ich habe nur einen + könnte es auch notfalls selbst editieren. Aber sämtliche ioBroker-eigen-Adapter erlauben es zum Beispiel
Aber dein Adapter, deine Entscheidung und es hat ja auch keinerlei Auswirkung auf die Funktionalität. Streich es einfach

...und nach dem ich getippt habe, könntest du nicht einfach "deviceModel oder -Name" in die Instanz-Beschreibung hineinschreiben lassen? "Dann wär der Käse gegessen...

Darauf hatte ich mich in meinem vorherigen Post bezogen - wir sind da jetzt ein bisschen durcheinander gekommen

[SBorg]

@mrbungle64 Ich weiß nicht wie es bei dir ist, aber mich "fuchst" das dann immer irgendwie an und ich will das hinkriegen. Im Grunde ist es aber nur ein "nice to have" (für wenige User mit mehreren Saugrobotern) und der Adapter erledigt seine Arbeit mehr als top.

Ggf. andere Idee (wie gesagt Adapter sind nicht meins): musst du die DPs sofort anlegen oder kann man das auf "später" verlagern? Könntest du dann nicht "einfach (^^)" in die Konfiguration noch ein Feld ala "Bezeichnung des Saugroboters" einfügen und dies dann als Name des Objekts benutzen und dann die DPs erzeugen?

btw: ja ich kenne "einfach" zur Genüge. Das letzte eines Users hat dann zwei Tage Entwicklung gekostet

...oder trete die ganze Idee einfach in die kleine, runde Papierablage auf dem Fußboden und gut is

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.12 Beta/Latest:

@mrbungle64 Ich weiß nicht wie es bei dir ist, aber mich "fuchst" das dann immer irgendwie an und ich will das hinkriegen. Im Grunde ist es aber nur ein "nice to have" (für wenige User mit mehreren Saugrobotern) und der Adapter erledigt seine Arbeit mehr als top.

Ja, deswegen habe ich es ja gestern auch mal probiert - ich hätte ja eben auch selbst Interesse dran, da ich 3 Geräte eingebunden habe (und manchmal zum Testen noch ein weiteres).

Ggf. andere Idee (wie gesagt Adapter sind nicht meins): musst du die DPs sofort anlegen oder kann man das auf "später" verlagern? Könntest du dann nicht "einfach (^^)" in die Konfiguration noch ein Feld ala "Bezeichnung des Saugroboters" einfügen und dies dann als Name des Objekts benutzen und dann die DPs erzeugen?

Habe ich auch schon dran gedacht. Müsste aber trotzdem erst mal weiter forschen, wie es überhaupt sauber (ohne zu tricksen) geht. Habe dazu werder was in der Doku noch im Forum gefunden.

btw: ja ich kenne "einfach" zur Genüge. Das letzte eines Users hat dann zwei Tage Entwicklung gekostet

So ist das

...oder trete die ganze Idee einfach in die kleine, runde Papierablage auf dem Fußboden und gut is

Ja, ich lege das zumindest erst mal auf Eis. Im Moment möchte ich diese Version erst mal stabil bekommen bzw. als solche veröffentlichen.

[mrbungle64]

Seit heute ist die Version 1.0.13 verfügbar.

Es gibt nur ein paar kleinere Verbesserungen:

• Die "Unhandled chargestatus: null" Warnung im Log sollte nun bei den 950er Modellen (920/950/T8) nicht mehr kommen
• Beim Datenpunkt "map.deebotPositionCurrentSpotAreaID" sollte jetzt der Wert "void" erscheinen, wenn bei der Library das Canvas Modul nicht (korrekt) installiert ist. Das wurde geändert, damit man das von "unknown" unterscheiden kann.
  Zur Erklärung: Bei "unknown" hat die Library versucht die SpotArea zu ermitteln - bei "void" nicht (da das Modul fehlt). Das Canvas Modul bleibt aber weiterhin eine optionale Abhängigkeit - daher diese Optimierung.

Ich hoffe weiterhin auf ein paar weitere Rückmeldung - auch positive Rückmeldungen natürlich sind willkommen

[SBorg]

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.13 Beta/Latest:

Die "Unhandled chargestatus: null" Warnung im Log sollte nun bei den 950er Modellen (920/950/T8) nicht mehr kommen

Zumindest beim 950er nicht mehr
Bisher lüppt alles, mal sehen was er am WE bei der nächsten Saug-/Wischrunde meint...

[mrbungle64]

Ich habe mal die Datenpunkte des Adapters (in deutscher Sprache) dokumentiert:
https://github.com/mrbungle64/ioBroker.ecovacs-deebot/wiki/Datenpunkte-(DE)

Auch hier würde ich mich über Rückmeldungen freuen, ob das alles so verständlich ist.

Falls Ihr Fragen dazu habt, kann ich die auch gerne hier beantworten

[SBorg]

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.13 Beta/Latest:

ob das alles so verständlich ist.

Ist es
Ev. höchstens "totalTime Dauer der Reinigungen (hh:mm:ss)" --> Gesamtdauer... oder wie bei "totalNumber" --> Dauer der Reinigungen (hh:mm:ss) gesamt
und
"Die Mehrzahl der Datenpunkte sind vom Modell anhängig." Wohl um ein Zeichen auf der Tastatur verrutscht

[mrbungle64]

@sborg said in Ecovacs Deebot Adapter v1.0.13 Beta/Latest:

@mrbungle64 sagte in Ecovacs Deebot Adapter v1.0.13 Beta/Latest:

ob das alles so verständlich ist.

Ist es
Ev. höchstens "totalTime Dauer der Reinigungen (hh:mm:ss)" --> Gesamtdauer... oder wie bei "totalNumber" --> Dauer der Reinigungen (hh:mm:ss) gesamt
und
"Die Mehrzahl der Datenpunkte sind vom Modell anhängig." Wohl um ein Zeichen auf der Tastatur verrutscht

Danke für das Feedback

Habe das entsprechend angepasst

[mrbungle64]

Habe heute die Version 1.1.0 veröffentlicht. Die ist nahezu identisch mit der 1.0.13.
Wenn heute nichts dramatisches mehr auffällt wird das dann die Stable Version