[Gelöst] Benötige Hilfe mit Node-Red in Verbindung mit homee
-
@oli Na das habe ich Dir ja gesagt, dass das nur funktioniert wenn ohne Batterie Eigenschaft.
Dann brauchst aber auch nichts abziehen, dann kannst doch gleich filtern
$count(payload[battery="Keine Batterie"]) -
@mickym Dann schreibst halt
@oli sagte in Benötige Hilfe mit Node-Red in Verbindung mit homee:
nochmals vielen Dank für deine Hilfe, hier jetzt mal der komplette Flow, sowie er jetzt aussieht
Ein paar Sachen sind aber noch unklar
- Bei mir werden die Batteriewerte nicht mehr abgefragt, um den Fehler einzugrenzen, habe ich mal einen extra Flow gebaut und etwas gespielt
Dabei ist mir aufgefallen, dass in der Funktions Node folgendes Global geladen wird:
const attributes = global.get('homee.attributes'); const nodes = global.get('homee.nodes');Es wird zwar im oberen Flow die "nodes" geladen, aber die "attributes" habe ich nicht gefunden, kann darin der Fehler liegen?
- Was ist der Unterschied "hole alle Geräte" und "Alle Geräte"? Nach meiner Ansicht machen die beide das Gleiche und man könnte "Alle Geräte" herausnehmen
Was ich bei Deiner funktion Node nicht verstehe, warum Du immer durch das Array iterierst, um die Anzahl der Geräte zu ermitteln. Du filterst doch immer, das sind doch Kopien deiner Arrays.
Also wie gesagt - wenn Du lieber JS Code schreiben willst, dann mach das halt:
var battery_devices = msg.payload.filter(device => (device.attributes.filter(attribute => ((attribute.type === 8) || (attribute.type === 69))).length)); var battery_devices_type8 = battery_devices.filter(device => (device.attributes.filter(attribute => (attribute.type === 8)).length)); msg.payload = { "Alle Batteriegeräte": battery_devices.length, "Nur Typ8 Geräte": battery_devices_type8.length } return msgMeines Erachtens geht das ganze auch ohne Schleifen zum zählen.
Wie gesagt - schreibe JS Code wenn Du willst, ich will Dich nicht davon abhalten.
Solche Schleifen
test.forEach(function (attribute){ if (attribute.type === 8) { batterie ++ } })sind doch unnötig wenn ich über ein filter das bereits gefilterte Array nochmal filtern und nur die Anzahl der Elemente ausgeben.
- Bei mir werden die Batteriewerte nicht mehr abgefragt, um den Fehler einzugrenzen, habe ich mal einen extra Flow gebaut und etwas gespielt
-
@oli sagte in Benötige Hilfe mit Node-Red in Verbindung mit homee:
Sorry, jetzt habe ich doch noch eine Frage, wie kann ich in Node Red rechnen?
Geräte Gesamt - Geräte Batterie = Geräte Strom
Anstelle Deine Tabelle zu analysieren kann Du aber auch direkt Deine Geräte abfragen.
Aber nochmal - wenn Du lieber codieren und JS schreiben willst, dann mach das ruhig, ich will Dich nicht dazu überreden JSONATA zu nutzen. Deswegen hier eine Version mit einer function Node:
-
@mickym
kann ich auch nach Wörtern im Namen suchen? So wie ich es versucht habe klappt es nicht"test" : $count(payload[note = note.indexof("Plug")]) -
@oli wenn du meine Flows importiert hättest: https://forum.iobroker.net/post/962432
dann hättest Du gesehen, dass ich aus der Tabelle den Status der aktiven, inaktiven Geräte über die Stringsuche $contains durchgeführt habe.
In meinem letzten Posting habe ich Dir übrigens auch mit einer function Node gezeigt, wie Du auf das Array mit den Geräten zugreifst. Dann nimm halt das, wenn Du lieber Javascript nimmst.
Es zwingt Dich niemand JSONATA zu nutzen:
Nur auch bei JavaScript musst Du bei indexOf auf ungleich -1 abprüfen, da -1 nicht falsch ist (also nicht 0) würde Deine Abfrage eh nicht gehen.
Hier der Javascript Code:
msg.payload = msg.payload.filter(device => (device.note.indexOf("Plug") !== -1)).length; return msg; -
aber wie setzte ich das in eine Change Node?
-
@oli Gar nicht. Wenn Du in Javascript programmieren willst dann nimmst du eine function Node so wie ich unten.
Wenn Du eine ChangeNode verwenden willst dann JSONATA und wie gesagt in dem Flow den ich Dir gepostet habe ist die Syntax mit $contains.
Wie gesagt, wenn Du mit Javascript besser zurecht kommst, dann das halt. In der function Node habe ich Dir ja auch gezeigt, wie Du auf die Objekte aus der Change Node zugreifst. Es ist alles in der msg.payload. Deine function Node - ist im Grunde Deine Change Node mit der Du das Nachrichtenobjekt so veränderst, wie Du es haben willst.
Oder ich verstehe Deine Frage nicht, aber letztlich ging es Dir doch darum, wie Du die Anzahl der Geräte zurückbekommst, in den "Plug" in der note des Gerätes enthalten ist und das gibt doch die function Node unten aus???
-
@mickym
Eigentlich wollte ich in der Tabelle unter Batterie noch zusätzlich überprüfen, ob in der "note" das Wort "Plug" vorkommt, wenn ja ,dann Text "Strombetrieben", wenn nein, dann Text "Keine Information vorhanden". -
@oli Wie gesagt mir wird das selbst unter JSONATA zu kompliziert, in diesem Fall nimmst Du wohl wirklich besser Javascript. Ich empfehle Dir dann einfach, wie ich mit der function Node direkt das Array mit allen Geräten als Basis zu nehmen und dann Dein JSON Array für die Tabelle in einer function Node aufzubauen. Da Du ja eh mit Javascript besser vertraut bist und das immer komplizierter wird, ist das sicher der bessere Ansatz.
-
@oli sagte in Benötige Hilfe mit Node-Red in Verbindung mit homee:
@mickym
Eigentlich wollte ich in der Tabelle unter Batterie noch zusätzlich überprüfen, ob in der "note" das Wort "Plug" vorkommt, wenn ja ,dann Text "Strombetrieben", wenn nein, dann Text "Keine Information vorhanden".Was steht denn jetzt in Deiner Tabelle für battery drin?
"Keine Batterie" oder gar nichts? Dann versuche ich es ein letztes Mal noch mit meinen Mitteln.
-
-
-
Vielen Dank, noch eine letzte Fragen und dann bist du mich los, versprochen!!!!!
Wenn der Flow nicht nur bei einer State Änderung ausgeführt werden soll, sondern auch, wenn sich an den Batterien was ändert, oder sich die Anzahl der Gräte ändert.
Wie muss ich das schreiben?
-
@oli Ich bin mir nicht mal sicher ob der Filter funktioniert, da ich keine Ahnung habe wann Du wo den globalen Kontext schreibst. Wenn der Kontext noch alt ist und das Gerät neue Information hat dann geht es. Es funktioniert auch jetzt schon bei neuen Geräten, da dann ja im Kontext das Gerät noch nicht enthalten ist und in sofern der state nicht gleich sein kann.
( payload.nodes[id=$$.device.node.id].state = device.node.state ) and ( payload.nodes[id=$$.device.node.id].attributes[type=8].current_value = device.node.attributes[type=8].current_value )Wie gesagt - ich habe momentan nicht das Gefühl dass Du was zu NodeRed lernen möchtest, sondern dass ich Deine vis Tabelle perfektioniere.
-
@mickym said in Benötige Hilfe mit Node-Red in Verbindung mit homee:
payload.nodes[id=$$.device.node.id].attributes[type=8].current_value = device.node.attributes[type=8].current_value
Jetzt noch eine Verständnisfrage, warum schreibst du "and" und nicht "or"
-
@oli Ja im Moment funktioniert mein Test nicht mehr - muss noch mal schauen. Ich denke es muss and sein, da ja nur wenn beide Bedingungen wahr sind, nichts geändert wurde. Ich muss das aber nochmal testen, wie es sich mit nicht batteriebetriebenen Geräten verhält.
-
@oli So ich habs jetzt getestet musste nochmal geändert werden. Wie gesagt es muss AND sein, weil ja nur dann nichts geändert wurde, wenn beide Parameter nicht geändert wurden.
So um das für strom und batteriegebundenen Geräten zu gültig zu machen, ist die Bedingung doch etwas komplizierter:
Du kannst selbst testen: Gerät 62 ist batteriegebunden, Gerät 71 strom. Wenn Du state änderst funktioniert es und wenn Du current_value im attribute type 8 änderst.
( payload.nodes[id=$$.device.node.id].state = device.node.state ) and ( device.node.attributes[type=8] ? payload.nodes[id=$$.device.node.id].attributes[type=8].current_value = device.node.attributes[type=8].current_value : true )Das Ganze sind aber langsam Logikfragen und haben weniger mit NodeRed zu tun.
Wie gesagt das Ganze steht und fällt aber damit, wann der globale Kontext geändert wird und ob die einzelnen Nodes vorher ankommen, bevor der globale Kontext geändert wird.
-
@mickym
sorry, aber ich kann das spielen nicht lassen und habe versucht noch etwas an der Tabelle zu ändernIch wollte, wenn nur der Batteriestatus 69 vorhanden ist keine Zahlen, sondern einen Text hinterlegt haben. In der JSONata klappt das auch
Heizungsthermostate haben beides, da wird mir der Prozentwert hinterlegt:
Netatmo Geräte haben nur 69, da wird mir der Text hinterlegt:
Im Flow bringt er mir aber folgenden Fehlermeldung:
Hier der Flow
Ich will es nur verstehen
-
gelöscht.
-
@mickym
verstehe ich jetzt nicht, da schreibe ich doch den value Wert rein, 0 = Batterie ok, 1 = Batterie wechseln
