homeassistant-apsystems_ecur icon indicating copy to clipboard operation
homeassistant-apsystems_ecur copied to clipboard

Published 20 hours ago • verified publisher icon ksheumaker

Enhancement: Add per inverter AC_power_total, per panel DC_voltage & DC_current

Open mi-hol opened this issue 5 months ago • 11 comments

It seems that currently APSystemsECU.py has some shortcomings.

From my view these are:

  • AC_voltage is reported in per panel data, where DC_voltage would be expected. Further outlined in this comment

  • AC_power for total inverter output is missing, but seems the most relevant value as AC_power is what gets charged by utility companies. Currently total DC input power is reported as "power" but that is a too high value due to DC to AC conversion losses.

  • per panel DC_voltage & DC_current are missing

From findings during reverse engineering APSystems ECU communication in https://community.home-assistant.io/t/apsystems-aps-ecu-r-local-inverters-data-pull/260835/238 I assume (but have yet to proof!) that above shortcomings could be fixed.

Unfortunately the fix could introduce compatibly issues for existing solutions. Eventually just a fix of "power" as "AC_power for total inverter output" & per panel DC_voltage could be feasible.

Therefore an alternative would be adding a new class with the new values (and eventually fix of "power" as "AC_power for total inverter output" in current class)

May I ask what course of action would be the best approach to address the shortcomings?

mi-hol avatar Sep 14 '24 10:09 mi-hol