Hilfe bei debuggen einer übernommenen Funktion

[BF0911]

@mickym

Ja, super. Vielen Dank. Ich habe jetzt alle MQTT-In und Out gegen Json und Iobroker-in und out getauscht.

Da werden auch schon die Daten in die Datenpunkte geschrieben. Das scheint zu klappen.

Nun muss ich das noch mit dem Dashboard verstehen und umsetzen.

[mickym]

@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:

Nun muss ich das noch mit dem Dashboard verstehen und umsetzen.

Na da brauchst dann die Init Node nicht mehr sondern konfigurierst Dir die iobroker-In Node so, dass die Parameter direkt eingelesen werden:

Spoiler

[
   {
       "id": "4e03066e4c048e62",
       "type": "ui_numeric",
       "z": "d25606275138920f",
       "name": "",
       "label": "PH Grenzwert",
       "tooltip": "",
       "group": "f173af108d4b6003",
       "order": 3,
       "width": 0,
       "height": 0,
       "wrap": false,
       "passthru": false,
       "topic": "topic",
       "topicType": "msg",
       "format": "{{value}}",
       "min": 0,
       "max": 10,
       "step": "0.1",
       "className": "",
       "x": 1160,
       "y": 2900,
       "wires": [
           [
               "f06b228b62354eb2"
           ]
       ]
   },
   {
       "id": "2dc1917de1a4f96a",
       "type": "ui_numeric",
       "z": "d25606275138920f",
       "name": "",
       "label": "Maximale Tageslaufzeit",
       "tooltip": "",
       "group": "f173af108d4b6003",
       "order": 0,
       "width": 0,
       "height": 0,
       "wrap": false,
       "passthru": false,
       "topic": "topic",
       "topicType": "msg",
       "format": "{{value}} min",
       "min": 0,
       "max": 10,
       "step": 1,
       "className": "",
       "x": 1190,
       "y": 2980,
       "wires": [
           [
               "9b305980fa6f1fe0"
           ]
       ]
   },
   {
       "id": "c0ebbc8c64b7b0cd",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "payload",
               "pt": "msg",
               "to": "payload.PH_Grenzwert",
               "tot": "msg"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 950,
       "y": 2900,
       "wires": [
           [
               "4e03066e4c048e62"
           ]
       ]
   },
   {
       "id": "4b28ddc52ba7ab15",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "payload",
               "pt": "msg",
               "to": "payload.MaxTagesLaufzeit_min",
               "tot": "msg"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 950,
       "y": 2980,
       "wires": [
           [
               "2dc1917de1a4f96a"
           ]
       ]
   },
   {
       "id": "f06b228b62354eb2",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "PH_Parameter.PH_Grenzwert",
               "pt": "flow",
               "to": "payload",
               "tot": "msg"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 1560,
       "y": 2900,
       "wires": [
           [
               "d4c5239e5ff47822"
           ]
       ]
   },
   {
       "id": "9b305980fa6f1fe0",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "PH_Parameter.MaxTagesLaufzeit_min",
               "pt": "flow",
               "to": "payload",
               "tot": "msg"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 1530,
       "y": 2980,
       "wires": [
           [
               "d4c5239e5ff47822"
           ]
       ]
   },
   {
       "id": "d4c5239e5ff47822",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "payload",
               "pt": "msg",
               "to": "PH_Parameter",
               "tot": "flow"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 1870,
       "y": 2940,
       "wires": [
           [
               "9f4b4cddf32c69fb"
           ]
       ]
   },
   {
       "id": "22fe457b460d3214",
       "type": "ioBroker out",
       "z": "d25606275138920f",
       "name": "Parameter",
       "topic": "0_userdata.0.Poolsteuerung.Parameter",
       "ack": "true",
       "autoCreate": "false",
       "stateName": "",
       "role": "",
       "payloadType": "",
       "readonly": "",
       "stateUnit": "",
       "stateMin": "",
       "stateMax": "",
       "x": 2210,
       "y": 2940,
       "wires": []
   },
   {
       "id": "9f4b4cddf32c69fb",
       "type": "json",
       "z": "d25606275138920f",
       "name": "",
       "property": "payload",
       "action": "",
       "pretty": false,
       "x": 2050,
       "y": 2940,
       "wires": [
           [
               "22fe457b460d3214"
           ]
       ]
   },
   {
       "id": "46f3c4494c51b112",
       "type": "ioBroker in",
       "z": "d25606275138920f",
       "name": "Parameter",
       "attrname": "payload",
       "topic": "0_userdata.0.Poolsteuerung.Parameter",
       "payloadType": "value",
       "onlyack": "",
       "func": "all",
       "gap": "",
       "fireOnStart": "true",
       "outFormat": "MQTT",
       "x": 560,
       "y": 2940,
       "wires": [
           [
               "3f76567ff2d9ebf0"
           ]
       ]
   },
   {
       "id": "3f76567ff2d9ebf0",
       "type": "json",
       "z": "d25606275138920f",
       "name": "",
       "property": "payload",
       "action": "",
       "pretty": false,
       "x": 730,
       "y": 2940,
       "wires": [
           [
               "c0ebbc8c64b7b0cd",
               "4b28ddc52ba7ab15"
           ]
       ]
   },
   {
       "id": "f173af108d4b6003",
       "type": "ui_group",
       "name": "Parameter",
       "tab": "5fb772105c6cc8b7",
       "order": 3,
       "disp": true,
       "width": "6",
       "collapse": false,
       "className": ""
   },
   {
       "id": "5fb772105c6cc8b7",
       "type": "ui_tab",
       "name": "Home",
       "icon": "dashboard",
       "disabled": false,
       "hidden": false
   }
]

[BF0911]

@mickym

Ahh, verstehe. Dann habe ich es verstanden.

Vielen, vielen Dank!

[mickym]

@bf0911 Einmal musst natürlich auch das komplette Objekt nach dem Einlesen in deine Flowvariable einlesen, das hatte ich noch vergessen:

Ich frage mich nur im Nachhinein, warum Dein Skript alles mit globalen Variablen und x unterschiedlichen Flows gemacht hat. Wenn ich mir das bis jetzt so anschaue

ist der Flow doch nicht so groß und komplex??

[BF0911]

@mickym

Warum globale Variable etc benutzt wurden kann ich dir nicht sagen.

Mein Flow ist etwas größer als deiner bzw. wird noch größer wegen dem Dashboard usw

Ich hab den Flow mit dem Dashboard zu übernommen, aber es lassen sich keine Zahlen einstellen
Das springt immer

[mickym]

@bf0911 Na wenn du es wirklich übernommen hast, dann sollte es nicht passieren.

Schau mal ob du die Nachrichten nicht durchreichst?

das muss natürlich deaktiviert sein.

Ggf. musst DU natürlich bei ALLEN Dashboard Feldern schauen, dass Du die Nachrichten nicht durchreichst!!! (Also auch nicht bei Schaltern(switches) etc.)

[mickym]

@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:

Ich hab den Flow mit dem Dashboard zu übernommen, aber es lassen sich keine Zahlen einstellen
Das springt immer

Und ggf. immer Debug Nodes an die letzte Node vorne und direkt dahinter machen. Prüfen, wenn Du das Kabel vorne trennst, ob es an ankommenden Nachrichten liegt oder nicht. Das ist ja nun das schöne, dass Du sowas gut debuggen kannst.

[BF0911]

@mickym

Ist es bei mir!

Ich hab deinen Flow natürlich mit meinen Variablen angepasst.

Jetzt scheint es zu funktionieren, obwohl ich nichts geändert habe

[mickym]

@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:

Jetzt scheint es zu funktionieren, obwohl ich nichts geändert habe

Na dann weiß ich auch nicht - und lässt sich ja auch nicht mehr nachvollziehen.

Wenn ein undefined drin ist (wie bei der Pumpe ) oder was das für eine Node ist

kann ich dir nicht sagen.

[mickym]

@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:

Jetzt scheint es zu funktionieren, obwohl ich nichts geändert habe

Vielleicht war auch noch was von Deinem alten Flow aktiv - dass den Kontext gesetzt hat? - Ich würde Dir raten den alten Flow komplett zu deaktivieren - damit Du nur noch die Nodes ggf. hast, um die raus zu kopieren.

[BF0911]

@mickym

Danke, habe den Flow direkt deaktiviert.

[mickym]

@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:

@mickym

Danke, habe den Flow direkt deaktiviert.

Wie gesagt auch für später ist es immer wichtig die Stellen, wo Du was ausgibst möglichst übersichtlich hälst. Im Idealfall einmal - manchmal auch an mehreren Stellen, um das Ganze übersichtlich zu halten.

[mickym]

Ich verabschiede mich dann mal, wenn Du noch was auf dem Herzen hast dann schreibs rein. Entweder bekommst dann eine Antwort direkt, aber ich geh ja mal davon aus, dass Du am Nachmittag dann wieder ins reale Leben abtauchst.

[BF0911]

@mickym

Ich bedanke mich total bei dir!

Ich habe soviel gelernt und verstanden! Stark!

Ja, genau, ich bin gleich auch erstmal wieder in der echten Welt.

Werde mich aber sicherlich nochmal melden, spätestens wenn der Flow komplett ist!

[BF0911]

Es scheint immer noch ein Problem mit dem "Vergleich" ist zu Gesamt-Laufzeit Ph-Pumpe zu geben

Egal, welche deiner beiden Nodes ich nutze, es wird kein true weitergeleitet.

Es kommt auf jeden Fall von "Chlor-Pumpe nicht aktiv?" noch ein True. Das habe debuggt.

[mickym]

@bf0911 Na sowas kann ich doch gar nicht beantworten.

Du verdoppelst die Nachricht und deaktivierst die Tageslaufzeit.

Was kommt den wo raus?

[BF0911]

Und bei den Schaltern für den Automatik-Modus Chlor und PH gibt es Problem.

Die Schalter werden bei den Strings "ON" und "OFF" geschaltet.

Da muss offensichtlich was anderes rein

[BF0911]

Der Debug-Node "Chlor-Pumpe nicht aktiv?" gibt ein True aus, das soll ja durch "<max. Laufzeit" oder "max.Laufzeit nicht erreicht?" weitergeleitet werden, auf die Abfrage "PH-Automatik aktiv?" und dann ja die PH-Pumpe einschalten.

Die beiden Nodes sind jetzt aus Testgründen zusammen drin.

Nehme ich die Max-Tageslaufzeit-Abfrage raus, funktioniert die Logik und die PH-Pumpe schaltet

[mickym]

@bf0911 Wie gesagt teste es in dem Du Dir die Switches aus dem Flow einzeln testest:

Test Deine Switch Nodes halt getrennt vom Flow:

Oben lässt durch, weil die Automatik auf OFF steht.

Spoiler

[
   {
       "id": "5a06d880e13db423",
       "type": "switch",
       "z": "d25606275138920f",
       "name": "",
       "property": "PH_Parameter.PH_Pumpe_Automatik",
       "propertyType": "flow",
       "rules": [
           {
               "t": "eq",
               "v": "OFF",
               "vt": "str"
           }
       ],
       "checkall": "true",
       "repair": false,
       "outputs": 1,
       "x": 1270,
       "y": 3120,
       "wires": [
           [
               "2a46d8527419d1de"
           ]
       ]
   },
   {
       "id": "2a46d8527419d1de",
       "type": "debug",
       "z": "d25606275138920f",
       "name": "test",
       "active": true,
       "tosidebar": true,
       "console": false,
       "tostatus": false,
       "complete": "payload",
       "targetType": "msg",
       "statusVal": "",
       "statusType": "auto",
       "x": 1410,
       "y": 3120,
       "wires": []
   },
   {
       "id": "228a6ad0adf15765",
       "type": "inject",
       "z": "d25606275138920f",
       "name": "",
       "props": [
           {
               "p": "payload"
           }
       ],
       "repeat": "",
       "crontab": "",
       "once": false,
       "onceDelay": 0.1,
       "topic": "",
       "payload": "Dies ist ein Test",
       "payloadType": "str",
       "x": 1100,
       "y": 3120,
       "wires": [
           [
               "5a06d880e13db423"
           ]
       ]
   },
   {
       "id": "b0e46ea62e68cf43",
       "type": "switch",
       "z": "d25606275138920f",
       "name": "",
       "property": "PH_Parameter.PH_Pumpe_Automatik",
       "propertyType": "flow",
       "rules": [
           {
               "t": "eq",
               "v": "ON",
               "vt": "str"
           }
       ],
       "checkall": "true",
       "repair": false,
       "outputs": 1,
       "x": 1270,
       "y": 3180,
       "wires": [
           [
               "9de1a93d06536ca7"
           ]
       ]
   },
   {
       "id": "9de1a93d06536ca7",
       "type": "debug",
       "z": "d25606275138920f",
       "name": "test",
       "active": true,
       "tosidebar": true,
       "console": false,
       "tostatus": false,
       "complete": "payload",
       "targetType": "msg",
       "statusVal": "",
       "statusType": "auto",
       "x": 1410,
       "y": 3180,
       "wires": []
   },
   {
       "id": "c6a98dc4bce67b4b",
       "type": "inject",
       "z": "d25606275138920f",
       "name": "",
       "props": [
           {
               "p": "payload"
           }
       ],
       "repeat": "",
       "crontab": "",
       "once": false,
       "onceDelay": 0.1,
       "topic": "",
       "payload": "Dies ist ein Test",
       "payloadType": "str",
       "x": 1100,
       "y": 3180,
       "wires": [
           [
               "b0e46ea62e68cf43"
           ]
       ]
   }
]

und bei der Laufzeit das Gleiche - wie gesagt die Konstante unten rein.

Wie du siehst wird bei 8 Minuten geblockt und unter 7 Minuten nicht.

Spoiler

[
   {
       "id": "b4a85244852fce78",
       "type": "switch",
       "z": "d25606275138920f",
       "name": "",
       "property": "PHPumpe_Laufzeit_Total.minutes",
       "propertyType": "flow",
       "rules": [
           {
               "t": "lte",
               "v": "PH_Parameter.MaxTagesLaufzeit_min",
               "vt": "flow"
           }
       ],
       "checkall": "true",
       "repair": false,
       "outputs": 1,
       "x": 1290,
       "y": 3340,
       "wires": [
           [
               "5ca9c435503069b0"
           ]
       ]
   },
   {
       "id": "265b29ab0aaaec2f",
       "type": "inject",
       "z": "d25606275138920f",
       "name": "",
       "props": [
           {
               "p": "payload"
           }
       ],
       "repeat": "",
       "crontab": "",
       "once": false,
       "onceDelay": 0.1,
       "topic": "",
       "payload": "Dies ist ein Test",
       "payloadType": "str",
       "x": 680,
       "y": 3300,
       "wires": [
           [
               "76b45739cd7ee3fa"
           ]
       ]
   },
   {
       "id": "76b45739cd7ee3fa",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "PHPumpe_Laufzeit_Total.minutes",
               "pt": "flow",
               "to": "0",
               "tot": "num"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 990,
       "y": 3300,
       "wires": [
           [
               "b4a85244852fce78"
           ]
       ]
   },
   {
       "id": "5ca9c435503069b0",
       "type": "debug",
       "z": "d25606275138920f",
       "name": "test",
       "active": true,
       "tosidebar": true,
       "console": false,
       "tostatus": false,
       "complete": "payload",
       "targetType": "msg",
       "statusVal": "",
       "statusType": "auto",
       "x": 1430,
       "y": 3340,
       "wires": []
   },
   {
       "id": "fff4e08af65548df",
       "type": "inject",
       "z": "d25606275138920f",
       "name": "",
       "props": [
           {
               "p": "payload"
           }
       ],
       "repeat": "",
       "crontab": "",
       "once": false,
       "onceDelay": 0.1,
       "topic": "",
       "payload": "Dies ist ein Test",
       "payloadType": "str",
       "x": 680,
       "y": 3360,
       "wires": [
           [
               "aed3d5cc1789e01b"
           ]
       ]
   },
   {
       "id": "aed3d5cc1789e01b",
       "type": "change",
       "z": "d25606275138920f",
       "name": "",
       "rules": [
           {
               "t": "set",
               "p": "PHPumpe_Laufzeit_Total.minutes",
               "pt": "flow",
               "to": "8",
               "tot": "num"
           }
       ],
       "action": "",
       "property": "",
       "from": "",
       "to": "",
       "reg": false,
       "x": 990,
       "y": 3360,
       "wires": [
           [
               "b4a85244852fce78"
           ]
       ]
   }
]

Was im Kleinen funktioniert, funktioniert dann auch im Flow.

Die Anzeige des Kontexts musst Du bei jeder Ansicht neu aktualisieren:

[BF0911]

@mickym

Node passt jetzt und der PH-Automatik-Zweig läuft (so wie er soll).

Danke!

Was ich eben noch festgestellt habe, ist dass wenn ich die Filter-Pumpe manuell per (Soft- oder Hardware)-Schalter einschalte, dass hier natürlich die Laufzeit auf eine Minute begrenzt ist, weil ja, die Zeitsteuerung nicht aktiv ist.

Das ist suboptimal. Was wäre hier ein sinnvoller Weg? Doch mit einem Betriebsmodus zu arbeiten?

Ideal wäre es, wenn ich die Pumpe zusätzlich zum Scheduler einschalten kann, sie dann aber so lange läuft bis der Scheduler ein False sendet.