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

New parameter jd #8

Merged
merged 12 commits into from
Aug 25, 2024
Merged

New parameter jd #8

merged 12 commits into from
Aug 25, 2024

Conversation

niltrip
Copy link
Owner

@niltrip niltrip commented Aug 24, 2024

refactoring
new parameter
faster polling per default (still problems with special parameters)
special icons for some parameters

niltrip and others added 12 commits August 17, 2024 15:28
noch ein Fehler in __get_sensors_battery (siehe Kommentar) > dadurch keine Parameter in der Oberfläche
parameter werden dargestellt
JTS1_EMS_CHANGE_REPORT bpSoc is the right one we have to use
refactoring
prefix only for batterie
JTS1_ENERGY_STREAM_REPORT not nessasarry
61 Parameter
disable some debug messages
@MvH-user
Copy link

Hallo eine frage was ist das den für ein neuer Parameter würde mich intressieren .

@niltrip
Copy link
Owner Author

niltrip commented Aug 25, 2024

Primär sind damit die Parameter gemeint, die angezeigt werden. Tatsächlich ist ein neuer Parameter entstanden, der die Summe aus den Leistungen der einzelnen Strings darstellt. Da die Information der Strings öfters aktualisiert werden, wird auch dieser Parameter öfters aktualisiert und bringt dann ein besseres Ergebnis als "mpptPwr".

@jdammers jdammers merged commit e6d0c5e into main Aug 25, 2024
@MvH-user
Copy link

Hallo ich hab mir mal die neue Integration herrunter geladen , läuft bei mir . Vielen Danke
Muss jetzt erstmal Schauen was alles neue ist bei mir, Top Job . Darauf habe ich gewartet ich hoffe das ich jetzt meine Hausverbrauch aus der Powerocean erechnen kann.

@niltrip
Copy link
Owner Author

niltrip commented Aug 26, 2024

Hi, hauptsächlich sind es neue Werte.
Im Webportal waren aktuell die Werte der einzelnen Strings enthalten. Diese sind jetzt auch hier sichtbar. Der Vorteil: diese Werte werden öfters aktualisiert. @jdammers hat daraus noch einen Summenparamter "mpptPv_pwrTotal" gemacht. Die Werte von mpptPwr, sysLoadPwr und sysGridPwr werden immer noch nicht aktualisiert, es sei denn Du hast das Portal oder die App offen. Was meinst Du mit errechnen? Für mich sind hauptsächlich die Parameter für das Energiedashboard interessant. Endlich beide Strings und mehr Genauigkeit.

Rückfrage:
Beim Update musstest Du das Gerät neu anlegen, richtig?

@MvH-user
Copy link

Hi, mit errechnen meinte ich den Hausnetz Verbrauch zu errechnen wie in der App aber das klappt leider nicht weil sie nicht immer aktuell sind . Wenn man den Solarertrag z.B. (31,70 Kwh) minus ins Netz (22,5 kwh) minus in die Batterie (6,52 kwh) plus aus der Batterie (3,68) plus ins Netz (0,01 kwh) kommt man auf den Hausverbrauch das ist dann in meinen fallbeispiel 6.62 kwh Verbrauch .
Zu deiner frage Update , ja ich mustte die Intigration neu installieren und HA neu starten . Aber sonst laüft alles .

@jdammers
Copy link
Collaborator

@MvH-user der (gesamte) Solarertrag sollte funktionieren, wenn man den ' mpptPv_pwrTotal' Sensor benutzt. Das Problem ist der Netzbezug/einspeisung ('sysgridpwr') und die Leistung der Batterie 'sysloadpwr'. Die letzten beiden werden nur selten aktualisiert. @niltrip wir könnten doch die gesamt Leistungleistung der Batterien auch berechnen mit (bpack1_bppwr + bpack2_bppwr) bzw. je nach dem wie viele Batterien da sind.

@niltrip
Copy link
Owner Author

niltrip commented Aug 27, 2024

Hi, der Parameter sysLoadPwr bildet den hausverbrauch ab, ob von Batterie, Netz oder Solar. Berechnen kann man es, aber dafür gibt es auch schöne UIs.

image

Dort wird rechts auch berechnet, wenn ich keinen Parameter für rechts eintrage. Das wichtigste ist aus meiner Sicht immer der Hauszähler...

@niltrip
Copy link
Owner Author

niltrip commented Aug 27, 2024

Aber mir ist gerade noch etwas anderes interessantes aufgefallen. Wenn man debug einschaltet kann man im Protokoll sich die aktuelle Response vom Server herausziehen. Die kann man sich gut mit pretty-print in eine Datei abspeichern und dann mal gegen eine andere vergleichen. Meine hatte ich mal nachts erzeugt. Jetzt gerade habe ich nochmal geschaut und merke das der Parameter "JTS1_EMS_HEARTBEAT_pcsBpPower" die Leistung aus der Batterie anzeigt. Das könnte ein Ersatz für bpPwr sein, welcher auch immer sehr sporadisch kommt.

"pcsMeterPower": -1466.125, zeigt die Einspeisung an.

Die Parameter sind alle im JTS1_EMS_HEARTBEAT

@jdammers
Copy link
Collaborator

Cool! Damit kann man doch arbeiten. Machen wir das im gleichen branch?

@jdammers
Copy link
Collaborator

Eigentlich brauchen wir diese Liste hier nur mit den neuen Entitäten erweitern, oder?
sens_select = [ "bpRemainWatth", "emsBpAliveNum", ]

@niltrip
Copy link
Owner Author

niltrip commented Aug 27, 2024

Ja das können wir machen. Bei so einer Änderung brauchen wir keinen branch machen. Man kann im main branch arbeiten und dann einen Tag setzen und ein Release draus machen. Das wollte ich jetzt auch endlich mal machen (ein bißchen Doku). Also ein richtiges Release. Mir war nur gerade noch was aufgefallen übers Wochenende... Bei mir waren nach einem Neustart manche Parameter ausgegraut. Jetzt aber wieder da. Das hatte ich noch nicht? Jemand von Euch? Benutzt ihr beide jetzt schon das Prerelease? Läuft es bei Euch?

Also auf jedenfall hat die Configänderung zur Folge das man das Gerät nochmal neu anlegen sollte ach einem Update. Das wollte ich zumindest auch aufschreiben.

@niltrip
Copy link
Owner Author

niltrip commented Aug 27, 2024

            "meterHeartBeat": [
                {
                    "meterAddr": 106,
                    "meterType": 11,
                    "meterData": [
                        0.0,
                        0.0,
                        0.0,
                        -1464.4297

Der zeigt auch einen ähnlichen wert. Hier sind Daten zum DTSU666 drin (ModBus Adresse).

            "bpErrCode": [
                {
                    "errCode": [
                        201
                    ],
                    "moduleSn": "codierte sn"
                },
                {
                    "errCode": [
                        201
                    ],
                    "moduleSn": "codierte sn"
                }
            ]

ErrorCodes, die ich auch schon mal im Webportal gesehen habe...

Soviel zu tun.

@MvH-user
Copy link

Ist mir noch nicht aufgefallen , die Prerelease laüft bei mir gut und konstant keine aussätzer

@niltrip
Copy link
Owner Author

niltrip commented Aug 27, 2024

@jdammers Auf einem branch könnte man mal folgendes machen:
Für Parameter mit Einheit suggested_display_precision = 1 (Ein Temperaturwert hat im Moment mehr als Stellen, andere 3-5)
DIAGNOSTIC Eigenschaft festlegbar pro Parameter

Da fällt einem immer mehr ein.

@MvH-user danke für die Rückmeldung

Wer möchte kann sich auch mal mein anderes REPO anschauen (https://github.com/niltrip/hassio-ecoflow-cloud). Hier habe ich jetzt auch die PowerOcean drin über die Public API. Da braucht man aber einen Developer Account. Aber dort sind wirklich wenige Werte vorhanden. Da arbeite ich doch vielleicht hier noch weiter.

@jdammers
Copy link
Collaborator

@niltrip ich fange morgen damit an ... melde mich bei dir

@jdammers
Copy link
Collaborator

@niltrip achso ja, die pre-release läuft bei mir auch stabil. Alte Integration löschen und neue wieder hinstellen hat problemlos geklappt. Neben den config Änderungen hatten wir ja auch die unique IDs verändert.

@noheton noheton mentioned this pull request Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants