Neue FHEM Statistik verfügbar unter fhem.de

Begonnen von Markus Bloch, 01 Juli 2017, 11:32:37

Vorheriges Thema - Nächstes Thema

betateilchen

Ich hadere immer noch mit der pauschalen 0,4% Grenze für alle pie-charts, die für mich so keinen wirklichen Sinn macht.

Man sollte diese Schwelle pro Chart definieren und davon abhängig machen, wieviele mögliche Varietäten im Chart auftreten können.

In der Darstellung der Update-Alter ist nun der Bereich 1-6 Monate rausgefallen, obwohl selbst  eine einzige Installation eine durchaus interessante Information darstellt.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

Markus Bloch

#61
Hallo Udo,

das kann ich so nicht bestätigen. Was (wieder) weg gefallen ist, ist der Teil "< 1 Tag". Der Teil "1 Monat - 6 Monate" ist nachwievor sichtbar. Es hat hier also nichts mit der 0,4% Regel zu tun, sonst würde "Other" in Grau in der Legende erscheinen.

Gruß
Markus
Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)

betateilchen

#62
Ja, Du hast Recht. Mir ist nur aufgefallen, dass plötzlich nur noch drei Einträge vorhanden waren.

Aber davon abgesehen: Wir sollten uns darum kümmern, warum die Installationen <= 1 Tag plötzlich rausgefallen sind...

(und das "to" ist immer noch falsch...)
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

betateilchen

Zitat von: betateilchen am 17 Juli 2017, 21:05:06
Aber davon abgesehen: Wir sollten uns darum kümmern, warum die Installationen <= 1 Tag plötzlich rausgefallen sind...

Durch das Runden auf 0 Nachkommastellen sind plötzlich alle Installationen älter als 36 Stunden heute im 20 Uhr in den Bereich 1-7 Tage gefallen.

Kannst Du bitte mal folgende Änderung einbauen:


         $countAll{'system'}{'age'}{'0'}++   if ($age <= 2);
         $countAll{'system'}{'age'}{'7'}++   if ($age > 2  && $age <= 7);
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

Markus Bloch

#64
Zitat von: betateilchen am 17 Juli 2017, 21:05:06
Aber davon abgesehen: Wir sollten uns darum kümmern, warum die Installationen <= 1 Tag plötzlich rausgefallen sind...

Das ist relativ einfach. Die Revision wird aus controls_fhem.txt genommen. Diese Revision ist die letzte Änderung bis 05:45. Wenn nun also die letzte Änderung am Vortag um 19:00 erfolgte, dann ist dies auch der Zeitstempel, der in der Statistik benutzt wird. Das bedeutet also, dass um 19:00 des aktuellen Tages alle Installationen "< 1Tag" nun wegfallen und ab dann in "1 Tag - 1 Woche" aufgehen.

EDIT: Vielleicht sollten wir hier die gemeldete Revision + 1 nehmen (controls_fhem.txt Check-In). Dann hast du immer den Zeitpunkt des Release eines Updates, was ja am selben Tag um 05:45 ist.

Zitat von: betateilchen am 17 Juli 2017, 21:05:06
(und das "to" ist immer noch falsch...)

Habe ich bereits gefixt.
Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)

Markus Bloch

Zitat von: betateilchen am 17 Juli 2017, 21:13:49
Kannst Du bitte mal folgende Änderung einbauen:


         $countAll{'system'}{'age'}{'0'}++   if ($age <= 2);
         $countAll{'system'}{'age'}{'7'}++   if ($age > 2  && $age <= 7);


Ist aktiv.

Gruß
Markus
Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)

Markus Bloch

Wenn man die Counter für das Update-Alter mit 0 initialisiert, dann behalten die entsprechenden Kuchenstücke ihre Farbe:

   
  $countAll{'system'}{'age'}{'0'} = 0;
   $countAll{'system'}{'age'}{'7'} = 0;
   $countAll{'system'}{'age'}{'30'} = 0;
   $countAll{'system'}{'age'}{'180'} = 0;
   $countAll{'system'}{'age'}{'365'} = 0;
   $countAll{'system'}{'age'}{'999'} = 0;


Gruß
Markus
Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)

betateilchen

Zitat von: Markus Bloch am 17 Juli 2017, 21:24:45
Vielleicht sollten wir hier die gemeldete Revision + 1 nehmen

Schau ich mir an.

Zitat von: Markus Bloch am 17 Juli 2017, 21:33:48
Wenn man die Counter für das Update-Alter mit 0 initialisiert,

Das auch.

Und ich will das Runden auf 1 Nachkommastelle ändern. Nach meinem Urlaub.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

betateilchen

@Markus: Eine überarbeitete Version der statistics2.cgi wurde heute eingecheckt.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

Markus Bloch

Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)

vuffiraa

Zitat von: betateilchen am 17 Juli 2017, 21:05:06
(und das "to" ist immer noch falsch...)

Hab mal jemanden gefragt, der sich damit auskennen sollte. Die Antwort ist
ZitatYou can help us to increase the quality... would be correct [emoji846]
FHEM 5.8 auf Cubietruck, Raspi B+

Weinzierl KNX IP BAOS 770, Homematic, EnOcean

betateilchen

Zitat von: vuffiraa am 20 Juli 2017, 22:27:53
Hab mal jemanden gefragt, der sich damit auskennen sollte.

Das habe ich auch gemacht, bevor ich den Textvorschlag gemacht hatte... Wenn Dein Auskenner recht hat, müssen jetzt zigtausende englischsprachige Webaeiten geändert werden.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

vuffiraa

Zitat von: betateilchen am 21 Juli 2017, 12:14:52
Das habe ich auch gemacht, bevor ich den Textvorschlag gemacht hatte... Wenn Dein Auskenner recht hat, müssen jetzt zigtausende englischsprachige Webaeiten geändert werden.
Naja, nur weil die Seiten behaupten englischsprachig zu sein, müssen sie ja nicht recht haben. Die Treffer bei Google scheinen aber auch auf deiner Seite zu sein.
Mal sehen, ob mein Auskenner dafür eine Erklärung hat [emoji6]
FHEM 5.8 auf Cubietruck, Raspi B+

Weinzierl KNX IP BAOS 770, Homematic, EnOcean

SabineT


Markus Bloch

Zurück zum eigentlichen Thema:

Zitat von: betateilchen am 17 Juli 2017, 20:45:21
Man sollte diese Schwelle pro Chart definieren und davon abhängig machen, wieviele mögliche Varietäten im Chart auftreten können.

Ich nehme an, Dir schwebt vor nur für Perl eine Schwelle zu definieren und bei alle anderen Pie-Charts keine Schwelle zu setzen. Geh ich richtig in der Annahme?

Gruß
Markus
Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)