Hilfe bei debuggen einer übernommenen Funktion

[mickym]

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

Das mit der Uhrzeit macht aus meiner Sicht keinen Sinn. Vielleicht wollte der ERsteller da so.

Die Frage ist warum, Du dass alles in globalen Variablen brauchst.

Die meisten globalen Variablen werden ja ausserhalb dieser function NOde gesetzt:

var FilterPumpeStatus = global.get('FilterPumpe'); 
var DurchflussschalterStatus = global.get('Paddelschalter');
var PH_Pumpe_Laufzeit_Tag = global.get('PH_Pumpe_Laufzeit_Tag'); 
var PH_Pumpe_Pause = global.get('PH_Pumpe_Pause'); 
var PH_Pumpe_Laufzeit_ms = flow.get('PH_Pumpe_Laufzeit_sec') * 1000; 
var Chlor_Pumpe_Status = global.get('Chlor_Pumpe'); 
var PoolAutomatikModus = global.get('PoolAutomatikModus'); 
var PH_Pumpe_MaxTagesLaufzeit_min = flow.get('PH_Pumpe_MaxTagesLaufzeit_min'); 
var PH_Pumpe_Grenzwert = flow.get('PH_Pumpe_Grenzwert'); 
var PH_Pumpe_Automatik =  flow.get('PH_Pumpe_Automatik');
var PH_Pumpe_Session_Laufzeit =  flow.get('PH_Pumpe_Session_Laufzeit');

also sind die ja bereits da und werden irgendwo anders in dem Flow gesetzt, also brauchst Du die doch erst mal gar nicht setzten.

Ich mache ja nur erst mal mit Dir als Übung durch DIESE function Node zu ersetzen. Wenn Du das kannst, dann kannst Du Dich später selbst an die anderen machen, wenn Du willst.

[BF0911]

@mickym

So, wie ich das interpretiere, werden die globalen Variablen aber noch anderen Stellen benötigt, daher vermutlich auch global.

Diese Function ist nur auch nur ein kleiner Teil des Flows.

Würde es ggf. zusätzlich helfen, wenn ich den ganzen Flow mal teile?

[mickym]

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

Das mit der Uhrzeit macht aus meiner Sicht keinen Sinn. Vielleicht wollte der ERsteller da so.

Sehr gut - dann kannst Du ja gleich Deine eigenen Vorstellungen verwirklichen.

[mickym]

@bf0911 Ja wir ersetz

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

@mickym

So, wie ich das interpretiere, werden die globalen Variablen aber noch anderen Stellen benötigt, daher vermutlich auch global.

Diese Function ist nur auch nur ein kleiner Teil des Flows.

Würde es ggf. zusätzlich helfen, wenn ich den ganzen Flow mal teile?

Ja wir ersetzen nur diese function NOde. Den ganzen Flow brauche ich im Moment noch nicht, aber ggf. komme ich darauf zurück.

Also das Zeitfenster brauchst Du nicht. Wie gesagt, der Ersteller dieser Node - liest erst mal alle globalen Variablen ein. Wir brauchen das nicht, da wir bei Bedarf auf die globalen Variablen zugreifen.

[BF0911]

@mickym

Aus meiner Sicht reicht, wenn die FIlterpumpe zeitgesteuert läuft, da die ja in Abhängigkeit des Betriebs der Dosierpumpen steht.

[mickym]

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

@mickym

Aus meiner Sicht reicht, wenn die FIlterpumpe zeitgesteuert läuft, da die ja in Abhängigkeit des Betriebs der Dosierpumpen steht.

Na durch das Filter über das Zeitfenster läuft sie ja zeitgesteuert. Ich würde vorschlagen. Du implementierst diese function Node, Schritt für Schritt in einen Flow. Damit verstehst Du das Ganze und kannst es dann selbst modifizieren? - OK? - dann mach Dir jetzt mal keinen Gedanken über die Logik, sondern übersetze das mal in einen Flow.

Die erste Anweisung ist also das hier:

msg.payload.ph = Number((msg.payload.ph_current + 0.2).toFixed(2));

Was macht die denn? - Ich meine nicht was sie programmtechnisch macht (sehe ich ja selbst), sondern was Du glaubst, was sie macht.

[BF0911]

@mickym

Ok, das klingt nach einem sehr guten Plan.

Ich würde sagen, dass diese Funktion (toFixed(2)) auf 2 Dezimalzahlen nach dem Komma kürzt. Das erste dürfe eine normale Addition sein?

[mickym]

@bf0911 Ja und zwar wird die Eigenschaft Deiner payload verwendet und in eine Zahl verwandelt, wobei das in meinen Augen überflüssig ist, sonst könnte er das ja nicht 0.2 zu der Eigenschaft ph_current dazu zählen.

Wir modifizieren also mittels einer Change Node das gleiche und fangen den Flow nun an.
Mit JSONATA kannst Du solche Operationen direkt machen

Deine Change Node wird also so aussehen:

Also du setzt die Eigenschaft ph in Deiner payload.

[BF0911]

@mickym

Das bedeutet, als Beispiel er addiert zu dem WErt 7,4 0,2 was 7,6 ergeben würde?

WEnn dem so wäre, würde das auch kein Sinn machen, weil der gemessene PH-Wert ja die Schwelle sein soll

[mickym]

@bf0911 Na ja das kannst Du ja später ändern, aber im Moment würde in Deinem Nachrichtenobjekt die Eigenschaft ph um 0.2 größer als ph_current sein. Warum auch immer.

[mickym]

@bf0911 Was Du mal machen kannst ist - an Deine Node VOR der function Node eine Debug Node machen und dir das GANZE Nachrichtenobjekt dirausgeben lassen und hier in CodeTags einstellen, damit DU siehst, was für Eigenschaft Dein Nachrichtenobjekt hat.

[BF0911]

@mickym

Ok, dann folge ich erstmal nur deinen "Anweisungen".

DAnke schon.

Ich entschuldige aber direkt, wenn es mal ein paar Stunden etc. keine Antwort gibt.

[mickym]

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

@mickym

Ok, dann folge ich erstmal nur deinen "Anweisungen".

DAnke schon.

Ich entschuldige aber direkt, wenn es mal ein paar Stunden etc. keine Antwort gibt.

Dann sag vorher Bescheid, ich sitze auch nicht immer am Rechner.

[mickym]

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

@bf0911 Was Du mal machen kannst ist - an Deine Node VOR der function Node eine Debug Node machen und dir das GANZE Nachrichtenobjekt dirausgeben lassen und hier in CodeTags einstellen, damit DU siehst, was für Eigenschaft Dein Nachrichtenobjekt hat.

Um ein Nachrichtenobjekt vollständig auszugeben, hängst Du wie gesagt eine Debug Node an die vorherige Node und gibst das vollständige Nachrichtenobjekt aus:

Das debug Tab ist das mit dem Käfer (Bug).

Der Vorteil an einem Flow ist nun - dass nach jedem Schritt / node - also auch nach der Change Node, die wir erstellt haben, eine Debug Node anhängen können und bei Bedarf sehen können, wenn sich was verändert hat.

[BF0911]

@mickym

Hier der Debug-Output.

{"payload":7.44,"topic":"esphome.0.F4CFA2D0A156.Sensor.2835673629.state","_msgid":"133efcbf8d3a671e","_event":"node:242d9d62b7b4c44a","acknowledged":true,"timestamp":1715429688000,"lastchange":1715429522976,"mydate":"Sat May 11 2024","myymd":"2024-05-11","myyear":"2024","mymonth":"May","mymonthn":"05","mydom":"11","mydoy":"132","myday":"Sat","myhourpm":"2","myhour":"14","mytime":"14:14","mytimes":"14:14:48","myminute":"14","myminutes":"14:48","mysecond":"48","mymillis":"893","myepoch":"1715429688893","myrawdate":"2024-05-11T12:14:48.893Z","mypm":"PM"}

Ich bin heute noch so ca. bis 15:30 Uhr am Rechner und dann leider morgen vormittag wieder.

[mickym]

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

{"payload":7.44,"topic":"esphome.0.F4CFA2D0A156.Sensor.2835673629.state","_msgid":"133efcbf8d3a671e","_event":"node:242d9d62b7b4c44a","acknowledged":true,"timestamp":1715429688000,"lastchange":1715429522976,"mydate":"Sat May 11 2024","myymd":"2024-05-11","myyear":"2024","mymonth":"May","mymonthn":"05","mydom":"11","mydoy":"132","myday":"Sat","myhourpm":"2","myhour":"14","mytime":"14:14","mytimes":"14:14:48","myminute":"14","myminutes":"14:48","mysecond":"48","mymillis":"893","myepoch":"1715429688893","myrawdate":"2024-05-11T12:14:48.893Z","mypm":"PM"}

Na wahrscheinlich bist Du in der Zeile etwas verrutscht, da Du eine payload- unter der payload hast.

Schaut Deine Debugausgabe so aus, dass die payload einklappbar ist oder so wie in der 2. Ausgabe?

Ich glaube eher wie in der 2. Ausgabe, da Du nur dort direkt in der function node auf msg.myhour referenzieren kannst.
Allerdings fehlt mit mir das payload objekt. - das ist ja nur eine Zahl.

Diese Zeile:

msg.payload.ph = Number((msg.payload.ph_current + 0.2).toFixed(2));

weißt aber darauf hin, dass Deine payload ein Objekt sein muss.

Also entweder ist das nicht das Nachrichtenobjekt was unmittelbar in die function NOde kommt, oder der Code Deiner function Node ist nicht vollständig?

[mickym]

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

Ich bin heute noch so ca. bis 15:30 Uhr am Rechner und dann leider morgen vormittag wieder.

Na ich dann wahrscheinlich erst am Abend wieder, melde Dich einfach via chat - musst Du ja nicht alles an einem Tag lernen. Jedenfalls muss die payload ein Objekt sein, dass da rein kommt und nicht nur eine Zahl (7.44), sprich die payload wurde bereits geändert.

[BF0911]

@mickym

{"payload":7.41,"topic":"esphome.0.F4CFA2D0A156.Sensor.2835673629.state","_msgid":"f2e56b586c288cd7","_event":"node:242d9d62b7b4c44a","acknowledged":true,"timestamp":1715432588486,"lastchange":1715432408502}

So sieht das aus. Hatte mich mit dem Debug vertan.

[mickym]

@bf0911 Nein das kann es auch nicht sein. Zeig mal Dein Flow, wo Du die Debug Node hingemacht hast.
Das erste war schon besser, aber wie gesagt Deine payload muss ein Objekt sein. Es schaut nun so aus, als ob Du das iobroker Objekt ausgibst und nicht das Nachrichtenobjekt.

[BF0911]

@mickym

debug3 und die Funktion ist Prüflogik