Zigbee-Geräte überwachen

bahnuhr

@ciddi89

ja du hast recht.
Beim 2 mal kam der Fehler nicht.
Und DP wurde angelegt.

mfg

bahnuhr

Unter "alias" geht createstate nicht.
Das hatte ich probiert.

Wäre gut wenn es dort auch gehen würde.
(gehört aber nicht hier rein )

Soo habe mal ein paar Fehler korrigiert und ein bisschen ausgebessert was mir so auf den ersten Blick aufgefallen ist. Bin ja auch kein Profi aber man lernt ja immer dazu. Bei den Datenpunkterstellung waren bei 2-3 DPs der type falsch und auch ob er das erste mal mit 0 oder " " gefüttert werden soll. Den Fehler beim ersten Start habe ich wegbekommen, das die Datenpunkte noch nicht existieren und zusätzlich habe ich mir eine Funktion reingesetzt das er mir beim Start vom Skript schon mal die Datenpunkte befüttert. Jetzt meckert der Editor noch bei 3 Punkten und unterstreicht die Rot aber ich denke das kann man Ignorieren da es ja funktioniert. Wenn das Okay für dich ist @JohannesA, würde ich dann die von mir korrigierte Version hier runter setzen.

paul53

@bahnuhr sagte:

if (getState(stateDevicesCount).notExist)

Ersetze es durch

if (!existsState(stateDevicesCount))

damit die Warnung auch beim ersten Mal ausbleibt.

JohannesA

@ciddi89 Ja bitte - her damit!
Bin schon gespannt wie das Script jetzt aussieht!
Sobald die Kinder im Bett sind und meine Frau General vorm Fernseher eingeschlafen ist, nutze ich die Zeit um dazuzulernen.

@paul53
Danke für den Hinweis! "exitsState" hab ich zwischendurch sogar schon mal verwendet um zu checken ob das Gerät den Datenpunkt für die Batterie hat. Da hätte ich selber auch drauf kommen können, dies gleich bei der Anlage zu verwenden! Aber wie @bahnuhr schon sagte, die Meldung kam dann ja nicht wieder.

@johannesa ok gerne! Es sind bis jetzt ja nur Kleinigkeiten gewesen und davon wirklich nicht viele. Das mit der existsObject Funktion war ja auch meine erste Vermutung und Maßnahme um die Fehler im Log wegzubekommen beim ersten Start. Und da der Rest ja so funktioniert, ist ja alles Tip top und waren Schönheitskorrekturen. Nichts wildes. Wüsste auch erstmal nicht was man noch verbessern könnte. Aber da gibst sicher noch Profis die da bestimmt ganz andere Dinge sehen.

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";


if (!existsObject(stateDevicesCount)) {createState(stateDevicesCount, 0, { read: true, write: true, desc: "Anzahl Geräte gesamt", name: "Anzahl Geräte gesamt",type: 'number' })};
if (!existsObject(stateDevicesLinkQuality)) {createState(stateDevicesLinkQuality, " ", { read: true, write: true, desc: "Liste Geräte Signalstärke", name: "Liste Geräte Signalstärke", type: 'string' })};
if (!existsObject(stateDevicesOfflineCount)) {createState(stateDevicesOfflineCount, 0, { read: true, write: true, desc: "Anzahl Geräte offline", name: "Anzahl Geräte offline",type: 'number' })};
if (!existsObject(stateDevicesOffline)) {createState(stateDevicesOffline, " ", { read: true, write: true, desc: "Liste Geräte offline", name: "Liste Geräte offline",type: 'string' })};
if (!existsObject(stateDevicesWithBattery)) {createState(stateDevicesWithBattery, " ", {read: true, write: true, desc: "Liste Geräte mit Batterie", name: "Liste Geräte mit Batterie", type: 'string'})};
if (!existsObject(stateDevicesWithBatteryCount)) {createState(stateDevicesWithBatteryCount, 0, {read: true, write: true, desc: "Anzahl Geräte mit Batterie", name: "Anzahl Geräte mit Batterie", type: 'number'})};
if (!existsObject(stateDevicesInfoList)) {createState(stateDevicesInfoList, " ", {read: true, write: true, desc: "Liste aller Geräte", name: "Liste aller Geräte", type: 'string'})};
if (!existsObject(stateDevicesLastCheck)) {createState(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 * * *", function () {
    log("Run Zigbee-Watchdog");
    zigbeeWatchdog();
});

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

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;

NEWS

Zigbee-Geräte überwachen

Support us

592
Online

31.6k
Users

79.5k
Topics

1.3m
Posts

NEWS

Zigbee-Geräte überwachen

Support us

592Online

31.6kUsers

79.5kTopics

1.3mPosts

592
Online

31.6k
Users

79.5k
Topics

1.3m
Posts