sfstar / hass-victron

Integration for Home Assistant to fetch data from the victron gx device via modbusTCP
Apache License 2.0
157 stars 24 forks source link

Please include addition ModBUS IDs for inverters #200

Closed AminShAT closed 1 month ago

AminShAT commented 2 months ago

As discussed please allow IDs above 36 for AC inverters.

a-wolter commented 2 months ago

Just added some opendtu inverters with id 34 and 35 and they also don’t show up. I think it’s maybe not only is 36 but I will add a third opendtu inverter soon with id36 I already redownloaded the integration set to “main” version. And restarted homeassistant.

In the system settings the slider is set to add new devices. But only inverter ID 20 is there. 34 and 35 are still missing (but ID34 and 35 show up in VRM portal).

a workaround could be to change the id in the opendtu config so that it will show up in the cerbo with a different id and the integration van pull it. But what would be the correct ID range for additional pv inverters? Btw: the open dtu default config starts with 34 for inverters. If that someone confirms this is wrong I can let them know to change it.

sfstar commented 2 months ago

@a-wolter please add your comment to #180 so that the opendtu related issues can be tracked in a single place.

sfstar commented 1 month ago

The issue has been shipped in the latest release (v0.3.0) closing issue as resolved. Feel free to re-open if the issue persists with the v0.3.0 release