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

BYD Atto 3: Add way to use SOC% sent by battery #463

Merged
merged 4 commits into from
Nov 18, 2024

Conversation

dalathegreat
Copy link
Owner

@dalathegreat dalathegreat commented Sep 17, 2024

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 line USE_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.

image

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

image

@dalathegreat
Copy link
Owner Author

We have three candidates, someone needs to test which one works best and make the integration accordingly;

  • Polled SOC
  • Highprecision SOC
  • Lowprecision SOC

@GraphicObession
Copy link

We have three candidates, someone needs to test which one works best and make the integration accordingly;

  • Polled SOC
  • Highprecision SOC
  • Lowprecision SOC

Hi Dala, I am happy to test this if it is still needed?

@dalathegreat
Copy link
Owner Author

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.
I still had a little left at the top did not go all the way up to 3400mV

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.

@GraphicObession
Copy link

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.

@dalathegreat
Copy link
Owner Author

@GraphicObession I suggest you try this PR instead, and report back if the various SOC% moves: #561

@dalathegreat
Copy link
Owner Author

This PR is IMO ready for merging, if it works well :)

@dalathegreat dalathegreat merged commit c5d0078 into main Nov 18, 2024
70 checks passed
@dalathegreat dalathegreat deleted the improvement/atto3-soc branch November 18, 2024 08:14
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

Successfully merging this pull request may close these issues.

2 participants