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

Add ability to do voltage and power control from experiments #174

Open
Jadhav-Sourabh opened this issue Jul 28, 2022 · 8 comments
Open

Add ability to do voltage and power control from experiments #174

Jadhav-Sourabh opened this issue Jul 28, 2022 · 8 comments
Labels
enhancement New feature or request

Comments

@Jadhav-Sourabh
Copy link

liionpack Version

0.3.2

Python Version

3.8

Describe the bug

Why this is not possible?
"Charge at 1C until 4.2V",
"Hold at 4.2V until C/20"

Steps to Reproduce

No response

Expected behaviour

No response

Relevant log output

No response

Additional context

No response

@Jadhav-Sourabh Jadhav-Sourabh added the bug Something isn't working label Jul 28, 2022
@wigging
Copy link
Collaborator

wigging commented Jul 28, 2022

I think the upper and lower cutoff voltage is 4.2 volts. Try your experiment with 4.1 V or lower.

@Jadhav-Sourabh
Copy link
Author

Jadhav-Sourabh commented Jul 28, 2022 via email

@wigging
Copy link
Collaborator

wigging commented Jul 28, 2022

In liionpack voltage limits are checked as shown below. So if your experiment exceeds these voltage limits then it will exit the simulation or at least show a warning.

# 06 Check if voltage limits are reached and terminate
if np.any(temp_v < self.v_cut_lower):
lp.logger.warning("Low voltage limit reached")
vlims_ok = False
if np.any(temp_v > self.v_cut_higher):
lp.logger.warning("High voltage limit reached")
vlims_ok = False
# 07 Step the electrochemical system

@Jadhav-Sourabh
Copy link
Author

In liionpack voltage limits are checked as shown below. So if your experiment exceeds these voltage limits then it will exit the simulation or at least show a warning.

# 06 Check if voltage limits are reached and terminate
if np.any(temp_v < self.v_cut_lower):
lp.logger.warning("Low voltage limit reached")
vlims_ok = False
if np.any(temp_v > self.v_cut_higher):
lp.logger.warning("High voltage limit reached")
vlims_ok = False
# 07 Step the electrochemical system

a1
a2
a3

@Jadhav-Sourabh
Copy link
Author

In liionpack voltage limits are checked as shown below. So if your experiment exceeds these voltage limits then it will exit the simulation or at least show a warning.

# 06 Check if voltage limits are reached and terminate
if np.any(temp_v < self.v_cut_lower):
lp.logger.warning("Low voltage limit reached")
vlims_ok = False
if np.any(temp_v > self.v_cut_higher):
lp.logger.warning("High voltage limit reached")
vlims_ok = False
# 07 Step the electrochemical system

a1 a2 a3

I want to implement CCCV in liionpack. I already implemented successfully CCCV in pybamm

@TomTranter
Copy link
Collaborator

Apologies for not checking these issues regularly. I need to sort out my github notifications. As discussed on the PyBaMM discussion this functionality needs to be added. Liionpack only works for CC and drive cycles not voltage or power driven experiments. Just FYI this will probably be quite a significant development and is not a high priority for me right now

@TomTranter TomTranter added enhancement New feature or request and removed bug Something isn't working labels Jul 29, 2022
@TomTranter TomTranter changed the title Experiment[Bug]: Add ability to do voltage and power control from experiments Jul 29, 2022
@wigging
Copy link
Collaborator

wigging commented Jul 29, 2022

@philgravy
Copy link

Were you ever able to resolve the issue? I'm experiencing the same problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants