How to use global variables (Version index 0x54 )

[Barpo]

I made this:

battery.py:
.
.
class Battery(object):

def __init__(self, port, baud):
    self.port = port

.
.
self.version = None
self.vers_index = None
self.hardware_version = None
self.soc = None
.
.
daly.py:
.
.
command_alarm = b"\x98"
command_vers_index = b"\x54"
command_version = b"\x62"
.
.
def refresh_data(self):
result = self.read_soc_data()
result = result and self.read_vers_index_data()
result = result and self.read_cell_voltage_range_data()
result = result and self.read_temperature_range_data()
result = result and self.read_fed_data()

    return result

.
.
def read_vers_index_data(self):
vers_index_data = self.read_serial_data_daly(self.command_vers_index)
# check if connection success
if vers_index_data is False:
return False

    self.vers_index = unpack_from('>h', vers_index_data)
    return True

.
.

dbushelper.py:
.
.
self._dbusservice.add_path('/ProductName', 'DalyBMS (' + self.battery.type + ') v' +
str(DRIVER_VERSION) + DRIVER_SUBVERSION)
self._dbusservice.add_path('/FirmwareVersion', self.battery.vers_index)
self._dbusservice.add_path('/HardwareVersion', self.battery.hardware_version)
self._dbusservice.add_path('/Connected', 1)
.
.

My question is, why is "self.battery.vers_index" not seen as global variable? It is shown as '...'(None) at the userinterface. dbushelper does not get the value from refresh_data

[Louisvdw]

Are you sure that read_vers_index_data() does get data from the BMS and does not just return False and never reaching your unpack?
You can put some temporary logging in to see if it is reached

[Barpo]

def read_vers_index_data(self):
    vers_index_data = self.read_serial_data_daly(self.command_vers_index)
    # check if connection success
    if vers_index_data is False:
        return False

    global self.vers_index 
    self.vers_index = vers_index_data
    logger.info(self.vers_index)
    return True

    self._dbusservice.add_path('/FirmwareVersion', self.battery.vers_index)

@40000000615b443516ce767c INFO:utils:20200306
@40000000615b443718cde82c INFO:utils:20200306
@40000000615b4439175ef2c4 INFO:utils:20200306

[Louisvdw]

It seems to get an value and I also cannot see why it should not populate. I suspect it might populate, but just not display corrently in the interface.

[Lumo1976]

Hi Louis, I have downloaded the driver to USB and for the life of me can’t get it to load to the Cerbo GX.. Any pointers you can give?? Thanks in advance.. Shaun From: Louis Van Der Walt ***@***.***> Sent: Tuesday, 5 October 2021 3:44 PM To: Louisvdw/dbus-serialbattery ***@***.***> Cc: Subscribed ***@***.***> Subject: Re: [Louisvdw/dbus-serialbattery] How to use global variables (Version index 0x54 ) (Discussion #57) It seems to get an value and I also cannot see why it should not populate. I suspect it might populate, but just not display corrently in the interface. — You are receiving this because you are subscribed to this thread. Reply to this email directly, <#57 (reply in thread)> view it on GitHub, or <https://github.com/notifications/unsubscribe-auth/AVWF7D2FPNFAGO5PKYLAVN3UFKGALANCNFSM5FHN2X5A> unsubscribe. Triage notifications on the go with GitHub Mobile for <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> iOS or <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub> Android. <https://github.com/notifications/beacon/AVWF7D7B3SNU2XXNRRCBQZ3UFKGALA5CNFSM5FHN2X5KYY3PNVWWK3TUL52HS4DFWFCGS43DOVZXG2LPNZBW63LNMVXHJKTDN5WW2ZLOORPWSZGOAAK4VWQ.gif>

[Louisvdw]

The Cerbo GX does not work with the auto installer. It's a known issue on that device.
https://github.com/Louisvdw/dbus-serialbattery/wiki/How-to-install