Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Impossible cell voltage #2

Open
najaB76 opened this issue Feb 22, 2023 · 2 comments
Open

Impossible cell voltage #2

najaB76 opened this issue Feb 22, 2023 · 2 comments

Comments

@najaB76
Copy link

najaB76 commented Feb 22, 2023

Hello @slim-bean ,

I've been using dbus-serialbattery which has forked from your code. First off, great job decoding the protocol, it's been really useful to get accurate battery state readings, better than a smart shunt!

One thing that I have noticed is that when battery gets close to full the individual cell voltages will return values that don't make sense. I've seen it reading 36V for the individual cells. My guess is that this is when balancing is active. In dbus-spy the values look like this:

Voltages/Cell1 3.512
Voltages/Cell2 36.276
Voltages/Cell3 36.281
Voltages/Cell4 36.277
Voltages/Cell5 3.493
Voltages/Cell6 3.509
Voltages/Cell7 3.513
Voltages/Cell8 36.278

I switched the RS485 across to my PC and read the values, so this was maybe a minute or so after the dbus-spy readings above:

image

I was thinking a first that it might simply be a case of subtracting 32 from the value but that still gives a cell voltage of 4.2 volts which is still high for a cell voltage. Any ideas what's going on?

Thanks

@pchiquit
Copy link

Hi @najaB76
I fixed that bug on the latest version of the EG4 Lifepower driver, but I don't think it was released yet. Louisvdw/dbus-serialbattery#390
I would suggest we move the discussion to Louisvdw/dbus-serialbattery as @slim-bean is not involved on the dbus-serialbattery development.

Basically I'm masking that bit that throws the voltage off, but your debugging gave me the idea of also reporting the cells that are balancing to the serial driver. I'll see if that is possible.

@slim-bean
Copy link
Owner

I have seen this spike in voltage myself a few times, good to know it's a masking issue! I'll see about making that change here as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants