Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature Request: Anzeige last Alarm: #608

Closed
Juergen2453 opened this issue Jan 20, 2023 · 10 comments
Closed

Feature Request: Anzeige last Alarm: #608

Juergen2453 opened this issue Jan 20, 2023 · 10 comments
Assignees
Labels
duplicate This issue or pull request already exists enhancement New feature or request fixed dev fixed

Comments

@Juergen2453
Copy link

Auf der Webseite von AhoyDTU gibt es dir Info "last Alarm: Inverter start"
Allerdings ändert sich die Anzeige nicht auch wenn es andere Alarme gibt.
Besser wäre es wenn es einen Punkt geben würde z.B. letzte Alarme oder Fehlerbericht...
Dann öffnet sich ein Fenster und die Meldungen werden angezeigt.
Der jetzige Anzeige "last Alarm: Inverter start" macht so kein sinn.

@jimknopf63
Copy link

Auf der Webseite von AhoyDTU gibt es dir Info "last Alarm: Inverter start"
Allerdings ändert sich die Anzeige nicht auch wenn es andere Alarme gibt.
Besser wäre es wenn es einen Punkt geben würde z.B. letzte Alarme oder Fehlerbericht...
Dann öffnet sich ein Fenster und die Meldungen werden angezeigt.
Der jetzige Anzeige "last Alarm: Inverter start" macht so kein sinn.

Also im Prinzip wie das bei OpenDTU gehandelt wird. Da werden 15 Alarme im Verlauf gezeigt

@Juergen2453
Copy link
Author

Genau so in der Art, oder überhaupt keine Anzeige.

@stefan123t
Copy link
Collaborator

Siehe auch #600 und #177

@stefan123t stefan123t added enhancement New feature or request duplicate This issue or pull request already exists labels Jan 20, 2023
@lumapu
Copy link
Owner

lumapu commented Jan 21, 2023

@stefan123t kannst du erklären warum es auch ein "AlarmUpdate" Command gibt und wann ich es nutzen sollte?

lumapu added a commit that referenced this issue Jan 22, 2023
@lumapu lumapu self-assigned this Jan 22, 2023
@stefan123t
Copy link
Collaborator

stefan123t commented Jan 22, 2023

@lumapu ich weiß es nicht, aber soweit ich es vermute kann man mit dem AlarmUpdate Command die alten bereits abgerufenen AlarmData Zeilen aus dem WR RAM löschen. Wir haben das Kommando m.W. bisher noch nicht bei einer Original DTU mit getraced daher ist die Anwendung hier nicht zweifelsfrei klar.
Alles was ich bisher dazu in Erfahrung bringen konnte findet sich hier https://github.com/lumapu/ahoy/wiki/Protocol#wie-sieht-das-device-info-kommando-alarm-data-0x11--alarm-update-0x12-aus-

@stefan123t stefan123t changed the title Anzeige last Alarm: Feature Request: Anzeige last Alarm: Jan 22, 2023
@lumapu
Copy link
Owner

lumapu commented Jan 23, 2023

@stefan123t danke für deine Antwort. Ins Wiki hatte ich schon geschaut, aber nichts brauchbares finden können

@stefan123t
Copy link
Collaborator

@lumapu eventuell hilft hier nochmal der Blick in die Hoymiles DTU Pro sources oder ein neues Trace mit einem vollen Event/AlarmLog. Dann sollte die Hoymiles DTU ja das AlarmData Log aufräumen und dazu braucht sie mE auch das AlarmUpdate.
Laut Thomas kann man mit schneller Abfrage des SystemConfig Commands nach einem PowerLimit Fehler im WR Log erzeugen.

@lumapu lumapu added the fixed dev fixed label Aug 15, 2023
lumapu added a commit that referenced this issue Aug 15, 2023
* add alarms overview to WebGui #608
* fix webGui total values #1084
@knickohr
Copy link

knickohr commented Aug 15, 2023

Endlich ! 😎

IMG_1083

(Auch wenn noch nix drin steht 🤪)

@lumapu
Copy link
Owner

lumapu commented Aug 15, 2023

Inverter start sollte schon drin stehen, allerdings fehlt noch das Zeitzonen Offset, das kommt heute noch dazu 😇

@knickohr
Copy link

Ein kleiner Schritt für einen Programmierer, aber ein großer Schritt für die Benutzer 😅

Damit wird Ahoy nochmals um einen großen Schritt aufgewertet 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request fixed dev fixed
Projects
None yet
Development

No branches or pull requests

5 participants