Solax X1-Fit_G4

[anthonyc2020]

Hi I have taken the Plugin_solax.py and added FIT in most places that Hybride is located - Serial is F450xxxxx and got it mostly working - Could I ask that you add FIT to your source for Plugin_solax.py when convenient otherwise it doesn't recognise my inverter.

[cleveHEX]

You can send a pull request if you wish, but you have to guarantee that it works, not just randomly adding stuff without testing.

[wills106]

The FIT range is already supported:

homeassistant-solax-modbus/custom_components/solax_modbus/plugin_solax.py

Lines 5663 to 5668 in 86308cd

	elif seriesnumber.startswith('F43'): invertertype = AC | GEN4 | X1 # RetroFit X1 3kW / 3.7kW?
	elif seriesnumber.startswith('F450'): invertertype = AC | GEN4 | X1 # RetroFit 5kW
	elif seriesnumber.startswith('F460'): invertertype = AC | GEN4 | X1 # RetroFit X1 6kW?
	elif seriesnumber.startswith('F475'): invertertype = AC | GEN4 | X1 # RetroFit X1 7.5kW?
	elif seriesnumber.startswith('PRE'): invertertype = AC | GEN4 | X1 # RetroFit
	elif seriesnumber.startswith('PRI'): invertertype = AC | GEN4 | X1 # RetroFit

If you are missing any entities from the AC / FIT range you need to add in an extra AC and not FIT

For example:

homeassistant-solax-modbus/custom_components/solax_modbus/plugin_solax.py

Lines 3495 to 3506 in 86308cd

	SolaXModbusSensorEntityDescription(
	name = "Today's Yield",
	key = "today_yield",
	native_unit_of_measurement = UnitOfEnergy.KILO_WATT_HOUR,
	device_class = SensorDeviceClass.ENERGY,
	state_class = SensorStateClass.TOTAL_INCREASING,
	register = 0x50,
	register_type = REG_INPUT,
	scale = 0.1,
	rounding = 2, # GEN4 might be 1
	allowedtypes = GEN3 | GEN4 | GEN | HYBRID,
	),

homeassistant-solax-modbus/custom_components/solax_modbus/plugin_solax.py

Line 3505 in 86308cd

allowedtypes = GEN3 | GEN4 | GEN | HYBRID,

Would become:
allowedtypes = GEN3 | GEN4 | GEN | AC | HYBRID,