New feature request: Ability to setup by config/flow modbus device unit

[marpi82]

Hello,

Most of inverters are running as first (unit 0x01) on modbus (this is Yours default also)
My SE inverter are running as second device (unit 0x02) and I've must change two valuse in init.py from 1 to 2 to get it work in my case.
It will be nice if we can setup unit number from config/flow

Marek

[binsentsu]

To be sure, do you mean this one: self.read_holding_registers(unit=1, address=40071, count=38)

[marpi82]

To be sure, do you mean this one: self.read_holding_registers(unit=1, address=40071, count=38)

Yes, this one on master repository:

home-assistant-solaredge-modbus/custom_components/solaredge_modbus/__init__.py

Line 182 in 351c4d3

inverter_data = self.read_holding_registers(unit=1, address=40071, count=38)

And this two lines on feature/meter1 (I have used this repository):

home-assistant-solaredge-modbus/custom_components/solaredge_modbus/__init__.py

Line 338 in 76a83ff

unit=1, address=start_address, count=103

home-assistant-solaredge-modbus/custom_components/solaredge_modbus/__init__.py

Line 632 in 76a83ff

inverter_data = self.read_holding_registers(unit=1, address=40071, count=38)

Marek

[johnpwhite]

Hi,

I'd love this feature too please! I to have to change these two values to equal 2 to match my meter device ID of 2.

THANKS IN ADVANCE!!

[marpi82]

Is there a chance that this parameter will appear in the configuration? :)

Now, these two lines contains unit/slave number

home-assistant-solaredge-modbus/custom_components/solaredge_modbus/__init__.py

Lines 337 to 339 in b395440

	meter_data = self.read_holding_registers(
	unit=1, address=start_address, count=103
	)

home-assistant-solaredge-modbus/custom_components/solaredge_modbus/__init__.py

Line 632 in b395440

inverter_data = self.read_holding_registers(unit=1, address=40071, count=38)

[binsentsu]

implemented