[Ovmsdev] Improvements to BMS command display
Michael Geddes
frog at bunyip.wheelycreek.net
Tue Nov 1 16:19:27 HKT 2022
Is there any reason I shouldn't create a pull request for this?
At the moment a multiple of 5 (but not a multiple of 4) will give you 5
columns ..
though I suspect this won't output nicely on the mobile app 'Messages'
which really will probably only work nicely with 4+1=5 total columns.
This is my latest output
with
Voltage
Cells = 180 (of 192 max)
Readings Per Module = 12 (so 15 modules of a possible 16)
Temperature
Readings = 16
Readings Per Module = 1
OVMS# bms
Voltage:
Average: 3.636V [3.620V - 3.640V]
Deviation: SD 7.98mV [max 0.00mV], 0 warnings, 0 alerts
Temperature:
Average: 17.2C [ 17.0C - 18.0C]
Deviation: SD 0.39C [max 0.39C], 0 warnings, 0 alerts
Cells:
+-----------------------------+--------+
1 | 3.640V 3.640V 3.640V 3.640V | 18.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
2 | 3.620V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
3 | 3.640V 3.640V 3.640V 3.640V | 18.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
4 | 3.620V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.620V 3.640V | |
+-----------------------------+--------+
5 | 3.620V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
6 | 3.640V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
7 | 3.640V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.620V 3.620V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
8 | 3.620V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.620V 3.620V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
9 | 3.640V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.640V 3.620V 3.620V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
10 | 3.640V 3.640V 3.640V 3.640V | 17.0C |
| 3.620V 3.620V 3.620V 3.640V | |
| 3.640V 3.640V 3.640V 3.640V | |
+-----------------------------+--------+
11 | 3.620V 3.640V 3.640V 3.640V | 17.0C |
| 3.620V 3.640V 3.640V 3.640V | |
| 3.620V 3.640V 3.620V 3.620V | |
+-----------------------------+--------+
12 | 3.620V 3.640V 3.620V 3.620V | 17.0C |
| 3.640V 3.620V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
13 | 3.620V 3.620V 3.640V 3.640V | 17.0C |
| 3.620V 3.640V 3.640V 3.620V | |
| 3.640V 3.620V 3.640V 3.620V | |
+-----------------------------+--------+
14 | 3.620V 3.640V 3.640V 3.640V | 17.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.620V 3.640V 3.620V | |
+-----------------------------+--------+
15 | 3.620V 3.640V 3.640V 3.640V | 18.0C |
| 3.640V 3.640V 3.640V 3.640V | |
| 3.640V 3.640V 3.640V 3.620V | |
+-----------------------------+--------+
16 | | 17.0C |
+-----------------------------+--------+
On Tue, 1 Nov 2022 at 07:29, Michael Geddes <frog at bunyip.wheelycreek.net>
wrote:
> .. As to whether this is correct for my i5: it might actually be correct.
> My i5 seems to have 180 of a possible 192 cells. It seems that it's likely
> there are 12 cell packs in a module (
> https://insideevs.com/news/539940/hyundai-ioniq5-battery-pack-opened/)
> Which possibly means they still have a temperature sensor for the unfilled
> module!? Which would make this correct!
> I wonder if I should cap the temperature sensors based on the number of
> battery packs?
> Anyway.
> Should I try and reduce the height of that last module entry?
>
> //.ichael
>
> On Tue, 1 Nov 2022 at 07:19, Michael Geddes <frog at bunyip.wheelycreek.net>
> wrote:
>
>> 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#
>>
>>
>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openvehicles.com/pipermail/ovmsdev/attachments/20221101/b92ed8d7/attachment-0001.htm>
More information about the OvmsDev
mailing list