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

Live-Daten: "Leistung / Netz" aktualisiert sich nicht #685

Closed
Forstgrund opened this issue Feb 21, 2024 · 19 comments · Fixed by #696
Closed

Live-Daten: "Leistung / Netz" aktualisiert sich nicht #685

Forstgrund opened this issue Feb 21, 2024 · 19 comments · Fixed by #696
Labels
bug Something isn't working

Comments

@Forstgrund
Copy link

What happened?

Die Version 2024.02.19.post1 aktualisiert die Live-Daten Anzeige von "Leistung / Netz" nicht.
Flasht man zurück auf 2024.02.09 funktioniert es perfekt

To Reproduce Bug

Normal Install 2024.02.19.post1 without Battery....

Expected Behavior

Die aktualisierung sollte automatisch erfolgen wie es immer war

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

2024.02.19.post1

Relevant log/trace output

No response

Anything else?

No response

@Forstgrund Forstgrund added the bug Something isn't working label Feb 21, 2024
@spcqike
Copy link

spcqike commented Feb 21, 2024

duplicate of #682 ?

@Ralph161065
Copy link

Gleiches Problem bei zwei verschiedenen Installationen

@der-peterh
Copy link

WebApps verstehe ich zwar nicht, aber einen 'Dreher' könnte es in webapp/src/components/InverterTotalInfo.vue Zeile 118 geben:
export default ....
hier werden props huaweiData und powerMeterData überkreuzt mit type: Object as PropType und type: Object as PropType
... hier ist dann aber auch das Ende meines Verständnisses erreicht - yarn, js und vue sind sowas von nicht meine Welt :(

@schlimmchen
Copy link
Member

Ich hab mir den Code im Frontend noch nicht angeschaut, aber es wäre wohl an der Zeit zu bemerken, dass der Fehler bei mir beispielsweise nicht auftritt (MQTT PowerMeter), und daher der Fehler in der Firmware zu suchen ist. Alle betroffenen melden Probleme mit dem HTTP PowerMeter. Da gab es auch Änderungen in den letzten Wochen, die durchaus dazu geführt haben können, dass aktualisierte Werte nicht mehr in die WebApp kommen oder überhaupt gar nicht abgeholt werden.

@der-peterh
Copy link

Mein ShellyEM3 läuft über MQTT und der Fehler tritt nur beim Auto Refresh auf: HM aktualisiert sich, PowerMeter nicht. Beim manuellen refresh (ziehen im GUI) wird hingegen aktualisiert.
Über MQTT schreibt die Firmware ebenfalls aktuelle Werte.
Ich vermute also entweder der (Websocket oder so was) wird falsch befüllt oder falsch ausgelesen...

@danny1605
Copy link

Gleiche Problem hier auch.

@Forstgrund
Copy link
Author

Meine Daten kommen vom Hichi am Zähler und in Tasmota kann ich sehen wie sich der Wert ständig ändert....nur mit der aktuellen Firmware bleibt der Wert stehen. Aktualisiere ich die Webseite zieht er sich den richtigen Wert vom Hichi. Ich hab jetzt wieder
Firmwareversion 2024.02.09 laufen....da geht alles wie es sein soll. Ich habe noch keine Batterie angeschlossen

@ingrimsch
Copy link

ingrimsch commented Feb 24, 2024

Kann ich bestätigen:

Site Refresh holt alle Live Daten im Browser - hier funktioniert es
Wert "Netz" vom OLED Display wird ebenfalls aktualisiert

Nur Autorefresh aktualisiert die "Leistung / Netz" Live Daten Kachel im Browser nicht

Version: 2024.02.19.post1,
Settings: ohne Batterie nur das http/json Powermeter (Tasmota / Hichi) konfiguriert

@cerise21
Copy link

Hier ebenso (mit 2024.02.19; selbstcompiliert wegen pinmapping für RS485-Anschluss eines SDM630-Powermeters):
Keine automatische Aktualisierung von 'Leistung/Netz'.
Zusätzlich ebenso keine Aktualisierung von 'MPPT-Gesamleistung' und (wahrscheinlich auch) von 'Ladezustand'.
In der Detail-Ansicht unter 'SmartSolar xxx' erfolgt die Aktualisierung der 'Eingang'- und 'Ausgang'-Werte wie gewohnt…

@schlimmchen
Copy link
Member

Ich sehe das Problem nun auch, wenn ich 2024.02.19.post1 installiere. Zuvor war ich auf einem Branch unterwegs, der die letzten Upstream-Änderungen noch nicht enthielt.

hier werden props huaweiData und powerMeterData überkreuzt mit type: Object as PropType und type: Object as PropType

Stimmt, das ist ein alter Fehler, der vermutlich deshalb keine Auswirkungen hat, weil die Objekte PowerMeter und Huawei gleich aufgebaut sind.

Der PowerMeter Wert im JSON des websockets wird aktualisiert, kommt also bis in die Web App, die im Browser läuft. Dann läuft irgendetwas schief.

Aha, hier: Im Upstream Projekt wurde die API insofern umgestellt, als dass die Daten der Inverter nur bröckchenhaft, oder genauer gesagt, nur pro Inverter übermittelt werden, um all zu große JSON Objekte in der Firmware zu vermeiden (wenn ich das alles richtig verstanden habe). In diesem Zusammenhang wird im Frontend nicht mehr die Nachricht aus dem Websocket stumpf als this.liveData interpretiert. Wir müssen die OnBattery-spezifischen Teilobjekte der Nachricht aus dem Websocket ausdrücklich übernehmen. Der Fehler betrifft also auch die Live-Header-Werte der Batterieschnittstelle, der Huawei-Schnittstelle, und der VE.Direct-Schnittstelle.

schlimmchen added a commit to schlimmchen/OpenDTU-OnBattery that referenced this issue Feb 25, 2024
schlimmchen added a commit to schlimmchen/OpenDTU-OnBattery that referenced this issue Feb 25, 2024
@mastertape
Copy link

Wollte es auch grade melden… Ich bedanke mich bereits jetzt für den fix

@hhoebel
Copy link

hhoebel commented Feb 26, 2024

Ich habe den Fix in der aktuellen Version 2024.02.19.post1 getestet und "Leistung/Netz" und "MPPT Gesamtleistung" aktualisieren bei mir immer noch nicht.

@schlimmchen
Copy link
Member

Also das hier diskutierte Problem ist mit der 2024.02.19.post1 erst in die Firmware reingekommen. Und meine Änderungen in #696 sind noch nicht veröffentlicht. Was hast du getestet, @hhoebel?

@hhoebel
Copy link

hhoebel commented Feb 26, 2024

Da ich wegen dem Problem mit dem falschen Kanalcount des DPL bei meinem HMT2250 aktuell die von dir erwähnte Firmware in gepatchter Version betreibe, habe ich deinen Fix dort mit eingespielt, compiliert und dann gestestet. Entweder ist das aber nicht alles, was man ändern muss oder die Änderungen benötigen noch was anderes als nur einen Build in VSCode, um im BIN-File zu landen (bin noch neu im Kompilieren des Projekts).

@schlimmchen
Copy link
Member

schlimmchen commented Feb 26, 2024

Du hast garantiert vergessen, die web app zu bauen. Gehe in einer Kommandozeile in den Unterorder "webapp" dieses Repos und führe "npm run build" aus (oder noch genauer: Installiere npm (nodejs), führe "npm install" aus, und dann "npm run build").

Und/Oder schau hier: https://www.opendtu.solar/firmware/compile_webapp/

Und dann die Firmware bauen (du wirst bemerken, dass drei Binäre Dateien nun verändert sind laut GIT).

@hhoebel
Copy link

hhoebel commented Feb 26, 2024

Danke Dir! Sowas in der Art habe ich auch irgendwann vermutet. Kaum macht man es richtig, funktioniert es auch schon :-)

@ingrimsch
Copy link

ingrimsch commented Feb 26, 2024

Getestet mit dem Build von https://github.com/helgeerbe/OpenDTU-OnBattery/actions/runs/8037760180
--> funktioniert! :)

@schlimmchen Vielen Dank!

@der-peterh
Copy link

Getestet mit dem Build von https://github.com/helgeerbe/OpenDTU-OnBattery/actions/runs/8037760180 --> funktioniert! :)

@schlimmchen Vielen Dank!

Same here - kudos @schlimmchen

Copy link

github-actions bot commented Apr 4, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants