[Ovmsdev] Improvements to BMS command display
Michael Balzer
dexter at expeedo.de
Tue Nov 1 16:49:57 HKT 2022
Michael,
regarding separate temperature / voltage display, I suggest adding sub
commands, i.e. "bms temp" / "bms volt".
Regarding the layout: it's difficult to tell what will fit, especially
in vertical orientation, as this depends on the display and font size
chosen. With a font size of 12.0 on my Huawei, max 4 columns would fit:
https://drive.google.com/file/d/1ra6QWcPh03rd5GO9xyykY3ZI_ofRrzj1/view?usp=share_link
https://drive.google.com/file/d/1rasYOBL8lY1NwM9oq_gL6pvcX-uinJjj/view?usp=share_link
Regarding your temperature sensors: it's possible you've actually got
<modules>+1 sensors if they are placed between the modules. You should
see that when heating up the battery by driving, an empty/reserved
module space wouldn't heat up as much as filled modules. For a layout of
temperature sensors between the cell modules, you could decide to derive
the module temperature by building the average of it's adjacent sensors.
Regards,
Michael
Am 01.11.22 um 09:19 schrieb Michael Geddes:
> 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#
>
>
>
>
>
> _______________________________________________
> OvmsDev mailing list
> OvmsDev at lists.openvehicles.com
> http://lists.openvehicles.com/mailman/listinfo/ovmsdev
--
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openvehicles.com/pipermail/ovmsdev/attachments/20221101/fea8b60b/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 203 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openvehicles.com/pipermail/ovmsdev/attachments/20221101/fea8b60b/attachment-0001.sig>
More information about the OvmsDev
mailing list