2.0.3

CHANGELOG.md

@@ -8,14 +8,13 @@ Additional data is now being retrieved, including warning, balancing, status ind
 <br>
 This script excludes COMMANDS and should only retrieve data. Nonetheless the author accepts no reponsibility whatsoever of your use, in any way, of this script / addon. Again, USE AT OWN RISK!
 <br>
-<br>
-2.0.0: Major rewrite using the official Pace RS232 Protocol Definition. Breaking changes including prefixing most data with its Pack number (to support multiple batteries in parallel). Temperatures values are now retrieved without names, in my case the first 4 values are cell temperatures, temp 5 the MOSFET, and temp 6 is ambient / environment.
-<br>
-<br>
-2.0.1: Bug fix
-<br>
-<br>
-2.0.2: Naming fixes. Possible fix for native and USB serial devices
-<br>
-<br>
-Known / possible issues: The overall Pack data collected under the root MQTT topic seems to follow the data of the first battery in the Pack.
+<h1>Changelog</h1>
+<h2>v2.0.0</h2> Major rewrite using the official Pace RS232 Protocol Definition. Breaking changes including prefixing most data with its Pack number (to support multiple batteries in parallel). Temperatures values are now retrieved without names, in my case the first 4 values are cell temperatures, temp 5 the MOSFET, and temp 6 is ambient / environment.
+<h2>v2.0.1</h2>
+Bug fixs
+<h2>v2.0.2</h2>
+Naming fixes. Possible fix for native and USB serial devices
+<h2>v2.0.3</h2>
+Possible bugfix for larger banks reading incorrect analog data
+<h1>Known / possible issues</h1>
+The overall Pack data collected under the root MQTT topic seems to follow the data of the first battery in the Pack.

bms.py

@@ -725,7 +725,16 @@ def bms_getAnalogData(bms,batNumber):
 
         for p in range(1,packs+1):
 
+            if p > 1:
+                cells_prev = cells
+
             cells = int(inc_data[byte_index:byte_index+2],16)
+
+            if p > 1:
+                if cells != cells_prev:
+                    byte_index += 2
+                    cells = int(inc_data[byte_index:byte_index+2],16)
+
             if print_initial:
                 print("Pack " + str(p) + ", Total cells: " + str(cells))
             byte_index += 2
@@ -1051,6 +1060,9 @@ def bms_getWarnInfo(bms):
     print("Error retrieving BMS and pack serial numbers. This is required for HA Discovery. Exiting...")
     quit()
 
+
+
+
 # time.sleep(0.1)
 # success, data = bms_getPackNumber(bms)
 # if success == True:

config.yaml

@@ -1,6 +1,6 @@
 name: "BMS Pace"
 description: "Pace BMS Interface"
-version: "2.0.2"
+version: "2.0.3"
 slug: "bms_pace"
 url: "https://github.com/Tertiush/bmspace"
 init: false