[Gelöst] Endlosschleife in ElectricityCalculator wegen Zeitumstellung?

[Nestor]

You don't need localtime and can simplify a bit with object chaining.
Maybe implement the timezone as a device attr, since it can be different for users. There is no global timezone support in Fhem I believe.

Code Auswählen Erweitern

use 5.030;
use strict;
use warnings;

use DateTime;

my $tz = 'Europe/Brussels'; # implement as device attribute ?
my $dt = DateTime->now()->set_time_zone($tz)->set_hour(0)->set_minute(0)->set_second(0);

say $dt;
say $dt->add(days => 1);

Result:

Code Auswählen Erweitern

2022-11-08T00:00:00
2022-11-09T00:00:00

[Sailor]

Hi Nestor

You don't need localtime and can simplify a bit with object chaining.

Ok, fair enough. I used you shorter version

Maybe implement the timezone as a device attr, since it can be different for users. There is no global timezone support in Fhem I believe.

I am obtaining the timzone from the system by

Code Auswählen Erweitern

my $TimeZone = DateTime::TimeZone->new( name => 'local' )->name();

I am testing the modules over night and then I will check them in.

Regards
    Sailor

[Sailor]

Ein herzerfrischendes "Moin" vom Achtern Diek vorweg!

Aufgrund eines Bugs in den
73_ElectricityCalculator
73_GasCalculator
73_WaterCalculator
mit den Anzahl der Sekunden pro Tag während der Zeitumstellung sowie dem Start der Mitternachtsroutine, bin ich gezwungen die Bibliothek "DateTime" zu verwenden.

Daher bitte unbedingt vor dem Update im linux shell die Bibliohek DateTime nachinstallieren:

Code Auswählen Erweitern

sudo cpan install DateTime

ausfuehren!

Ansonsten schmiert Euch euer fhem ab und legt sich in eine Dauer-Startschleife.

Sorry für die Unannehmlichkeiten, aber es ging leider nicht anders.

Gruß
    Sailor

[Beta-User]

Doch... nocheck!

[Nestor]

Code Auswählen Erweitern

my $TimeZone = DateTime::TimeZone->new( name => 'local' )->name();

You can use the string "local" directly (instead of creating a TimeZone object and converting to string).

Code Auswählen Erweitern

my $dt = DateTime->now()->set_time_zone('local');
say $dt->time_zone;


Result:

Code Auswählen Erweitern


DateTime::TimeZone::Europe::Brussels=HASH(0x5628649f3a00)

[Beta-User]

Können wir bitte in Ruhe nochmal checken, welche Lösung ggf. Sinn macht, ohne für die kommenden Monate einen unabsehbaren support-Aufwand zu generieren?

[betateilchen]

Ansonsten schmiert Euch euer fhem ab und legt sich in eine Dauer-Startschleife.

Sorry für die Unannehmlichkeiten, aber es ging leider nicht anders.

Sorry Herr Nachbar, aber das ist Quatsch. Du kannst doch einfach prüfen, ob das benötigte Modul geladen werden kann und wenn das nicht der Fall ist, eine Logmeldung ausgeben und die Verarbeitung beenden.

Da muss man doch keine Dauerschleife für das gesamte System provozieren.

[KölnSolar]

Können wir bitte in Ruhe nochmal checken, welche Lösung ggf. Sinn macht, ohne für die kommenden Monate einen unabsehbaren support-Aufwand zu generieren?

Sehe ich ähnlich. Nach Nestors Analyse nutzen ja eine Menge Module einen "Tageswechselcheck" per 86.400s. Dann sollte aber auch eine allgemeine Lösung(zentrale Funktion) gefunden und mit den anderen maintainern diskutiert werden. Dauert zwar länger, aber eine Hals-über-Kopf-Lösung ist doch gar nicht notwendig. Das "Problem" kommt doch frühestens Ende März wieder. 

Ein herzerfrischendes "Moin" vom Achtern Diek vorweg!

Aufgrund eines Bugs in den
73_ElectricityCalculator
73_GasCalculator
73_WaterCalculator
mit den Anzahl der Sekunden pro Tag während der Zeitumstellung sowie dem Start der Mitternachtsroutine, bin ich gezwungen die Bibliothek "DateTime" zu verwenden.

Daher bitte unbedingt vor dem Update im linux shell die Bibliohek DateTime nachinstallieren:

Code Auswählen Erweitern

sudo cpan install DateTime

ausfuehren!

Ansonsten schmiert Euch euer fhem ab und legt sich in eine Dauer-Startschleife.

Sorry für die Unannehmlichkeiten, aber es ging leider nicht anders.

Gruß
    Sailor

Viele sind wenig bedarft im Umgang mit dem OS. Und viele fahren mit Debian und nutzen apt zur "Systempflege".  DateTime ist wohl in libdatetime-perl enthalten und dann braucht es kein cpan zur Installation.
Grüße Markus

[Sailor]

Hi BetaUser

Doch... nocheck!

Habe sie noch nicht eingecheckt.
Ich will da noch ein wenig Zeit in Land streichen lassen...

Es gibt hier noch interessante Verbesserungsvorschläge von betateilchen und Nestor die ich wohl erst mit einarbeiten will.

Gruß
    Sailor

[Sailor]

Hallo betateilchen

Sorry Herr Nachbar, aber das ist Quatsch. Du kannst doch einfach prüfen, ob das benötigte Modul geladen werden kann und wenn das nicht der Fall ist, eine Logmeldung ausgeben und die Verarbeitung beenden.
Da muss man doch keine Dauerschleife für das gesamte System provozieren.

Seit 2 Jahren suche ich mir im fhem - Developer Board nach genau dieser Möglichkeit einen Wolf.
Das ist generell genau ein Problem mit allen Modulen und mich hat es ohnehin gewundert, warum das nicht auf übergeordneter Ebene bereits abgefangen wird.
Dort könnte man dann auch gleich die entsprechenden Module automatisch nachinstallieren.

Wenn du mir ein Code-Schnipsel nennen oder auch nur einen Hinweis zum Lesen geben kannst wie man dieses Problem löst, dann hast du bei mir einen Whiskey gut. 

Gruss
    Sailor

[Sailor]

Hi Markus

Sehe ich ähnlich. Nach Nestors Analyse nutzen ja eine Menge Module einen "Tageswechselcheck" per 86.400s. Dann sollte aber auch eine allgemeine Lösung(zentrale Funktion) gefunden und mit den anderen maintainern diskutiert werden. Dauert zwar länger, aber eine Hals-über-Kopf-Lösung ist doch gar nicht notwendig. Das "Problem" kommt doch frühestens Ende März wieder. 
Viele sind wenig bedarft im Umgang mit dem OS. Und viele fahren mit Debian und nutzen apt zur "Systempflege".  DateTime ist wohl in libdatetime-perl enthalten und dann braucht es kein cpan zur Installation.

Da gebe ich dir vollkommen Recht!

Aus meiner Sicht sollte es in der fhem.pl einige nützliche Funktionen mehr geben.

Zum Beispiel die besagte "getSecondsOfDay()" mit der Aufruf-Option
getSecondsOfDay(0) -> heute
getSecondsOfDay(+1) -> Morgen
getSecondsOfDay("2023-03-26") -> Vom 26. März 2023

Damit wären die 86400 endgültig Mausetot!

Anderes Beispiel zum Starten von Mitternachtsroutinen ein "getEpochOfMidnight()"
Diese werden beim InternalTimer benötigt.

Gruß
    Sailor

[KölnSolar]

Hi Sailor,
bin zwar zu weit weg für den Whisky(den kannst Du ja Udo rüber bringen), aber so mach ich es

Code Auswählen Erweitern

eval {require Time::HiRes; 1;}
or do {
return "......Module Time:HiRes not installed. Check perl libraries'";
};


Da gebe ich dir vollkommen Recht!

Schön. Machst Du dann einen neuen Faden im Development Bereich ?

Grüße Markus

[Sailor]

Hi Markus

Hi Sailor,
bin zwar zu weit weg für den Whisky(den kannst Du ja Udo rüber bringen)

Der wird sich sicher freuen!

so mach ich es

Code Auswählen Erweitern


eval {require Time::HiRes; 1;}
or do {
return "......Module Time:HiRes not installed. Check perl libraries'";
};


OK, hatte immer wieder mit dem Standard Modul ExtUtils::Installed eingelesen.

Aber die Frage die sich bei beiden Lösungen stellt ist die, ob ich das Ganze am geschicktesten über oder in die Subfunktion "X_Initialize()" stelle...

Schön. Machst Du dann einen neuen Faden im Development Bereich ?

Sobald ich dieses Problem gelöst habe, mache ich mich ran...

Die derzeitig eingecheckten Module werden wohl am 30.11. Schwierigkeiten bereiten weil sie versuchen den 31.11. zu berechnen.
Ein saudummer Fehler meinerseits im letzten Update.
Bis dahin habe ich noch 21 Tage Zeit...

Gruß
   Sailor

[KölnSolar]

Aber die Frage die sich bei beiden Lösungen stellt ist die, ob ich das Ganze am geschicktesten über oder in die Subfunktion "X_Initialize()" stelle...

Initialize, Define... lasse ich durchlaufen. Dann hat der Anwender wenigstens sein device anlegen können(schlimmstenfalls ja ein altes device, welches dann nach einem update/restart "plötzlich" verschwunden ist. Und man merkt es erst einmal nicht.
Erst wenn ich das Modul bei einer Interaktion benötige, prüfe ich und der User sieht die Meldung ggfs. hoffentlich online.

Ist bei Dir natürlich anders, da es ja keine Interaktion gibt. Ich würde das Log fluten.....

Grüße Markus

[Beta-User]

Hi BetaUser

Habe sie noch nicht eingecheckt.

So war es nicht gemeint, das bezog sich auf die "nocheck"-Variante von timelocal...

Für die Kommandozeile:

Code Auswählen Erweitern

{ my $ts=1669836392;; my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) = localtime($ts);; return localtime(Time::Local::timelocal_nocheck(0,0,0,$mday+1,$mon,$year)) }