Skip to content

Fix multi-inverter entity collisions#89

Open
scruysberghs wants to merge 1 commit into
redpomodoro:mainfrom
smartenergycontrol-be:fix/multi-inverter-hub-ids
Open

Fix multi-inverter entity collisions#89
scruysberghs wants to merge 1 commit into
redpomodoro:mainfrom
smartenergycontrol-be:fix/multi-inverter-hub-ids

Conversation

@scruysberghs

Copy link
Copy Markdown

Summary

  • fix entity and device ID collisions between multiple fronius_modbus config entries
  • expose all configured smart meters instead of only Meter 1
  • fix Discharge limit so it writes the discharge setter

Why

Multiple inverter setups currently collide in Home Assistant because entity IDs are not scoped per hub. This prevents additional config entries from registering cleanly.

Validation

Tested on Home Assistant Core 2026.3.4 with three Fronius Modbus entries. All three inverters were added successfully, with battery and meter entities visible for each.

Attribution

This builds on the original approach proposed in #36 by @larsxschneider.
You can use it here:

Scope entity and device identifiers per hub so multiple Fronius Modbus entries can coexist, and align meter polling and discharge control behavior with the configured hardware. Document multi-inverter support and credit the original idea from PR redpomodoro#36.
@Julian0021

Copy link
Copy Markdown

this repo is dead, switch over to califos fork where its already fixed

@scruysberghs

Copy link
Copy Markdown
Author

would be nice if this repo could be archived and a link to https://github.com/callifo/fronius_modbus added in the readme? @redpomodoro ?

This repo still shows up first in Google searches

@scruysberghs

Copy link
Copy Markdown
Author

this repo is dead, switch over to califos fork where its already fixed

sadly with callifos repo the problem also exist, only the first inverter you add has entities
afbeelding

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

Successfully merging this pull request may close these issues.

2 participants