search

DieBieEngineering / DieBieMS-Firmware

Firmware for the DieBieMS
49 stars 69 forks source link

Cell voltage reporting error in terminal #1

Closed DieBieEngineering closed 5 years ago

DieBieEngineering commented 6 years ago

Sometimes when using the terminal to print all cell voltages ( "cells" command ) the BieBieMS lists the voltages sorted by voltage instead of sorted by cell number.

Example: `----- Cell voltages ----- Cell voltage 0 : 3.843V Cell voltage 1 : 3.953V Cell voltage 2 : 3.809V Cell voltage 3 : 3.840V Cell voltage high : 3.953V Cell voltage low : 3.809V Cell voltage average : 3.861V Cell voltage mismatch : 0.144V ----- End Cell voltages -----

----- Cell voltages ----- Cell voltage 0 : 3.842V Cell voltage 1 : 3.953V Cell voltage 2 : 3.809V Cell voltage 3 : 3.842V Cell voltage high : 3.953V Cell voltage low : 3.809V Cell voltage average : 3.861V Cell voltage mismatch : 0.144V ----- End Cell voltages -----

----- Cell voltages ----- Cell voltage 0 : 3.954V Cell voltage 1 : 3.843V Cell voltage 2 : 3.843V Cell voltage 3 : 3.807V Cell voltage high : 3.954V Cell voltage low : 3.807V Cell voltage average : 3.862V Cell voltage mismatch : 0.147V ----- End Cell voltages -----`

The first two rapports are OK, the last is sorted by highest to lowest cell voltage. This happens in the "modPowerElectronicsSubTaskBalaning()" function.

DieBieEngineering commented 5 years ago

Solved long time ago, but forgot to update here.