Unifying registry definitions for different inverter models

[canton7]

Hi,

I've been thinking about finishing off the H3 support, and shuddering at the thought of adding and maintaining another set of largely duplicate register definitions.

I think there might be some mileage in tagging the entity descriptions with the inverter models which provide that register. That way we just have a single list to maintain.

I haven't worked out a nice syntax yet... Maybe something like:

ModbusSensorDescription(
    key="pv1_voltage",
    address_specs=[
        AddressSpec(type=AUX, models=[H1, AC1, AIO_H1], addresses=[11000]),
        AddressSpec(type=LAN, models=[H1, AC1, AIO_H1], addresses=[31000]),
    ],
    ...
)

Or perhaps:

ModbusSensorDescription(
    key="pv1_voltage",
    address_specs=[
        AddressSpec(models=[H1, AC1, AIO_H1], lan=[11000], aux=[31000]),
    ],
    ...
)

Or maybe:

ModbusSensorDescription(
    key="pv1_voltage",
    address_specs=[
        AddressSpec(type=LAN, model=H1, variants=[BASE, AC1, AOI], addresses=[11000]),
        AddressSpec(type=AUX, model=H1, variants=[BASE, AC1, AOI], addresses=[31000]),
    ],
    ...
)

@nathanmarlor What do you think?

[canton7]

I think I'm currently favouring number 2...

[canton7]

Implemented in #167