Audit Panel/iOS: sechs Befunde behoben, Zuendung aufs Trip-Signal (2026.8.31.5)
ZUENDUNG_SENSOR liegt jetzt auf dem Trip-Signal des FMM003 statt auf dem rohen, prellenden Zuendungseingang. Damit kommt die Pausentoleranz vom Geraet - eine Fahrt beginnt erst bei Zuendung UND Bewegung UND Start Speed. MINDESTDAUER_S wandert nach verlauf.py und gilt jetzt auch im Livepfad. Vorher verwarf nur der Import Zuendphasen unter 60 s; live entstanden daraus am 31.08. elf Fahrten mit 0 km, zwei davon mit null Sekunden Dauer. Das Panel erfand einen Literpreis von 0,00 EUR, wenn die Kosten noch nicht feststanden (null/liter ist 0 in JavaScript) - neue Hilfsfunktion literpreis() prueft beides, wie die App seit jeher. schnittpreis() teilte ohne Nennerpruefung und konnte "NaN" oder "unendlich" anzeigen. Der Setup-Dialog ist wirklich modal: die Tab-Leiste hatte einen eigenen Zuhoerer vor der Sperre und liess einen Tabwechsel bei offenem Fenster zu. Verwaister Code entfernt: drei Funktionen und vier Regeln im Panel, 20 Regeln in der App. Offen bleibt allein die leere Erstzulassung - eine Angabe, keine Aenderung. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -26,7 +26,7 @@ from typing import TYPE_CHECKING
|
||||
from . import screening
|
||||
from .ablage import neue_id
|
||||
from .veroeffentlichung import zustand_oder_none
|
||||
from .verlauf import verlauf_lesen, zeit_normalisiert
|
||||
from .verlauf import MINDESTDAUER_S, verlauf_lesen, zeit_normalisiert
|
||||
|
||||
if TYPE_CHECKING:
|
||||
from .koordinator import Koordinator
|
||||
@@ -170,6 +170,20 @@ async def _letztes_lebenszeichen(
|
||||
async def fahrt_beenden(
|
||||
k: Koordinator, start_ts: datetime.datetime, ende_ts: datetime.datetime
|
||||
) -> None:
|
||||
dauer_s = int((ende_ts - start_ts).total_seconds())
|
||||
if dauer_s < MINDESTDAUER_S:
|
||||
# Zündung an, aber nicht gefahren - oder ein prellender
|
||||
# Zündungseingang. Der Import verwirft solche Fenster seit jeher;
|
||||
# hier taten wir es nicht, und der Bestand füllte sich mit
|
||||
# Nulleinträgen (siehe MINDESTDAUER_S in verlauf.py). Der laufende
|
||||
# Start wird trotzdem zurückgesetzt, sonst hinge er für immer.
|
||||
await k.fahrt_start_setzen(None)
|
||||
_LOGGER.info(
|
||||
"Zündung war nur %s s an - keine Fahrt angelegt (Grenze %s s)",
|
||||
dauer_s,
|
||||
MINDESTDAUER_S,
|
||||
)
|
||||
return
|
||||
fahrt = leere_fahrt(start_ts, ende_ts, "ha")
|
||||
await k.ablage.fahrt_anhaengen(fahrt)
|
||||
await k.fahrt_start_setzen(None)
|
||||
|
||||
@@ -1 +1 @@
|
||||
{"version":"2026.8.31.4","sha256":"48d4965f198c06dbcad882c76eca656de6cc8c18a17c7ff24a39c5c9a1e8c5a9","bytes":266032,"gebaut":"2026-08-31T13:30:52Z"}
|
||||
{"version":"2026.8.31.5","sha256":"3f297e72a5b215e54d1eb17d134d6d66ef65d1d29a5793220ff2456b6c6cc5c8","bytes":265697,"gebaut":"2026-08-31T18:03:21Z"}
|
||||
Binary file not shown.
@@ -124,11 +124,29 @@ const $ = (s) => ROOT.querySelector(s);
|
||||
|
||||
/* ---------------------------------------------------------------- Format */
|
||||
const de = (n, d = 0) => Number(n ?? 0).toLocaleString("de-DE", { minimumFractionDigits: d, maximumFractionDigits: d });
|
||||
/* Preis je Liter, oder null.
|
||||
|
||||
Die Kosten muessen ausdruecklich geprueft werden, nicht nur die Liter: bei
|
||||
einem automatisch erkannten Tankvorgang steht der Preis noch nicht fest, und
|
||||
`null / 11.5` ist in JavaScript `0`. Die Liste, der Einzelbeleg und die
|
||||
CSV-Ausgabe zeigten daraufhin "0,00 €/l" - einen Preis, den niemand bezahlt
|
||||
hat. Die App prueft an derselben Stelle seit jeher auf beides. */
|
||||
const literpreis = (f) => (f && f.liters && f.fuel_total_eur != null ? f.fuel_total_eur / f.liters : null);
|
||||
const eur = (n) => de(n, 2);
|
||||
const esc = (s) => String(s ?? "").replace(/[&<>"]/g, (c) => ({ "&": "&", "<": "<", ">": ">", '"': """ }[c]));
|
||||
const sum = (a, f) => a.reduce((s, x) => s + f(x), 0);
|
||||
/* Durchschnittspreis IMMER mengengewichtet: Summe Kosten / Summe Liter */
|
||||
const schnittpreis = (list) => sum(list, (f) => f.fuel_total_eur ?? f.summe ?? 0) / sum(list, (f) => f.liters ?? f.liter ?? 0);
|
||||
/* Ohne die Nennerpruefung liefert ein Zeitraum, in dem kein Tankvorgang Liter
|
||||
traegt, 0/0 = NaN - und einer mit Kosten aber ohne Liter eine Division durch
|
||||
null. de() reicht beides ungefiltert durch, die Kopfzeile las dann
|
||||
"Ø NaN €/l" bzw. "Ø ∞ €/l". Einzelne solche Datensaetze gibt es real; es
|
||||
braucht nur einen Monat, der ausschliesslich daraus besteht. Die App hat
|
||||
diesen Schutz samt Test seit jeher (format.ts). */
|
||||
const schnittpreis = (list) => {
|
||||
const liter = sum(list, (f) => f.liters ?? f.liter ?? 0);
|
||||
if (liter <= 0) return 0;
|
||||
return sum(list, (f) => f.fuel_total_eur ?? f.summe ?? 0) / liter;
|
||||
};
|
||||
|
||||
/* -------------------------------------------------------- Adapter-Schicht
|
||||
Übersetzt die Fahrzeugprofil-Struktur aus dem Backend (fahrzeug/
|
||||
@@ -333,7 +351,6 @@ function bildInfo(i) {
|
||||
}
|
||||
return { label: b.label, datei: b.datei, src: `/local/bilder/${b.datei}?v=${bildVersion}` };
|
||||
}
|
||||
function bildSrc(i) { return bildInfo(i).src; }
|
||||
function carImg(modus) {
|
||||
if (modus === "fest") {
|
||||
const info = bildInfo(startIndex());
|
||||
@@ -460,19 +477,6 @@ function ics(titel, datum, ort, notiz) {
|
||||
a.download = titel.replace(/[^\wäöüÄÖÜß ]/g, "").replace(/\s+/g, "-") + ".ics";
|
||||
document.body.appendChild(a); a.click(); a.remove();
|
||||
}
|
||||
function ring(pct) {
|
||||
const r = 64, c = 2 * Math.PI * r, on = (c * pct) / 100;
|
||||
return `<div class="ringwrap">
|
||||
<svg width="150" height="150" viewBox="0 0 150 150">
|
||||
<circle cx="75" cy="75" r="${r}" fill="none" stroke="var(--line)" stroke-width="2"/>
|
||||
<circle cx="75" cy="75" r="${r}" fill="none" stroke="var(--red)" stroke-width="2"
|
||||
stroke-linecap="round" stroke-dasharray="${on} ${c}" transform="rotate(-90 75 75)"/>
|
||||
</svg>
|
||||
<div class="mid">
|
||||
<div class="fig" style="font-size:40px">${pct}<span class="unit">%</span></div>
|
||||
<span class="label" style="margin-top:8px">${de(CONFIG.tankvolumen * pct / 100, 1)} Liter</span>
|
||||
</div></div>`;
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------------ Karte
|
||||
Leaflet weiterhin per CDN geladen (siehe Kopfkommentar). */
|
||||
@@ -3925,7 +3929,7 @@ function tankFelder(praefix, f) {
|
||||
const kraftstoff = q && q.fuel_type ? q.fuel_type : KRAFTSTOFFSORTEN[0];
|
||||
const station = q && q.station_name ? q.station_name : "";
|
||||
const preisText = q && q.liters && q.fuel_total_eur != null && q.liters > 0
|
||||
? `Preis je Liter: ${de(q.fuel_total_eur / q.liters, 2)} €` : "Preis je Liter: wird berechnet";
|
||||
? `Preis je Liter: ${de(literpreis(q), 2)} €` : "Preis je Liter: wird berechnet";
|
||||
return `
|
||||
<div class="feld"><label>Datum</label><input type="date" id="${praefix}Datum" value="${datum}"></div>
|
||||
<div class="feld"><label>Uhrzeit</label><input type="time" id="${praefix}Zeit" value="${zeit}"></div>
|
||||
@@ -4006,7 +4010,7 @@ function vFuel() {
|
||||
<span style="color:var(--fg)">${eur(sum(fm, (f) => f.fuel_total_eur || 0))} €</span>
|
||||
<span>Ø ${de(schnittpreis(fm), 2)} €/l</span></span></button>
|
||||
<div class="acc-b ${mi === 0 ? "on" : ""}" id="fm${j}${m}">`;
|
||||
fm.forEach((f) => { const preis = f.liters ? f.fuel_total_eur / f.liters : null; h += swipeZeile(`fill:${f.tank_id}`, `<button class="leaf" data-go="fill:${f.tank_id}">
|
||||
fm.forEach((f) => { const preis = literpreis(f); h += swipeZeile(`fill:${f.tank_id}`, `<button class="leaf" data-go="fill:${f.tank_id}">
|
||||
<span class="k">${dedat(new Date(f.ts))}<small>${esc(f.station_name || "")}</small></span>
|
||||
<span class="v">${f.liters != null ? de(f.liters, 2) + " l" : ""}<small>${preis != null ? de(preis, 2) + " €/l" : ""}</small></span>
|
||||
<svg class="chev" viewBox="0 0 6 10"><polyline points="1,1 5,5 1,9"/></svg></button>`); });
|
||||
@@ -4021,7 +4025,7 @@ function vFuel() {
|
||||
function vFill(id) {
|
||||
const f = FILLS.find((x) => x.tank_id === id);
|
||||
if (!f) return `<div class="tile">Tankvorgang nicht gefunden</div>`;
|
||||
const berechneterPreis = f.liters ? f.fuel_total_eur / f.liters : null;
|
||||
const berechneterPreis = literpreis(f);
|
||||
|
||||
if (tankBearbeitenOffen) {
|
||||
return `
|
||||
@@ -4323,7 +4327,7 @@ function datensatzExportieren(art) {
|
||||
TRIPS.map((t) => [t.trip_id, t.ts_start, t.ts_end, t.distance_km != null ? de(t.distance_km) : "", t.art, t.status]));
|
||||
} else if (art === "tanken") {
|
||||
csvDatei("tankvorgaenge.csv", ["ID", "Zeitpunkt", "Station", "Liter", "€/l", "Kosten €"],
|
||||
FILLS.map((f) => [f.tank_id, f.ts, f.station_name || "", f.liters != null ? de(f.liters, 2) : "", f.liters ? de(f.fuel_total_eur / f.liters, 2) : "", f.fuel_total_eur != null ? eur(f.fuel_total_eur) : ""]));
|
||||
FILLS.map((f) => [f.tank_id, f.ts, f.station_name || "", f.liters != null ? de(f.liters, 2) : "", literpreis(f) != null ? de(literpreis(f), 2) : "", f.fuel_total_eur != null ? eur(f.fuel_total_eur) : ""]));
|
||||
} else if (art === "service") {
|
||||
csvDatei("wartungsplan.csv", ["Datum", "km", "Art", "Werkstatt", "Kosten €"],
|
||||
CAR.service.buch.map((e2) => [e2.datum, de(e2.km), e2.art, e2.werkstatt, e2.kosten != null ? eur(e2.kosten) : ""]));
|
||||
@@ -4940,7 +4944,17 @@ function ereignisseVerdrahten() {
|
||||
.replace(/audiRingsClip/g, "audiRingsClipNav");
|
||||
tabbar.prepend(marke);
|
||||
}
|
||||
ROOT.getElementById("tabbar").addEventListener("click", (e) => { const b = e.target.closest(".tab"); if (b) go(b.dataset.tab); });
|
||||
/* Der eigene Zuhoerer der Tab-Leiste haengt vor der Sperre des grossen
|
||||
click-Handlers (`if (setupOffen) return`), umging sie deshalb und liess
|
||||
einen Tabwechsel zu, waehrend das Setup-Fenster offen stand - obwohl es
|
||||
aria-modal="true" traegt. Das Fenster schwebte danach ueber einer anderen
|
||||
Seite, mitsamt der nicht gespeicherten Zuordnung. Dieselbe Sperre also
|
||||
auch hier. */
|
||||
ROOT.getElementById("tabbar").addEventListener("click", (e) => {
|
||||
if (setupOffen || sheet) return;
|
||||
const b = e.target.closest(".tab");
|
||||
if (b) go(b.dataset.tab);
|
||||
});
|
||||
ROOT.getElementById("back").addEventListener("click", () => zurueck());
|
||||
ROOT.getElementById("ringsbtn").addEventListener("click", () => go("einst"));
|
||||
/* In Telefonbreite sind die Ringe oben rechts der Zugang zu den
|
||||
@@ -5807,10 +5821,6 @@ function ereignisseVerdrahten() {
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------ Datenladen */
|
||||
function zustandsDaten(entityId) {
|
||||
const st = HASS && HASS.states[entityId];
|
||||
return st && st.attributes ? st.attributes.daten : undefined;
|
||||
}
|
||||
|
||||
let letzterProfilState = null, letzterFahrtenState = null, letzterTankState = null, letzterStatusState = null, letzterBelegState = null, letzterBattState = null, letzterVersionState = null;
|
||||
|
||||
|
||||
@@ -142,7 +142,6 @@ main#view{--seitenrand:16px;padding:0 var(--seitenrand) 30px;scroll-behavior:smo
|
||||
.row dt{font-size:16px;color:var(--fg)}
|
||||
.row dd{font-size:16px;color:var(--fg2)}
|
||||
.row dd small{font-size:12.5px;margin-top:2px}
|
||||
.row-fussnote{font-size:12.5px}
|
||||
.rows-fest dt{flex:0 0 150px}
|
||||
.chev{stroke:var(--ios-grau);stroke-width:1.6}
|
||||
|
||||
@@ -352,7 +351,6 @@ main#view{--seitenrand:16px;padding:0 var(--seitenrand) 30px;scroll-behavior:smo
|
||||
/* Feste 220px hier wuerden das Foto wieder beschneiden - die Hoehe
|
||||
bestimmt seit dem 2026-08-31 das Bild selbst (siehe audi-dashboard.css). */
|
||||
.bildbox.carfix.klein{height:auto}
|
||||
.bildgrid{grid-template-columns:repeat(4,1fr)}
|
||||
.quad{gap:14px}
|
||||
|
||||
/* Design-Audit 2026-08-13: .setup-popup/.sdpopup/.sheet/.standortmenu sind
|
||||
@@ -392,7 +390,6 @@ main#view{--seitenrand:16px;padding:0 var(--seitenrand) 30px;scroll-behavior:smo
|
||||
deutlich sichtbares transluzentes Grau, dieselbe Konvention wie
|
||||
.feld select/.aktion ("das hier ist eine Flaeche"). */
|
||||
.bildbox.bild-fehlt .platzhalter{border:0;background:var(--ios-fill);border-radius:var(--r-tile)}
|
||||
.platzhalter-datei{display:none}
|
||||
|
||||
/* Zeilen mit Wert plus Statuspunkt rechts (Sicherheitscheck) ruhiger setzen */
|
||||
.row dd .dot{width:8px;height:8px}
|
||||
|
||||
@@ -418,12 +418,10 @@ button.tile, .tilebtn { transition: background .15s, transform .1s; }
|
||||
.row dd { margin: 0; font-size: 14px; color: var(--fg); text-align: right; }
|
||||
.row dd small { display: block; font-size: 12.5px; color: var(--fg3); margin-top: 3px; }
|
||||
.rows-fest dd { min-width: 120px; }
|
||||
/* Fussnote unter einer Zeile, ueber die volle Kachelbreite statt in der
|
||||
rechten Spalte: dort blieben nur ~217px, in die eine Prognose wie
|
||||
"9.029 km, voraussichtlich am 12.04.2027" zwangslaeufig umbrach.
|
||||
:has() grenzt das Umbrechen auf genau die Zeilen ein, die eine Fussnote
|
||||
fuehren - alle anderen .row bleiben unveraendert einzeilig. */
|
||||
.row:has(.row-fussnote) { flex-wrap: wrap; }
|
||||
/* .row-fussnote ist mit dem Umbau der Service-Seite auf .termingruppe
|
||||
entfallen (2026-08-18) - dort gibt es keine dt/dd-Spalten mehr, die um
|
||||
Platz konkurrieren. Die beiden Regeln dafuer standen seither ohne
|
||||
passendes Markup da. */
|
||||
/* Sammelzeilen der Sicherheit-Ansicht (vSicherheit()): Kreis-Häkchen + Text
|
||||
+ optionaler Chevron statt dt/dd. Erbt Abstand/Trennlinie von .row, ersetzt
|
||||
aber dessen space-between-Layout und setzt die Button-Vorgaben zurück, wenn
|
||||
@@ -449,13 +447,6 @@ button.sicherheitszeile:active { background: var(--tile-2); }
|
||||
und Farbe wie jede andere Nebenzeile. */
|
||||
.row dd.row-hinweis { font-size: 12.5px; color: var(--fg3); }
|
||||
|
||||
.row-fussnote {
|
||||
flex: 1 0 100%;
|
||||
text-align: left;
|
||||
font-size: 12.5px;
|
||||
color: var(--fg3);
|
||||
margin-top: 4px;
|
||||
}
|
||||
|
||||
.leaf {
|
||||
display: flex; align-items: center; justify-content: space-between;
|
||||
|
||||
@@ -45,6 +45,7 @@ from .batterie import SPANNUNG_MIN_V
|
||||
from .fahrterkennung import leere_fahrt
|
||||
from .tankerkennung import LITER_SCHWELLE, leerer_tankvorgang, schwelle_prozent
|
||||
from .verlauf import (
|
||||
MINDESTDAUER_S,
|
||||
durchschnitt_kmh,
|
||||
hoechstwert_im_fenster,
|
||||
route_aus_verlauf,
|
||||
@@ -68,10 +69,8 @@ _LOGGER = logging.getLogger(__name__)
|
||||
# Import und Live-Erkennung am Rand überschneiden.
|
||||
TANK_DUBLETTE_MIN = 90
|
||||
|
||||
# Fahrten, die kürzer sind, sind Zündung-an-ohne-Fahrt (Radio, Tür öffnen mit
|
||||
# Zündung, Diagnose) - die Live-Erkennung legt sie zwar an, im Rückblick
|
||||
# fluten sie den Bestand aber mit Nulleinträgen. Bewusst konservativ.
|
||||
MINDESTDAUER_S = 60
|
||||
# MINDESTDAUER_S liegt seit dem 31.08.2026 in verlauf.py - die Live-Erkennung
|
||||
# prüft seitdem dieselbe Grenze, siehe die Begründung dort.
|
||||
|
||||
# UNPLAUSIBLE_KMH selbst lebt in verlauf.py - gemeinsam mit screening.py, das
|
||||
# dieselbe Prüfung seit 2026.8.27 auch für die Live-Vervollständigung nutzt
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
{
|
||||
"domain": "audi_dashboard",
|
||||
"name": "Audi Dashboard",
|
||||
"version": "2026.8.31.4",
|
||||
"version": "2026.8.31.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": ["@paul"],
|
||||
|
||||
@@ -118,6 +118,18 @@ def zahl(wert: object) -> float | None:
|
||||
# bewerten.
|
||||
UNPLAUSIBLE_KMH = 300
|
||||
|
||||
# Kuerzer als das ist Zuendung ohne Fahrt: Radio, Tuer oeffnen, Diagnose - oder,
|
||||
# seit die Pausentoleranz entfallen ist, ein prellender Zuendungseingang. Am
|
||||
# 31.08.2026 sind daraus an einem einzigen Tag elf Fahrten mit 0 km entstanden,
|
||||
# zwei davon mit null Sekunden Dauer.
|
||||
#
|
||||
# Die Zahl stand vorher nur in historienimport.py, dessen eigener Kommentar den
|
||||
# Widerspruch sogar benannte ("die Live-Erkennung legt sie zwar an"): derselbe
|
||||
# Vorgang wurde beim Import verworfen und live gespeichert. Sie lebt deshalb
|
||||
# jetzt hier, gemeinsam mit UNPLAUSIBLE_KMH und aus demselben Grund - beide
|
||||
# Wege duerfen denselben Vorgang nie unterschiedlich bewerten.
|
||||
MINDESTDAUER_S = 60
|
||||
|
||||
|
||||
def durchschnitt_kmh(distanz_km: float | None, dauer_s: float | None) -> float | None:
|
||||
"""km/h aus Strecke und Dauer, oder None wenn eine der beiden fehlt oder
|
||||
|
||||
Reference in New Issue
Block a user