Hilfe bei debuggen einer übernommenen Funktion

[BF0911]

Hallo zusammen,

ich hab im letzten Herbst, mit Zustimmung, einen Flow zur Steuerung einer Dosieranlage für die Pool übernommen.
Jetzt ist es soweit, dass das Ganze "live" gehen soll, aber wie, von mir erwartet, funktioniert das nicht.

Ich habe das Flow hier und da natürlich auf meine Gegebenheiten angepasst.

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');

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

flow.set('PH_Actual',msg.payload.ph); 
flow.set('ORP_Actual',msg.payload.orp_current); 

if (myStunde >= 20 && myStunde <= 23  )
{

msg.delay = PH_Pumpe_Laufzeit_ms;

// for test without Logic
//return [msg];

if (PoolAutomatikModus == "ON" && PH_Pumpe_Automatik == "ON")
{
        if (msg.payload.ph_current < 6 || msg.payload.ph_current > 8 )
        {
            return [null];
        }
        else if (msg.payload.ph_current >= PH_Pumpe_Grenzwert && FilterPumpeStatus == "ON" && PH_Pumpe_Laufzeit_Tag < PH_Pumpe_MaxTagesLaufzeit_min && PH_Pumpe_Pause == "NO" && Chlor_Pumpe_Status == "OFF" && DurchflussschalterStatus == "ON" && PH_Pumpe_Session_Laufzeit > 10) 
            {
                return [msg];
            }
        else
            {
                return [null];
            }
}
else
 {
    return [null];
}
}
else
 {
    return [null];
}

Jetzt gibt von dem Function-Nodes so einige in diesem Flow. Bei einigen bin ich mir (ziemlich) sicher, die komplett verstanden zu haben.

Das hier eingefügte nicht so. Vielleicht kann mir jemand bei der Interpretation helfen bzw. als erstes sagen, wo z. B. der msg.payload.orp_current herkommen muss?

Dann habe ich hier schon öfter gelesen, dass ja der Sinn einer grafischen Programmierung die Vermeidung der Function-Node sein sollte, vielleicht kann mir auch anhand der eingefügten Funktion erklären, wie man das komplett grafisch programmiert.

Vielen Dank, und wie ihr euch vielleicht denken könnte, bin ich relativ unerfahren in Node-Red.

[mickym]

@bf0911 Das ist genau der Grund warum man keine function Nodes verwendet, weil du dir die tollen Möglichkeiten des Debuggens nimmst und dich durch Code quälen musst. Entweder du schmeißt die function nodes weg und erstellst anständige flows, anderenfalls bleibt dir nichts anderes übrig, als mit

node.warm("…")

dir Zwischenachrichten oder Teilergebnisse im Debugfenster ausgeben zu lassen. und das nächste Mal machst du es lieber selbst, als Flows it Javascript zu codieren. Wenn ich mir den Code anschaue dann gibt es nichts was den Einsatz von Code zumindest nicht in dem Ausmaß rechtfertigt, wenn du schon lieber mit Javascript arbeitest.

im Grunde tust du dir einen Gefallen, wenn du diese function Node durch einen Flow ersetzt,

Ansonsten kannst auch deinen ganzen Flow in eine function Node packen, dann ist das „noch übersichtlicher“. Ein trigger alles andere in eine function node und dann noch eine Ausgabe. Sehr übersichtlich

[mickym]

msg.payload.orp_current

bedeutet, dass deine eingehende payload ein Objekt ist, die eine eigene Eigenschaft orp_current hat. Woher es diese bekommt, musst du halt vorher im flow suchen.

[BF0911]

@mickym

Danke für den Hinweis. Da ich wie schon geschrieben, kein Node-Red-Profi bin, weiß ich nicht so recht wie anfangen, die Funktion gegen einzelne Nodes zu tauschen.

Das fängt schon damit an, wie produziert man per Node eine globale Variable?
Wie geht If-Else? Gibt es da eine vernüftige Lektüre, WEbsite, Youtube-Channel etc, der mir da helfen kann?

[mickym]

@bf0911 Na ich kann Dir schon auch helfen - wenn Du Dich da einarbeiten willst.

Ansonsten gibts verschiedene Tutorials oder auch Bücher. Zum Beispiel hier https://stevesnoderedguide.com/ und auch Bücher oder https://haus-automatisierung.com/nodered-tutorial-reihe/

So nun fangen wir aber mal an mit Deinen Fragen.

Dein Freund mit der Du zu 95% Deinen Code ersetzen kannst ist die Verarbeitung der payload - also Änderung der payload mit Change Nodes.

Dort setzt Du auch eine globale Variable

If Abfragen machst Du einfach in dem Du Deinen Flow mit switch Nodes verzweigst.

[mickym]

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

if (myStunde >= 20 && myStunde <= 23 )

Fangen wir mal an.

var myStunde = Number(msg.myhour);

So irgendwo in seinem Nachrichtenobjekt ermittelt er also eine Uhrzeit ich gehe mal davon aus, dass der ganze Flow nur dann was machen soll, wenn es zwischen 20 Uhr und 23 Uhr ist ? - Die Frage ist, willst Du Dich darauf einlassen, dass auch so zu verstehen?

Um so was grafisch zu machen, was Zeitverarbeitung betrifft - könntest Du Dir zum Beispiel die light scheduler Node nehmen.

https://flows.nodered.org/node/node-red-contrib-light-scheduler

Damit kannst Du Dir dann grafisch sogar einstellen, wann das ganze aktiv sein soll:

Du kannst Dir auch meinen Thread zur Datums und Zeitverarbeitung anschauen, um zu sehen, wie man das mit Standardmitteln macht.

https://forum.iobroker.net/topic/50086/datum-und-zeitverarbeitung-mit-nodered

Machen wir es also mit Standardmitteln:

$moment().isBetween($moment("20:00", "HH:mm"),$moment("23:00", "HH:mm"),'minute','[)')

Das verfrachten wir nun in eine Switch Node. Da Dein Freund, der diese function NOde erstellt hat, ansonsten nichts ausgibt,

else
 {
    return [null];
}

kann man sich das else auch sparen und die Nachricht einfach verschlucken.

So mit einer Switch Node filtern wir also mal Deine Nachrichten:

Wenn Du jetzt die Inject Node betätigst, dann wird Deine Nachricht ausgefiltert und es kommt nichts raus. Das meine ich mit debuggen. Du siehst also sofort, dass die Nachricht ausgefiltert wird. Änderst Du nun das Zeitfenster zwischen 13 und 23 Uhr so wird die Nachricht durchgelassen.

[BF0911]

@mickym

Das wäre natürlich noch besser, wenn du mich ein bisschen unterstützt.

Ich bin durchaus bereit, mich in Node-Red einzuarbeiten.

Ok, das würde bedeuten, ich erzeuge erstmal auf diesen Weg alle globale Variable? Der Inject-Node ist, vermutlich nur auf einmal inizieren, richtig?

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

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