evcc icon indicating copy to clipboard operation
evcc copied to clipboard

Custom charger: add more apis

Open andig opened this issue 1 year ago • 9 comments

Aktuell kann der custom charger noch keine Leistung. Stattdessen wäre ein separates meter am Loadpoint zu verwenden. Das ist nicht ganz offensichtlich und könnten wir hinzu fügen- dann müssten wir uns allerdings fragen, welche der vielen meter APIs wir hier noch anbieten wollen?

Originally posted by @andig in https://github.com/evcc-io/evcc/issues/14504#issuecomment-2183952954

andig avatar Jun 22 '24 09:06 andig

Guter Punkt. Daran hab ich überhaupt nicht gedacht.

premultiply avatar Jun 22 '24 09:06 premultiply

Meter hätte heute:

Power    provider.Config
Energy   *provider.Config  // optional
Currents []provider.Config // optional
Voltages []provider.Config // optional
Powers   []provider.Config // optional

// battery
capacity    `mapstructure:",squash"`
battery     `mapstructure:",squash"`
Soc         *provider.Config // optional
LimitSoc    *provider.Config // optional
BatteryMode *provider.Config // optional

Davon wäre BatteryMode nicht relevant und LimitSoc erwarten wir für bestimmte Arten von Battery Control einen Setter. Bliebe also alles über dem Strich.

andig avatar Jun 22 '24 09:06 andig

Würde damit evtl auch https://github.com/evcc-io/evcc/issues/12276 lösbar?

VolkerK62 avatar Jun 22 '24 10:06 VolkerK62

Nein. Deine Schaltsteckdose ist kein custom device.

andig avatar Jun 22 '24 11:06 andig

Ah, Mist. Erst denken, dann schreiben 😄

VolkerK62 avatar Jun 22 '24 11:06 VolkerK62

Power und Energy sollten schon sein, Currents und Voltages könnten auch noch rein. Den Rest halte ich erstmal für nicht relevant.

premultiply avatar Jun 22 '24 19:06 premultiply

Na dann brauchts Powers auch fürs Vorzeichen, oder? Also alles „über dem Strich“?

andig avatar Jun 22 '24 20:06 andig

Wozu beim Charger? Da gibt es nur eine Richtung: Rein in's Fahrzeug. ⚡️➡️🚗 Daher immer alles unsigned.

premultiply avatar Jun 22 '24 20:06 premultiply

Solange es kein Bidi gibt. Aber verstanden, danke 🙏

andig avatar Jun 22 '24 20:06 andig