ahoy
ahoy copied to clipboard
[Bug]Anzeige der Alarme so korrekt?
Platform
ESP8266
Assembly
I did the assebly by myself
nRF24L01+ Module
No response
Antenna
circuit board
Power Stabilization
Elko (~100uF)
Connection picture
- [ ] I will attach/upload an Image of my wiring
Version
0.8.88
Github Hash
2bc6610
Build & Flash Method
AhoyDTU Webinstaller
Setup
V08.88
Debug Serial Log output
No response
Error description
Ist die Alarmanzeige korrekt? bei mir werden die Alarme doppelt und 3-fach angezeigt und die Reihenfolge ist verwirrend (nicht chronologisch)
Das ist weder ein Bug noch ein Feature, das kommt leider genau so aus dem WR raus. Wenn man das zusammen ziehen wollte, müßte man alle Meldungen speichern und dann die Frage wie lange ?
Das ist weder ein Bug noch ein Feature, das kommt leider genau so aus dem WR raus. Wenn man das zusammen ziehen wollte, müßte man alle Meldungen speichern und dann die Frage wie lange ?
Werden im WR eigentlich nur 10 Fehlermeldungen gespeichert oder wurde die Begrenzung auf 10 wegen dem Speicherverbrauch im ESP gemacht?
Nein, im WR können mehr sein. Wieviele weiß ich nicht.
die Grenze in Wechselrichter kennen wir bis dato nicht, ist aber auch nicht wichtig, da wird zyklisch die neuesten Meldungen abholen und das sich in Grenzen hält. Die Überlegung wäre die Limitierung von 10 Meldungen beim ESP32 zu erhöhen.
meine Frage wäre trotzdem, wieso die Reihenfolge in der Darstellung nicht chronologisch ist, ist das ein FIFO oder wie werden sie angezeigt?
frage bei Hoymiles nach, wie geschrieben geben wir nur das wieder was der Wechselrichter uns an Infos bereitstellt.
meine Frage wäre trotzdem, wieso die Reihenfolge in der Darstellung nicht chronologisch ist, ist das ein FIFO oder wie werden sie angezeigt?
Die 10 Einträge sind hardcoded und aktuell als Ringpuffer implementiert. Ja, wäre durchaus überlegenswert das zu ändern und chronologisch anzuzeigen.
ich prüfe was möglich ist, schön wäre es, wenn dadruch die Code-Size nicht unverhältmismäßig steigt.
ich prüfe was möglich ist, schön wäre es, wenn dadruch die Code-Size nicht unverhältmismäßig steigt.
Meine letzte Programmiererfahrung ist lange her, aber ich glaube es ist alles dafür bereits vorhanden ohne extra sortieren zu müssen. Mit mAlarmNxtWrPos und alarmCnt und den fixen 10 (ich würde hier eine Konstante einführen statt hardcoden) sollte man in der Webausgabe die richtige Startposition im Array berechnen können. Quasi ein FifoBuffer realisiert mit Ringpuffer und Zeiger.
die Frage ist, ob der Wechselrichter die Reihenfolge einhält, aber ja den Zeiger haben wir schon, den müsste man nur noch über die API verfügbar machen - danke für den Hinweis
Alarmanzeige jetzt ok, über eine Erweiterung mit mehr Einträgen würde ich mich freuen! (in Whishlist aufnehmen?)
Das Problem ist das diese Alarme zwischengespeichert werden müssen um sie zeitlich zu sortieren und auf Anfang und Endezeit zu warte.
ich glaube er meint diesen Zwischenspeicher, den können wir auf alle Fälle anheben
noch nicht getestet, da z.Zt. nicht so viele Meldungen auflaufen 😉
habe jetzt 15 Meldungen angezeigt, allerdings andere als in alter Version mit 10 Meld. Was stimmt jetzt?
oder sind die Meld. auf 15 stattt 50 begrenzt und die weiteren werden weggelassen oder nicht angezeigt?
die Meldungen starten jeden Tag aufs neue
weiss ich doch, sind aber vom gleichen Tag! nur mit 2 versch. Versionen ausgelesen
hab gerade nochmal deine Screenshots angeschaut. Die Alarme sind doch die gleichen, nur in der neueren Version sind es mehr (alle) und sortiert.
einer fehlt, das wäre der 16. Deswegen meine Frage
oder gibt es ne scrollfunktion, die ich nicht sehe?
Na schieb das Ganze doch mal nach oben.
https://github.com/user-attachments/assets/ee22cc91-b30c-482f-b638-c2c73fad085c
hab ich versucht, ging aber nicht...🤷♂️