[Ovmsdev] Improvements to BMS command display

[Michael Geddes]

.. 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/a372e4c1/attachment.htm>