Fehler im 98_statistics->Tendency->singuralreadings

Begonnen von Icinger, 04 Januar 2015, 10:02:39

Vorheriges Thema - Nächstes Thema

Icinger

Guten Morgen,

verwende schon länger das statistics-Modul für meine Temp-Fühler.

Vor 2 Tagen habe ich das SingularReagings-Attr um den pressure-Tendency ergänzt:
Temp_.*:temperature:(Min|Max|Avg):(Hour|Day)|Temp_.*:pressure:Max:Hour|Regenmesser:rain_total:Delta:(Hour|Day|Month|Year)|Temp_.*:pressure:Tendency:(1h|2h|3h|6h)

Interessanterweise habe ich beim kombinierten Tendency-Reading die richtigen Werte für 1, 2, 3 und 6 Stunden, aber bei den Einzelreadings in allen 4 Readings den selben Wert:

     2015-01-04 08:59:58   statPressureTendency 1h: +0 2h: +1 3h: +1 6h: +4
     2015-01-04 08:59:58   statPressureTendency1h +0
     2015-01-04 08:59:58   statPressureTendency2h +0
     2015-01-04 08:59:58   statPressureTendency3h +0
     2015-01-04 08:59:58   statPressureTendency6h +0


Ich denke, der Fehler im statistics-Modul dürfte hier zu finden sein, alle Readings werden mit dem Wert aus $stat[1] gefüllt.
$
      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","1h",$stat[1],0,0);
      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","2h",$stat[1],0,0);
      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","3h",$stat[1],0,0);
      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","6h",$stat[1],0,0);


Hier ein Patch, der das behebt:

--- 98_statistics.pm    2015-01-04 08:05:24.066283100 +0100
+++ 98_statistics_neu.pm        2015-01-04 09:04:57.166284804 +0100
@@ -642,9 +642,9 @@
    if ($singularReadings ne "") {
       # statistics_storeSingularReadings $hashName,$singularReadings,$dev,$statReadingName,$readingName,$statType,$period,$statValue,$lastValue,$saveLast
       statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","1h",$stat[1],0,0);
-      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","2h",$stat[1],0,0);
-      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","3h",$stat[1],0,0);
-      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","6h",$stat[1],0,0);
+      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","2h",$stat[3],0,0);
+      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","3h",$stat[5],0,0);
+      statistics_storeSingularReadings ($name,$singularReadings,$dev,$statReadingName,$readingName,"Tendency","6h",$stat[7],0,0);
    }

    $result = join( " ", @hidden );
@@ -1359,4 +1359,4 @@

=end html_DE

-=cut
\ No newline at end of file
+=cut


lg, Ici
Verwende deine Zeit nicht mit Erklärungen. Die Menschen hören (lesen) nur, was sie hören (lesen) wollen. (c) Paulo Coelho

santalaus

Hallo,

wenn ich das richtig sehe ist der Fehler noch im Modul, evtl. könnte das jemand fixen.

Kann es sein, das bei der Tendency Funktion ein weiterer Fehler drin ist?

statHumidityDay Min: 64 Avg: 70 Max: 74 2015-07-21 11:23:11
statHumidityHour Min: 64 Avg: 65 Max: 67 2015-07-21 11:23:11
statHumidityTendency 1h: -9 2h: -8 3h: -7 6h: +2 2015-07-21 10:59:55

Zur vollen Stunde sah man das besser ;) aber IMHO  müssten die Werte 1,2,3,6 gedreht werden 1h: +2 2h: -7 3h: -8 6h:-9

Nico

tupol

#2
Habe den Beitrag gerade entdeckt. Ist gefixt, sobald Sourceforge wieder geht.

Den zweiten Fehler kann ich anhand der Angaben nicht nachvollziehen.

santalaus

Danke für den Fix.

Bei dem 2ten bin ich mir auch nicht mehr so sicher. Ist halt schwierig zu sagen.

An dem Tag sah es von Graph her so aus als wäre die Tendency falsch.