Why do you append strings to self?

[Giermann]

Is there any useful reason to append strings to a class (self)?

And if so, what is the meaning of the values?
If it is the register number, it needs another change to complete commit #9f47563:

class SungrowInverter(SungrowProduct):
    def __init__(self, client, servicename, deviceinstance):
        super().__init__(client, 'Sungrow Inverter', servicename, deviceinstance)
    
        # Fixed values
        maxpower = 10 # self.read(XXX)
        self._dbusservice.add_path('/Ac/MaxPower', maxpower)
        position = 0 # where it's connected to the inverter. 0=AC input 1; 1=AC output; 2=AC input 2
        self._dbusservice.add_path('/Position', position)

        self += '/Ac/Energy/Forward', 'kWh', 5004 #Total produced energy over all phases = Total power yields
        self += '/Ac/Power', 'W', 5009 # Total apparent power

to

        self += '/Ac/Power', 'W', 5031 # Total active power

[strocode]

HI Sven, Lazy design on my part I'm sorry. It just calls the __iadd__ function. The right hand side is a tuple containing (path, units, register number). I think in the end I don't actually use the values, as I parse it all again in the update function.

…

On Wed, 29 Jan 2025 at 23:00, Sven Giermann ***@***.***> wrote: Is there any useful reason to append strings to a class (self)? And if so, what is the meaning of the values? If it is the register number, it needs another change to complete commit #9f47563 <9f47563> : class SungrowInverter(SungrowProduct): def __init__(self, client, servicename, deviceinstance): super().__init__(client, 'Sungrow Inverter', servicename, deviceinstance) # Fixed values maxpower = 10 # self.read(XXX) self._dbusservice.add_path('/Ac/MaxPower', maxpower) position = 0 # where it's connected to the inverter. 0=AC input 1; 1=AC output; 2=AC input 2 self._dbusservice.add_path('/Position', position) self += '/Ac/Energy/Forward', 'kWh', 5004 #Total produced energy over all phases = Total power yields self += '/Ac/Power', 'W', 5009 # Total apparent power to self += '/Ac/Power', 'W', 5031 # Total apparent power — Reply to this email directly, view it on GitHub <#1>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAOS22ONPDENYDYJNNBNT4T2NC7D7AVCNFSM6AAAAABWCVQOVGVHI2DSMVQWIX3LMV43ASLTON2WKOZSHAYTQMBRGEZDSMY> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

-- Keith Bannister