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

[WernerS]

@ro75

habe ich gemacht, aber die fehler sind immer noch im log

Gruß
Werner

[Ro75]

@werners im JS-Adapter ist "sendto" erlaubt/aktiviert?

Ro75

[WernerS]

@ro75

habe sendtohost aktiviert. Die Fehler sind aber immer noch im log

Gruß
Werner

[fastfoot]

also die Fehlermeldung deuted darauf hin dass kein Ergebnis zurück kommt. Hier tippe ich darauf dass als Datenbank SQLITE verwendet wird. Dann allerdings muss die DB Bezeichnung iobroker. vor den Tabellennamen wegfallen.

wenn schon moment(der Nachfolger ist luxon) verwendet wird, dann sollte man es auch verwenden!

const moment = require('moment');
let tsNow = moment().startOf('month').unix() * 1000;

es geht aber auch mit Bordmitteln:

const ts1 = new Date();
ts1.setDate(1);
ts1.setHours(0, 0, 0, 0);
log(ts1.getTime());

letztlich noch das query, das kann man vereinfachen und sich somit 1 Abfrage spareb

const query = `select sum(ts_number.val) as monatssumme 
                 from datapoints join ts_number 
                   on datapoints.id = ts_number.id and datapoints.name = "${Traffic_Tag_IN}"
               where ts_number.ts >= ${tsNow}`;
sendTo('sql.0', 'query', query, (res) => {
    log(res.result[0].monatssumme)
})

[WernerS]

@fastfoot

als das ist eine MySQL Datenbank
ich kann das mal testen, aber was muss ich wie jetzt ersetzen?

Gruß
Werner

[fastfoot]

@werners sagte in Daten aus der Fritzbox visualisieren:

@fastfoot

als das ist eine MySQL Datenbank
ich kann das mal testen, aber was muss ich wie jetzt ersetzen?

Gruß
Werner

zeige zuerst mal die Einstellungen deines SQL Adapters

[WernerS]

@fastfoot

diese?

[fastfoot]

@werners mich hätte dein Datenbankname interessiert, ist der iobroker ? Wenn nicht muss das auch in den Selects geändert werden

[WernerS]

@fastfoot

Nein die heist anderst.
muss ich überall wo was mit
SELECT id FROM iobroker.datapoints
steht das iobroker dann entsprechend umbenennen?

Gruß
Werner

[fastfoot]

@werners genau, MySql verlangt den DB Namen vor den Tabellen. Es gibt eig. keinen Grund die DB anders zu benennen, und auch keinen den dann zu verheimlichen :grin:

[WernerS]

@fastfoot

Also er hat jetzt was gerechnet aber die Werte sind die vom 30.09. Ich habe mal in der Datenbank geschaut und die Werte gecheckt. Die sind nicht 100% gleich. Irgendwas stimmt da noch nicht. Kann es sein er hat was gerechnet und der Unterschied ist ein Rundungsfehler?

0_userdata.0.FritzBox.Traffic.Monat_IN 38509464882
0_userdata.0.FritzBox.Traffic.Tag_IN 38509510547

Gruß
Werner

[fastfoot]

@werners ich kenne weder dein Skript noch deine Daten, wie soll ich da helfen? Hast du denn die Änderungen von Ro75 eingepflegt?

[Ro75]

@werners hast du den Code von Post 62 übernommen?

Ro75

[WernerS]

@ro75
@fastfoot

Sorry fastfoot das wusste ich nicht. Also mit Post 62 kann ich hier nichts finden.
Ich hänge mal mein Script an vielleicht hilft das weiter

const Traffic_IN            = '0_userdata.0.FritzBox.Traffic.Traffic_IN';  // protokollieren mit SQL! Ggfs. zusätzlich mit Influx
const Traffic_OUT           = '0_userdata.0.FritzBox.Traffic.Traffic_OUT';  // protokollieren mit SQL! Ggfs. zusätzlich mit Influx
const Traffic_Tag_IN        = '0_userdata.0.FritzBox.Traffic.Tag_IN';  // protokollieren mit SQL! Ggfs. zusätzlich mit Influx
const Traffic_Tag_OUT       = '0_userdata.0.FritzBox.Traffic.Tag_OUT';  // protokollieren mit SQL! Ggfs. zusätzlich mit Influx
const Traffic_Monat_IN      = '0_userdata.0.FritzBox.Traffic.Monat_IN';  // protokollieren mit SQL! Ggfs. zusätzlich mit Influx
const Traffic_Monat_OUT     = '0_userdata.0.FritzBox.Traffic.Monat_OUT';  // protokollieren mit SQL! Ggfs. zusätzlich mit Influx

// die oberen 6 DP müssen protokolliert werden!

const sTraffic_Monat_IN     = '0_userdata.0.FritzBox.Traffic.Traffic_Monat_IN';
const sTraffic_Monat_OUT    = '0_userdata.0.FritzBox.Traffic.Traffic_Monat_OUT';

createState(Traffic_IN, 0, {type: 'number', read: true, write: true});
createState(Traffic_OUT, 0, {type: 'number', read: true, write: true});
createState(Traffic_Tag_IN, 0, {type: 'number', read: true, write: true});
createState(Traffic_Tag_OUT, 0, {type: 'number', read: true, write: true});
createState(Traffic_Monat_IN, 0, {type: 'number', read: true, write: true});
createState(Traffic_Monat_OUT, 0, {type: 'number', read: true, write: true});

createState(sTraffic_Monat_IN, 0, {type: 'number', read: true, write: true});
createState(sTraffic_Monat_OUT, 0, {type: 'number', read: true, write: true});

var IPV4='';
var IPV4I='';

var tIN_NEW = 0;
var tIN_OLD = 0;
var tIN_DIFF = 0;
var tIN_SUM = 0;

var tOUT_NEW = 0;
var tOUT_OLD = 0;
var tOUT_DIFF = 0;
var tOUT_SUM = 0;

//Summierung der Tageswerte via SQL für Monatsergebnis

function TrafficlfdMonat() {
    const moment = require("moment");
    let dateX=moment().format();
    var Stunden = ((parseFloat(formatDate(dateX, "TT"))-1)*24)*-1;
    
    var Ergebnis = ErstelleZahl(getState(Traffic_IN).val);
    sendTo('sql.0', 'query', 'SELECT id FROM iobrokerslave02.datapoints WHERE name="0_userdata.0.FritzBox.Traffic.Tag_IN"', function (result) {
        if (!result.error) {
            var now = new Date();
            now.setHours(Stunden);
            sendTo('sql.0', 'query', 'SELECT SUM(val) AS val FROM iobrokerslave02.ts_number WHERE ts >= ' + now.getTime() + ' AND id=' + result.result[0].id, function (result) {
                Ergebnis = Ergebnis + ErstelleZahl(parseFloat(result.result[0].val));
            });
        }
    });
    setTimeout(function(){
        setState(sTraffic_Monat_IN, Ergebnis, true);
    },100);
 
    var ErgebnisII = ErstelleZahl(getState(Traffic_OUT).val);
    sendTo('sql.0', 'query', 'SELECT id FROM iobrokerslave02.datapoints WHERE name="0_userdata.0.FritzBox.Traffic.Tag_OUT"', function (result) {
        if (!result.error) {
            var now = new Date();
            now.setHours(Stunden);
            sendTo('sql.0', 'query', 'SELECT SUM(val) AS val FROM iobrokerslave02.ts_number WHERE ts >= ' + now.getTime() + ' AND id=' + result.result[0].id, function (result) {
                ErgebnisII = ErgebnisII + ErstelleZahl(parseFloat(result.result[0].val));
            });
        }
    });
    setTimeout(function(){
        setState(sTraffic_Monat_OUT, ErgebnisII, true);
    },100);
}

//um die Monatssumme "laufend" zu halten
schedule('*/2 * * * *', function Auswertung7() {
    TrafficlfdMonat();
});

// Berechnung Empfangsrichtung
on({id: 'upnp.0.WANDevice_-_FRITZ!Box_7490.WANDevice.WANCommonInterfaceConfig.TotalBytesReceived', change: 'any'}, function(dp) { //DP ggfs. anpassen
    if (IPV4 == '') {
        IPV4 = getState('tr-064.0.states.externalIP').val;
    }

    tIN_OLD=tIN_NEW;
    tIN_NEW = parseFloat(dp.state.val);
    if (tIN_OLD == 0) {
        tIN_OLD=tIN_NEW;
    } else {
        if (IPV4 == getState('tr-064.0.states.externalIP').val) {
            tIN_DIFF=tIN_NEW-tIN_OLD;
        } else {
            IPV4 = getState('tr-064.0.states.externalIP').val;
            tIN_DIFF=tIN_NEW;
        }

        if (tIN_DIFF < 0) {
            tIN_DIFF=(4294967295-tIN_OLD)+tIN_NEW; //der Counter geht nur bis 4294967295, danach wieder ab 0 >> Berechnung
        }
        tIN_SUM=tIN_SUM+tIN_DIFF;
        var oIn = parseFloat(getState(Traffic_IN).val);
        setState(Traffic_IN, (oIn+tIN_DIFF)*1);
    }
});

// Berechnung Senderichtung

on({id: 'upnp.0.WANDevice_-_FRITZ!Box_7490.WANDevice.WANCommonInterfaceConfig.TotalBytesSent', change: 'any'}, function(dp) {

    if (IPV4I == '') {

        IPV4I = getState('tr-064.0.states.externalIP').val;

    }

 

    tOUT_OLD=tOUT_NEW;

    tOUT_NEW = parseFloat(dp.state.val);

 

    if (tOUT_OLD == 0) {

        tOUT_OLD=tOUT_NEW;

    } else {

        if (IPV4I == getState('tr-064.0.states.externalIP').val) {

            tOUT_DIFF=tOUT_NEW-tOUT_OLD;

        } else {

            IPV4I = getState('tr-064.0.states.externalIP').val;

            tOUT_DIFF=tOUT_NEW;

        }

 

        if (tOUT_DIFF < 0) {

            tOUT_DIFF=(4294967295-tOUT_OLD)+tOUT_NEW; //der Counter geht nur bis 4294967295, danach wieder ab 0 >> Berechnung

        }

        tOUT_SUM=tOUT_SUM+tOUT_DIFF;

        var oOUT = parseFloat(getState(Traffic_OUT).val);

        setState(Traffic_OUT, (oOUT+tOUT_DIFF)*1);

    }

});

 

//Tages und Monatswert in DP schreiben

schedule('30 59 23 * * *', function TrafficTagesWert() {

    setState(Traffic_Tag_IN, getState(Traffic_IN).val);

    setState(Traffic_Tag_OUT, getState(Traffic_OUT).val);

 

    //Reset für neuen Tag

    tOUT_SUM=0;

    tIN_SUM=0;

    setState(Traffic_IN, 0);

    setState(Traffic_OUT, 0);

 

    //herausfinden ob +- 10 Minuten immer noch den selben Monat oder letzter Tag im Monat ist

    const moment = require("moment");

    let datePlus=moment().add(10, 'm').format();

    let dateMinus=moment().subtract(10, 'm').format();

 

    if (formatDate(datePlus, "MM") != formatDate(dateMinus, "MM")) {

        //jetzt ist noch der letzte Tag des Monat

 

        TrafficlfdMonat();

 

        setState(Traffic_Monat_IN, getState('0_userdata.0.FritzBox.Traffic.Traffic_Monat_IN').val, true);

        setState(Traffic_Monat_OUT, getState('0_userdata.0.FritzBox.Traffic.Traffic_Monat_OUT').val, true);

 

    }

});



function ErstelleZahl(wert) {
    if(isNaN(wert)) {
        wert = 0;
    }
    return wert;
}

Gruß
Werner

[Ro75]

@WernerS

Das Skript ist so schon OK. Es gibt 2 Timeout-Bereiche mit je ",100".

    setTimeout(function(){
        xxxxxxxx
    },100);

Möglicherweise (ich habe da was mit SLAVE gesehen) ist dieser Wert zu gering. Ändere es mal so - ACHTUNG 2 Stellen ändern.

    setTimeout(function(){
        xxxxxxxx
    },500);

Ro75.

[WernerS]

@ro75

Ja Javascript läuft auf einem slave und die Sql auf einem anderen. habe die lasten etwas verteilen müssen.
Ich ändere das mal ab und dann warten wir mal vier Wochen
Danke für die Hilfe

Gruß
Werner

[fastfoot]

@werners sagte in Daten aus der Fritzbox visualisieren:

und dann warten wir mal vier Wochen

wozu? hiermit kannst du dir Monats- oder Tageswerte abrufen

Spoiler

const db = 'iobrokerslave02'
const stateID = '0_userdata.0.FritzBox.Traffic.Tag';

//summe(8, 3); // Der 3. September

summe(8); // Der gesamte September

function summe(monat = null, tag = null) {
   const monatsStart = new Date();
   const monatsEnde = new Date();

   monatsStart.setDate(tag || 1);
   monatsStart.setHours(0, 0, 0, 0);
   monatsStart.setMonth(monat || new Date().getMonth());

   monatsEnde.setMonth(monatsStart.getMonth() + (tag ? 0 : 1));
   monatsEnde.setDate(tag || 0);
   monatsEnde.setHours(23, 59, 59, 999);

   log(monatsStart.toString());
   log(monatsEnde.toString());

   const query = `
       SELECT b.name, sum(a.val) AS summe
         FROM ${db}.ts_number a
         JOIN ${db}.datapoints b ON a.id = b.id AND b.name LIKE "${stateID}%"
         WHERE a.ts BETWEEN ${monatsStart.getTime()} AND ${monatsEnde.getTime()}
         GROUP BY b.name`

   sendTo('sql.0', 'query', query, (res) => {
       if (/*!res.error &&*/ res.result?.length) {
           log(res.result[0].name.split('.')[4] + ' : ' + res.result[0].summe)
           log(res.result[1].name.split('.')[4] + ' : ' + res.result[1].summe)
       } else {
           log('Keine Daten!', 'warn');
           res.error && log(res.error, 'error')
       }
   })
}

[WernerS]

@fastfoot

Danke für das Script :+1: wusste nicht dass das auch nachträglich geht
Allerdings bringt das Script bei mir einen Fehler

javascript.0
	2024-10-04 10:20:12.837	error	at processImmediate (node:internal/timers:476:21)
javascript.0
	2024-10-04 10:20:12.836	error	at Immediate._onImmediate (file:///opt/iobroker/node_modules/@iobroker/db-states-redis/src/lib/states/statesInRedisClient.ts:381:37)
javascript.0
	2024-10-04 10:20:12.835	error	at change (/opt/iobroker/node_modules/@iobroker/js-controller-adapter/src/lib/adapter/adapter.ts:10886:45)
javascript.0
	2024-10-04 10:20:12.835	error	at Object.cb (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:3097:38)
javascript.0
	2024-10-04 10:20:12.834	error	at Object.<anonymous> (script.js.common.FritzBox_Daten_nachträglich_neuberechnen:33:23)
javascript.0
	2024-10-04 10:20:12.833	error	Error in callback: TypeError: Cannot read properties of undefined (reading 'length')
javascript.0
	2024-10-04 10:20:12.829	error	at processImmediate (node:internal/timers:476:21)
javascript.0
	2024-10-04 10:20:12.829	error	at Immediate._onImmediate (file:///opt/iobroker/node_modules/@iobroker/db-states-redis/src/lib/states/statesInRedisClient.ts:381:37)
javascript.0
	2024-10-04 10:20:12.829	error	at change (/opt/iobroker/node_modules/@iobroker/js-controller-adapter/src/lib/adapter/adapter.ts:10886:45)
javascript.0
	2024-10-04 10:20:12.829	error	at Object.cb (/opt/iobroker/node_modules/iobroker.javascript/lib/sandbox.js:3097:38)
javascript.0
	2024-10-04 10:20:12.828	error	at Object.<anonymous> (script.js.common.FritzBox_Daten_nachträglich_neuberechnen:33:23)
javascript.0
	2024-10-04 10:20:12.827	error	Error in callback: TypeError: Cannot read properties of undefined (reading 'length')

[fastfoot]

@werners Das kommt davon wenn man keine Fehlerbehandlung einbaut, my bad!

Kopiere bitte nochmal das verbesserte Skript und achte darauf dass der Datenbankname in Zeile 1 richtig ist. Dass der Statename in Zeile 2 verkürzt scheint ist Absicht, nicht abändern!

const db = 'iobrokerslave02'
const stateID = '0_userdata.0.FritzBox.Traffic.Tag';

//summe(8, 3); // Der 3. September

summe(8); // Der gesamte September

function summe(monat = null, tag = null) {
    const monatsStart = new Date();
    const monatsEnde = new Date();

    monatsStart.setDate(tag || 1);
    monatsStart.setHours(0, 0, 0, 0);
    monatsStart.setMonth(monat || new Date().getMonth());

    monatsEnde.setMonth(monatsStart.getMonth() + (tag ? 0 : 1));
    monatsEnde.setDate(tag || 0);
    monatsEnde.setHours(23, 59, 59, 999);

    log(monatsStart.toString());
    log(monatsEnde.toString());

    const query = `
        SELECT b.name, sum(a.val) AS summe
          FROM ${db}.ts_number a
          JOIN ${db}.datapoints b ON a.id = b.id AND b.name LIKE "${stateID}%"
          WHERE a.ts BETWEEN ${monatsStart.getTime()} AND ${monatsEnde.getTime()}
          GROUP BY b.name`

    sendTo('sql.0', 'query', query, (res) => {
        if (/*!res.error &&*/ res.result?.length) {
            log(res.result[0].name.split('.')[4] + ' : ' + res.result[0].summe)
            log(res.result[1].name.split('.')[4] + ' : ' + res.result[1].summe)
        } else {
            log('Keine Daten!', 'warn');
            res.error && log(res.error, 'error')
        }
    })
}

[danwir]

@ro75 entschuldige, kannst du mir sagen wo ich den Code reinkopieren muss ? Stehe da gerade ein wenig auf dem Schlauch. Hab iobroker drauf und den Adapter avm 64, welcher auch verbunden ist, aber nichts anzeigt.

Danke
Grüße
Daniel