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

Issue with more than 1 inverter. #4

Closed
wishie opened this issue Sep 21, 2020 · 3 comments
Closed

Issue with more than 1 inverter. #4

wishie opened this issue Sep 21, 2020 · 3 comments

Comments

@wishie
Copy link

wishie commented Sep 21, 2020

I have 2 x GoodWe GW5048-EM inverters. When setting them up in Home Assistant with the following config:

sensor:                                                  
  - platform: goodwe                                     
    ip_address: 192.168.100.6                            
    scan_interval: 30                                    
    sensor_name_prefix: Inverter1                  
  - platform: goodwe                                     
    ip_address: 192.168.100.22                     
    scan_interval: 30                                    
    sensor_name_prefix: Inverter2

I do not get 2 sets of sensors as expected.

Below is list of sensors found in HA for the above config:
Screen Shot 2020-09-21 at 9 06 58 am

@wishie
Copy link
Author

wishie commented Sep 21, 2020

@mletenay perhaps you can just add your single inverter twice, but using a different sensor_name_prefix for each.. that would simulate 2 inverters well enough for this testing, no?

@wishie
Copy link
Author

wishie commented Sep 22, 2020

Oddly enough.. I updated the code this morning with your latest version and 2 inverters seem to be working.

@mletenay
Copy link
Owner

I believe I have found a problem with the multiple inverter setup.
The required change was however substantial and a lot of internal code was redesigned, leading to a breaking change.
The way how individual sensors get their unique_id and entity_id values was changed.
Now they are properly "multi instance" capable and more stable (the default entity names no longer play a role there and can be easily changed).
However, the unfortunate side-effect means the sensors will now get different uuids, so if you already created custom automations and/or UI tweaks, you have to re-apply them again, now with proper/stable IDs.

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

2 participants