-
Notifications
You must be signed in to change notification settings - Fork 43
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
Enhancement: Add per inverter AC_power_total, per panel DC_voltage & DC_current #268
Comments
Example from (restricted to installers) APS EMA report: Data same time from (slightly modified) ECU_B.py:
Difference "Current power (DC): 945 W" to "actual AC power: 897 W (from EMA)" is 5.2% (= DC to AC conversion loss) |
So this goes for an ECU-B with an YC600? |
for an ECU-B with a DS3 and after this works with a QT2. Most important from my view is an answer to question:
because this will drive my next activities |
The query retrieves other data fields than what is being sent to EMA. The query in the integration does not contain DC voltages from individual panels (afaik). The data pushed to EMA does so that's why you see them in the EMA site. We might have to break it down a bit to fully understand what you mean. |
correct |
So that data is correct, I mean - I only see AC voltages that are within the standard for the country ~230V AC in The Netherlands |
These are the data fields (YC600) which are being sent to EMA (currently zero because the inverter is down) but these voltages are actual DC, per panel voltages. |
the data I'm after is apparently extracted in api.py function get_inverters. |
Unfortunately not. What we've done in the beginning is looking and interpreting data dat is shown in the ECU app (later EMA manager app) and there it does not contain the data you're after. There is no documentation or further reference on what commands can be send to the ECU. |
The data is available in EMA web but only shown to privileged "installer" accounts (see screenshots in #268 (comment) showing the data I'm after) |
Yes but it ain't in the apps (unless you discover something else) and therefor is not included in this integration. The proxy method intercepts data that is sent to EMA and it does include the data. |
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?
The text was updated successfully, but these errors were encountered: