Skip to content

Update LFP properties#18

Merged
c-randall merged 2 commits intomainfrom
feature/lfp-params
May 30, 2025
Merged

Update LFP properties#18
c-randall merged 2 commits intomainfrom
feature/lfp-params

Conversation

@c-randall
Copy link
Collaborator

@c-randall c-randall commented May 30, 2025

Description

Update the LFP material properties for both Ds and Eeq to match new values from ICI measurements. Note that new diffusivity values were 3-4 orders of magnitude lower than previous values, which caused the model only to achieve a small fraction (~1/10) of the capacity, even when simulated at low rates (C/20).

The new diffusivities were scaled until simulations at C/10 could run for approximately 10 hours, and radial solid-phase lithium concentrations were roughly uniform. This required a scaling of 1,000-10,000. The diffusivity value included in the new LFP material file is also a constant because the functional form from ICI varied at both ends of lithiation (near xs=0 and xs=1) by 3-4 orders of magnitude and created some unphysical results.

Type of change

  • New feature (non-breaking change which adds functionality)

Key checklist:

  • No style issues: $ nox -s linter [-- format]
  • Code is free of misspellings: $ nox -s codespell [-- write]
  • All tests pass: $ nox -s tests
  • Badges are updated: $ nox -s badges

Further checks:

  • The documentation builds: $ nox -s docs.
  • Code is commented, particularly in hard-to-understand areas.
  • Tests are added that prove fix is effective or that feature works.

@c-randall c-randall marked this pull request as ready for review May 30, 2025 19:33
@c-randall c-randall merged commit df71593 into main May 30, 2025
18 checks passed
@c-randall c-randall deleted the feature/lfp-params branch May 30, 2025 19:33
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.

1 participant

Comments