CJNE / ha-myenergi

Home Assistant integration for MyEnergi devices
MIT License
150 stars 33 forks source link

Grid Voltage taken from Zappi Slave, no Eddi vHub grid voltage #534

Closed japaho closed 5 months ago

japaho commented 6 months ago

Version of the custom_component

Configuration

Add your logs here.

Describe the bug

Eddi (as vHub) + Zappi 2 - grid voltage sensor is taken from Zappi - no Eddi voltage sensor.

Debug log


Add your logs here.
G6EJD commented 6 months ago

I don’t understand this, it makes no sense. The grid voltage at the Zappi and Eddi measurements are identical and always will be.

japaho commented 6 months ago

Except they aren't, my eddi is the hub device and the voltage readings are from the zappi, which is reading about 3-4v higher due to drop on the cable or calibration issues. But the question is why does the reading come from the zappi (slave), not the eddi (master) and could there be individual readings on a device basis rather than at the hub entity

G6EJD commented 6 months ago

It’s a design choice because if there is a Zappi present, that makes a more logical choice for voltage readings to cater for 3-phase installations.

The Eddi is a single phase device the Zappi can be 1 or 3-phase.

If a Zappi is not fitted then the voltage source comes from the Eddi.

In your case your getting a better voltage reading from the Zappi as it’s not suffering from cable resistance and hence a reduced voltage.

Whatever the device source, it would be easy to achieve a voltage calibration in HA by adding in an offset, ideally in percentage terms to keep the reduction of voltage aligned for the Eddi. A constant offset would introduce more errors and should be avoided.

G6EJD commented 6 months ago

This isn’t an issue for this integration and can never be solved unless MyEnergi make a change - highly unlikely.

This issue needs to be closed by the instigator.

japaho commented 5 months ago

Hi,

Are you saying the eddi voltage is not available to be displayed? I accept your point that the zappi is the logical choice for a holistic system view being 3ph capable but if the other devices are submitting voltage readings is it possible to add those as another sensor measurement?

On Mon, 27 May 2024 at 17:04, G6EJD @.***> wrote:

This isn’t an issue for this integration and can never be solved unless MyEnergi make a change - highly unlikely.

This issue needs to be closed by the instigator.

— Reply to this email directly, view it on GitHub https://github.com/CJNE/ha-myenergi/issues/534#issuecomment-2133773950, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACWBP6USGDPE7VC3AVXXVYDZENKSPAVCNFSM6AAAAABH2MK5S6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZTG43TGOJVGA . You are receiving this because you authored the thread.Message ID: @.***>

G6EJD commented 5 months ago

Only the master returns the voltage readings and you can’t have two masters. If you check the Eddi response you’ll see it returns 0.0volts when it’s not the master.

japaho commented 5 months ago

Thanks for your help.

japaho commented 5 months ago

.