Site: publish meter values as map (BC)

[andig]

Individual pv and battery values are published as a map with device name as key instead of an array.

Before:

"pv": [
  { "power": 1000, "energy": 1235 },
  { "power": 1000, "energy": 1235 }
],
"battery": [
  { "power": -1000, "soc": 50 },
  { "power": -2000, "soc": 80 }
]

After

"pv": {
  "pv_north": { "power": 1000, "energy": 1235 },
  "pv_south": { "power": 1000, "energy": 1235 }
},
"battery": {
  "sungrow": { "power": -1000, "soc": 50 },
  "powerwall": { "power": -2000, "soc": 80 }
}

TODO

• [x] Check UI impact @naltatis

[andig]

@naltatis hier fallen wieder die Tests um- evtl. berechtigt?

[naltatis]

UI ist nun so angepasst, dass sie mit Object statt Array bei pv und battery klar kommt.

🚨 Müssen wir auf jeden Fall mit in den Release Notes als Breaking API Change erwähnen. Da werden einige Integrationen (inkl. meiner eigenen PV-Einzelauswertung in Home Assistant) für angepasst werden müssen. Find aber, dass das ne gute Änderung ist. Da wir damit nicht mehr von der Konfigurationsreihenfolge abhängig sind.

[andig]

Wirklich unschön ist allerdings, dass die keys jetzt mehr und mehr die db-Namen sein werden. Ausnahme wäre hier grid weil da die Verwendung des Zählers klar ist. So ganz konsistent erscheint mir das noch nicht.

[naltatis]

Wirklich unschön ist allerdings, dass die keys jetzt mehr und mehr die db-Namen sein werden.

Wie meinst du das? Das ist doch jetzt so, dass der key der yaml-Name bzw. der DB-Identifiert ist:

[andig]

Heute im master?

[naltatis]

Ah, ich hab deinen Satz falsch gelesen. Ja, die Keys werden/müssen mehr und mehr die DB Namen werden. Bei Fahrzeugen ist das ja im Master schon so.

Wir könnten auch zusätzlich noch nen Identifiert pro Gerät per UI pflegen lassen, glaube aber, dass das wenig Mehrwert hat. Dann lieber das title Feld für die Meter angehen 😁

[andig]

Since name will be placed with title for database devices this PR no longer provides much value.