-
Notifications
You must be signed in to change notification settings - Fork 167
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
BYD Atto 3: Add way to use SOC% sent by battery #463
Conversation
We have three candidates, someone needs to test which one works best and make the integration accordingly;
|
Hi Dala, I am happy to test this if it is still needed? |
Those who tested this on the Discord ran into the following issues: The problem I now seem to be encountering is that the span of the SOC sent by battery is drifting. It used to be 100% when I had at about 96 scaled but know hitting the same cell voltages at 80% and then the deviation starts growing. The cell information does not work for me in HA so can not trend. I'm going to go back to estimate to try to discharge the battery lower. If I was to try and reconnect the donar car to the battery what is the best way to monitor the canbus. I have a spare lilygo but no special software. Ran the battery down using SOC inverter stopped at 5%. Changed to Estimated and was able to run battery down to 16% estimated approx 3200mv. No warnings unable to go lower. As soon as it started charging the Hi started counting up. It has reset the battery range from here so I was able to get it back up to 100% with the scale in the software set to 1.3 . The hi got back up to 760. I think there is still room to take the battery lower and the Hi count should be higher at the top. Basically it is behaving a bit strange when using the real SOC sent by battery. Not sure if we want to merge this or not. |
I think it would be good to merge version as I would be keen to try it as the current version does not seem to give any real view of the SOC. |
@GraphicObession I suggest you try this PR instead, and report back if the various SOC% moves: #561 |
This PR is IMO ready for merging, if it works well :) |
What
This PR adds a configurable option to use SOC% sent by battery
Why
When Atto 3 packs are crashed hard, they lock themselves and stop updating SOC%. Our previous implementation instead estimates SOC% via battery voltage. This solution is bad, since putting load on the battery will make the reading swing, this making it impossible to use the whole battery.
How
We add a configurable option to the
BYD-ATTO-3-BATTERY.h
file. If you are certain that your battery is not crashed hard, you can remove the lineUSE_ESTIMATED_SOC
, and the code will then instead use the periodically transmitted SOC value. ℹ️ If your battery IS locked and you remove this line, SOC% will stay stuck at the value it had when the battery was crashed and airbags popped.The "More battery info" page also now shows what is used. It will either show Estimated from voltage OR Measured by BMS , based on the configuration