Adapter mihome-vacuum anpassungen

[RkcCorian]

@dirkhe Moin! Hab jetzt mal passende Skripte in Blockly erstellt. Für den JS code ggf. importieren und die Ansicht auf JS ändern.

Vorbedingungen

DPs anlegen wie folgt

alias.0.Wohnung.Saugwischroboter.ROOM_START_COMMAND mit

{
  "common": {
    "name": "ROOM_START_COMMAND",
    "desc": "Manuell erzeugt",
    "role": "text",
    "type": "string",
    "read": true,
    "write": true,
    "def": "",
    "alias": {
      "id": "mihome-vacuum.0.control.X_send_command",
      "write": "\"app_segment_clean;[\"+val+\"]\""
    }
  },
  "type": "state",
  "_id": "alias.0.Wohnung.Saugwischroboter.ROOM_START_COMMAND",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  },
  "from": "system.adapter.admin.0",
  "user": "system.user.admin",
  "ts": 1656176031942
}

alias.0.Wohnung.Saugwischroboter.ZONE_START_COMMAND mit

{
  "common": {
    "name": "ZONE_START_COMMAND",
    "desc": "Manuell erzeugt",
    "role": "text",
    "type": "string",
    "read": true,
    "write": true,
    "def": "",
    "alias": {
      "id": "mihome-vacuum.0.control.X_send_command",
      "write": "\"app_zoned_clean;[\"+val+\"]\""
    }
  },
  "type": "state",
  "_id": "alias.0.Wohnung.Saugwischroboter.ZONE_START_COMMAND",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  },
  "from": "system.adapter.admin.0",
  "user": "system.user.admin",
  "ts": 1657567662511
}

Raumzuordnung über enum "rooms"

Blockly

Blockly.zip

[RkcCorian]

Gerne feedback was ich noch besser machen kann.
Änderungen inzw.:

• BUGFIX: Initialer Start der Queue-Abarbeitung geändert
• COSMETIC: CLEANING_QUEUE in COMMAND_QUEUE umbenannt > Geschmackssache

Bin aber mit dem initialen Start/Trigger immer noch nicht zufrieden. Muss da nochmal überlegen. Es gehen Befehle verloren, wenn das nächste Kommando bspw. während dem Absaugen kommt.

Version 2:
Blockly_v2.zip

[RkcCorian]

@dirkhe : Falls Du reinschaust... denkst Du, Du könntest im Adapter was anpassen, sodass weite Teile meines eigenen Skripts nicht mehr notwendig wären?

[dirkhe]

@rkccorian Ich habe mal reingeschaut (musste mir das in JS wandeln, weil Blocky mir zu kompliziert ist), sehe aber den Unterschied nicht wirklich zum Adapter.
Was du brauchst ist die alexa Geschichte, weil du da ja spezielle Fälle für reinigen, saugen usw. abfängst, das ist eh individuell, das bekommen wir so nicht in den Adapter.
Was du aber auf jeden Fall machen kannst, ist ja die roomFanPower und roomWaterBoxMode einstellen und dann roomclean triggern, dann kannst du dir das mit der queue sparen.
Was halt nicht geht, ist den selben Raum erst saugen und dann wischen, weil er FanPower und WaterboxMode halt beim starten abfragt. -> das könnte man vlt. nochmal einbauen, dass es zb sowas wie addToQueue(RoomOrZone, FanPower,WaterBoxMode) gibt. Könnte man dann auch bei den Timern setzten.
Für die Zonen kannst du übrigens per "addRoom" einen eigenen Raum anlegen, indem du die ZonenKoordinaten angibst.

[RkcCorian]

@dirkhe Danke Dir!
roomFanPower und roomWaterBoxMode speichere ich zusammen mit dem Raum / der Zone. Unabhängig (so wie jetzt) bekomme ich das aktuell mit dem Adapter nicht hin.
Bzgl. Timer... die laufen nach einer bestimmten Zeit?! Sind also kein Ersatz für eine Queue?!
Ahhh, Zone kann über addRoom angelegt werden, das wusste ich nicht.

[RkcCorian]

@dirkhe Wenn Du roomFanPower und roomWaterBoxMode mit in der Queue speicherst könnte ich auch das Reinigen des Essbereich 2 mal triggern. Einmal mit der Einstellung max. Saugen und einmal mit der Einstellung max. Wischen.

[dirkhe]

@rkccorian Timer kannst du anlegen, die dann zu einer bestimmten Zeit laufen, dort kannst du dann auch sofort mehr als einen Raum zuweisen. Diesen Timer kann dann man disablen, einmalig aussetzr oder auch direkt starten, geht dann sehr gut mit einer VIS.
Wenn ich das hunzufügen würden, wäre es wahrscheinlich erstmal nur ein customcommand, welches du mit sendTo senden könntest, also sowas wie

 sendTo("mihome-vacuum.0", "addToQueue", {cleanRooms:'enum.rooms.wohnzimmer', FanPower:102, WaterBoxMode:5}
, async function (result) {
    console.log(result);
  });

oder

 sendTo("mihome-vacuum.0", "addToQueue", {cleanSegments: '12', FanPower:102, WaterBoxMode:5}
, async function (result) {
    console.log(result);
  });

oder

 sendTo("mihome-vacuum.0", "addToQueue", {cleanZone: '24117,26005,25767,27205,1', FanPower:102, WaterBoxMode:5}
, async function (result) {
    console.log(result);
  });

also quasi ein wrapper für die 3 zielobjekte.
Würdest du dafür ein Issue auf github aufmachen?

[RkcCorian]

@dirkhe Nichts was mir lieber wäre
Als Bug oder als Geräte-Request?
Hinweis: cleanZone müsste m.W. als Array übergeben werden...

 sendTo("mihome-vacuum.0", "addToQueue", {cleanZone: '[24117,26005,25767,27205,1]', FanPower:102, WaterBoxMode:5}
, async function (result) {
    console.log(result);
  });

Dann geht auch bspw. ...

 sendTo("mihome-vacuum.0", "addToQueue", {cleanZone: '[24117,26005,25767,27205,1],[24223,26777,25767,27205,1]', FanPower:102, WaterBoxMode:5}
, async function (result) {
    console.log(result);
  });

Bzgl. cleanRoom würde dann auch Folgendes gehen?!

 sendTo("mihome-vacuum.0", "addToQueue", {cleanRooms:'enum.rooms.wohnzimmer, enum.rooms.esszimmer', FanPower:102, WaterBoxMode:5}
, async function (result) {
    console.log(result);
  });

[dirkhe]

@rkccorian ist ja ein feature request, weiß gerade gar nicht, was genau du da auswählen kannst. Im Zweifel labeln wir die sowoeso entsprechend.
Also im Prinzip würde ich dasselbe wie bei den originären command cleanzone gehen und da wird nur ein String erwartet, weil der dann anschlissend in ein [] gepackt wird. Ich könnte aber auch abfragen, ob es ein array ist, aber dann müsste es ohne die ' übergeben werden

[RkcCorian]

@dirkhe Moin! So ungefährt?! > https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum/issues/620

RkcCorian created this issue in iobroker-community-adapters/ioBroker.mihome-vacuum

closed Feature Request: Queue shall contain fan_power and water_box_mode for each command given #620

[dirkhe]

@rkccorian ja genau, ich habs nochmal kommentiert. Bin aber nächste Woche im Urlaub, danach mache ich mich daran

[RkcCorian]

@rkccorian Ein letztes Update, bevor es ein Adapter-Update von dirkhe gibt...

Änderungen inzw.:

• BUGFIX: Initialer Start der Queue-Abarbeitung geändert
• BUGFIX: Semaphore für konkurrierenden Zugriff
• BUGFIX: Div. kl. Bugfixes

Version 3:
Blockly_v3.zip

[dirkhe]

So, ich habe mal eine Testverion auf github geladen, siehe https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum/issues/620
Wer testen möchte....

RkcCorian created this issue in iobroker-community-adapters/ioBroker.mihome-vacuum

closed Feature Request: Queue shall contain fan_power and water_box_mode for each command given #620

[MP_Trixi]

@dirkhe
Kurze Frage: Irgendwie bin ich zu blöd timer in der Adapterinstanz zu setzen. Gefunden habe ich dazu nichts. Ich kann keine Tage und keine Räume auswählen (beide Dropdownlisten sind ausgeraut). Ein channel ist vorausgewählt, weitere gehen nicht. Enable/Disable geht. Ebenso h und min:

Was mache ich falsch?


Admin 6.2.22
Roborock S7 Pro Ultra
Danke für die Hilfe

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?

[Neuschwansteini]

@mp_trixi

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..

[MP_Trixi]

@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?

[dirkhe]

@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?

[MP_Trixi]

@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.

[dirkhe]

@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

RkcCorian created this issue in iobroker-community-adapters/ioBroker.mihome-vacuum

closed Feature Request: Queue shall contain fan_power and water_box_mode for each command given #620

[MP_Trixi]

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: