Skip to content
New issue

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

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

Already on GitHub? # to your account

Multiple PV Inverter in HomeAssistant #36

Closed
benedictjaeger opened this issue Jan 29, 2023 · 3 comments
Closed

Multiple PV Inverter in HomeAssistant #36

benedictjaeger opened this issue Jan 29, 2023 · 3 comments
Labels
shipped This issue has been shipped. 🚢 specsheet mismatch Issue related to the mismatch between the victron specsheet and the actual implementation.

Comments

@benedictjaeger
Copy link

Hi,
I run 3 PV inverters on my Victron Multiplus. These are also recognized in Venus and have different VRM instances (34,35,36).
However, when I run a scan in HomeAssistant, only the first inverter (34) is detected. I can recognize this from the displayed values.
Any solution for this?
BR
Ben

@sfstar
Copy link
Owner

sfstar commented Feb 1, 2023

Hello and thank you for opening this issue.
I haven't tested the integration with multiple PV inverters present.
The specification for the modbus service states the following unit id's to be valid id's for modbus queryable devices
Screenshot 2023-02-01 at 19 10 49

As can be seen in the provided screenshot the id's of these PV inverters should have been in the range of 20 until 29.
Unit id 34 is a valid id for a gavazzi meter but not a pv inverter instance.
However, the integration currently doesn't enforce only searching for meters on unit id 34.
That's why only 1 of the PV inverters gets detected.
On the one hand adding the undefined unit ids 35 and 36 to this integration should solve your issue.
But on the other hand there are increasingly more and more cases reported in which real world situations don't match with the specification.
Rather than patch hunting each issue i'm trying to gather a list of specification divergences to present to victron.
So that the modbus document is the definitive specification for this integration.
Since, patching and diverging from the specification won't ensure that the integration will keep working once victron releases another patch

@benedictjaeger
Copy link
Author

Thanks for this solution. 3 Inverters know are found in HA. Would be great to provide this table also in the documentation. Now its dark outside and I will test the connection tomorrow when the inverters start producing.

@sfstar
Copy link
Owner

sfstar commented Mar 29, 2024

The release (v0.2.0) has been shipped.
Closing issue, feel free to re-open if the issue persists with the new release

@sfstar sfstar closed this as completed Mar 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
shipped This issue has been shipped. 🚢 specsheet mismatch Issue related to the mismatch between the victron specsheet and the actual implementation.
Projects
None yet
Development

No branches or pull requests

2 participants