PRESENCE: "maybe absent" nach Neustart von FHEM

[CoolTux]

Ok, wahrscheinlich muss man sich Mal den ganzen Code anschauen. Werde das mal so einbauen und testen bei mir. Ausserdem schaue ich morgen mal in den Code.

[DeeSPe]

Habe gerade vor $count noch das -- ergänzt damit der Zähler stimmt.
Bei mir überleben die absent Devices so schon mal sauber einen Neustart.

Gruß
Dan

[DeeSPe]

Ganz rund ist die Sache noch nicht, denn der Counter muss ja tatsächlich wieder auf 0 gesetzt werden.
Ich denke so geht es, die Readings waren einfach vertauscht:

Index: FHEM/73_PRESENCE.pm
===================================================================
--- FHEM/73_PRESENCE.pm   (revision 14249)
+++ FHEM/73_PRESENCE.pm   (working copy)
@@ -1289,7 +1289,7 @@
         {
             if(++$count >= $absenceThreshold)
             {
-                readingsBulkUpdate($hash, ".absenceThresholdCounter", 0);
+                readingsBulkUpdate($hash, ".presenceThresholdCounter", 0);
                 readingsBulkUpdate($hash, "state", "absent");
                 readingsBulkUpdate($hash, "presence", "absent");
             }
@@ -1334,7 +1334,7 @@
         {
             if(++$count >= $presenceThreshold)
             {
-                readingsBulkUpdate($hash, ".presenceThresholdCounter", "0");
+                readingsBulkUpdate($hash, ".absenceThresholdCounter", 0);
                 readingsBulkUpdate($hash, "state", "present");
                 readingsBulkUpdate($hash, "presence", "present");

Gruß
Dan

EDIT:
Nein immer noch nicht!
Die Mischung aus beiden klappt nun gut bei mir:

Code Auswählen Erweitern

Index: FHEM/73_PRESENCE.pm
===================================================================
--- FHEM/73_PRESENCE.pm (revision 14249)
+++ FHEM/73_PRESENCE.pm (working copy)
@@ -1289,7 +1289,8 @@
         {
             if(++$count >= $absenceThreshold)
             {
-                readingsBulkUpdate($hash, ".absenceThresholdCounter", 0);
+                readingsBulkUpdate($hash, ".presenceThresholdCounter", 0);
+                readingsBulkUpdate($hash, ".absenceThresholdCounter", $count);
                 readingsBulkUpdate($hash, "state", "absent");
                 readingsBulkUpdate($hash, "presence", "absent");
             }
@@ -1334,7 +1335,8 @@
         {
             if(++$count >= $presenceThreshold)
             {
-                readingsBulkUpdate($hash, ".presenceThresholdCounter", "0");
+                readingsBulkUpdate($hash, ".absenceThresholdCounter", 0);
+                readingsBulkUpdate($hash, ".presenceThresholdCounter", --$count);
                 readingsBulkUpdate($hash, "state", "present");
                 readingsBulkUpdate($hash, "presence", "present");


[DeeSPe]

Hast Du es mal getestet Marko?

Gruß
Dan

[CoolTux]

Hast Du es mal getestet Marko?

Gruß
Dan

Leider noch nicht. Bin noch nicht dazu gekommen. Stecke mitten im Umbau von AMAD fest 
Läuft es bei Dir?


Grüße

[DeeSPe]

Es tut erfolgreich seinen Dienst, überlebt die Neustarts problemlos und setzt auch den Zähler zurück.
Sogar wenn man mitten in einem absence Cycle neustartet, startet auch PRESENCE wieder beim richtigen Zähler.
Ganz perfekt ist der Zähler am Ende (nach Ablauf des Cycle) irgendwie teilweise noch nicht!
Ist aber nur ein Schönheitsfehler, der nicht wirklich stört. Markus wird da sicher gleich die Lösung für sehen.
Habe allerdings bisher auch nur den absence Cycle getestet. presenceThreshold habe ich nicht im Einsatz.

Gruß
Dan

[CoolTux]

Klingt auf jeden Fall gut. PresenceThreshold habe ich auch nicht im Einsatz.

[Markus Bloch]

Hallo zusammen,

mein Fehler war, dass ich beim erreichen des finalen Zustands (absent/present), den Counter mit dem Wert 0 gespeichert habe, was bei einem Neustart ein erneutes durchgehen zufolge hat. Jetzt wird der Counter im korrekten Zustand gespeichert, sodass nach einem Neustart der Zustand sofort final gesetzt wird.

Vielen Dank für eure Hilfe.

Gruß
Markus

[DeeSPe]

Danke, und freut mich dass Du das so schnell gefixt hast.
Dann war ja meine vorgeschlagene Änderung gar nicht sooo schlecht.
Und eigentlich hattest Du ja, entgegen Deiner eigenen Vermutung, bereits alles vorbereitet. Nur war die Umsetzung noch nicht ganz perfekt.

Gruß
Dan

[Markus Bloch]

Und eigentlich hattest Du ja, entgegen Deiner eigenen Vermutung, bereits alles vorbereitet. Nur war die Umsetzung noch nicht ganz perfekt.

Man wird mit dem zunehmenden Alter eben auch vergesslicher

[CoolTux]

Und attraktiver, vergiss das attraktiver nicht!!!