Hi all,
I'll create a PR soon, but just wanted to ask something first.
I've made the 'bms' command so that it will work with temperature OR
voltages and only display that one column. For example:
OVMS# bms
Temperature:
Average: 3.6C [ 3.6C - 3.6C]
Deviation: SD 0.00C [max 0.48C], 0 warnings, 0 alerts
Cells:
+--------+
1 | 14.0C |
+--------+
2 | 14.0C |
+--------+
3 | 14.0C |
+--------+
4 | 15.0C |
+--------+
5 | 14.0C |
+--------+
6 | 3.6C |
+--------+
7 | 3.6C |
+--------+
8 | 3.6C |
+--------+
9 | 3.6C |
+--------+
10 | 3.6C |
+--------+
11 | 3.6C |
+--------+
12 | 3.6C |
+--------+
13 | 3.6C |
+--------+
14 | 3.6C |
+--------+
15 | 3.6C |
+--------+
16 | 3.6C |
+--------+
The other improvement I'm trying, and want some feedback on is when you
have readings-per-module set > 4. Here I don't have it set quite right..
but you get the idea. The setting is at 12, which ends up being quite wide
(yes, there's one temperature sensor off on its own, so my setting is not
quite right yet).
At the moment the column count is capped at 4 (and will cope with it not
dividing properly), but the idea would be something along the lines of
trying n % 3 and n % 5 first and then 4. So cap the column count at 3, 5 or
4 depending. The idea is that this becomes readable in the APP!
Thoughts?
OVMS# bms
Voltage:
Average: 3.632V [3.620V - 3.640V]
Deviation: SD 9.75mV [max 0.00mV], 0 warnings, 0 alerts
Temperature:
Average: 14.0C [ 13.0C - 15.0C]
Deviation: SD 0.35C [max 0.48C], 0 warnings, 0 alerts
Cells:
+-----------------------------+--------+
1 | 3.640V 3.640V 3.640V 3.640V | 14.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
2 | 3.620V 3.620V 3.640V 3.640V | 14.0C |
| 3.620V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
3 | 3.620V 3.640V 3.640V 3.640V | 14.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
4 | 3.620V 3.640V 3.640V 3.640V | 14.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.620V 3.620V | |
+-----------------------------+--------+
5 | 3.620V 3.620V 3.640V 3.640V | 14.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.620V 3.620V | |
+-----------------------------+--------+
6 | 3.640V 3.640V 3.640V 3.640V | 15.0C |
| 3.640V 3.620V 3.620V 3.640V | |
| 3.640V 3.620V 3.620V 3.620V | |
+-----------------------------+--------+
7 | 3.640V 3.640V 3.640V 3.640V | 13.0C |
| 3.620V 3.620V 3.620V 3.620V | |
| 3.640V 3.640V 3.620V 3.620V | |
+-----------------------------+--------+
8 | 3.620V 3.640V 3.620V 3.640V | 14.0C |
| 3.640V 3.620V 3.620V 3.620V | |
| 3.640V 3.640V 3.620V 3.620V | |
+-----------------------------+--------+
9 | 3.640V 3.640V 3.640V 3.640V | 14.0C |
| 3.620V 3.620V 3.620V 3.620V | |
| 3.640V 3.620V 3.640V 3.640V | |
+-----------------------------+--------+
10 | 3.620V 3.640V 3.640V 3.620V | 14.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
11 | 3.620V 3.640V 3.640V 3.640V | 14.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.620V 3.620V 3.640V 3.620V | |
+-----------------------------+--------+
12 | 3.620V 3.620V 3.620V 3.620V | 14.0C |
| 3.620V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
13 | 3.620V 3.620V 3.640V 3.640V | 14.0C |
| 3.640V 3.620V 3.620V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
14 | 3.620V 3.620V 3.640V 3.640V | 14.0C |
| 3.620V 3.620V 3.620V 3.620V | |
| 3.620V 3.620V 3.640V 3.620V | |
+-----------------------------+--------+
15 | 3.620V 3.640V 3.640V 3.640V | 14.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
16 | | 14.0C |
| | |
| | |
+-----------------------------+--------+
OVMS#
