Fix for #450

Louisvdw/dbus-serialbattery#450

etc/dbus-serialbattery/bms/ecs.py

@@ -46,6 +46,9 @@ def test_connection(self):
 
                 self.find_LiPro_cells()
 
+                # get first data to show in startup log
+                self.refresh_data()
+
                 return self.get_settings()
         except IOError:
             return False

etc/dbus-serialbattery/bms/jkbms_ble.py

@@ -91,6 +91,11 @@ def test_connection(self):
             return False
 
         logger.info("JK BMS found!")
+
+        # get first data to show in startup log
+        self.get_settings()
+        self.refresh_data()
+
         return True
 
     def get_settings(self):
@@ -150,8 +155,8 @@ def refresh_data(self):
         self.to_temp(0, st["cell_info"]["temperature_mos"])
         self.to_temp(1, st["cell_info"]["temperature_sensor_1"])
         self.to_temp(2, st["cell_info"]["temperature_sensor_2"])
-        self.current = st["cell_info"]["current"]
-        self.voltage = st["cell_info"]["total_voltage"]
+        self.current = round(st["cell_info"]["current"], 1)
+        self.voltage = round(st["cell_info"]["total_voltage"], 2)
 
         self.soc = st["cell_info"]["battery_soc"]
         self.cycles = st["cell_info"]["cycle_count"]

etc/dbus-serialbattery/bms/lltjbd.py

@@ -67,6 +67,9 @@ def test_connection(self):
         result = False
         try:
             result = self.read_hardware_data()
+            # get first data to show in startup log
+            if result:
+                self.refresh_data()
         except Exception as err:
             logger.error(f"Unexpected {err=}, {type(err)=}")
             result = False

etc/dbus-serialbattery/bms/renogy.py

@@ -47,6 +47,9 @@ def test_connection(self):
         result = False
         try:
             result = self.read_gen_data()
+            # get first data to show in startup log
+            if result:
+                self.refresh_data()
         except Exception as err:
             logger.error(f"Unexpected {err=}, {type(err)=}")
             result = False

etc/dbus-serialbattery/bms/revov.py

@@ -54,8 +54,12 @@ def test_connection(self):
         result = False
         try:
             result = self.read_gen_data()
-        except:
-            pass
+            # get first data to show in startup log
+            if result:
+                self.refresh_data()
+        except Exception as err:
+            logger.error(f"Unexpected {err=}, {type(err)=}")
+            result = False
 
         return result
 

etc/dbus-serialbattery/utils.py

@@ -36,7 +36,7 @@ def _get_list_from_config(
 
 # Constants - Need to dynamically get them in future
 DRIVER_VERSION = "1.0"
-DRIVER_SUBVERSION = ".0-jkbms_ble (20230501)"
+DRIVER_SUBVERSION = ".0-jkbms_ble (20230502)"
 zero_char = chr(48)
 degree_sign = "\N{DEGREE SIGN}"