Hilfe bei debuggen einer übernommenen Funktion
-
@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.
-
@bf0911 Ja wir ersetz
@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:
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.
-
Aus meiner Sicht reicht, wenn die FIlterpumpe zeitgesteuert läuft, da die ja in Abhängigkeit des Betriebs der Dosierpumpen steht.
-
@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:
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.
-
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?
-
@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.
-
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
-
@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.
-
@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.
-
Ok, dann folge ich erstmal nur deinen "Anweisungen".
DAnke schon.
Ich entschuldige aber direkt, wenn es mal ein paar Stunden etc. keine Antwort gibt.
-
@bf0911 sagte in Hilfe bei debuggen einer übernommenen Funktion:
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 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.
-
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.
-
@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.myhourreferenzieren 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?
-
@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.
-
{"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.
-
@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. -
debug3 und die Funktion ist Prüflogik
-
@bf0911 Na das kann es nicht sein. Ist denn der Code Deiner function node vollständig oder wird da die payload noch verschoben.
Steht da so eine Zeile wie
msg.payload.ph_current = msg.payloadoder wie ist denn Deine iobroker-get Node konfiguriert?
-
Ok ich verabschiede mich dann mal - und wir können dann weiter machen, wenn Du wieder da bist.
mach mal in Deine function Node - zum Debuggen noch folgende Zeile an 1. Stelle bzw. nach dem Setzen der Variablen rein und schau was im Debug Fenster auftaucht.
node.warn(msg.payload.ph_current);Wenn Du das siehst, dann sehen wir ja woher das kommt bzw. ob die Eigenschaft definiert ist:
Am Besten Du fügst das node.warn(...) direkt vor die Anweisung, die die msg.payload.ph setzt. Eventuell hast Du die ganze function Node an die falsche Stelle gesetzt
