Solax Gen3 RetroFit - half way between an AC and a Hybrid #1350
Replies: 2 comments
-
|
I noticed that there was a previous discussion on this in topic #579 . You recommended to add AC into the allowedtypes to resolve this. What about where the function register mapping for AC is not applicable to FIT but HYBRID is? For example
Will this approach be ok to merge? |
Beta Was this translation helpful? Give feedback.
-
|
@wills106 I have created a fork and made some changes to plugin_solax.py that fixes a handful of issues with controls and sensors not working properly or being unavailable for the X1 G3 RetroFit model. Working well now for me. The fixes are -
Do you want me to open a pull request to merge this into the integration? You can see what changes I made here |
Beta Was this translation helpful? Give feedback.
-
Hey @wills106 and all other contributors, thanks for creating and continuing to enhance an amazing HA integration. It is a key part of my home energy management setup in HA.
I have a Solax X1 G3 RetroFit (serial PRI502H...) and I have noticed that it is recognised by the integration as an AC inverter. However it seems to share most but not all features and registers with the Hybrid inverter according to the Solax docs, such as the grid-tied minimum SOC which is not accessible at present with the integration but can be with changes. The main difference seems to be that the FIT doesn't have the PV related registers, most else is the same.
I looked at modding the plugin_solax.py file by changing what the AC inverter supports to line up with my inverter, but those changes would then alter how it works for the AC if you were to merge changes. I also looked at making it be recognised as a HYBRID but that makes all the irrelevant PV related sensor appear, and a few others that are not supported by the FIT. Maybe another inverter type like 'FIT' would be better? What do you think? I'd be happy to build on that to create the assignments for the Gen3 FITs once there's a way forward.
Beta Was this translation helpful? Give feedback.
All reactions