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

[deifel]

Danke Euch Beiden! Es klappt jetzt mit beiden Varianten. Wobei evtl. das von dslraser sicherer ist, da es nicht nur noch low* abfragt - keine Ahnung, was in Zukunft sonst noch "low" wird ;)

Vielen Dank für die tatkräftige Unterstützung!!!

[deifel]

Kleine Ergänzung: Bei der Lösung von dslraser werden die cux-Geräte gefunden und eine Warnung angezeigt...
Ich lass es jetzt trotzdem so drin

[paul53]

@deifel sagte:

werden die cux-Geräte gefunden

Es hätte mich auch sehr gewundert, wenn nicht.

[Fogg]

@Xanon said in Skript LOWBAT:

Danke, das Skript sieht gut aus und funktionierte auf anhieb!!

Um Deine Frage zu beantworten, wie wäre es mit:

addDeviceCounter("lowbat",      $('channel[state.id=*.LOW_BAT]') + $('channel[state.id=*.LOWBAT]'),                     true);

Habs aber nicht ausprobiert, habe keine IP Komponenten.

[markphilip]

Hallo zusammen,

sorry das ich den alten thread noch mal auspacke. Ich habe allerdings das problem das meine Homematic Ip Komponenten nicht erkannt werden.
Ich habe das Script folgendermaßen geändert:

createState('zählenLowbat.möglicheLOWBAT', 0); // wenn benötigt: Anzahl der Geräte mit Batterie
createState('zählenLowbat.anzahlLOWBAT', 0); // wenn benötigt: Anzahl der vorhandenen LOWBAT
createState('zählenLowbat.textLOWBAT', " "); // Namen LOWBAT, die an sind als Variable unter Javascript.0 anlegen

var cacheSelectorLOWBAT = $('channel[state.id=*0.LOWBAT]'); // für Homematic
var cacheSelectorLOW_BAT = $('channel[state.id=*0.LOW_BAT]'); // für Homematic IP

function countLowbat(obj) {
// Setzt die Zähler vor dem Durchlauf aller Elemente *.LOWBAT auf 0
var moeglicheLOWBAT = 0;
var textLOWBAT = [];

if (obj) {
log('Auslösender Aktor: ' + obj.id + ': ' + obj.newState.val); // Info im Log, welcher Zustand sich geändert hat
} else {
log('Ausgelöst bei Timer');
}

cacheSelectorLOWBAT.each(function (id, i) { // Schleife für jedes gefundenen Element *0.LOWBAT
var status = false;
if(getState(id)) {
status = getState(id).val; // Zustand *0.LOWBAT abfragen (jedes Element)
}
var obj = getObject(id);
if (status === true) { // wenn Zustand = true, wird das Array erweitert
textLOWBAT.push(obj.common.name); // Zu Array hinzufügen
}
log("Geräte Nr. " + i + ": " + getObject(id).common.name + ": " + status);
++moeglicheLOWBAT; // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
});

// Schleife ist durchlaufen. Im Log wird der aktuelle Status (Anzahl, davon LOWBAT zutreffend) ausgegeben
log("Text: " + textLOWBAT);
log("Anzahl Geräte: " + moeglicheLOWBAT + " # davon LOWBAT erkannt: " + textLOWBAT.length);

// die ermittelten Werte werden als javascript.0. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
setState("zählenLowbat.textLOWBAT", textLOWBAT.join(',<br>')); // Schreibt die Namen der Geräte mit LOWBAT Meldung
setState("zählenLowbat.anzahlLOWBAT", textLOWBAT.length); // Schreibt die aktuelle Anzahl der LOWBAT-Meldungen
setState("zählenLowbat.möglicheLOWBAT", moeglicheLOWBAT); // Schreibt die Anzahl der vorhandenen Batterie-Geräte
}

cacheSelectorLOWBAT.on(function(obj) { // bei Zustandänderung . LOWBAT in allen Gewerken
countLowbat(obj);
});
schedule("/60 * * * *", function () { //oder!! soll entweder ausgelöst werden alle 10 Minuten
log("===>Will be triggered every 60 minutes!");
countLowbat();
});

muss ich sonst noch was ändern?

[paul53]

@markphilip sagte:

var cacheSelectorLOWBAT = $('channel[state.id=*0.LOWBAT]'); // für Homematic
var cacheSelectorLOW_BAT = $('channel[state.id=*0.LOW_BAT]'); // für Homematic IP

Du verwendest anschließend nur cacheSelectorLOWBAT. Ändere den mal in

const cacheSelectorLOWBAT  = $('hm-rpc.*.0.LOW*');

Javascript-Code bitte immer in Code tags </> posten.

[opossum]

Hallo, @paul53,

ich habe das Batteriescript im Einsatz, läuft auch und zeigt mir zuverlässig die LowBats an.
Jetzt habe ich ein Homematicgerät in RaspberryMatic abgelernt, weil es mit dem Gerät nicht funktioniert hat. Es handelt sich um einen Magnet-Tür-Kontakt.
Leider wirft mir nun das Script eine Warnung aus:

getState "hm-rpc.1.OEQ0167836.1.LOWBAT" not found (3)

Das ist genau das abgelernte Gerät. Wo zieht sich das Script die Daten für die Geräte her? Ich suche schon ziemlich lange. Hier scheint es ein Problem mit der Löschverfolgung zu geben. Wenn Du nähere Angaben benötigst, lass' es mich bitte wissen. Vielen Dank für Deine Hilfe.

[paul53]

@opossum
Starte die Javascript-Instanz neu, damit deren Puffer aktualisiert wird.

[opossum]

Hallo, @paul53,

manche Dinge sind so einfach, und man vergisst es immer wieder. Das Problem saß mal wieder vorm Bildschirm. Danke Dir.

[Semmy]

@bluefox sagte in Skript LOWBAT:

@Homoran:

Was muss ich tun??

Das Lowbat-Script wäre unbedingt auch etwas für die Homepage! `
nimm das:

createState('zählenLowbat.möglicheLOWBAT', 0);   // wenn benötigt: Anzahl der vorhandenen LOWBAT
createState('zählenLowbat.anzahlLOWBAT', 0);     // wenn benötigt: Anzahl der vorhandenen LOWBAT
createState('zählenLowbat.textLOWBAT', " ");     // Anzahl LOWBAT, die an sind als Variable unter Javascript.0 anlegen

var cacheSelectorLOWBAT  = $('channel[state.id=*.LOWBAT]');

function countLowbat(obj) {
	// Setzt die Zähler vor dem Durchlauf aller Elemente *.LOWBAT auf 0
	var moeglicheLOWBAT = 0;
	var anzahlLOWBAT    = 0;
	var textLOWBAT      = [];

	if (obj) {
		log('Auslösender Aktor: ' + obj.id + ': ' + obj.newState.val);  // Info im Log, welcher Zustand sich geändert hat
	} else {
		log('Ausgelöst bei Timer'); 
	} 

	cacheSelectorLOWBAT.each(function (id, i) {                         // Schleife für jedes gefundenen Element *.LOWBAT
		var status = getState(id).val;                                  // Zustand *.LOWBAT abfragen (jedes Element)
		var obj    = getObject(id);
		if (status === true) {                                          // wenn Zustand = true, dann wird die Anzahl der Geräte hochgezählt
			textLOWBAT.push(obj.common.name);                           // Zu Array hinzufügen
		}                
		log("Geräte Nr. " + i + ": " + getObject(id).common.name + ": " + status);
		++anzahlLOWBAT;                                                 // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
	}); 

	// Schleife ist durchlaufen. Im Log wird der aktuelle Status (Anzahl, davon LOWBAT zutreffend) ausgegeben
	log("Text: " + textLOWBAT);
	log("Anzahl Geräte: " + moeglicheLOWBAT + " # davon LOWBAT erkannt: " +  anzahlLOWBAT);

	// die ermittelten Werte werden als javascript.0\. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
	setState("zählenLowbat.textLOWBAT",     textLOWBAT.join(',
')); // Schreibt die aktuelle Namen der Geräte mit LOWBAT Meldung
	setState("zählenLowbat.anzahlLOWBAT",   textLOWBAT.length);        // Schreibt die aktuelle Anzahl der Geräte im System
	setState("zählenLowbat.möglicheLOWBAT", moeglicheLOWBAT);          // Schreibt die aktuelle Anzahl der vorhandene Geräte 
}

cacheSelectorLOWBAT.on(function(obj) {    // bei Zustandänderung *. LOWBAT in allen Gewerken
   countLowbat(obj);
});
schedule("*/60 * * * *", function () {                                  //oder!! soll entweder ausgelöst werden alle 10 Minuten
   log("===>Will be triggered every 60 minutes!"); 
   countLowbat();
});

Ich bekomme immer diese Fehlermeldung

17:59:05.589	error	javascript.0 (574) script.js.Haus.Test_Batterie compile failed: at script.js.Haus.Test_Batterie:38

[paul53]

@semmy sagte: Fehlermeldung

Füge mal Zeilen 34 und 35 zu einer Zeile zusammen.

[Homoran]

@semmy du referenzierst tatsächlich einen Post von 2015?
bist du Archäologe?

Bist du sicher, dass das Skript (wie genau sieht das jetzt bei dir aus??) heute noch passt?

[Semmy]

@paul53 Fehlermeldung ist weg. Werden mit den Script auch IP-Geräte angezeigt?

[Homoran]

@semmy sagte in Skript LOWBAT:

Werden mit den Script auch IP-Geräte angezeigt?

eher nicht.
ich weiß gar nicht, ob es die schon 2015 gab.
Außerdem ist die Datenstruktur da anders.

Aber @liv-in-sky hat dazu was tolles gebastelt. musst du mal suchen

[bahnuhr]

@semmy

Schau dir den Selektor an, dann kannst du die Frage selber beantworten!

bei HM: LOWBAT
bei IP: LOW_BAT

Also nein, nur HM

alternative: Script umschreiben.

oder du nimmst mein geändertes Script:
DP natürlich anpassen.

// Batterie Meldung über leere Batterien

    var textLOWBAT = []; 
    var bat_arr = [];
    var logging = false;
    var cacheSelectorLOWBAT  = $('channel[state.id=hm-rpc.*0.LOW*]').each(function(id, i) {
        if(id.indexOf('CUX') == -1 && id.indexOf('ALARM') == -1) { bat_arr.push(id); }       // Objekte mit "cux" oder "Alarm" im Namen nicht in array schreiben
    });

weiter();

function weiter() {
    textLOWBAT = []; 
    bat_arr.forEach(Ermittlung);

    function Ermittlung(id, i) {
        var status = getState(id).val;                                  // Zustand *.LOWBAT abfragen (jedes Element)
        var obj = getObject(id);
        var sname = getObject(id).common.name;
        var sdevicename = sname.replace(/.LOWBAT/g, '');   // .lowbat aus Text entfernen
        sdevicename = sdevicename.replace(/.LOW_BAT/g, '');   // .lowbat aus Text entfernen
        sdevicename = sdevicename.replace(/:0/g, '');     // :0 aus Text entfernen

        if (logging) log("a) " + i + " : " + id + " : " + status + " : " + sdevicename); 

            if (status == true) {      
                    if (logging) log("c) dazu fügen: " + sdevicename); 
                    textLOWBAT.push('- ' + sdevicename);                           // Zu Array hinzufügen
                    log("Batterie leer: Gerät: " + sdevicename);
            }
    }

    // Schleife ist durchlaufen. 
        if (logging) log ("d) " + textLOWBAT.length);
        //textLOWBAT = deleteDuplicates(textLOWBAT);   // es geht nicht -> warum weiß ich nicht !
        textLOWBAT.sort(); 
        if (logging) log ("e) " + textLOWBAT.join('<br>').replace(/- /g,'') );
        setState('javascript.0.System.Batterie.Batterie_LOWBAT', textLOWBAT.join('<br>').replace(/- /g,'') );    // Schreibt die aktuelle Namen der Geräte mit LOWBAT Meldung
        setState('javascript.0.System.Batterie.Batterie_Anzahl', textLOWBAT.length); 
    // Telegram Meldung
        var sText = "Batterie Meldung vom:   " + Datum_holen("tmj") + "\n--------------------------------------------------------" + "\n";
        if (textLOWBAT.length === 0) {
            sText = sText + "Keine leeren Batterien in Homematic Geräten gefunden.";
        } else {
            sText = sText + "Bei folgenden Homematic Geräten ist die Batterie leer:\n";
            sText = sText + textLOWBAT.join('\n');
        }
        sendTo('telegram.0', sText); 
        //log (sText);
}

on({id: bat_arr, change: "ne"}, function(){        // bei Zustandänderung im "bat_arr*
    log("Auslöser: bat_arr"); 
    weiter();
});


schedule("15 0 * * *", function () {        //  nachts Batterie Status überprüfen
    log("Auslöser: Schedule"); 
    weiter();
});

[Homoran]

@bahnuhr sagte in Skript LOWBAT:

bei HM: LOWBAT
bei IP: LOW_BAT

Bei HM auch bool und bei IP in Volt, oder?

[paul53]

@semmy sagte: Werden mit den Script auch IP-Geräte angezeigt?

Nein, es sei denn, der $Selector (Zeile 5) wird angepasst:

var cacheSelectorLOWBAT  = $('hm-rpc.*.0.LOW*');

@homoran sagte in Skript LOWBAT:

bei IP in Volt, oder?

LOW_BAT ist auch boolean.

[Semmy]

@paul53 Ich habe das folgende Script schon Jahre am Laufen, es wird mir zwar angezeigt, dass Batterien leer sind, aber das Gerät dazu nicht.

 var debug = true;
 
createState('zählenLowbat.möglicheLOWBAT', 0);   // wenn benötigt: Anzahl der vorhandenen Geräte
createState('zählenLowbat.anzahlLOWBAT', 0);     // wenn benötigt: Anzahl der vorhandenen LOWBAT
createState('zählenLowbat.textLOWBAT', " ");     // Anzahl LOWBAT, die an sind als Variable unter Javascript.0 anlegen
 
var cacheSelectorLOWBAT  = $('channel[state.id=*.LOWBAT]');
var cacheSelectorLOW_BAT  = $('channel[state.id=*.LOW_BAT]');   // für Homematic IP, dort LOWBAT in LOW_BAT umbenannt
 
function countLowbat(obj) {
   // Setzt die Zähler vor dem Durchlauf aller Elemente *.LOWBAT auf 0
   var moeglicheLOWBAT = 0;
   var anzahlLOWBAT    = 0;
   var textLOWBAT      = [];
 
   if (obj) {
      if(debug) log('Auslösender Aktor: ' + obj.id + ': ' + obj.newState.val);  // Info im Log, welcher Zustand sich geändert hat
   } else {
      if(debug) log('Ausgelöst bei Timer'); 
   } 
 
   cacheSelectorLOWBAT.each(function (id, i) {                         // Schleife für jedes gefundenen Element *.LOWBAT
      var status = getState(id).val;                                  // Zustand *.LOWBAT abfragen (jedes Element)
        if (status === true) {                                              // wenn Zustand = true, dann wird die Anzahl der Geräte hochgezählt
        var name = getObject(id).common.name;                                   // neu eingefügt
        var devicename = name.substring(0, name.indexOf(":0.LOWBAT"));    // neu eingefügt, LOWBAT aus Bezeichnung entfernen   
        textLOWBAT.push(devicename);                                   // Zu Array hinzufügen; aktualisiert, vormals obj.common.name
        ++anzahlLOWBAT;
      }                
      if(debug) log("Geräte Nr. " + i + ": " + getObject(id).common.name + ": " + status);
      ++moeglicheLOWBAT;                                                 // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
   }); 
 
   cacheSelectorLOW_BAT.each(function (id, i) {                         // Schleife für jedes gefundenen Element *.LOW_BAT, Homematic IP
      var status = getState(id).val;                                    // Zustand *.LOW_BAT abfragen (jedes Element)
        if (status === true) {                                            // wenn Zustand = true, dann wird die Anzahl der Geräte hochgezählt      
        var name = getObject(id).common.name;                             // neu eingefügt
        var devicename = name.substring(0, name.indexOf(":0.LOW_BAT"));   // neu eingefügt, LOW_BAT aus Bezeichnung entfernen         
        textLOWBAT.push(devicename);                                   // Zu Array hinzufügen; aktualisiert, vormals obj.common.name
        ++anzahlLOWBAT;
      }                
      if(debug) log("Geräte Nr. " + i + ": " + getObject(id).common.name + ": " + status);
      ++moeglicheLOWBAT;                                                 // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
   }); 
 
 
 
   // Schleife ist durchlaufen. Im Log wird der aktuelle Status (Anzahl, davon LOWBAT zutreffend) ausgegeben
   if(debug) log("Text: " + textLOWBAT);
   log("Anzahl Geräte: " + moeglicheLOWBAT + " # davon LOWBAT erkannt: " +  anzahlLOWBAT);
 
   // die ermittelten Werte werden als javascript.0. Variable in ioBroker gespeichert (z.B. für die Verarbeitung in VIS)
   setState("zählenLowbat.textLOWBAT",     textLOWBAT.join('<br>')); // Schreibt die aktuelle Namen der Geräte mit LOWBAT Meldung
   setState("zählenLowbat.anzahlLOWBAT",   anzahlLOWBAT);        // Schreibt die aktuelle Anzahl der Geräte im System
   setState("zählenLowbat.möglicheLOWBAT", moeglicheLOWBAT);          // Schreibt die aktuelle Anzahl der vorhandene Geräte 
}
 
cacheSelectorLOWBAT.on(function(obj) {    // bei Zustandänderung *. LOWBAT in allen Gewerken
   countLowbat(obj);
});
 
cacheSelectorLOW_BAT.on(function(obj) {    // bei Zustandänderung *. LOW_BAT in allen Gewerken
   countLowbat(obj);
});
 
 
// Bei Start
 
countLowbat();

[paul53]

@semmy sagte: das Gerät dazu nicht.

Der DP "zählenLowbat.textLOWBAT" enthält nicht die Gerätenamen? Dann ist die Struktur der Datenpunktnamen wahrscheinlich abweichend.
Man kann auch direkt auf die Gerätenamen zugreifen (Zeilen 25, 26 und 37, 38 ersetzen):

        let idDev = id.substring(0, id.lastIndexOf('.'));
        idDev = id.substring(0, idDev.lastIndexOf('.'));
        let devicename = getObject(idDev).common.name;

[Semmy]

@paul53 sagte in Skript LOWBAT:

let idDev = id.substring(0, id.lastIndexOf('.'));
idDev = id.substring(0, idDev.lastIndexOf('.'));
let devicename = getObject(idDev).common.name;

Ok, jetzt funktioniert es wieder wie es soll.
Vielen Dank

Gruß
Semmy