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
Adding customized function for tortuosity #2111
Comments
You could create a new submodel in this folder. The simplest thing would be to copy the Bruggeman submodel and edit it to whatever function you want. You could either hard-code some functional form with some new scalar parameters, or allow a general function of porosity/active material by adding a new |
If I copy the Bruggeman submodel and add my function, how should I have the model use this custom submodel and not the Bruggeman? Also, can you please elaborate on how this I'm also wondering how you defined your effective properties e.g. electrolyte diffusivity/conductivity? I get tortuosity smaller than 1 which is confusing. Where should I define my effective properties the way I want? |
I'm currently working on adding extra documentation around how to use submodels, but there is an example of swapping out submodels here. If you wanted to integrate your new submodel into the code you could make it so that you pass an option to the model to set the tortuosity. Take a look at the base battery model to see how options are handled. For See this discussion around how the properties are defined (we actually define a "transport efficiency", not a tortuosity). |
Hi @amirsh213 , how's it going? We are running a hackathon on 29th of March in Oxford and this issue is potentially one we could work on together. You'd be welcome to join us if you can make it. Tickets are free but limited availability https://www.eventbrite.co.uk/e/pybamm-hackathon-tickets-517334430207?keep_tld=1 |
Hackathon info
|
Description
I was wondering if it is possible to enable adding customized non-Bruggeman functions to define tortuosity or MacMullin number as a function of porosity?
Thanks.
Motivation
No response
Possible Implementation
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: