search

ksheumaker / homeassistant-apsystems_ecur

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

Please add inverter type 702 für DS3-S #85

Closed alpapst closed 2 years ago

alpapst commented 2 years ago

Adding ECU-B gave me the following log entry:

ERROR (MainThread) [custom_components.apsystems_ecur.config_flow] APSystemsInvalidData exception: Unsupported inverter type 702 please create GitHub issue.

imonlinux commented 2 years ago

My APsystem solar was just installed and I'm getting the same HA log entry as @alpapst above.

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.10/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.

Thanks for your work.

HAEdwin commented 2 years ago

Sorry for my late response, I was on vacation. There are two ways for solving this.

  1. Edit APSystemsSocket.py on line 48 add the 702 inverter type (restart home assistant afterwards)
  2. recommended method: download my repository at https://github.com/HAEdwin/homeassistant-apsystems_ecur/archive/refs/heads/main.zip and replace APSystemsSocket.py (restart home assistant afterwards)
HAEdwin commented 2 years ago

@alpapst @imonlinux Did my response solve you issues? If so @alpapst please close this issue for I'm not able to moderate this. Thank you!

imonlinux commented 2 years ago

Thanks for the instructions. I just made the change and it is now working as expected.

Thanks again for your work.

HAEdwin commented 2 years ago

@alpapst please close this issue for I'm not able to moderate this. Thank you!

alpapst commented 2 years ago

@HAEdwin Sorry for the late response, I was on vacation and had no access to my GitHub account. Thank you for your support!