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.

Sign up for GitHub

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

Jump to bottom

Electrical conductivity as function of temperature #1506

Closed
brosaplanella opened this issue Jun 10, 2021 · 0 comments · Fixed by #1570
Closed

Electrical conductivity as function of temperature #1506

brosaplanella opened this issue Jun 10, 2021 · 0 comments · Fixed by #1570
Assignees
@brosaplanella

Comments

@brosaplanella
Copy link
Member

brosaplanella commented Jun 10, 2021 •
edited
Loading

Summary
Add the possibility to define the electrical conductivities of the electrodes as a function of temperature (like other parameters).

Motivation
Some chemistries show a non-negligible dependence on temperature. It will also help to keep consistency across all the parameters.
@brosaplanella brosaplanella changed the title Electrical conductivity as function of the stoichiometry and temperature Electrical conductivity as function of temperature Jun 10, 2021
@brosaplanella brosaplanella self-assigned this Jun 10, 2021
brosaplanella added a commit to brosaplanella/PyBaMM that referenced this issue Jul 27, 2021
@brosaplanella
pybamm-team#1506 added temperature dependence, tests still not working
7ac08d8
brosaplanella added a commit to brosaplanella/PyBaMM that referenced this issue Jul 27, 2021
@brosaplanella
pybamm-team#1506 make electronic conductivity function of temperature
1a54286
brosaplanella added a commit to brosaplanella/PyBaMM that referenced this issue Jul 27, 2021
@brosaplanella
pybamm-team#1506 fix tests
c72ff9d
@brosaplanella brosaplanella mentioned this issue Jul 27, 2021
Merged
8 tasks
brosaplanella added a commit to brosaplanella/PyBaMM that referenced this issue Jul 27, 2021
@brosaplanella
pybamm-team#1506 added tests for coverage
208294b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brosaplanella brosaplanella

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Temperature dependent electrode conductivity brosaplanella/PyBaMM
1 participant
@brosaplanella