SQL - Wert per Script speichern

fastfoot

@bahnuhr sagte in SQL - Wert per Script speichern:

@fastfoot

du Teufelskerl ;-)

Ich hatte nicht auf "Änderung" gestellt.
Jetzt sehe ich die Punkte auch.

Vielleicht liest ja @apollon77 mit.
Denn ansonsten ist bei der nächsten Aktualisierung ja wieder alles weg.

Danke.

Evtl. ist das mit dem Rückgängig machen des Hacks gar nicht so einfach, obwohl ich denke dass der nie funktioniert hat. Habe da aber auch zu wenig Einblick. Jedenfalls ist mein Post nicht die Lösung sondern nur ein Workaround. So oft wird sich aber der SQL Adapter als auch Flot nicht ändern. Mir ging es hauptsächlich darum apollon77 zu zeigen dass er Recht hatte mit seinen Vermutungen. Für einen PR reicht es leider noch nicht, kommt aber auch noch irgendwann

bahnuhr

@fastfoot

Was ist PR ?

fastfoot

@bahnuhr sagte in SQL - Wert per Script speichern:

@fastfoot

Was ist PR ?

Pull Request, das wäre eine Lösung auf Github bereit zu stellen, so dass die Entwickler sie nur noch akzeptieren und einpflegen könnten. Da ich aber weder weiss wie das konkret funktioniert, noch die sonstigen Auswirkungen einer solchen Änderung abschätzen kann, wage ich mich da nicht heran. Apollon wird schon irgendwann die Zeit finden, sich das genau anzuschauen. Mit den hier gemachten Änderungen dürfte es jedenfalls nicht getan sein

apollon77

ALs überflogen habt Ihr das gemacht wa sich gesagt habe oder ?! Die Änderung und Flot macht aus dem "*1000" einfach ein "*1" - also das gleiche wie das rausnehmen.

Bitte schaut das es GitHub issues gibt - am besten bei flot, history. sql und influxdb ( :-) ) ... muss an die eh die Tage alle ran

fastfoot

@apollon77 sagte in SQL - Wert per Script speichern:

ALs überflogen habt Ihr das gemacht wa sich gesagt habe oder ?! Die Änderung und Flot macht aus dem "*1000" einfach ein "*1" - also das gleiche wie das rausnehmen.

genau, ohne deinen Input hätte ich das ja nicht gefunden :-) Die Änderung in Flot war für mich so am einfachsten da das ja kein 'normales' Sourcecodefile ist. Ich hatte es nichtmal geschafft da ein console.log() reinzubringen :-(

Bitte schaut das es GitHub issues gibt - am besten bei flot, history. sql und influxdb ( :-) ) ... muss an die eh die Tage alle ran

Das überlasse ich gerne @bahnuhr. Dennoch, soll das issue beim SQL Adapter auch das Schreiben in die DB beinhalten? Da wird ja in Zeile 1547 auch mit 1000 multipliziert, was in meinen Augen auch nicht wirklich Sinn macht. Hatte hier schon gefragt aber du bist nicht darauf eingegangen

apollon77

@fastfoot sagte in SQL - Wert per Script speichern:

Da wird ja in Zeile 1547 auch mit 1000 multipliziert, was in meinen Augen auch nicht wirklich Sinn macht. Hatte hier schon gefragt aber du bist nicht darauf eingegangen

Naja da wird versucht "am ende alles gleich" zu machen ... ich muss mal schauen wo sowas alles gemacht wird ...

fastfoot

@apollon77 sagte in SQL - Wert per Script speichern:

@fastfoot sagte in SQL - Wert per Script speichern:

Da wird ja in Zeile 1547 auch mit 1000 multipliziert, was in meinen Augen auch nicht wirklich Sinn macht. Hatte hier schon gefragt aber du bist nicht darauf eingegangen

Naja da wird versucht "am ende alles gleich" zu machen ... ich muss mal schauen wo sowas alles gemacht wird ...

ja schon, aber wenn der ts auch nur eine ms kleiner ist wird mit 1000 multipliziert was das Datum in eine Zukunft katapultiert welche wir alle nicht erleben. Wenn da aus Sekunden Millisekunden werden sollen dann wäre eher eine Prüfung auf < 946681200 Sek sinnvoll nicht auf <946681200000 Millisekunden. Aber schau halt mal :-)