Review-Dokument als umgesetzt kennzeichnen
Haelt fest, was bewusst anders geloest wurde als vorgeschlagen, welche drei Fehler beim Umsetzen dazukamen und welcher pyscript-Fallstrick dabei aufgefallen ist.
This commit is contained in:
@@ -10,6 +10,34 @@ nicht im Browser reproduziert.
|
||||
|
||||
---
|
||||
|
||||
> ## ✅ Umgesetzt am 2026-08-13
|
||||
>
|
||||
> Alle Befunde dieses Dokuments sind behoben — siehe die fünf Commits von
|
||||
> `Setup-Zuordnung: Zuruecksetzen wirkt…` bis `Kleinere Befunde und Dokumentation…`.
|
||||
> Das Dokument bleibt als Befundprotokoll stehen: es begründet, warum die
|
||||
> Änderungen so aussehen, wie sie aussehen.
|
||||
>
|
||||
> **Zwei Punkte wurden bewusst anders gelöst als hier vorgeschlagen:**
|
||||
>
|
||||
> 1. *Karte nur bei echtem Ansichtswechsel neu aufbauen* — nicht umgesetzt. `render()` ersetzt den
|
||||
> Inhalt per `innerHTML`; eine überlebende Leaflet-Instanz zeigte danach auf ein abgehängtes
|
||||
> Element und bliebe leer. Der Neuaufbau hängt am 20-Sekunden-Takt des Backends und ließe sich
|
||||
> nur durch einen Umbau der Render-Architektur ändern. Die Begründung steht im Code.
|
||||
> Behoben ist dagegen die Standortabfrage, die tatsächlich ungedrosselt lief.
|
||||
> 2. *`SPECIFICATION.md`* wurde nicht in Teilen umgeschrieben, sondern hat einen datierten
|
||||
> Statushinweis am Anfang bekommen. Ein Umschreiben der §4/§5-Tabellen hätte neue
|
||||
> Ungenauigkeiten riskiert.
|
||||
>
|
||||
> **Drei Fehler kamen beim Umsetzen dazu**, sichtbar erst im laufenden Panel: ein unlesbares Datum
|
||||
> ergab „NaN/N" statt eines Strichs, ein fehlender Tanksensor „0 %" statt „unbekannt", und neben dem
|
||||
> Typenschild stand die Ausführung doppelt. Alle drei sind mit behoben.
|
||||
>
|
||||
> Und ein pyscript-Fallstrick ist dabei aufgefallen und im Code vermerkt: **Generatorausdrücke sind
|
||||
> nicht implementiert** (`not implemented ast ast_generatorexp`), Mengen-, Listen- und
|
||||
> Dict-Comprehensions dagegen schon.
|
||||
|
||||
---
|
||||
|
||||
## Gesamteindruck
|
||||
|
||||
Handwerklich stark. Das Setup-Menü ersetzt „vor der Installation `einstellungen.py` editieren"
|
||||
|
||||
Reference in New Issue
Block a user