AC Battery CT clamp on Eddi not added

godfreym29 commented 2 years ago

I've installed ha-myenergi intergration and it has located all my 3 devices and created 31 enteries. I have a Eddi, Harvi and a Hub

The Eddi has a CT clamp attached to ectt3 and configured as "AC Battery", however it's not being found by the intergration. Using acurl I am able to retrieve the data for this CT clamp fine

I have redownloaded the intergration and restarted HA but nothing has updated. both the System Options for myenergi are venabled. const.py lists the version as VERSION = "0.0.16", which is the same as on the latest version on github.

Data retrieved from Eddi using curl:

"eddi": [
    {
        "sno": 14467978,
        "dat": "21-12-2021",
        "tim": "18:42:43",
        "ectp2": 582,
        "ectp3": -14,
        "ectt1": "Internal Load",
        "ectt2": "Grid",
        "ectt3": "AC Battery",
        "bsm": 0,
        "bst": 0,
        "cmt": 254,
        "dst": 1,
        "div": 0,
        "frq": 50.02,
        "fwv": "3200S3.051",
        "gen": -4,
        "grd": 578,
        "pha": 1,
        "pri": 1,
        "sta": 1,
        "tz": 0,
        "vol": 2402,
        "hpri": 1,
        "hno": 1,
        "ht1": "Tank 1",
        "ht2": "Tank 2",
        "r1a": 0,
        "r2a": 0,
        "rbc": 0,
        "rbt": 1,
        "tp1": 127,
        "tp2": 127
    }
]

G6EJD commented 2 years ago

I’ve noticed that HA does not update the integration, so I use the HA file manager and upload the files separately to the custom folder, it works that way. I also manually install the Python files.

godfreym29 commented 2 years ago

G6EJD

I've manually downloaded the folders & files and copied them to the custom_components folder and restarted etc. But no change, i still do not have the battery showing under the Eddi.

I believe that the const.py file may not have been updated with the new version number.

G6EJD commented 2 years ago

Johan afaik doesn’t have an Eddi so it could be there is no CT detection included yet, I haven’t checked the code yet to see if that’s so. Works ok for a Harvi, but maybe not Zappi too.

godfreym29 commented 2 years ago

G6EJD

The intergration is picking up:

sensor.myenergi_eddi_xxxx_power_ct_internal_load - "ectt1": "Internal Load" - which is "ectp1" on the eddi - the internal CT clamp which measures what is diverted to the immersion
sensor.myenergi_eddi_xxxx_power_ct_grid - "ectt2": "Grid" - which is "ectp2" on the eddi - import/export to the grid

but not "ectt3": "AC Battery" - "ectp3"

sparks1372 commented 1 year ago

These will hopefully resolve the issue - #309 and https://github.com/CJNE/pymyenergi/pull/4

CJNE commented 1 year ago

Thanks @sparks1372, will include this in the next release. @G6EJD is correct, I don't have an Eddi so I missed this :)

G6EJD commented 4 days ago

This issue needs to be closed...

CJNE / ha-myenergi

AC Battery CT clamp on Eddi not added #59