UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

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

[Ein ehemaliger Benutzer]

@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. :D

@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

Spoiler

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:

Spoiler

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

[Ein ehemaliger Benutzer]

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 🙈

Spoiler

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

[Ein ehemaliger Benutzer]

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

[liv-in-sky]

@ciddi89 ok - thread inhaber und mit thread-poster verwechselt - sorry

überlege gerade , ob man noch andere devices einbinden kann- wäre dann ja fast ein adapter wert :-)

hatte erst gestern ein problem mit einem sonoff device - habe zwar eigene tabellen für die geräte, aber eine sammlung aller geräte, die offline sind, wäre schon nicht schlecht

z.b https://forum.iobroker.net/topic/51217/html-tabelle-für-zigbee-devices-vis-übersicht

theoretisch die verschieden device-selektoren als array eintragen - in diesem array definiert man, zum einen das device (sonoff,zigbee) und noch zusätzlich die definition des z.b device name - dann das ganze existierende in einer schleife über alle im array

[Ein ehemaliger Benutzer]

@liv-in-sky said

überlege gerade , ob man noch andere devices einbinden kann- wäre dann ja fast ein adapter wert :-)

Die Idee hatte der threadersteller auch schon. Da könnte man wenigstens alle Devices zusammen fassen. (Ble, hm-ip, Zigbee usw.) Aber leider reichen unsere skills dafür nicht aus 😂

[liv-in-sky]

@ciddi89 glaube ich nicht :-)

das schwierige ist nur - wie man die verschiedenen dp reinbringt und in den listen kommt dann noch ein punkt dazu - sonoff, zigbee,..), damit man das auch weiß, welches device es ist

da ja der name des devices bei zigbee vom common.name kommt und bei sonoff vom dp ....hostname (sonoff ist da etwas doof, weil es drei verschiedene möglichkeiten geben kann, wo der hostname dp ist), wahrscheinlich muss dann wiederum mit selektoren arbeiten, die die richtigen dp finden

liegt an euch, ob ihr lust habt, dass anzugehen - unterstützung bekommt ihr sicher von uns hier im forum

[paul53]

@liv-in-sky sagte: der name des devices bei zigbee

Die Frage ist, ob es immer der Name des Device-Objektes sein soll, der angezeigt wird. Da es unterschiedlich viele Ebenen vom Datenpunkt bis zum Device-Objekt gibt, kann man das in einer Funktion berücksichtigen.

[liv-in-sky]

@paul53 genau darüber denke ich gerade nach :-)

[Ein ehemaliger Benutzer]

@liv-in-sky said in Zigbee-Geräte überwachen:

glaube ich nicht

Danke :)

Sagen wir mal so, abgeneigt wäre ich nicht. Wollte mich eh früher oder später mit sowas beschäftigen da ich auch noch andere Ideen habe die evtl. einen Adapter wert währen. Müsste mich erst mal darin ein Arbeiten. 😃

[liv-in-sky]

@ciddi89

ein beispiel: eine constante, die man als user definieren muss

theSektor ist der dp, der vom selektor gesucht wird
the Name ist die art, wo der hostname gesucht werden muss .. da habe ich noch nicht weiter überlegt,

zeile 1:

const myArrDev=[{"theSelektor":"zigbee.0.*.link_quality","theName":"common"}]

dann vor der each schleife:

 for(let x=0; x<myArrDev.length;x++){

    var zigbee = $(myArrDev[x].theSelektor);

 

    zigbee.each(function (id, i) {

        currDeviceString = id.slice(0, (id.lastIndexOf('.') + 1) - 1);
        //hier braucht man eine function, die den hostnamen findet:
        if (myArrDev[x].theName=="common")  deviceName=getObject(currDeviceString).common.name

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

 .... }

dann fehlt nur noch eine funktion, die mit "myArrrDev.theName" auf den richtigen dp kommt. das kann in das script so eingesetzt werden und funktioniert noch)

[Ein ehemaliger Benutzer]

@liv-in-sky okay das sollte ja machbar sein. Mal gucken ob sich Johannes das mal annehmen möchte aber ich versuche mich evtl auch mal daran. Werde mir mal die Umgebung fertig machen, mich ein bisschen einlesen und probiere mich da mal dran aus. :D Das Wetter hier in Ostfriesland ist sowieso inmoment total fürn a*** :D

[liv-in-sky]

@ciddi89

ich habe nur noch ein wenig gespielt:

sonoff ist soweit drin, dass keine fehler kommen - warte aber auf eure entscheidung und entwicklung

nur soviel - evtl kann man das besser machen:

zeile 1:

const myArrDev=[{"theSelektor":"zigbee.0.*.link_quality","theName":"common"},
                {"theSelektor":"sonoff.1.*.Wifi_RSSI","theName":"dp","thedpName":"Hostname"}
                ]

dann als beispiel , den hostnamen zu finden, der im gegesatz zu zigbee in einem dp zu finden ist

 for(let x=0; x<myArrDev.length;x++){

    var zigbee = $(myArrDev[x].theSelektor);

 

    zigbee.each(function (id, i) {

        currDeviceString = id.slice(0, (id.lastIndexOf('.') + 1) - 1);
        //hier braucht man eine function, die den hostnamen findet:
        if (myArrDev[x].theName=="common")  deviceName=getObject(currDeviceString).common.name
        if (myArrDev[x].theName=="dp") {
                                       let ida=id.split('.');
                                       let mySelect=$(ida[0]+'.'+ida[1]+'.'+ida[2]+'.*');
                                       mySelect.each(function (ad, i) {
                                           if (ad.includes(myArrDev[x].thedpName)) deviceName=getState(ad).val
                                       });
                                       


        }

wenn unter "theName" "dp" steht , wird das ganze über einen selektor gesucht - wenn es einen dp mit Hostname gibt, nimm den namen

so etwas braucht man auch noch für die batterie und auch link_quality
in den listen sollte dann auch noch ein extra punkt rein - welche art von device

[Ein ehemaliger Benutzer]

@liv-in-sky vielen vielen dank für den Input. Ich probiere mich mal in nächster Zeit ein wenig daran aus. :) Mal schauen was man daraus so machen kann.