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

Separate electrode dimensions #4796

Open
aabills opened this issue Jan 23, 2025 · 3 comments
Open

Separate electrode dimensions #4796

aabills opened this issue Jan 23, 2025 · 3 comments

Comments

@aabills
Copy link
Contributor

aabills commented Jan 23, 2025

It's pretty common in practice for the anode and cathode to have different heights and widths (usually by a small margin, admittedly). However, PyBaMM forces them to be the same by not providing separate dimensions for the negative and positive electrodes. This can be compensated by changing the maximum concentration, but this approach is unphysical. Should we change them to be separate?

@rtimms
Copy link
Contributor

rtimms commented Jan 23, 2025

The interpretation here is that height times width gives the overlapping electrode pair area (i.e., it doesn't include the overhang).

@aabills
Copy link
Contributor Author

aabills commented Jan 23, 2025

Yes. But that means that the assumption is that the overhang doesn't participate at all, which I doubt is true, though surprisingly, I can't find a 3d study revealing the answer

@alighato
Copy link

alighato commented Feb 7, 2025

I agree with Alec. See for example https://www.sciencedirect.com/science/article/pii/S2590116819300116 p.13.

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

No branches or pull requests

3 participants