ahoy icon indicating copy to clipboard operation
ahoy copied to clipboard

[Bug]Anzeige der Alarme so korrekt?

Open Gubi2023 opened this issue 11 months ago • 21 comments

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)

Screenshot 2024-03-02 162311

Gubi2023 avatar Mar 02 '24 15:03 Gubi2023

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 ?

knickohr avatar Mar 03 '24 09:03 knickohr

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?

Zordandi avatar Mar 03 '24 09:03 Zordandi

Nein, im WR können mehr sein. Wieviele weiß ich nicht.

knickohr avatar Mar 03 '24 10:03 knickohr

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.

lumapu avatar Mar 03 '24 10:03 lumapu

meine Frage wäre trotzdem, wieso die Reihenfolge in der Darstellung nicht chronologisch ist, ist das ein FIFO oder wie werden sie angezeigt?

Gubi2023 avatar Mar 03 '24 10:03 Gubi2023

frage bei Hoymiles nach, wie geschrieben geben wir nur das wieder was der Wechselrichter uns an Infos bereitstellt.

lumapu avatar Mar 03 '24 10:03 lumapu

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.

Zordandi avatar Mar 03 '24 10:03 Zordandi

ich prüfe was möglich ist, schön wäre es, wenn dadruch die Code-Size nicht unverhältmismäßig steigt.

lumapu avatar Mar 03 '24 11:03 lumapu

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.

Zordandi avatar Mar 03 '24 11:03 Zordandi

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

lumapu avatar Mar 03 '24 20:03 lumapu

Alarmanzeige jetzt ok, über eine Erweiterung mit mehr Einträgen würde ich mich freuen! (in Whishlist aufnehmen?)

Gubi2023 avatar Aug 14 '24 08:08 Gubi2023

Das Problem ist das diese Alarme zwischengespeichert werden müssen um sie zeitlich zu sortieren und auf Anfang und Endezeit zu warte.

knickohr avatar Aug 14 '24 09:08 knickohr

ich glaube er meint diesen Zwischenspeicher, den können wir auf alle Fälle anheben

lumapu avatar Aug 14 '24 14:08 lumapu

noch nicht getestet, da z.Zt. nicht so viele Meldungen auflaufen 😉

Gubi2023 avatar Sep 10 '24 09:09 Gubi2023

habe jetzt 15 Meldungen angezeigt, allerdings andere als in alter Version mit 10 Meld. Was stimmt jetzt?

Screenshot 2024-09-23 154747 2024-09-23 15_39_08-Greenshot

oder sind die Meld. auf 15 stattt 50 begrenzt und die weiteren werden weggelassen oder nicht angezeigt?

Gubi2023 avatar Sep 23 '24 13:09 Gubi2023

die Meldungen starten jeden Tag aufs neue

lumapu avatar Sep 24 '24 14:09 lumapu

weiss ich doch, sind aber vom gleichen Tag! nur mit 2 versch. Versionen ausgelesen

Gubi2023 avatar Sep 24 '24 14:09 Gubi2023

hab gerade nochmal deine Screenshots angeschaut. Die Alarme sind doch die gleichen, nur in der neueren Version sind es mehr (alle) und sortiert.

lumapu avatar Sep 24 '24 15:09 lumapu

einer fehlt, das wäre der 16. Deswegen meine Frage
Screenshog

oder gibt es ne scrollfunktion, die ich nicht sehe?

Gubi2023 avatar Sep 24 '24 15:09 Gubi2023

Na schieb das Ganze doch mal nach oben.

https://github.com/user-attachments/assets/ee22cc91-b30c-482f-b638-c2c73fad085c

knickohr avatar Sep 24 '24 15:09 knickohr

hab ich versucht, ging aber nicht...🤷‍♂️

Gubi2023 avatar Sep 24 '24 19:09 Gubi2023