Problem mit "Fenster offen Script"

Negalein

@paul53 sagte in Problem mit "Fenster offen Script":

Gesamtanzahl der Fenster.

Hallo Paul!

Jetzt steh ich wieder an und bräuchte dein (oder jeder der helfen könnte) Fachwissen.

Hab ein paar Fenstersensoren neu montiert und der Funktion Tuer zugewiesen.
Neuen DP in 0_userdata.0.Tuer.Anzahl als Zahl angelegt.

Trotz mMn passendem Script (musste ja nur DP und Funktion tauschen) meint er es sind immer 0.

const ids = Array.prototype.slice.apply($("(functions=Tuer)"));
setState("0_userdata.0.Tuer.Anzahl", ids.length, true);

paul53

@negalein sagte: der Funktion Tuer zugewiesen.

Lautet die ID "enum.functions.Tuer"? Oder ist "tuer" in der ID klein geschrieben?
Nach dem Erstellen von "Tuer" mal die Javascript-Instanz neu gestartet?

Negalein

@paul53 sagte in Problem mit "Fenster offen Script":

Lautet die ID "enum.functions.Tuer"? Oder ist "tuer" in der ID klein geschrieben?

Danke, das war der Fehler.

Ich hatte auch das Beispiel von @Codierknecht versucht (sogar klein geschrieben), weil mir mit dem uralt-Script jetzt die Anzahl der offenen nicht mehr angezeigt wird.
Da bekam ich bei .Anzahl und .Offen auch nur 0.

Im Log war auch nichts.

const ids = Array.prototype.slice.apply($("(functions=tuer)"));
 
log(ids);
 
on(ids, function () {
  let open = 0;
  for (let id of ids) {
    if (getState(id).val) open++;
  }
 
  let count = ids.length;
  log(open + ' von ' + count + ' Tür geöffnet');
 
  setState("0_userdata.0.Tuer.Anzahl", count, true);
  setState("0_userdata.0.Tuer.Offen", open, true);
 
});

Adnim

Hallo,
hänge mich mal hier an, da das Thema das gleiche ist :)

benutze noch das Device Counter Script. Da gibt es auch noch so eine schöne Liste welche Fenster offen sind.
Würde das aber gerne unter Userdata haben und nicht unter Javascript.

Bekomme es aber einfach nicht hin. Kann mir da mal jemand helfen?

//Add device rules
//addDeviceCounter("lowbat",         $('channel[state.id=*LOWBAT]'),                     true);
//addDeviceCounter("lowbat",         $('channel[state.id=*lowbat]'),                      true);
//addDeviceCounter("sticky_unreach", $('channel[state.id=*.STICKY_UNREACH]'),              true);
//addDeviceCounter("unreach",        $('channel[state.id=*.UNREACH]'),                     true);
//addDeviceCounter("config_pending", $('channel[state.id=*.CONFIG_PENDING]'),              true);
addDeviceCounter("Motion",  $('channel[state.id=*.MOTION]'),               true);
//addDeviceCounter("DutyCycle",  $('channel[state.id=*.DUTYCYCLE]'),               true);
//addDeviceCounter("Spannungsüberwachung",  $('channel[state.id=*.VOLTAGE]'),           function(state){return state<200;});
addDeviceCounter("Auskühlschutz",  $('channel[state.id=*.ACTUAL_TEMPERATURE]'),           function(state){return state<16;});
//addDeviceCounter("ERROR",  $('channel[state.id=*.ERROR*]'),               function(state){return state>0;});
//addDeviceCounter("ERROR1", $('channel[state.id=*.ERROR*]').each(function(id, i) {
 // log(id))};
addDeviceCounter("Heizbedarf",  $('channel[state.id=*.VALVE_STATE]'),           function(state){return state>13;});
//addDeviceCounter("Alive",         $('channel[state.id=*alive]'),                     false);
//addDeviceCounter("CONNECTED",         $('channel[state.id=*.connected]'),                     false);
//addDeviceCounter("open_shutters",  $('channel{TYPE=BLIND}[state.id=*.LEVEL]'),           function(state){return state>0;});
addDeviceCounter("open_windows",   $('channel{TYPE=SHUTTER_CONTACT}[state.id=*.STATE]'), true);





/******************************************
 * Code - do not change anything bellow this line *
 ******************************************/

function addDeviceCounter(name, selector, val_to_count) {

    //Total amount of devices found by selector
    var state_total = 'deviceCounter.' + name + '.total';
    createState(state_total, 0);
    //List of devices that match condition
    var state_list = 'deviceCounter.' + name + '.list';
    createState(state_list, "");
    //Number of devices that match condition
    var state_counter = 'deviceCounter.' + name + '.counter';

    createState(state_counter, 0);
    function countDevs(obj) {
        var d_count = 0;
        var d_list  = [];

        selector.each(function (id, i) {
            var status = getState(id).val;
            var obj    = getObject(id);
            if ((typeof val_to_count === 'function' && val_to_count(status)) ||
                (typeof val_to_count === 'boolean'  && val_to_count == status)) {
                //Get parent object
               var parentChannelId = id.slice(id.lastIndexOf("."));
                var parentDevicelId = id.split(".").slice(0,-2).join(".");
                var deviceObject = getObject(parentDevicelId);
                //remember device name
                d_list.push(deviceObject.common.name);
            }                
            ++d_count;                                                 // Zählt die Anzahl der vorhandenen Geräte unabhängig vom Status
        }); 

//if ("state_counter" > 0) {event(name + "(" + d_count + "): " + d_list);
//}

        setState(state_list,     d_list.join('<br>'));
        setState(state_counter,  d_list.length);
        setState(state_total,    d_count);
    }

    //Re-evaluate when a device status changes
    selector.on(function(obj) {    // bei Zustandänderung *. LOWBAT in allen Gewerken
        countDevs(obj);
    });

    //Re-evaluate each hour
    schedule("*/60 * * * *", countDevs);
    
    //Evaluate for the first time
    countDevs();
}

paul53

@adnim sagte: Würde das aber gerne unter Userdata haben und nicht unter Javascript.

In Zeilen 31, 34 und 37 vor 'deviceCounter.' zusätzlich '0_userdata.0.' einfügen.
Beispiel Zeile 31:

    var state_total = '0_userdata.0.deviceCounter.' + name + '.total';

Adnim

@paul53

danach kommt aber das im Log:


javascript.0
2024-04-21 09:04:49.847	warn	at script.js.common.DeviceCounter1:79:3

javascript.0
2024-04-21 09:04:49.847	warn	at script.js.common.DeviceCounter1:10:1

javascript.0
2024-04-21 09:04:49.847	warn	at addDeviceCounter (script.js.common.DeviceCounter1:76:5)

javascript.0
2024-04-21 09:04:49.847	warn	at countDevs (script.js.common.DeviceCounter1:64:9)

javascript.0
2024-04-21 09:04:49.847	warn	at setState (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1929:20)

javascript.0
2024-04-21 09:04:49.847	warn	State "0_userdata.0.deviceCounter.Auskühlschutz.total" not found

javascript.0
2024-04-21 09:04:49.847	warn	at script.js.common.DeviceCounter1:79:3

javascript.0
2024-04-21 09:04:49.847	warn	at script.js.common.DeviceCounter1:10:1

javascript.0
2024-04-21 09:04:49.847	warn	at addDeviceCounter (script.js.common.DeviceCounter1:76:5)

javascript.0
2024-04-21 09:04:49.847	warn	at countDevs (script.js.common.DeviceCounter1:63:9)

javascript.0
2024-04-21 09:04:49.847	warn	at setState (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1929:20)

javascript.0
2024-04-21 09:04:49.846	warn	State "0_userdata.0.deviceCounter.Auskühlschutz.counter" not found

javascript.0
2024-04-21 09:04:49.846	warn	at script.js.common.DeviceCounter1:79:3

javascript.0
2024-04-21 09:04:49.846	warn	at script.js.common.DeviceCounter1:10:1

javascript.0
2024-04-21 09:04:49.846	warn	at addDeviceCounter (script.js.common.DeviceCounter1:76:5)

javascript.0
2024-04-21 09:04:49.846	warn	at countDevs (script.js.common.DeviceCounter1:62:9)

javascript.0
2024-04-21 09:04:49.846	warn	at setState (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:1929:20)

javascript.0
2024-04-21 09:04:49.846	warn	State "0_userdata.0.deviceCounter.Auskühlschutz.list" not found

javascript.0
2024-04-21 09:04:49.067	info	Start javascript script.js.common.DeviceCounter1

Edit: Scheint aber nur beim ersten Start zu sein, skript funktioniert. Danke!

werner2000x

Bei mir werden keine offene Fenster angezeigt.
Ich habe in den objekten den Fenstern die Rolle "sensor. Window" gegeben.
Ist das richtig oder muss das "Fenster" sein?
Gibt es zu dem script ggf. eine Anleitung?

paul53

@werner2000x sagte: die Rolle "sensor. Window" gegeben.

Richtige Rolle: "sensor.window"
Wenn nach der Rolle selektiert werden soll, dann mit so einem Selektor

$('*[role=sensor.window]')

werner2000x

@paul53 Ich bekomme kein Ergebnis, immer 0 erkannte Fenster.
Ich habe allerdings kein Homematic.

paul53

@werner2000x sagte: immer 0 erkannte Fenster.

Zeige bitte Datenpunkte.
Was ergibt?

log($('*[role=sensor.window]'));

@werner2000x sagte in Problem mit "Fenster offen Script":

Ich habe allerdings kein Homematic.

Das hat nichts mit HomeMatic zu tun.

werner2000x

@paul53 sagte in Problem mit "Fenster offen Script":

log($('*[role=Fenster]'));

keine Ausgabe, die Rolle habe ich auf "Fenster" geändert

Homoran

@werner2000x und bitte noch

@paul53 sagte in Problem mit "Fenster offen Script":

Zeige bitte Datenpunkte.

werner2000x

@homoran hier 2 Datenpunkte der Fenstersensoren.

ein Datenpunkt:

{
  "type": "state",
  "common": {
    "name": "Contact event",
    "type": "boolean",
    "read": true,
    "write": false,
    "role": "Fenster"
  },
  "native": {},
  "_id": "zigbee.0.00158d008b158119.contact",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  },
  "from": "system.adapter.admin.0",
  "user": "system.user.admin",
  "ts": 1731498455840
}

paul53

@werner2000x sagte: die Rolle habe ich auf "Fenster" geändert

Gerade getestet: Bei mir funktioniert der Selektor mit der unzulässigen Rolle "Fenster".

log($('*[role=Fenster]').toArray());

liefert

javascript.0	14:14:00.756	info	script.js.common.Neuer_Test: [ '0_userdata.0.Test.on' ]

Codierknecht

@werner2000x sagte in Problem mit "Fenster offen Script":

keine Ausgabe, die Rolle habe ich auf "Fenster" geändert

Warum macht man sowas?
Die role eines State hat eine semantische Bedeutung. Die sollte man nicht willkürlich ändern.

Alternative wäre eine Aufzählung (Kategorie). Die kann man frei vergeben.
Hinzu kommt, dass man dazu gar nicht erst im Expertenmodus in den Objekten rumfummeln muss.

Auf die kann man dann auch ganz elegant zugreifen:

log($('(functions=window)'));

https://www.iobroker.net/#en/documentation/dev/stateroles.md

werner2000x

Ich habe ja die Rolle nur geändert weil nichts ankam.
Jetzt wieder rolle als "sensor.window".
Es wird noch kein offenes Fenster gefunden. Außerdem habe ich Fenstersensoren
bei denen bei 1 die Fenster auf sind und andere Sensoren bei denen bei 1 die Fenster zu sind.

Meine Kategorie sieht allerdings anders aus. Dort fehlt "Fenster".

Codierknecht

@werner2000x sagte in Problem mit "Fenster offen Script":

Meine Kategorie sieht allerdings anders aus. Dort fehlt "Fenster".

Wenn nicht vorhanden: Selbst anlegen ;-)

Außerdem habe ich Fenstersensoren bei denen bei 1 die Fenster auf sind und andere Sensoren bei denen bei 1 die Fenster zu sind.

Dazu bieten sich Aliase an, um das zu harmonisieren.

werner2000x

@codierknecht Habe ich gemacht:

Doch werden noch immer keine offenen Fenster gefunden.

Homoran

@werner2000x sagte in Problem mit "Fenster offen Script":

Habe ich gemacht:

was genau?

werner2000x

@homoran eine Kategorie mit der Funktion Fenster eröffnet und dort alle Fenster hereingebracht.

Community Forum

NEWS

Problem mit "Fenster offen Script"

Support us

784

32.4k

81.5k

1.3m