Align grid measurements structure with pv and battery (BC)

[andig]

Follow-up to #17943

Use identical structure for grid measurements, drop unused energy total.

Before:

"gridConfigured": true,
"gridPower": -300,
"gridPowers": [-100, -200, 0],
"gridEnergy": 10000,
"gridCurrents": [1,2,3],

After:

"gridConfigured": true,
"grid": {
  "power": -300,
  "powers": [-100, -200, 0],
  "energy": 10000,
  "currents": [1,2,3]
},

TODO

• ui @naltatis
• ha @marq24 could you kindly take a look, too?
• check influx @andig InfluxDB: publish structs and pointers #18173
• check mqtt @andig Mqtt: improve publishing structs #18171

[marq24]

first of all THANKS for the mention! - this helps a lot (to avoid trouble with the integration) - I am going to review the PR this evening and will provide feedback

[marq24]

I have added now the following to the integration:
at startup it will be checked, if /api/state will include a grid object (if yes I do check additional, if the keys "pwer" & "currents" is present)...

if YES -> new Code: I will extract sensor data from ["grid"]["power"] & ["grid"]["currents"][0-2]

if NO -> old Code: I will extract sensor data from ["gridPower"] & ["gridCurrents"][0-2]

[andig]

@naltatis mqtt/influx sollten jetzt vorbereitet sein

[naltatis]

@andig wir haben leider einen Fall übersehen. Es kann heute durchaus vorkommen, dass kein Grid-Meter konfiguriert ist, wir aber dennoch eine gridPower haben. Grund ist, dass wir die Gridpower bei fehlendem Meter aus Ladezustand oder Ladeleistung ableiten und auch veröffentlichen. Das ist in dem Anwendungsfall relevant wenn ich evcc bspw. nur mit dynamischen Strompreisen (ohne PV) einsetze.

Siehe:
https://github.com/evcc-io/evcc/blob/master/tests/smart-cost-only.spec.js
https://github.com/evcc-io/evcc/blob/master/tests/smart-cost-only.evcc.yaml

In der UI wird in diesem Fall [PV], [Min+PV] durch die option [Smart] ersetzt.

Daher würde ich vorschlagen die gridConfigured-Änderung zu reverten.

[naltatis]

@andig ich hab den gridConfigured-Revert gerade gepusht. Wenn das für dich passt kann der PR rein.

[andig]

Gut, dass Du dafür einen Test hast!

[StefanSchoof]

@TheNinth7 Betrifft dieser Breaking Change deine Garmin App?

[TheNinth7]

@TheNinth7 Betrifft dieser Breaking Change deine Garmin App?

Ja, vielen Dank für den Hinweis. Werde auch eine Abfrage einbauen um die alte und die neue Struktur zu unterstützen. Ab welcher evcc Version wird die Änderung denn umgesetzt?

[StefanSchoof]

Die aktuelle 0.132.1 hat das noch nicht. Also mit der Version nach der 0.132.1.

[TheNinth7]

Noch eine Frage dazu:

Für Batterie und PV gibt es ja aggregierte Werte pvPower und batteryPower direkt unter result. Fürs Netz wird es aber den gridPower direkt unter result nicht mehr geben, sondern nur noch als "power" unter dem Element grid?

Also so schaut es jetzt aus:

{ "result": { "battery": [ { "power":956.3000000000001,"energy":1957.15122,"capacity":0,"soc":17.900000000000002,"controllable":true}], "batteryPower":956.3, "batterySoc":17.9, "gridPower":-1.6, "homePower":2305, "pv":[{"power":1350.3000000000002}], "pvPower":1350.3 } }

Und so wird es in Zukunft aussehen?

{ "result": { "battery": [ { "power":956.3000000000001,"energy":1957.15122,"capacity":0,"soc":17.900000000000002,"controllable":true}], "batteryPower":956.3, "batterySoc":17.9, "grid": { "power":-1.6 } "homePower":2305, "pv":[{"power":1350.3000000000002}], "pvPower":1350.3 } }

[naltatis]

Ja, genau so ist das.

[TheNinth7]

Danke! Hab gerade die neue Version der Garmin-App online gestellt. Hab es mit einem Mockup getestet, hoffe es klappt dann auch mit der nächsten evcc-Version (das jq-Filtering kann ich mit einem Mockup nicht testen).

[Maddin2020]

Noch eine Frage dazu. Der Syntax für eine Pushnachricht scheint dann nicht mehr zu gehen. Muss ich den auch anpassen

Eigenverbrauch: {{round (divf .homePower 1000) 3 }} kW geht noch richtig

Netz-Leistung: {{round (divf .gridPower 1000) 3 }} kW
Solar-Leistung: {{round (divf .pvPower 1000) 3 }} kW funktionieren nicht mehr sondern es kommt immer nur 0 in die Anzeige