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

balancing? #8

Open
Energyfuzzi opened this issue Aug 19, 2019 · 2 comments
Open

balancing? #8

Energyfuzzi opened this issue Aug 19, 2019 · 2 comments

Comments

@Energyfuzzi
Copy link

Hi, great job!
I try to implement it on a Volkswagen Beetle with 5 Tesla Bricks, looks good!
While checking the balancing, I found that there is something strange inside your code:
You start balancing at 3.9V, and stop when the cellVoltage is below this Voltage minus a balancing hysteresis.
There are 2 problems I see, because "balancing(i)" seems to mean: burn the energy at the resistors at cell(i)!

  1. So the Resistors from each cell burn energy every time the Voltage is higher than 3.9 V unless it is below 3.86V, even if the voltage should be at 4.1V.
  2. During balancing the Voltagemeasurement does not work properly, because of the structure of the electronic.

I would suggest starting balancing, if the difference to the neighbour cell is to high, maybe via the difference between avgCellVoltage and cellVoltage(i), or just by comparing, and a minimum voltage is guaranteed (as already done).
Then the balancing should last 2-5 min, should stop, then after a short time start the measurement and the balancing again.
What do you think?

@collin80
Copy link
Owner

You're pretty much right all the way around. The balancing stuff found in the public code is only to be used as an example of how one might be able to turn on balancing. It certainly isn't a very good algorithm. So, yes, I would recommend you implement something better.

@Energyfuzzi
Copy link
Author

thanks, I´ll change it for my needs on the Volkswagen beetle.
And by the way: thanks once more for finding out, how to communicate with the original TeslaBrickElectronic. It really helps to use the original Brick instead of using a custom BMS!
Has there been something new about the communication in the last 1-2 years (since you posted the code?) All the best from Austria!

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

2 participants