Adapter mihome-vacuum anpassungen
-
Hi,
klick mal bei dem Wochentag NEBEN die Schrift, also rechts davon, dann wird der Haken gesetzt.
Genauso wie beim Raum etc... n bisserl bloed gemacht.. glaub da gabs mal beim Admin n Issuee...@dirkhe kannst du da mal schauen, bitte ? Ist nachvollziehbar und ich fahr Node 16.x und latest Relaeses..
-
@ilovegym
Super - danke, hat so funktioniert. Glaub da wäre ich so nicht draufgekommen. Hatte immer auf den Text oder die Checkbox geklickt.Hier hierzu was bekannt? --> Noch ne kurze Frage (evtl. ein Feature request): Beim o.g. S7 kann man noch in der App eine "Wischroute" definieren: Standard, Gründlich und Gründlich+. Das habe ich in den Objekten nicht gefunden. Ist hierzu etwas geplant?
-
@mp_trixi sagte in Adapter mihome-vacuum anpassungen:
Noch ne kurze Frage (evtl. ein Feature request): Beim o.g. S7 kann man noch in der App eine "Wischroute" definieren: Standard, Gründlich und Gründlich+. Das habe ich in den Objekten nicht gefunden. Ist hierzu etwas geplant?
Ist das evtl. Der waterboxmode? Hadt du mal das o.g. feature branch getestet?
-
@dirkhe
Danke für die Antwort. Hier ein Screenshoot aus der App:
Die "Wischintensität" findet sich im Datenpunkt waterboxmode wieder. Ändert man "Wischroute" passiert da nichts. Scheint also ein eigener Datenpunkt zu sein. Zumindest keine neue Stati im Datenpunkt waterboxmode.Welches Feature Branch? Habe dazu nichts gefunden (auch kein Datenpunkt).
Wenn man einen Raum mit Zonenkoordinaten mit addRoom anlegt. Kann man da auch mehrere Zonen(koordinaten) angeben? Habe die Zone [18300,32000,29000,24900,1] und [21300,24900,25000,15000,1] und hätte gerne in einem Channel/Raum. Habs mit [18300,32000,29000,24900,1],[21300,24900,25000,15000,1] versucht. Da legt er aber nur den ersten an. Bei [[18300,32000,29000,24900,1],[21300,24900,25000,15000,1]] macht er gar nichts.
-
@mp_trixi probiere mal https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum/issues/620 da ist waterboxmode auf jeden fall verfügbar.
Mehrere zonen in einem raum geht nicht. Du kannst aber 2 anlegen und die dann gemeinsam reinigen lassen, oder du machst es per script, zb via sendto und clean segments -
Hallo @dirkhe
danke - ich habs mir angeschaut und es so angelegt (2 Zonen). Die werden getriggert und beide nacheinander abgefertigt
Das funktioniert.
Die Parameter water_box_mode habe ich und kann diese auch setzen. Allerdings bekomme ich damit "nur" die Wischintensität (Aus=Off=200,Sanft=Low=201, Mittel=Medium=202 und High=Intensiv=203) hin. Die Wischroute habe ich so noch nicht geschafft zu setzen (s. Bild oben). Was mir auch aufgefallen ist: Ihm ist egal welche Werte für fan_power und water_box_mode in den Räumen/Zonen gesetzt sind. Er nimmt immer die unter control. So habe ich z.B. in den zwei Zonen den water_box_mode auf off, er wischt aber wenn man diese Zonen triggert:
-
@mp_trixi kannst du deinen adapter mal auf debug stellen.
- Was die intensitivität angeht, müsstedt du mal die get status abfragen beobachten, wenn du die in der app verstellst. Wenn du den parameter rausgefunden hast und welche werte es gibt, wäre es gut, wenn du einen neuen feature request erstellen könntest. Dann kann ich das einbauen.
- Was die anderen beiden parameter angeht. Wenn man genau einen raum oder zone antriggert, wird versucht, den passenden datenpunkt rauszufinden und dann dessen einstellungen zu nehmen. Bei mehreren gleichzeitig geht das natürlich nicht, da nimmt er die werte aus control. Mit dem neuen feature kannst du die werte aber direkt mitgeben. Wenn du das gemacht hattest, dann ist da noch ein bug drin, müsste ich mir dann nochmal anschauen
-
doppelpost
-
@dirkhe
zu 1:
Du meinst das, richtig?
setGetStatus {"msg_ver":2,"msg_seq":2903,"state":8,"battery":100,"clean_time":1624,"clean_area":17307500,"error_code":0,"map_present":true,"in_cleaning":false,"in_returning":0,"in_fresh_state":1,"lab_status":3,"water_box_status":1,"back_type":-1,"wash_phase":0,"wash_ready":0,"fan_power":105,"dnd_enabled":false,"map_status":3,"is_locating":0,"lock_status":0,"water_box_mode":203,"water_box_carriage_status":1,"mop_forbidden_enable":1,"adbumper_status":[0,0,0],"water_shortage_status":0,"dock_type":3,"dust_collection_status":0,"auto_dust_collection":1,"mop_mode":303,"debug_mode":0,"switch_map_mode":0,"dock_error_status":0,"charge_status":1,"error_text":"No error"}
Wenn ja, glaube habe ich den Wert gefunden: mop_mode
300=Standard
301=Gründlich
303=Gründlich+
komischerweise gibt es kein 302?!? Ist es das was Du meintest?zu 2)
ich habe die Werte in die Datenpunkte der Räume geschrieben, in der Annahme dass die beim "Aufbau" der Queue verwendet werden. Wenn ich das jetzt richtig verstanden habe, geht das nur via Script/Blockly mit Sendto, richtig? Ich versuche mal im nächsten Durchlauf die zwei Werte (fan und waterbox) im Control Channel zu schreiben, bevor ich die Räume trigger, das sollte ja auch gehen, oder nicht? -
@mp_trixi dass der 301 fehlt, muss nichts zu bedeuten haben. Aber dann mach für den mop_mode bitte mal ein feature request auf github auf.
wenn du das in die dp geschrieben und die beiden auch nacheinander angetriggert hast, sollten deren einstellungen übernommen werden, sonst ist es ein bug.
Also die frage ist, wie hast du die beiden räume dann angetriggert? Einzeln und nacheinander? -
@dirkhe
getriggert habe ich das erste mal manuell (1. auf den Button im Raum 1 und dann danach gleich auf den Button im Raum 2). Mittlerweile habe ich ein Blockly dafür gemacht. Da habe ich festgestellt, dass ich String statt Number übergeben habe:
2022-10-12 09:00:00.139 - info: javascript.0 (877) State value to set for "mihome-vacuum.0.rooms.manual_18300_32000_29000_24900.roomFanPower" has to be type "number" but received type "string"
2022-10-12 09:00:00.141 - info: javascript.0 (877) State value to set for "mihome-vacuum.0.rooms.manual_18300_32000_29000_24900.roomWaterBoxMode" has to be type "number" but received type "string"
2022-10-12 09:00:00.142 - info: javascript.0 (877) State value to set for "mihome-vacuum.0.rooms.manual_21300_24900_25000_15000.roomFanPower" has to be type "number" but received type "string"
2022-10-12 09:00:00.169 - info: javascript.0 (877) State value to set for "mihome-vacuum.0.rooms.manual_21300_24900_25000_15000.roomWaterBoxMode" has to be type "number" but received type "string"
2022-10-12 09:00:00.204 - info: mihome-vacuum.0 (1083) State value to set for "mihome-vacuum.0.rooms.manual_18300_32000_29000_24900.roomFanPower" has to be type "number" but received type "string"
2022-10-12 09:00:00.210 - info: mihome-vacuum.0 (1083) State value to set for "mihome-vacuum.0.rooms.manual_18300_32000_29000_24900.roomWaterBoxMode" has to be type "number" but received type "string"
2022-10-12 09:00:00.213 - info: mihome-vacuum.0 (1083) State value to set for "mihome-vacuum.0.rooms.manual_21300_24900_25000_15000.roomFanPower" has to be type "number" but received type "string"
2022-10-12 09:00:00.215 - info: mihome-vacuum.0 (1083) State value to set for "mihome-vacuum.0.rooms.manual_21300_24900_25000_15000.roomWaterBoxMode" has to be type "number" but received type "string"In den Datenpunkten hat er aber die Werte doch richtig gesetzt und den Saugvorgang gestartet (Raum 2 in Warteposition):
Ich werde das nochmals beobachten. Das Blockly habe ich trotzdem "gefixt" und auf Zahl umgestellt. Es sieht jetzt so aus:
Bezgl. Feature request: Denke nicht dass ich den Wert brauche. War nur die Frage ob es den schon gibt. Soll ich trotzdem einen aufmachen (wie gesagt brauchst Dir wegen mir die Mühe nicht machen)?
-
@mp_trixi ich brauche den mop_mode auch nicht, wenn dann nur die s7 besitzer. Sollte aber kein grosser skt sein.
Das mit number und stijng hat nichts mit dem adapter zu tun. Generell setzt iobroker controller mehr und nehr auf typesicherheit, daher die warnung.
Wenn du die räume nacheinander triggerst, er also die warteschlange nutzt, sollte er für jeden raum die eingestellten werte nehmen. Das bitte nochmal beobachten
-
@dirkhe
dann habe ich mal, damit das Herausgefundene nicht verloren geht :), im github ein Issue aufgemacht:
https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum/issues/670Das andere werde ich beobachten. Hoffe komme zum Wochenende hin dazu.
closed Datenpunkt für Wischroute (mop_mode) hinzufügen #670
-
Ich habe die Suche schon verwendet, finde aber keinen brauchbaren Eintrag, ich habe folgendes Problem (ne, nur eine Info Meldung, nervt aber trotzdem) im Log:
Alle 20 Sekunden spukt der Adapter diese Meldung ins Log:
State value to set for "mihome-vacuum.0.info.water_box" has to be type "string" but received type "boolean"(mihome-vacuum.0 v3.4.2, Roborock S5 Max)
Woran kann das liegen?
-
@dpcfan ach komm .. im ernst
-
@dpcfan Mit v3.6.0 habe ich diese Meldung nicht im Log
-
@rkccorian Habe nun im Objekt von string zu boolean geändert und es ist weg... das meint evtl. arteck, die Lösung war zu einfach für das Forum. Ob nach einem Update es so bleibt, keine Ahnung, kenne jetzt ja die Lösung zum "Nacharbeiten".
-
@dirkhe
Hi - also habe jetzt mal ein Weilchen beobachtet und getestet. Bei mir funktioniert das nun alles mit den Zonen. Auch die Trigger mit FanPower und WaterBoxMode gehen. Einzig was noch auffällt - beim Start bekomme ich eine Warnung, dass der Typ nicht unterstützt wird (aber wie gesagt, es funzt alles):
mihome-vacuum.0 (266915) Model roborock.vacuum.a62 not supported! Please open issue on git: https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum/issues -
@mp_trixi ist in der aktuellen 3.7.0 drin
-
@dirkhe hab gerade noch einen letzten commit beim mihome-adapter gemacht für die Map: Karte zeigt jetzt Teppich und Wischroute an (neuere Modelle wie S7, s5 Max glaub ich auch), Performance wurde verbessert (schnellere Kartenerstellung) und die Map wird gezoomt auf den tatsächlichen Bereich ich übernehme aber keinen Support mehr falls noch ein Bug drin ist ggf könnte man den zoom auch noch in der config aus wählbar machen, so war mein Gedanke... Wollte es nicht auf meinem Rechner verstauben lassen, da das decodieren viel Aufwand war und es diese Features noch nirgends gibt und die Kartendaten auch noch nirgends dokumentiert sind.
