Problem with setting ip adress ECU-R After install integration on new HA

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

Problem with setting ip adress ECU-R After install integration on new HA #60

Closed RobertOz20 closed 2 years ago

RobertOz20 commented 2 years ago

Hello,

i install integration 1.2.11 by HACS, but i haven't any page, prompt to enter ip adress my ECU-R. MY Home Assistant 2022.3.5. I don't know how configure it ...

HAEdwin commented 2 years ago

You have to install the integration also, that's where the IP-Address is being entered.

RobertOz20 commented 2 years ago

I'v install ... but i havent any page to enter parameters ..

HAEdwin commented 2 years ago

In HA go to [Settings] > [Devices and Services](Integrations, devices,entities and Areas) > on the right bottom select [Add Integration] and look for APSystems PV solar ECU, select it and you will be able to add an IP-Address In case you have ECU-R-Pro firmware installed (look for the logo on the back of the ECU), you might run into compatibility problems.

RobertOz20 commented 2 years ago

Thanks, it works !!! I'm new in HA. I have ECU-R, and 6 inverters.

gregyyc commented 2 years ago

@HAEdwin you mention above if you have the IEEE 2030.5 CSIP logo you have ECU-R-Pro firmware and may have compatibility problems. That might be where I am at. I do have that logo on the on the back and when adding the IP address I get "Can't connect to ECU, check log" and the following in the logs ....

APSystemsInvalidData exception: Error getting checksum int from 'Inverter data' data=b''

Is there a work around for that firmware?

Thanks!

HAEdwin commented 2 years ago

@gregyyc The challenge with this firmware is that the response to commands appears to be slightly different or unexpected. It is difficult to anticipate this if you as a developer do not have an ECU-R with ECU-pro firmware.

The initial starting point for this integration was the (apparently) obsolete standard ECU-R without Sunspec logo on the back which also happens to have compatibility with the ECU-B.

Solving the problem requires a lot of patience and troubleshooting. Also, the owner of the ECU-R-Pro firmware must have some Python knowledge to indicate what to change in the code or configuration. Apart from this integration for HA, there are alternatives in the form of scraping or direct Zigbee reception from the inverters without ECU. I myself have thought about an ECU proxy where the data to EMA is first intercepted for use in HA.

gregyyc commented 2 years ago

@HAEdwin I completely understand the challenges for this. Happy to see any solutions existed and installed HA just for this for this solution.

I am a software developer with some limited python experience and a ECU-R with ECU-pro firmware so I'd be happy to help. I'd need a little guidance from you on where the important areas are and what methods you use to debug by dumping the response data (I'm assuming to a log or something along those lines).

That said, If you're thinking about changing your approach dramatically I'd also understand not wanting to continue investing time to this approach.

HAEdwin commented 2 years ago

@gregyyc The integration works great for me and other owners of an ECU-R (without Sunspec logo) or ECU-B so for me personally there is currently no need to change the approach.

We could give it a try. If you would like to raise a new issue with the subject something like: ECU-R with ECU-R-PRO firmware compatibility. Then we can start with the latest 1.2.11 version to work on. The first step is to solve "when adding the IP address I get Can't connect to ECU, check log and the following in the logs APSystemsInvalidData exception: Error getting checksum int from 'Inverter data' data=b'"

HAEdwin commented 2 years ago

@RobertOz20 Since your issue is solved, can you close this issue please?

HAEdwin commented 2 years ago

@gregyyc Look at this issue if you'd like to try to solve it: https://github.com/ksheumaker/homeassistant-apsystems_ecur/issues/67

HAEdwin commented 2 years ago

@gregyyc Was the issue ever solved? If not you might want to try my fork, is has a lot of improvements: https://github.com/HAEdwin/homeassistant-apsystems_ecur/archive/refs/heads/main.zip

gregyyc commented 2 years ago

@gregyyc Was the issue ever solved? If not you might want to try my fork, is has a lot of improvements: https://github.com/HAEdwin/homeassistant-apsystems_ecur/archive/refs/heads/main.zip

Hey - last time I tried it worked out well. I've since moved to a different solution interacting with my Sense monitor but will probably dig back into this again soon. Thanks for the follow up!