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]: ElectrodeSOH implementation doesn't update Q_Li #339

Closed
BradyPlanden opened this issue Jun 3, 2024 · 1 comment · Fixed by #349
Closed

[Bug]: ElectrodeSOH implementation doesn't update Q_Li #339

BradyPlanden opened this issue Jun 3, 2024 · 1 comment · Fixed by #349
Assignees
Labels
bug Something isn't working

Comments

@BradyPlanden
Copy link
Member

BradyPlanden commented Jun 3, 2024

Python Version

N/A

Describe the bug

  • electodeSOH in echem models doesn't update Q_Li in the pybamm_model.
  • This causes issues in the design cost functions as they commonly modify electrode capacity parameters
  • pybamm code ref

Steps to reproduce the behaviour

run:

python examples/scripts/spme_max_energy.py

Grey areas are cost values of np.inf, which can be a result of Q_p > Q_Li.

Relevant log output

No response

@BradyPlanden BradyPlanden added the bug Something isn't working label Jun 3, 2024
@NicolaCourtier NicolaCourtier moved this to Todo in v24.6 Jun 6, 2024
@NicolaCourtier NicolaCourtier self-assigned this Jun 6, 2024
@NicolaCourtier
Copy link
Member

To see a grey area, change the design parameters to e.g.

parameters = pybop.Parameters(
    pybop.Parameter(
        "Positive electrode thickness [m]",
        prior=pybop.Gaussian(7.56e-05, 0.1e-05),
        bounds=[65e-06, 10e-05],
    ),
    pybop.Parameter(
        "Negative electrode thickness [m]",
        prior=pybop.Gaussian(7.56e-05, 0.1e-05),
        bounds=[65e-06, 10e-05],
    ),
)

@NicolaCourtier NicolaCourtier linked a pull request Jun 6, 2024 that will close this issue
15 tasks
@NicolaCourtier NicolaCourtier moved this from Todo to In Progress in v24.6 Jun 18, 2024
@BradyPlanden BradyPlanden removed this from v24.6 Jun 30, 2024
@NicolaCourtier NicolaCourtier moved this to Done in v24.6 Jul 4, 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
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants