Fixes Louisvdw#1094 and smaller fixes

Louisvdw#1094

CHANGELOG.md

@@ -60,6 +60,7 @@
 
 ## v1.4.x
 
+* Changed: JKBMS BLE - Fixes wrong max battery voltage https://github.com/Louisvdw/dbus-serialbattery/issues/1094 by @mr-manuel
 * Changed: JKBMS PB Model fixes by @KoljaWindeler
 
 

etc/dbus-serialbattery/bms/jkbms_ble.py

@@ -96,7 +96,7 @@ def get_settings(self):
         self.cell_count = st["cell_count"]
         self.max_battery_charge_current = st["max_charge_current"]
         self.max_battery_discharge_current = st["max_discharge_current"]
-        self.max_battery_voltage = st["cell_ovp"] * self.cell_count
+        self.max_battery_voltage = utils.MAX_CELL_VOLTAGE * self.cell_count
         self.min_battery_voltage = st["cell_uvp"] * self.cell_count
 
         # Persist initial OVP and OPVR settings of JK BMS BLE

etc/dbus-serialbattery/bms/jkbms_pb.py

@@ -196,16 +196,17 @@ def read_status_data(self):
         temp3 = unpack_from("<h", status_data, 254)[0] / 10
         temp4 = unpack_from("<h", status_data, 256)[0] / 10
         temp5 = unpack_from("<h", status_data, 258)[0] / 10
-        if(unpack_from("<B", status_data, 214)[0] & 0x02):
-                self.to_temp(1, temp1 if temp1 < 99 else (100 - temp1))
-        if(unpack_from("<B", status_data, 214)[0] & 0x04):
-                self.to_temp(2, temp2 if temp2 < 99 else (100 - temp2))
-        if(unpack_from("<B", status_data, 214)[0] & 0x08):
-                self.to_temp(3, temp3 if temp3 < 99 else (100 - temp3))
-        if(unpack_from("<B", status_data, 214)[0] & 0x10):
-                self.to_temp(4, temp4 if temp4 < 99 else (100 - temp4))
-        if(unpack_from("<B", status_data, 214)[0] & 0x20):
-                self.to_temp(5, temp5 if temp5 < 99 else (100 - temp5))
+
+        if unpack_from("<B", status_data, 214)[0] & 0x02:
+            self.to_temp(1, temp1 if temp1 < 99 else (100 - temp1))
+        if unpack_from("<B", status_data, 214)[0] & 0x04:
+            self.to_temp(2, temp2 if temp2 < 99 else (100 - temp2))
+        if unpack_from("<B", status_data, 214)[0] & 0x08:
+            self.to_temp(3, temp3 if temp3 < 99 else (100 - temp3))
+        if unpack_from("<B", status_data, 214)[0] & 0x10:
+            self.to_temp(4, temp4 if temp4 < 99 else (100 - temp4))
+        if unpack_from("<B", status_data, 214)[0] & 0x20:
+            self.to_temp(5, temp5 if temp5 < 99 else (100 - temp5))
 
         # Battery voltage
         self.voltage = unpack_from("<I", status_data, 150)[0] / 1000

etc/dbus-serialbattery/dbushelper.py

@@ -834,10 +834,10 @@ def publish_battery(self, loop):
                 if time_since_first_error >= 60 * 20 and not utils.BLOCK_ON_DISCONNECT:
                     loop.quit()
 
-            # This is to mannage CVCL
+            # This is to manage CVCL
             self.battery.manage_charge_voltage()
 
-            # This is to mannage CCL\DCL
+            # This is to manage CCL\DCL
             self.battery.manage_charge_current()
 
             # Manage battery state, if not set to error (10)

etc/dbus-serialbattery/utils.py

@@ -37,7 +37,7 @@ def _get_list_from_config(
 
 
 # Constants
-DRIVER_VERSION = "1.3.20240710dev"
+DRIVER_VERSION = "1.4.20240712dev"
 zero_char = chr(48)
 degree_sign = "\N{DEGREE SIGN}"