From b9e93babc7e4bd79d0884a9fdd2f9a49880ed92d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20=C3=96ster?= Date: Sun, 12 Jan 2025 23:20:40 +0200 Subject: [PATCH] Make all 96 cells work --- Software/src/battery/RENAULT-ZOE-GEN1-BATTERY.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Software/src/battery/RENAULT-ZOE-GEN1-BATTERY.cpp b/Software/src/battery/RENAULT-ZOE-GEN1-BATTERY.cpp index 28a69df6..f3f10bdd 100644 --- a/Software/src/battery/RENAULT-ZOE-GEN1-BATTERY.cpp +++ b/Software/src/battery/RENAULT-ZOE-GEN1-BATTERY.cpp @@ -8,7 +8,6 @@ https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/blob/master/vehicle/OVMS.V3/components/vehicle_renaultzoe/src/vehicle_renaultzoe.cpp The Zoe BMS apparently does not send total pack voltage, so we use the polled 96x cellvoltages summed up as total voltage Still TODO: -- Fix the missing cell96 issue (Only cells 1-95 is shown on cellmonitor page) - Automatically detect if we are on 22 or 41kWh battery (Nice to have, requires log file from 22kWh battery) /* @@ -397,7 +396,7 @@ void handle_incoming_can_frame_battery(CAN_frame rx_frame) { case 0x29: if (requested_poll == GROUP1_CELLVOLTAGES_1_POLL) { cellvoltages[30] = (rx_frame.data.u8[1] << 8) | rx_frame.data.u8[2]; - cellvoltages[21] = (rx_frame.data.u8[3] << 8) | rx_frame.data.u8[4]; + cellvoltages[31] = (rx_frame.data.u8[3] << 8) | rx_frame.data.u8[4]; cellvoltages[32] = (rx_frame.data.u8[5] << 8) | rx_frame.data.u8[6]; highbyte_cell_next_frame = rx_frame.data.u8[7]; }