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

Current control oscillation #34

Closed
luckylinux opened this issue Mar 26, 2024 · 10 comments
Closed

Current control oscillation #34

luckylinux opened this issue Mar 26, 2024 · 10 comments
Assignees
Labels
wontfix This will not be worked on

Comments

@luckylinux
Copy link
Contributor

image

image

image

OVPR is set at 3.52 VDC:
image

Detail of max cell voltage
image

@MrPabloUK MrPabloUK self-assigned this Mar 26, 2024
@MrPabloUK MrPabloUK added the bug Something isn't working label Mar 26, 2024
@MrPabloUK
Copy link
Collaborator

I can't quite see what your float voltage is in your screenshot above, can you confirm please?

@luckylinux
Copy link
Contributor Author

Sure, I removed the overvoltage protection from the plots that are for single cells (much lower of course)

image

@MrPabloUK
Copy link
Collaborator

So at the time of the current oscillation, your float voltage was 56.1v, or 3.506v per cell?

@luckylinux
Copy link
Contributor Author

luckylinux commented Mar 26, 2024

So at the time of the current oscillation, your float voltage was 56.1v, or 3.506v per cell?

Yeah, I needed to raise it because otherwise it will never balance out ...

And until now bulk -> float transitioned immediately. I'll try to go to the garage and implement the new changes now ...

@luckylinux
Copy link
Contributor Author

Now it's around 3.55 VDC for the cells that are on the high end. Still 3.39 VDC for the 2 remainining ones ...

@MrPabloUK
Copy link
Collaborator

MrPabloUK commented Mar 26, 2024

With the following settings:

  • Float = 3.506v
  • OVPR = 3.520v
  • End of Curve Factor = 2.0
  • Curve Shape = 0.4
  • Max Charge Current = 180A

You end up with a curve like this:
image

Those settings result in a request to reduce current 180A over 14mV. That's not realistic, even if it were linear then it'd be nearly 13A per mV.

You should set float back to 3.37v or so, expanding the range over which the current will be reduced.
I know you've had issues keeping the charge state in bulk or balancing - my suggestion is to lower the balance trigger voltage to below 3.45v as previously stated. You can also increase OVPR to a higher value if needed.
The changes mentioned in #35 may also help.

@MrPabloUK MrPabloUK added wontfix This will not be worked on and removed bug Something isn't working labels Mar 26, 2024
@luckylinux
Copy link
Contributor Author

But if I recalled correctly, previously (yesterday ?), this was also happening during Bulk, not only Float ...

@MrPabloUK
Copy link
Collaborator

The code for this feature uses the cell float voltage as the start point of current tapering.
This is why I suggest setting your float voltage back to a more reasonable setting. Artificially increasing the float voltage in order to facilitate balancing shouldn't be necessary, assuming the balance trigger voltage is appropriately set.

@luckylinux
Copy link
Contributor Author

I agree. Problem is that even though I properly balanced it 3 weeks ago, it unbalanced quite quickly since then (and I don't think I went above the required knee point of the charge characteristic, when there was no communication between Inverter and BMS).

@MrPabloUK
Copy link
Collaborator

For now, I'll close this issue out as the code itself is not causing the problem.
Thanks for testing out an edge case!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants