Replies: 3 comments
-
Yes, that's a bug, can you open a PR to fix? |
Beta Was this translation helpful? Give feedback.
0 replies
-
No problem. Will do later. (After I figure that out with Simon) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Dear PyBaMM,
I just found that in Line 154 in /pybamm/parameters/lithium_ion_parameters.py, "D_e__c_e" is used to set tolerances for kappa_e_dimensional. Would it be possible that it is "kappa_e__c_e"? Though they are both 10.
Thanks!
Beta Was this translation helpful? Give feedback.
All reactions