Flot in 0.2.5

[sissiwup]

Hallo,

da hat sich ein alter Fehler wieder eingeschlichen.

flot läuft nicht ohne history-Adapter.

Den brauche ich aber nicht, da ich nur den SQL-nutze.

[sissiwup]

Achtung, in der 0.2.5 sind nicht die Optimierungen zur Anzeige enthalten, die ich gepostet hatte.

Das sieht also alles wieder recht zerstückelt aus.

Bin wieder zurück auf mein Backup.

(siehe meinen Beitrag vom 3.1.)

 `function readOneChart(id, instance, index, callback) {

        var option = getStartStop(index);
        option.instance = instance;

        socket.emit('getHistory', id, option, function (err, res, step) {
            if (!err && res) {
                //for (var i = 0, len = res.length; i < len; i++) {

                if (config.longestStep < step) {
                    config.longestStep = step;
                }
                var lval = null;  //qf
                for (var i = 0; i < res.length; i++) {
       //qf             if (res[i].val != null || !config.ignoreNull) {
                        // Convert boolean values to numbers
                        if (res[i].val === 'true' || res[i].val === true) {
                            res[i].val = 1;
                        } else if (res[i].val === 'false' || res[i].val === false) {
                            res[i].val = 0;
                        }
                        if (typeof res[i].val == 'string') res[i].val = parseFloat(res[i].val);
       //qf -- start
                    if (res[i].val == null) {
                        if (config.ignoreNull || i == res.length-1) {
                             res[i].val=lval;
                        }
                    } else {
                        lval=res[i].val;
                    }
                    if (res[i].val != null || i == 0 || i == res.length-1) {
       //qf -- ende
                        seriesData[index].push([res[i].ts * 1000, res[i].val]);
                    }

                }
            }

            // Add first tick with required range
            // todo muss das sein ??
            // seriesData[index] = fillRange(seriesData[index]);

            if (callback) callback(id, index);
        });
    }` 

Bei dieser Variante werden nie Null-Werte geschrieben. Wenn man ignore-Null aktiviert werden null-Werte durch den letzten bekannten Wert ersetz. Sieht ab und an besser aus.[/i][/i][/i][/i][/i][/i][/i][/i][/i][/i][/i][/i][/i][/i][/i]

[sissiwup]

Hab den fix mal ins repository eingearbeitet. Mal sehen ob das so ok ist…

(Abhängigkeit zu history hab ich allerdings nicht gefunden, wo die aufzulösen ist).

[Pman]

Ich hatte in der letzten Version lediglich das "ignore null" auf die State-Ebene gezogen, also so dass man pro State einstellen kann ob Null ignoriert werden soll. An der Funktion selber hatte ich eigentlich nichts verändert. Auch an der History-Abhängigkeit nicht.

[Pman]

Okay ich habe keine Ahnung warum es bei mir funktioniert hatte aber das "ignore Null" pro State ist broken :x .

EDIT: ich baue kurz die ignore null Änderungen zurück und mache einen pull request. Das muss ich nochmal neu angehen. Ich denke dein pull request wird damit unnötig?

[sissiwup]

Hallo,

mein Request hat etwas damit zu tun,

das der erste und letzte Wert eines Intervalls

benötigt wird um eine saubere x-Axenberechnung durchführen zu können.

Die Werte müssen, egal ob Null oder nicht, zur Berechnung der x-Skalierung herangezogen werden,

sonst funktioniert es nicht sauber mit mehr als einer Kurve.

Außerdem habe ich die Interpretation von "null" etwas geändert, so dass man auch durchgezogene Kurven bekommen kann.

[Pman]

Alles klar. Wie gesagt, meine Änderungen bezüglich null habe ich zurückgenommen. Grundsätzlich erscheint es mir aber trotzdem sinnvoller die "ignore null"-Option pro Datenpunkte/Kurve einzustellen.

[nobody]

So richtig scheint der Rückbau zumindest bei mir nicht funktioniert zu haben.

Bei mir ist nun ignoreNull immer an, auch wenn ich es explizit nicht will (ignoreNull=false), weil ich mit 0 und 1 Werten Schaltübergänge darstellen möchte. Mit dem ersten 1-Wert bleibt die Linie nun fest auf 1.

[m4rcel]

Hab das gleiche Problem.

Es werden nun keinerlei null-Werte mehr angezeigt, obwohl ignoreNull aus ist.