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

ECU-R Can't Connect - Inverter Type 702 #83

Closed esloyer closed 2 years ago

esloyer commented 2 years ago

Hello, first of all, thank you for creating this integration, I'm excited to have my ECU-R data in HomeAssistant.

I have 23 DS3-S inverters of type 702, however, when trying to connect, I get an error as seen below. Any advice?

`Logger: custom_components.apsystems_ecur.config_flow Source: custom_components/apsystems_ecur/APSystemsSocket.py:348 Integration: APSystems PV solar ECU (documentation) First occurred: 7:20:50 AM (1 occurrences) Last logged: 7:20:50 AM

APSystemsInvalidData exception: Unsupported inverter type 702 please create GitHub issue. Traceback (most recent call last): File "/config/custom_components/apsystems_ecur/config_flow.py", line 45, in async_step_user test_query = await self.hass.async_add_executor_job(ap_ecu.query_ecu) File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) File "/config/custom_components/apsystems_ecur/APSystemsSocket.py", line 158, in query_ecu data = self.process_inverter_data() File "/config/custom_components/apsystems_ecur/APSystemsSocket.py", line 348, in process_inverter_data raise APSystemsInvalidData(error) custom_components.apsystems_ecur.APSystemsSocket.APSystemsInvalidData: Unsupported inverter type 702 please create GitHub issue.`

HAEdwin commented 2 years ago

You can try my fork in the meantime: https://github.com/HAEdwin/homeassistant-apsystems_ecur/archive/refs/heads/main.zip and replace the excisting APSystemsSocket.py.

esloyer commented 2 years ago

It appears that the fork worked, thanks!