Defining new functions in ParameterValues #3103
-
Hello,
in
but I get the following error
I wonder if you would please help to find the reason. Thank you. Regards, |
Beta Was this translation helpful? Give feedback.
Replies: 5 comments 3 replies
-
Dear @brosaplanella |
Beta Was this translation helpful? Give feedback.
-
This is because the current collector conductivity is a This should be a super easy fix if you want to try it |
Beta Was this translation helpful? Give feedback.
-
Hi @tinosulzer, As per your suggestion, I tried to implement temperature dependent EC diffusivity, by replacing
I tried different ways, defining the function outside/inside of the |
Beta Was this translation helpful? Give feedback.
-
Hi @brosaplanella, As suggested, I tried implementing a method for D_ec(T) as below:
Also, I commented the D_ec line at l454 But, when I load the model with the following options, I get this NotImplementedError: Do I have to do anything after defining the method Thank you. |
Beta Was this translation helpful? Give feedback.
-
@brosaplanella Hello, I managed to change D_ec to D_ec(T) and get it to work. However, if I follow a similar approach to define inner_sei_proportion to be SOC dependent, I got errors. In lithium_ion_parameters.py, I defined the following equation inside class ParticleLithiumIonParameters(BaseParameters):
And in the sei_growth.py codes, I added the following codes: I believe the errors are associated with sto, could you please suggest how to modify sto in the above codes? |
Beta Was this translation helpful? Give feedback.
This is because the current collector conductivity is a
Parameter
, not aFunctionParameter
It needs to be change from this to this
This should be a super easy fix if you want to try it