[gelöst] SQL getHistory Abfrage

[ostseeskipper]

@apollon77

wenn die Beispiele schon nicht funktionieren was kann soll man dann tun?
Der SQL Adapter zeichnet Datenpunkte wie gewünscht auf und stellt die Infos auch bereit, also muss er doch eine Datenbank haben.

Könnt Ihr bitte euer Beispiel fixen damit man das nutzen und daran lernen kann.
lieben Dank

[ostseeskipper]

@apollon77

auch mit addId: true ... gibt es undefined

[apollon77]

@ostseeskipper bitte ganz locker bleiben. Was denkst du was wir in dieser Konversation gerade tun?! Du hast auf ein Problem hingewiesen und ich versuche zu helfen das wir die Beispiele fixen können und ich hab noch nicht mal um ein github issue gebeten.

Ich nehme also jetzt mit das es auch mit addId nicht geht. Dann muss ich wohl mal genauer schauen.

[ostseeskipper]

@apollon77
ich bleib locker und hab gesucht
hab den Fehler gefunden und auf Github was geschrieben

edit: Sorry wenn es zu emotional rüber kam, Ihr macht schon tolle arbeit

[apollon77]

@ostseeskipper Hi,

ok, aber das was Du jetzt af GitHub hast ist was anderes ... oder ist es quasi kombiniert?

Weil "query" ist was anderes als "getHistory"

[fastfoot]

• Bei query fehlt die Angabe der Datenbank vor den Tabellennamen, steht auch so im Readme. Also SELECT id FROM iobroker.datapoints....
• Bei getHistory wird id nicht mit zurück geliefert, allerdings würde ich mich fragen wozu, denn die Id ist ja Bestandteil der Abfrage. Es werden nur ts und val geliefert.
• addId hat tatsächlich keinen Effekt, habe es im source auch nur einmal gefunden, genutzt scheint es nicht zu werden.

[ostseeskipper]

@apollon77

Es ist kombiniert, da ja query und gethistory BuildIn Funktionen sind

Da ich mit gethistory nicht weitergekommen bin hatte ich query versucht und auch da fehler bekommen
Query sagte mir das die Datenbank nicht angegeben ist.
also die Datenbank im ersten sendto für die id's hinzugefügt,

dann ging es weiter, aber nun kam wieder diese 'undefined' Fehlermeldung wie aus der gethistory funktion

dann habe ich im 2ten sendto der 'query' funktion zum holen der values auch die Datenbank hinzugefügt.

Wenn es in der internen gethistory Funktion auch so abläuft nur halt intern im programmcode, vermute ich, das da der Datenbankname nicht sauber aus den Adaptereinstellungen ermittelt wird oder in die integrierten BuildIn-Funktionen wie gethistory übergeben werden.

[fastfoot]

@ostseeskipper sagte in SQL getHistory Abfrage:

Wenn es in der internen gethistory Funktion auch so abläuft nur halt intern im programmcode, vermute ich, das da der Datenbankname nicht sauber aus den Adaptereinstellungen ermittelt wird oder in die integrierten BuildIn-Funktionen wie gethistory übergeben werden.

Schmarrn. Deine Abfrage funktionierte doch für das Datum, id ist einfach nicht im Resultset vorhanden. Weshalb erwartest du das überhaupt? logge mal val statt id

        console.log(result.result[i].id);
        console.log(new Date(result.result[i].ts).toISOString());

[ostseeskipper]

@fastfoot
ich hatte erst mal nur erwartet das das Beispiel in der readme funktioniert,

        console.log(result.result[i].id + ' ' + new Date(result.result[i].ts).toISOString());

für jemanden der nicht aus der JS Sprache kommt und lernen will etwas frustriend.

Auch im anderen Beispiel zu queries ist die Datenbank im Beispielscript nicht enthalten, aber ja du hast Recht es steht ein paar zeilen drunter.

Was ich möchte, ist von einem Datenpunkt den gleitenden Durchschnitt der letzten x Minuten in eine Variable bzw. einen Datenpunkt schreiben.
.ts muss ich vermutlich durch .val ersetzen um den Wert zu bekommen

console.log(new Date(result.result[i].ts).toISOString());

[apollon77]

Naja, schau mal hier https://github.com/ioBroker/ioBroker.sql#structure-of-the-dbs ... da steht elche Tabelle für welchen DB Typ wie heisst ... Ja ich kann gern in der Doku darauf hinweisen das das Beispiel zB für ne PostGres bzw sqlite ist und ggf andere Tabellennamen genutzt werden müssen.

.ts muss ich vermutlich durch .val ersetzen um den Wert zu bekommen

ts ist der Timestamp ... und val ist was anderes.

Logge doch einfach mal console.log(JSON.stringify(result.result[i])) dann sehen wir was wirklich zurückommt und raten nicht rum

[apollon77]

@fastfoot sagte in SQL getHistory Abfrage:

addId hat tatsächlich keinen Effekt, habe es im source auch nur einmal gefunden, genutzt scheint es nicht zu werden.

Jupp scheint so zu sein, dann wäre das ein Bug

[ostseeskipper]

@apollon77 said in SQL getHistory Abfrage:

Logge doch einfach mal console.log(JSON.stringify(result.result[i])) dann sehen wir was wirklich zurückommt und raten nicht rum

In der query oder gethistory ?

[fastfoot]

@apollon77 sagte in SQL getHistory Abfrage:

val ist was anderes.

val ist der Wert, wir müssen nicht raten, ich habe das schon geprüft
{'ts':1648832706691,'val':2.029}

[ostseeskipper]

@fastfoot und @apollon77

sendTo('sql.0', 'query', 'SELECT id FROM iobroker.datapoints WHERE name="smartmeter.0.1-0:16_7_0__255.value"', function (result) {
    if (result.error) {
        console.error(result.error);
    } else {
        var now = new Date();
        now.setMinutes(-3);
        sendTo('sql.0', 'query', 'SELECT AVG(val) AS Wert FROM iobroker.ts_number WHERE ts >= ' + now.getTime() + ' AND id=' + result.result[0].id, function (result) {
            console.log(JSON.stringify(result.result[0]));
        });
    }
});

ergibt

21:54:35.208	info	javascript.0 (149) Start javascript script.js.Logik.GleitDurchschnitt
21:54:35.226	info	javascript.0 (149) script.js.Logik.GleitDurchschnitt: registered 0 subscriptions and 0 schedules
21:54:35.242	info	javascript.0 (149) script.js.Logik.GleitDurchschnitt: {"Wert":3.6997549019607843}

[ostseeskipper]

Wenn mir nun noch einer von euch Beiden bitte sagen könne wie ich die Zahl 3.699 in eine Variable extrahiert bekomme wäre mein Abend gerettet

[fastfoot]

@ostseeskipper probiere mal mit getHistory average statt minmax. Dann kannst du mit:
log(i+' : ' + new Date(result.result[i].ts).toLocaleString() + ' => ' + result.result[i].val); das Ergebnis anschauen

22:01:19.335	info	javascript.1 (527) script.js.Forum.getHistory: 0 : 1.4.2022, 19:14:39 => 2.0199999999999996
22:01:19.336	info	javascript.1 (527) script.js.Forum.getHistory: 1 : 1.4.2022, 20:29:39 => 2.01
22:01:19.337	info	javascript.1 (527) script.js.Forum.getHistory: 2 : 1.4.2022, 22:01:19 => 1.9977777777777777

[paul53]

@ostseeskipper sagte: wie ich die Zahl 3.699 in eine Variable extrahiert bekomme

let wert = result.result[0].Wert;

[fastfoot]

@ostseeskipper sagte in SQL getHistory Abfrage:

Wenn mir nun noch einer von euch Beiden bitte sagen könne wie ich die Zahl 3.699 in eine Variable extrahiert bekomme wäre mein Abend gerettet

mit let DeineVariable = result.result[0].Wert weist du das zu

[ostseeskipper]

@paul53
@fastfoot
@apollon77

Danke Euch erst mal.

@fastfoot ist es mit gethistory performanter als über die letzte von mir gepostete Lösung mit queries und avg ?
fand das now.setMinutes(-3) relativ charmat auch wenn ich es noch nicht wirklich verstehe wie das alles mit den result.result verkapselt ist.

Aber ein Auto muss ja auch nur fahren
schönen Abend euch

[apollon77]

Ich stimme aber zu das die Beispiele vllt nicht die besten sind. Ich kann mal schauen das ich das hier gesagt dort einfließen lasse.

Ich bin aber auch ehrlich das mir persönlich Gerade die Zeit fehlt das alles sinnvoll und vollständig durchzuspielen und zu überlegen welche Beispiele in welcher Form aus „User“ Sicht Sinn machen. Da freuen wir uns über konkrete Vorschläge Bzw auf github einem Pull request“ für die readme (ist ganz easy: Stift bei der readme klicken, ändern, Änderung vorschlagen, pull request erstellen)