Batteriehistorie: stehende Werte gelten nicht mehr als frische Messungen

Der Vorschlag aus Abschnitt CB (CAN-Wert innerhalb 300 s verlangen) wurde gemessen und trifft nicht: er haette zwei eindeutige Werkbank-Tage durchgelassen (CAN 5 Minuten neben dem Minimum) und echte Fahrzeugtage verworfen - am stehenden Fahrzeug schweigt CAN, eine CAN-Frist verwirft also ausgerechnet die Ruhespannung. Der wirkliche Mechanismus: zustand_oder_none() liefert den zuletzt bekannten Zustand unabhaengig vom Alter, und pruefen() stempelte ihn mit "jetzt". Ein schweigender Dongle "misst" damit alle fuenf Minuten weiter seinen letzten Wert. Am Bestand belegt: der 16.08.2026 trug einen Tageseintrag, obwohl der Sensor an dem Tag kein einziges Mal gemeldet hatte (naechste echte Meldung 885 Minuten entfernt); sieben von 19 Eintraegen standen mehr als fuenf Minuten von jeder echten Meldung entfernt. messzeitpunkt() beantwortet jetzt beide Fragen an einer Stelle: ist das eine neue Messung, und wann wurde sie gemacht. Gestempelt wird mit der Geraetezeit, wenn zuordenbar - so wie der Rueckblick es laengst tut, der damit die richtige Umsetzung war und keine Aenderung brauchte. Der Tag kommt aus der Messung, nicht aus date.today(). Die Frischepruefung steht vor der Plausibilitaetsgrenze, sonst warnt ein offline gegangener Dongle alle fuenf Minuten ueber denselben Wert (77 identische Zeilen in 3000 Protokollzeilen gezaehlt). Die erste Fassung mass das Alter an last_updated und hatte damit ein Loch, das erst die Live-Pruefung zeigte: Home Assistant setzt beim Hochfahren jede Entitaet neu, last_updated steht danach auf der Neustartzeit. Gemessen: nach dem Neustart um 21:12:54 trug der Spannungssensor last_updated 21:12:54, waehrend die Meldezeit desselben Datensatzes 14:46:47 sagte. Das Alter wird deshalb an der Messzeit gemessen. Geprueft: 18 Tests in tests/batterie/, davon fallen 16 gegen HEAD; alle 116 Backend-Tests gruen; live im Testcontainer belegt - der erste Takt nach dem Neustart meldet "keine neue Messung" statt der bisherigen Warnung. Offen: Werkbank und Fahrzeug lassen sich aus diesen Daten nicht sicher trennen. Abgestellt sind die erfundenen Eintraege, nicht die Frage, wo der Dongle steckt.
2026-09-06 23:30:08 +02:00
parent f53469f197
commit 7bbd236a17
7 changed files with 528 additions and 13 deletions
@@ -35,6 +35,22 @@ Liste und im Diagramm landet.
 Ohne zugeordneten BATTERIE_SENSOR passiert hier bewusst nichts, statt
 Platzhalter-Nullwerte aufzuzeichnen.

+Aufgezeichnet wird nur eine NEUE Messung. zustand_oder_none() liefert den
+zuletzt bekannten Zustand unabhängig von seinem Alter - ohne diese Prüfung
+"misst" ein schweigender Dongle alle fünf Minuten weiter seinen letzten Wert,
+gestempelt mit "jetzt". Am Bestand nachgewiesen (06.09.2026): der 16.08.2026
+trug einen Tageseintrag, obwohl der Spannungssensor an diesem Tag kein einziges
+Mal gemeldet hatte - die nächste echte Meldung lag 885 Minuten entfernt; sieben
+von 19 Einträgen standen mehr als fünf Minuten von jeder echten Meldung
+entfernt, die unstrittigen Fahrzeugtage dagegen exakt bei 0 s.
+
+Der Zeitstempel ist deshalb auch nicht mehr "jetzt", sondern der der Messung
+selbst, und der Tag wird aus ihm abgeleitet - sonst landete eine Messung von
+23:58 im Folgetag. Damit verhält sich der Livepfad wie der Rückblick
+(historienimport.py's _batterie_importieren), der seit jeher jede Messung mit
+ihrem eigenen Zeitstempel einträgt: dieselbe Messung, gleich auf welchem Weg
+erfasst, ergibt denselben Eintrag.
+
 Optional wird bei jeder neuen Tagesminimum-Messung auch die aktuelle
 Außentemperatur (AUSSENTEMP_SENSOR, CAN ambient air temperature) mit
 aufgezeichnet - für die Messwertliste (Datum, Uhrzeit, Außentemperatur), nicht
@@ -49,7 +65,7 @@ import logging
 from typing import TYPE_CHECKING

 from .veroeffentlichung import zustand_oder_none
-from .verlauf import NEBENWERT_MAX_ABSTAND_S
+from .verlauf import NEBENWERT_MAX_ABSTAND_S, ZUORDNUNG_MAX_S, geraetezeit_plausibel

 if TYPE_CHECKING:
    from .koordinator import Koordinator
@@ -75,6 +91,76 @@ SPANNUNG_MIN_V = 10.0
 # Wert.
 AGM_RUHE_MAX_V = 13.0

+# Wie alt darf eine Messung sein, um noch als neu zu gelten? Der Takt liegt bei
+# 5 Minuten (TAKT_BATTERIE in koordinator.py); das Fenster muss deshalb
+# mindestens eine Taktlänge betragen, sonst ginge eine Messung verloren, die
+# kurz nach einem Takt eintrifft. Verdoppelt, damit auch ein verspäteter Takt
+# sie noch fasst - dieselbe Messung zweimal einzutragen ist folgenlos, weil sie
+# ihren eigenen Zeitstempel trägt und die Ablage min/max nur erweitert.
+MESSUNG_MAX_ALTER_S = 600
+
+
+def messzeitpunkt(
+    spannung_zustand,
+    meldezeit_zustand,
+    jetzt: datetime.datetime,
+) -> datetime.datetime | None:
+    """Wann wurde diese Spannung gemessen - und ist es überhaupt eine neue Messung?
+
+    `None` heißt: die Messung ist älter als MESSUNG_MAX_ALTER_S, es gibt also
+    nichts aufzuzeichnen. Siehe Moduldocstring, warum das nötig ist.
+
+    Ohne zugeordnete Meldezeit bleibt nur die Ankunftszeit, und die wird von
+    einem Neustart zurückgesetzt - dann kann unmittelbar danach einmalig ein
+    veralteter Wert durchgehen. Mit MELDEZEIT_SENSOR passiert das nicht.
+
+    Gestempelt wird mit der **Gerätezeit**, wenn sie sich zuordnen lässt, sonst
+    mit der Ankunftszeit. Zugeordnet wird über ZUORDNUNG_MAX_S: die flespi-
+    Integration setzt die Entitäten eines Datensatzes nacheinander, aber
+    innerhalb weniger Millisekunden - liegt die Meldezeit weiter weg, gehört sie
+    zu einem anderen Datensatz und sagt über diese Messung nichts. Der Rückblick
+    stempelt Batteriewerte ebenfalls mit Gerätezeit (verlauf_lesen() bekommt in
+    historienimport.py die Meldezeit-Entität mit); ohne dieselbe Wahl hier
+    ergäben Livepfad und Rückblick für dieselbe Messung verschiedene Einträge.
+
+    Gemessen wird gegen `last_updated`, nicht gegen `last_reported` - und das
+    ist Absicht: `last_updated` springt nur bei einem echten Wertwechsel, und
+    genau solche Wechsel schreibt auch der recorder, aus dem der Rückblick
+    liest. Ein Sensor, der unverändert weitermeldet, gilt hier deshalb als
+    "nichts Neues" - was richtig ist, denn ein unveränderter Wert erweitert
+    weder min noch max. Auf `last_reported` umzustellen würde den Livepfad
+    wieder vom Rückblick trennen.
+    """
+    if spannung_zustand is None:
+        return None
+
+    gemessen = spannung_zustand.last_updated
+    if meldezeit_zustand is not None:
+        abstand_s = abs(
+            (meldezeit_zustand.last_updated - spannung_zustand.last_updated).total_seconds()
+        )
+        if abstand_s <= ZUORDNUNG_MAX_S:
+            try:
+                kandidat = datetime.datetime.fromtimestamp(
+                    float(meldezeit_zustand.state), datetime.UTC
+                )
+            except (TypeError, ValueError, OSError, OverflowError):
+                kandidat = None
+            if kandidat is not None and geraetezeit_plausibel(kandidat, jetzt):
+                gemessen = kandidat
+
+    # Das Alter wird an `gemessen` gemessen, nicht an last_updated - und das ist
+    # der Unterschied zwischen "funktioniert" und "funktioniert bis zum nächsten
+    # Neustart". Home Assistant setzt beim Hochfahren jede Entität neu, ihr
+    # last_updated steht danach auf der Neustartzeit, auch wenn der Wert Stunden
+    # alt ist. Am 06.09.2026 im Testcontainer gemessen: nach dem Neustart um
+    # 21:12:54 trug der Spannungssensor last_updated 21:12:54, während die
+    # Meldezeit desselben Datensatzes 14:46:47 sagte - 395 Minuten alt. Die
+    # Gerätezeit übersteht den Neustart, die Ankunftszeit nicht.
+    if (jetzt - gemessen).total_seconds() > MESSUNG_MAX_ALTER_S:
+        return None
+    return gemessen
+

 async def pruefen(k: Koordinator) -> None:
    sensor = k.zuordnung.werte.BATTERIE_SENSOR
@@ -87,6 +173,25 @@ async def pruefen(k: Koordinator) -> None:
        spannung = float(wert)
    except (TypeError, ValueError):
        return
+    # Ist das überhaupt eine neue Messung? Ohne diese Frage zeichnet ein
+    # schweigender Dongle alle fünf Minuten weiter seinen letzten Wert auf -
+    # siehe Moduldocstring und messzeitpunkt().
+    jetzt = datetime.datetime.now(datetime.UTC)
+    spannung_zustand = k.hass.states.get(sensor)
+    meldezeit_sensor = k.zuordnung.werte.MELDEZEIT_SENSOR
+    meldezeit_zustand = k.hass.states.get(meldezeit_sensor) if meldezeit_sensor else None
+    gemessen = messzeitpunkt(spannung_zustand, meldezeit_zustand, jetzt)
+    if gemessen is None:
+        _LOGGER.debug(
+            "Batteriespannung %s V ist keine neue Messung (Sensor meldet seit "
+            "über %s s nichts) - nicht erfasst",
+            spannung, MESSUNG_MAX_ALTER_S,
+        )
+        return
+
+    # Erst hier, nach der Frischeprüfung: sonst warnt ein offline gegangener
+    # Dongle alle fünf Minuten über denselben Wert (im Testcontainer 77 Mal in
+    # 3000 Protokollzeilen gezählt). So warnt jede echte Messung genau einmal.
    if spannung < SPANNUNG_MIN_V:
        _LOGGER.warning(
            "Batteriespannung %s V unter der Plausibilitätsgrenze (%s V) - nicht erfasst",
@@ -108,7 +213,6 @@ async def pruefen(k: Koordinator) -> None:
    temp_sensor = k.zuordnung.werte.AUSSENTEMP_SENSOR
    temp_wert = zustand_oder_none(k.hass, temp_sensor)
    if temp_wert is not None:
-        spannung_zustand = k.hass.states.get(sensor)
        temp_zustand = k.hass.states.get(temp_sensor)
        abstand_s = abs((temp_zustand.last_updated - spannung_zustand.last_updated).total_seconds())
        if abstand_s <= NEBENWERT_MAX_ABSTAND_S:
@@ -122,13 +226,14 @@ async def pruefen(k: Koordinator) -> None:
                round(abstand_s),
            )

-    jetzt = datetime.datetime.now(datetime.UTC)
-    heute = datetime.date.today().isoformat()
+    # Tag und Zeitstempel kommen aus der Messung selbst, nicht aus der Uhr des
+    # Servers - sonst landete eine Messung von 23:58 im Folgetag.
+    tag = gemessen.date().isoformat()
    await k.ablage.batterieverlauf_tageswert_aktualisieren(
-        heute, jetzt.isoformat(), spannung, aussentemp
+        tag, gemessen.isoformat(), spannung, aussentemp
    )
    await k.batterieverlauf_veroeffentlichen()
-    _LOGGER.debug("Batteriespannung erfasst (%s: %s V)", heute, spannung)
+    _LOGGER.debug("Batteriespannung erfasst (%s: %s V)", tag, spannung)


 async def eintrag_loeschen(k: Koordinator, datum: str | None) -> None:
@@ -1 +1 @@
-{"version":"2026.9.6.4","sha256":"1d9d562b8215a64e176861d3de52d5f53a17fe590d73814fa065748a7e0e43e9","bytes":378713,"gebaut":"2026-09-06T15:30:15Z"}
+{"version":"2026.9.6.5","sha256":"d7373ee38b4b6e97f5d20e5225bca6981d21f60fc34d7b2658b8571650b52726","bytes":378720,"gebaut":"2026-09-06T21:29:15Z"}
@@ -1,7 +1,7 @@
 {
  "domain": "audi_dashboard",
  "name": "Audi Dashboard",
-  "version": "2026.9.6.4",
+  "version": "2026.9.6.5",
  "documentation": "https://gitea.nothaft.cloud/paul/audi-app/src/branch/main/README.md",
  "issue_tracker": "https://gitea.nothaft.cloud/paul/audi-app/issues",
  "codeowners": [