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

[Bug]: Only Current Operations are Supported #317

Open
philgravy opened this issue Jul 29, 2024 · 0 comments
Open

[Bug]: Only Current Operations are Supported #317

philgravy opened this issue Jul 29, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@philgravy
Copy link

liionpack Version

Latest

Python Version

3.12.4

Describe the bug

Experiments ran with Liionpack seem to not have the same capabilities as PyBaMM. Running experiments such as "Charge at 1 C until 4.1 V" are met with the error "Only Current Operations are Supported", meaning that experiments can only be ran with current and nothing more. If I'm simply trying to charge a battery from 0% to 100%, how can I do so with Liionpack if lithium ion batteries charge using CC/CV charging? I'm open to modifying source code, but I'd appreciate some direction with this endeavor. Thanks

Steps to Reproduce

Attempt running an experiment that involved voltage or C-rate, such as
"Charge at 1C until 4.1 V"

Expected behaviour

Fully charging a battery

Relevant log output

31 termination = step.termination
     32 if typ not in ["current", "power"]:
---> 33     raise ValueError("Only current operations are supported")
     34 else:
     35     if not step.is_drive_cycle:

ValueError: Only current operations are supported

Additional context

No response

@philgravy philgravy added the bug Something isn't working label Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant