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

Nach Upgrade von 245 auf 254 ist Web GUI fehlerhaft #2041

Closed
hannesrh opened this issue Feb 17, 2022 · 4 comments
Closed

Nach Upgrade von 245 auf 254 ist Web GUI fehlerhaft #2041

hannesrh opened this issue Feb 17, 2022 · 4 comments

Comments

@hannesrh
Copy link

Kernel: Linux 5.10.63-v7+ GNU/Linux
Python 3.7.3
openWB Version: 1.9.254

Alle Daten werden über den ioBroker / Node Red / MQTT an die openWB standalone übertragen.
Seit dem Upgrade von 245 auf 254 ist die Anzeige fehlerhaft.

  1. Bei der PV Leistung fehlt die Einheit W, außer die PV Leistung ist 0, dann steht dort 0 W
  2. Die PV Leistung wird wie in der MQTT Schnittstellen Beschreibung als openWB/set/pv/1/W PV-Leistung in Watt, int, negativ übertragen. Hat sich da etwas geändert?
  3. Die geladene Leistung in der GUI oben und beim Ladepunkt sind unterschiedlich, warum?

Bild1

@yankee42
Copy link
Contributor

yankee42 commented Feb 17, 2022

Ja, da hat sich was geändert durch #1879 + #1942. Vorher wurde das Vorzeichen komplett ignoriert, jetzt wird ein positives Vorzeichen erwartet. Eigentlich war es garnicht meine Absicht das Verhalten dort zu ändern. Ich glaube ich hatte garnicht realisiert, dass das Vorzeichen komplett ignoriert wurde. In #2030 wurde auch die Dokumentation dazu angepasst.

Ich finde es eigentlich sinnvoller/logischer das Vorzeichen nicht zu ignorieren (so wie es jetzt ist), da ansonsten eine übertragene Standby-Last als Erzeugung gewertet wird. Allerdings ist die Änderung so natürlich ein "breaking change". Muss @LKuemmel oder @benderl was zu sagen ob das "won't fix" ist oder da wieder absolute Werte genommen werden sollen.

@hhoefling
Copy link
Contributor

hhoefling commented Feb 17, 2022

Da bin ich auch schon drauf reingefallen.
Kompatiblität ade.

vorher stand da mal

                if (float(msg.payload) >= -10000000 and float(msg.payload) <= 100000000):
                    if (float(msg.payload) > 1):
                        pvwatt=int(float(msg.payload.decode("utf-8"))) * -1
                    else:
                        pvwatt=int(float(msg.payload.decode("utf-8")))

was einem abs() gleich kommt.
Also wurde der negative-Absolutbetrag in die Ramdisk geschreiben egal was auf das mqtt topic geschrieben wurde.

Ich hatte ja schon mal darum gebeten solche "Designe Änderungen" nur in der 2.0 vorzunhemen.

@hhoefling
Copy link
Contributor

Wenn ich dran denke mit welcher velementz mein 80=80% Änderungsvorschlag abgeschmettert wurde dann erstaunt mich die einfachheit dieser verschlimmbesserung doch schon ein wenig.

@benderl
Copy link
Collaborator

benderl commented Jul 7, 2022

Die Anzeige sollte mit der aktuellen Version wieder passen. Wenn es noch zu Problemen kommt, bitte ein neues Issue öffnen.

@benderl benderl closed this as completed Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants