Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New feature request: Multiple inverter units and meter #5

Closed
julezman opened this issue Jun 18, 2020 · 8 comments
Closed

New feature request: Multiple inverter units and meter #5

julezman opened this issue Jun 18, 2020 · 8 comments

Comments

@julezman
Copy link

Thanks for this great HACS component. I'd like to request two new features:

  1. Enhance the component so that it can read data from multiple inverters.
  2. Add enhancement to read a Wattnode\SolarEdge modbus meter attached to the inverter. This would primarily allow users to keep track of imported and exported power from the house as well as numerous other attributes the meter records.

Didn't know if any work was going on in this regard and happy to help via a pull if not.

@binsentsu
Copy link
Owner

Hi.

  1. Reading data from multiple inverters should be possible. Just go through the integration UI configuration again for the second inverter.
  2. I don't own a meter, so can't test this out myself. I think the fork of goergch /
    home-assistant-solaredge-modbus includes meter implementation as far as I can see. Mayb you can check this out and test it. If it is ok, I can merge it.

@miguelrjim
Copy link

I've tested goergch fork and it works exporting the meter information into HA; but the way the code is set up it will fail reading the registers if there's not a meter installed so I wouldn't recommend merging its fork as it is.

@binsentsu
Copy link
Owner

@miguelrjim Thanks for the info. Are the meter and inverter hosted on the same IP address in such a setup?

@stuarts0011
Copy link

@miguelrjim Thanks for the info. Are the meter and inverter hosted on the same IP address in such a setup?

yes,

@binsentsu
Copy link
Owner

There is currently a prerelease V1.2.0-A2 which has meter support. It would be great if it would be tested by some more people before release...

@marpi82
Copy link

marpi82 commented Jul 30, 2020

There is currently a prerelease V1.2.0-A2 which has meter support. It would be great if it would be tested by some more people before release...

I am also testinh this release, I have one meter connected to the inverter - I have not noticed any problems so far

@zeeuk
Copy link

zeeuk commented Aug 29, 2020

Thank you very much! The added function works perfectly (meter)

@binsentsu
Copy link
Owner

Finally released, closing this one...

binsentsu pushed a commit that referenced this issue Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants