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

Chemical reactions with FEniCS #278

Merged
merged 31 commits into from
Sep 21, 2022
Merged

Conversation

boris-martin
Copy link
Contributor

@boris-martin boris-martin commented Jun 16, 2022

Description of the case

Added a new tutorial for partitioned advection-diffusion-reaction of chemical species in a incompressible fluid.

  • Fluid participant solves the NS equations and sends its velocity to Chemical participant
  • Chemical participants computes diffusion, production and reaction of chemical species (A + B -> C with sources of A, B) as well as advection through the fluid, reading velocity from the Fluid participant.

Coupling is unidirectional, serial explicit.

Due to tight time stepping constraints on the fluid flow and high cost of the non-linear chemical reactions, different time scales are used: the fluid solver is subcycling.

This test case was made for showcasing Linear Cell Interpolation data mapping.

Todo-list

  • Writing README
  • add run.sh and clean.sh scripts
  • add some measurements: the chemical case keeps track of total quantities of A, B and C species over time (through FEniCS built-in support of integrals)
  • General cleaning, formatting, etc
  • Investigate better discretization choices (e.g. more accurate schemes, like mid-point rule for velocities, choose between 2nd order and 1st order elements for chemistry)

@uekerman
Copy link
Member

Nice 👍
Alternative names could be channel-chemistry or channel-transport-reaction. It is a channel flow with no-slip walls, right?
This would put the case close to the existing channel-transport.

@boris-martin
Copy link
Contributor Author

Yes it's a channel. I imagine channel-transport-reaction is the most appropriate since it describes what is coupled. On the other hand, no reference to chemistry at all doesn't seem right.

@uekerman
Copy link
Member

You could say that "chemistry" is already covered by "reaction"? 😁

@boris-martin boris-martin marked this pull request as ready for review July 19, 2022 12:31
@uekerman
Copy link
Member

Just saw right now: PR should point to develop, not master.

@boris-martin boris-martin changed the base branch from master to develop July 19, 2022 15:17
@precice-bot
Copy link
Collaborator

This pull request has been mentioned on preCICE Forum on Discourse. There might be relevant details there:

https://precice.discourse.group/t/iceberg-ahead-and-new-precice-release-v2-5/1133/1

boris-martin and others added 6 commits September 20, 2022 16:18
Co-authored-by: Ishaan Desai <ishaan.desai@ipvs.uni-stuttgart.de>
Co-authored-by: Ishaan Desai <ishaan.desai@ipvs.uni-stuttgart.de>
Co-authored-by: Ishaan Desai <ishaan.desai@ipvs.uni-stuttgart.de>
Co-authored-by: Ishaan Desai <ishaan.desai@ipvs.uni-stuttgart.de>
@IshaanDesai IshaanDesai merged commit 700c5e2 into precice:develop Sep 21, 2022
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

Successfully merging this pull request may close these issues.

4 participants