Zigbee-Geräte überwachen

JohannesA

@ciddi89
Vielen Dank für die "Version 2"!
Mit der Ergänzung bei der Datenpunkterstellung ist es jetzt wirklich plug&play!

Freut mich dass auch ich meinen Beitrag zu diesem tollen Gewerk leisten kann! Bisher hab ich ja nur "geschnorrt".

Die meisten Änderungen die du bei den Deklarationen gemacht hast, hat mir duckduckgo gerade erklärt.
Warum "const" statt "var" ist mir jetzt klar -> Anfängerfehler. Auch den Unterschied zwischen var und let hab ich theoretisch verstanden. Praktisch - naja, das zeigt sich vielleicht noch.

Aber kannst du mir noch erklären, was diese Codezeilen machen? Die kapier ich nicht:

let javascript_start = null


javascript_start = setTimeout (function () {
    log("Run Zigbee-Watchdog");
    zigbeeWatchdog();
}, 300);

Vielleicht magst mir das noch kurz erläutern.

Und auf dein Lob hin ist bei mir die Motivation soeben durch die Decke gegangen.

(hatte sogar kurz überlegt ob ich da vielleicht einen Watchdog-Adapter bauen sollte. Aber in Anbetracht des steinigen Weges vom Auspacken des Raspi bis jetzt wär das wie die Umsetzung einer bemannten Mars-Mission - deshalb schnell wieder zurück auf den Boden der Realität. )

@johannesa said in Zigbee-Geräte überwachen:

Warum "const" statt "var" ist mir jetzt klar -> Anfängerfehler.

Naja, Theoretisch hätte man das auch alles in "var" lassen können. Aber ich habe mir das alles nach und nach angewöhnt Variabeln die auf Datenpunkte zeigen mit "const" zu deklarieren und alle anderen mit "let". Wieso, weshalb, warum hast du ja schon selbst nachgeforscht

@johannesa said in Zigbee-Geräte überwachen:

Aber kannst du mir noch erklären, was diese Codezeilen machen? Die kapier ich nicht:

Aber natürlich! Du hast ja schon die shedule Funktion, welche in der 6ten Minute nach der vollen Stunde ausgeführt wird, eingesetzt. Aber ich bin ein sehr ungeduldiger Mensch Daher habe ich diese Funktion mit reingenommen. Nach dem das Skript gestartet wurde, macht diese nach 300ms schon einmal ein Durchlauf mit deiner Hauptfunktion und befüllt die Datenpunkte. Das wird einmal gemacht nach Skriptstart und danach kommt halt der nächste Durchlauf wieder nach deinem Zeitplan.

@johannesa said in Zigbee-Geräte überwachen:

Und auf dein Lob hin ist bei mir die Motivation soeben durch die Decke gegangen.

Naja ich bin selber am lernen. Und wie lernt man am besten? In dem man auch anderen Hilft und sich anguckt was die so machen und wie die ihren Code schreiben. Bin erstaunt wie schnell man immer wieder Verbesserungen findet und diese dann an seinen eigenen Skripten ausprobiert und nutzt.

paul53

@johannesa sagte: was diese Codezeilen machen?

Da der Timer nicht mit clearTimeout(javascript_start) gestoppt wird (oder übersehe ich es?), kann man auf die Timer-Variable verzichten.

@paul53 said

Da der Timer nicht mit clearTimeout(javascript_start) gestoppt wird (oder übersehe ich es?), kann man auf die Timer-Variable verzichten.

Nein ist kein stopper drin. Aber danke für den Hinweis Dann kann ich die ja wieder rausnehmen.

JohannesA

@ciddi89
Guten Morgen!
Danke für die Erklärung! Diese Funktion wird so also nur einmal ausgeführt. richtig? Also wenn ich zum Beispiel den Raspi neu starte, oder auch nur den JavaScript-Adapter.
Das ist perfekt, genau das brauche ich noch wo anders um den Zeitpunkt des letzten Neustarts umzuwandeln. So kommt eines zum anderen.

Mir ist heute früh noch aufgefallen, dass die Visualisierung die JSON-Table in Jarvis nicht richtig dargestellt hat. Hatte zwar keine Zeit mehr dem genauer nachzugehen bevor ich ins Büro musste, aber es lässt mir keine Ruhe. Ich hab schnell die Quelltexte verglichen und vermute, dass es daran liegt:

Zugeordneten Raum suchen - bei mir:

currRoom = getObject(id, 'rooms').enumNames[0].de;

Zugeordneten Raum suchen - bei dir:

currRoom = getObject(id, 'rooms').enumNames[0];

Ich vermute, dass es daran liegt, denn genau die Raumzuordnung hat mich ganz schön Nerven gekostet.

Weil ich Räume selber angelegt habe (Kinderzimmer den Namen zugeordnet) musste ich im Expertenmodus unter "enums.rooms" von ioBroker bei den selber erstellten Räumen die "de"-Bezeichnung in den Objektdaten dazu anlegen. (das werde mein Leben lang auswendig wissen ).

Den Thread wo ich das herhab finde ich gerade nicht, aber es entstand dadurch dass ich selbst erstellte und vordefinierte Räume gemischt hab.

Wenn ich mich recht erinnere haben die "Standard"-Räume den Namen mit "name: (de=Küche,en=Kitchen,etc...)" angegeben und selber definierte nur "name:eigenerRaum" oder "name:(en=eigenerRaum)" oder so ähnlich. da müsste ich am Abend nochmal im Detail schauen.

Jedenfalls schätze ich dass das ".de" hinten dran soll. oder man fragt ab was bei deinem Code retour kommt und nimmt gegebenenfalls das Ergebnis von meinem.

Werds mir am Abend nochmal ansehen und den Beitrag ergänzen.

@johannesa said in Zigbee-Geräte überwachen:

Diese Funktion wird so also nur einmal ausgeführt. richtig?

Genau das ist richtig! Habe es in meinen Codetags auch nochmal so angepasst wie Paul geschrieben hatte. Also ohne Variabel da diese dann ja wirklich unnötig ist.

Ah okay danke für deine Erklärung. Das kann dann auf jeden fall daran liegen. Ich habe es nur rausgenommen, da bei mir ein Fehler kommt wenn ich es drin lasse. Wahrscheinlich weil es bei mir nirgends so deklariert ist wie du es beschrieben hast. Dann müsstest du für dich das '.de' wieder da hintersetzen.

bahnuhr

@ciddi89

Du kannst es auch noch einfacher machen.
Timeout brauchst du dort nicht.

    log("Run Zigbee-Watchdog");
    zigbeeWatchdog();

@bahnuhr danke das weiss ich. Ich weiss ehrlich gesagt in Moment garnicht mehr woher ich das habe aber irgendwie habe ich das bei allen mein Skripten, das ich den paar ms zeit lasse nach Skriptstart um ausgeführt zu werden. Ich glaube ich hatte da mal was aufgeschnappt das beim neu starten nicht alles auf einmal dann abgefragt wird wegen Auslastung und so. Ob es wirklich Sinn macht ist die andere Sache. Aber das kann ja bestimmt einer sagen der mehr Ahnung davon hat.

bahnuhr

@ciddi89 sagte in Zigbee-Geräte überwachen:

das beim neu starten nicht alles auf einmal dann abgefragt

Das stimmt natürlich. Wenn das komplette System neu startet, dann starten die Scripte gleichzeitig.

frana120500

Moin zusammen,
wie kann ich denn aus der json, die jetzt beispielsweise in "devices_offline_list" steht, das so extrahieren, dass er mir eine Telegram Benachrichtigung sendet?

paul53

@ciddi89 sagte: paar ms zeit lasse nach Skriptstart um ausgeführt zu werden.

Das ist wegen der asynchronen createState(), die fertig sein müssen, wenn auf die Datenpunkte mit getState() / setState() zugegriffen wird.

@johannesa sagte in Zigbee-Geräte überwachen:

die Raumzuordnung hat mich ganz schön Nerven gekostet.

Mache es so:

        currRoom = getObject(id, 'rooms').enumNames[0];
        if(typeof currRoom == 'object') currRoom = currRoom.de;

Damit sind beide Fälle abgedeckt.

@paul53 Vielen dank für deine Erklärung und dein Hinweis. Habe es mit eingefügt. Ich hoffe ja das ich irgendwann auch nur einen Bruchteil von dem kann was du in Javascript kannst.

@JohannesA Habe es in den Script, welches ich weiter oben gepostet habe ergänzt. Vielleicht kannst du ja prüfen ob es jetzt bei dir funktioniert?

amg_666

@JohannesA
Beim Starten lief das Skirpt mit den oben erwähnten diversen Warnungen durch und die DP wurden angelegt.
Dann passiert aber nichts mehr, die DP bleiben leer und er loggt jede Menge Fehler:

javascript.0
	2022-02-04 12:06:00.036	error	at processTimers (internal/timers.js:500:7)
javascript.0
	2022-02-04 12:06:00.035	error	at listOnTimeout (internal/timers.js:557:17)
javascript.0
	2022-02-04 12:06:00.035	error	at Timeout._onTimeout (/opt/iobroker/node_modules/node-schedule/lib/Invocation.js:228:7)
javascript.0
	2022-02-04 12:06:00.035	error	at /opt/iobroker/node_modules/node-schedule/lib/Invocation.js:268:28
javascript.0
	2022-02-04 12:06:00.035	error	at Job.invoke (/opt/iobroker/node_modules/node-schedule/lib/Job.js:168:15)
javascript.0
	2022-02-04 12:06:00.035	error	at Job.job (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1316:34)
javascript.0
	2022-02-04 12:06:00.035	error	at Object.<anonymous> (script.js.Tests.zigbee_watchdog:243:5)
javascript.0
	2022-02-04 12:06:00.034	error	at zigbeeWatchdog (script.js.Tests.zigbee_watchdog:83:12)
javascript.0
	2022-02-04 12:06:00.034	error	at Object.result.each (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:814:29)
javascript.0
	2022-02-04 12:06:00.034	error	at script.js.Tests.zigbee_watchdog:89:55
javascript.0
	2022-02-04 12:06:00.033	error	script.js.Tests.zigbee_watchdog: TypeError: Cannot read property 'de' of undefined
javascript.0
	2022-02-04 12:06:00.021	info	script.js.Tests.zigbee_watchdog: Run Zigbee-Watchdog

Das ist der Teil des Scripts:

    zigbee.each(function (id, i) {

        currDeviceString = id.slice(0, (id.lastIndexOf('.') + 1) - 1);

        deviceName=getObject(currDeviceString).common.name

        currRoom = getObject(id, 'rooms').enumNames[0].de;

wo es bei mir keine Raumzuordnung auf die zigbee devices gibt. Vielleicht hab ichs überlesen, aber der Hinweis, dass das gefüllt sein muss wäre gut...

paul53

@amg_666 sagte: wo es bei mir keine Raumzuordnung auf die zigbee devices gibt.

Dann erweitere um die Abfrage, ob Raum vorhanden / ein Objekt ist:

        currRoom = getObject(id, 'rooms').enumNames[0];
        if(typeof currRoom == 'object') currRoom = currRoom.de;

liv-in-sky

@johannesa

ich habe mal die erstellung der datenpunkte geändert - evtl willst du das so einfügen - dann kommt auch kein warning beim ersten mal

dii dp-erstellung werden in einer function abgelegt

async function doStates(){

if (!(await existsStateAsync(stateDevicesCount))) await createStateAsync(stateDevicesCount, 0, { read: true, write: true, desc: "Anzahl Geräte gesamt", name: "Anzahl Geräte gesamt",type: 'number' });

if (!(await existsStateAsync(stateDevicesLinkQuality))) await createStateAsync(stateDevicesLinkQuality, "", { read: true, write: true, desc: "Liste Geräte Signalstärke", name: "Liste Geräte Signalstärke", type: 'string' });

if (!(await existsStateAsync(stateDevicesOfflineCount))) await createStateAsync(stateDevicesOfflineCount, 0, { read: true, write: true, desc: "Anzahl Geräte offline", name: "Anzahl Geräte offline",type: 'number' });

if (!(await existsStateAsync(stateDevicesOffline))) await createStateAsync(stateDevicesOffline, " ", { read: true, write: true, desc: "Liste Geräte offline", name: "Liste Geräte offline",type: 'string' });

if (!(await existsStateAsync(stateDevicesWithBattery))) await createStateAsync(stateDevicesWithBattery, "", {read: true, write: true, desc: "Liste Geräte mit Batterie", name: "Liste Geräte mit Batterie", type: 'string'});

if (!(await existsStateAsync(stateDevicesWithBatteryCount))) await createStateAsync(stateDevicesWithBatteryCount, 0, {read: true, write: true, desc: "Anzahl Geräte mit Batterie", name: "Anzahl Geräte mit Batterie", type: 'number'});

if (!(await existsStateAsync(stateDevicesInfoList))) await createStateAsync(stateDevicesInfoList, " ", {read: true, write: true, desc: "Liste aller Geräte", name: "Liste aller Geräte", type: 'string'});

if (!(await existsStateAsync(stateDevicesLastCheck))) await createStateAsync(stateDevicesLastCheck, " ", {read: true, write: true, desc: "Zeitpunkt letzter Überprüfung", name: "Zeitpunkt letzter Überprüfung", type: 'string'});
}

der aufruf im schedule ist dann anders:


schedule("6 */1 * * *", async function () {

   log("Run Zigbee-Watchdog");
   await doStates().then(zigbeeWatchdog);
  // zigbeeWatchdog()

});

das .de musste ich auch raus tun bei den enums - evtl noch wissenswert - wenn man die enums noch anlegen muss, muss die javascript instanz neugestartet werden, sonst kommt undefined zurück

ps: danke fürs script

War mal so Frei und habe nun alle Vorschläge schonmal mit reingenommen. Das mit der async Funktion hatte ich auch erst in betracht gezogen aber dann gelassen weil es dann ja auch ohne ging. Zumindest bei mir. Habe es nun aber übernommen. Danke @liv-in-sky

const basePath = "0_userdata.0.Steuerzentrale.ZigbeeWatchdog.";
const stateDevicesCount = basePath + "devices_count_all";
const stateDevicesLinkQuality = basePath + "devices_link_quality_list";
const stateDevicesOfflineCount = basePath + "devices_offline_count";
const stateDevicesOffline = basePath + "devices_offline_list";
const stateDevicesWithBatteryCount=basePath + "devices_battery_count";
const stateDevicesWithBattery=basePath + "devices_battery_list";
const stateDevicesInfoList=basePath + "devices_list_all";
const stateDevicesLastCheck=basePath + "lastCheck";


async function doStates(){

   if (!(await existsStateAsync(stateDevicesCount))) await createStateAsync(stateDevicesCount, 0, { read: true, write: true, desc: "Anzahl Geräte gesamt", name: "Anzahl Geräte gesamt",type: 'number' });
   if (!(await existsStateAsync(stateDevicesLinkQuality))) await createStateAsync(stateDevicesLinkQuality, "", { read: true, write: true, desc: "Liste Geräte Signalstärke", name: "Liste Geräte Signalstärke", type: 'string' });
   if (!(await existsStateAsync(stateDevicesOfflineCount))) await createStateAsync(stateDevicesOfflineCount, 0, { read: true, write: true, desc: "Anzahl Geräte offline", name: "Anzahl Geräte offline",type: 'number' });
   if (!(await existsStateAsync(stateDevicesOffline))) await createStateAsync(stateDevicesOffline, "", { read: true, write: true, desc: "Liste Geräte offline", name: "Liste Geräte offline",type: 'string' });
   if (!(await existsStateAsync(stateDevicesWithBattery))) await createStateAsync(stateDevicesWithBattery, "", {read: true, write: true, desc: "Liste Geräte mit Batterie", name: "Liste Geräte mit Batterie", type: 'string'});
   if (!(await existsStateAsync(stateDevicesWithBatteryCount))) await createStateAsync(stateDevicesWithBatteryCount, 0, {read: true, write: true, desc: "Anzahl Geräte mit Batterie", name: "Anzahl Geräte mit Batterie", type: 'number'});
   if (!(await existsStateAsync(stateDevicesInfoList))) await createStateAsync(stateDevicesInfoList, "", {read: true, write: true, desc: "Liste aller Geräte", name: "Liste aller Geräte", type: 'string'}); 
   if (!(await existsStateAsync(stateDevicesLastCheck))) await createStateAsync(stateDevicesLastCheck, "", {read: true, write: true, desc: "Zeitpunkt letzter Überprüfung", name: "Zeitpunkt letzter Überprüfung", type: 'string'});

}



function zigbeeWatchdog() {

   let maxMinutes = 300; // "Gerät offline" - Wert in Minuten: Gilt erst, wenn Gerät länger als X Minuten keine Meldung gesendet hat 
   let arrOfflineDevices = []; //JSON-Info alle offline-Geräte
   let arrLinkQualityDevices = []; //JSON-Info alle offline-Geräte
   let arrBatteryPowered = []; //JSON-Info alle batteriebetriebenen Geräte
   let arrListAllDevices = []; //JSON-Info Gesamtliste mit Info je Gerät
   let currDeviceString;
   let currDeviceBatteryString;
   let currRoom;
   let deviceName;
   let linkQuality;
   let lastContact;
   let lastContactString;
   let offlineDevicesCount;
   let deviceCounter=0;
   let batteryPoweredCount=0;
   let batteryHealth;

   const zigbee = $('zigbee.0.*.link_quality');

   zigbee.each(function (id, i) {
       currDeviceString = id.slice(0, (id.lastIndexOf('.') + 1) - 1);
       deviceName=getObject(currDeviceString).common.name
       currRoom = getObject(id, 'rooms').enumNames[0];
       if(typeof currRoom == 'object') currRoom = currRoom.de;



       // 1. Link-Qualität des Gerätes ermitteln
       //---------------------------------------
       
       linkQuality=parseFloat((100/255 * getState(id).val).toFixed(2)) + "%"; // Linkqualität in % verwenden
       //linkQuality=getState(id).val; // ALTERNATIV: Echt-Wert der Linkqualität (0-255) verwenden
       
       arrLinkQualityDevices.push({device: deviceName, room: currRoom, link_quality: linkQuality});


       // 2. Wann bestand letzter Kontakt zum Gerät
       //------------------------      
       lastContact = Math.round((new Date() - new Date(getState(id).ts)) / 1000 / 60);
       // 2b. wenn seit X Minuten kein Kontakt mehr besteht, nimm Gerät in Liste auf
       //Rechne auf Tage um, wenn mehr als 48 Stunden seit letztem Kontakt vergangen sind
       lastContactString=Math.round(lastContact) + " Minuten";
       if (Math.round(lastContact) > 100) {
           lastContactString=Math.round(lastContact/60) + " Stunden";
       } 
       if (Math.round(lastContact/60) > 48) {
           lastContactString=Math.round(lastContact/60/24) + " Tagen";
       } 
       if ( lastContact > maxMinutes) {
           arrOfflineDevices.push({device: deviceName, room: currRoom, lastContact: lastContactString});
       }

       // 3. Batteriestatus abfragen
       currDeviceBatteryString=currDeviceString+".battery";
       if (existsState(currDeviceBatteryString)) {
           batteryHealth=getState(currDeviceBatteryString).val + "%"; // Batteriestatus in %
           arrBatteryPowered.push({device: deviceName, room: currRoom, battery: batteryHealth});
       } else batteryHealth="-";
      
   arrListAllDevices.push({device: deviceName, room: currRoom, battery: batteryHealth, lastContact: lastContactString, link_quality: linkQuality});

   });


   // 1b. Zähle, wie viele Zigbee-Geräte existieren
   //---------------------------------------------       
   deviceCounter=arrLinkQualityDevices.length;
       //falls keine Geräte vorhanden sind, passe Datenpunkt-Inhalt der Geräte-Liste an
       if (deviceCounter == 0) { 
           arrLinkQualityDevices.push({device: "--keine--", room: "", link_quality: ""})
           arrListAllDevices.push({device: "--keine--", room: "", battery: "", lastContact: "", link_quality: ""});
       }

   // 2c. Wie viele Geräte sind offline?
   //------------------------   
   offlineDevicesCount=arrOfflineDevices.length;
       //falls keine Geräte vorhanden sind, passe Datenpunkt-Inhalt der Geräte-Liste an
       if (offlineDevicesCount == 0) { 
           arrOfflineDevices.push({device: "--keine--", room: "", lastContact: ""})
       }

   // 3c. Wie viele Geräte sind batteriebetrieben?
   //------------------------   
   batteryPoweredCount=arrBatteryPowered.length;
       //falls keine Geräte vorhanden sind, passe Datenpunkt-Inhalt der Geräte-Liste an
       if (batteryPoweredCount == 0) { 
           arrBatteryPowered.push({device: "--keine--", room: "", battery: ""})
       }

   // SETZE STATES
   setState(stateDevicesCount, deviceCounter);
   setState(stateDevicesLinkQuality, JSON.stringify(arrLinkQualityDevices));
   setState(stateDevicesOfflineCount, offlineDevicesCount);
   setState(stateDevicesOffline, JSON.stringify(arrOfflineDevices));
   setState(stateDevicesWithBatteryCount, batteryPoweredCount);
   setState(stateDevicesWithBattery, JSON.stringify(arrBatteryPowered));
   setState(stateDevicesInfoList, JSON.stringify(arrListAllDevices));
   setState(stateDevicesLastCheck, [formatDate(new Date(), "DD.MM.YYYY"),' - ',formatDate(new Date(), "hh:mm:ss")].join(''));
}

schedule("6 */1 * * *", async function () {
   log("Run Zigbee-Watchdog");
   await doStates().then(zigbeeWatchdog);
});

setTimeout (async function () {
   log("Run Zigbee-Watchdog");
   await doStates().then(zigbeeWatchdog);
}, 300);

liv-in-sky

@ciddi89

ich mache updates der scripte immer im ersten post - dann weiß man immer wo die aktuelle version liegt

dann verweiße ich zum ersten post , bei einer änderung

@liv-in-sky ja letztendlich liegt es ja an @JohannesA ob er das so übernimmt und wenn ja, kann er es ja im ersten Post ändern. Aber der kann erst heut Abend darüber schauen. Wollte nur schon mal die Arbeit abnehmen.

amg_666

@paul53 Irgendwie kapier ich es nicht. Es liegt an

  currRoom = getObject(id, 'rooms').enumNames[0].de;

Wieso ".de" ? Ich hab das gelöscht und jetzt werden dei Datenpunkte korrekt befüllt.

paul53

@amg_666 sagte: Ich hab das gelöscht und jetzt werden dei Datenpunkte korrekt befüllt.

Das ist o.k., wenn die Räume nicht benötigt werden / nicht zugewiesen sind.
Andernfalls mach die vorgeschlagene Änderung.

NEWS

Zigbee-Geräte überwachen

Support us

611
Online

31.6k
Users

79.5k
Topics

1.3m
Posts

NEWS

Zigbee-Geräte überwachen

Support us

611Online

31.6kUsers

79.5kTopics

1.3mPosts

611
Online

31.6k
Users

79.5k
Topics

1.3m
Posts