search

ksheumaker / homeassistant-apsystems_ecur

Home Assistant custom component for local querying of APSystems ECU-R Solar System
Apache License 2.0
166 stars 42 forks source link

Support for DS3D-L inverter #135

Closed dclobato closed 1 year ago

dclobato commented 1 year ago

Today, I added 4 more panels to my setup. Now, I have 4 micro inverters generating 8.64kWp

Web capture_14-11-2022_175844_10 0 1 245

I had 3 QS1 microinverters, and they have all details on the integration

Web capture_14-11-2022_175715_10 0 1 245

The new inverter has no details at all...

Web capture_14-11-2022_175741_10 0 1 245

The new inverter is a DS3D-L.

I saw, on the code, that there is support for DS3 inverters. Is it possible to add support for the DS3D model? What information should I collect/provide to allow this new model be added to the integration?

HAEdwin commented 1 year ago

Have you tried uninstalling the integration (Settings>Devices & Services>Integrations and then using the menu to delete the integration) restart HA and add the integration again. The entities might than get added/discovered. If this works it would mean that the system options don't. image

dclobato commented 1 year ago

Have you tried uninstalling the integration (Settings>Devices & Services>Integrations and then using the menu to delete the integration) restart HA and add the integration again.

I understand that the integration detected the new entity.

I believe that the problem lies on the fact that the DS3D-L inverter has a different code. From the code, each inverter has a different code: 01 for YC600/DS3, 02 for YC1000 and 03 for QS1. Maybe the DS3D-L has a different code on the response string...

HAEdwin commented 1 year ago

Yes, that's a good starting point if entities from the inverter are not discovered. You can uncomment line 262 by removing the first # . In /config/home-assistant.log you will find the results from the inverter query. Can you share that string for further analysis?

dclobato commented 1 year ago

Here it is

2022-11-15 17:33:17.001 WARNING (SyncWorker_1) [custom_components.apsystems_ecur.APSystemsSocket] b'415053313130313139303030323030303100042022111517284180400004919001303302580089001800d90025002c00448040000512700130330258008d006900d9006e001d000a8040000513600130330259008d000e00da000c000d000b7060000305790130340258009500e400d7009700d7454e440a'
HAEdwin commented 1 year ago

Daniel, I've made to changes to APSystemsSocket.py to add compatibility with this new DS3D-L inverter. Before a new release could you check if this works out fine for you? Follow link for changes. https://github.com/ksheumaker/homeassistant-apsystems_ecur/commit/bcd62f98a08fdbbc5a9d3ed1deb11fab37a035c2

dclobato commented 1 year ago

Before a new release could you check if this works out fine for you?

I believed it's working now!

Web capture_16-11-2022_15125_10 0 1 245

Web capture_16-11-2022_15151_10 0 1 245

Can I close the issue?

Thanks!

HAEdwin commented 1 year ago

Yes you can, glad it worked out fine. Thanks for supplying info on this relatively new inverter so that I could add it. It will be included in the new release.