Skip to content

[WIP] Add support for electron impact ionization in binary collisions (DSMC module) #5524

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

Closed
wants to merge 31 commits into from

Conversation

oshapoval
Copy link
Contributor

@oshapoval oshapoval commented Dec 20, 2024

*Description edited by @RemiLehe *

This adds support for electron impact ionization, within the DSMC module.

Note: could we do binary charge exchange in the same way?

TODO:

  • Use the proper energy cost: store the energy cost in the SplitAndScatter functor
  • Avoid creating a particle of the neutral species
  • Add support for excitation in DSMC (right now, it prevents us from running the capacitive discharge test)
  • Test on GPU
  • Add a dedicated CI test for DSMC ionization. Note: for now, setting very high tolerances on purpose. This needs to be modified once all features are implemented in the PR.

@RemiLehe RemiLehe changed the title Added ionization DSMC CI test [WIP] Add support for electron impact ionization in binary collisions (DSMC module) Dec 20, 2024
@roelof-groenewald
Copy link
Member

This PR has been incorporated into #5654, so I'll close this one.

RemiLehe added a commit that referenced this pull request Mar 4, 2025
This PR extends the work already done by @RemiLehe and @oshapoval in
#5524.

Some changes were made to how the ionization process is initialized, for
example, the user must now specify the `target_species` (i.e., the
species which undergoes ionization) as well as the `product_species`.
The product species can include the colliding species (for example
electron + neutral -> 2 x electron + ion), but does not have to (for
example H$^+$ + D -> H$^+$ + D$^+$ + electron).

The test created by @archermarx is now passing (at least early on):

![image](https://github.com/user-attachments/assets/344476b8-bc63-4395-92c1-d795183048b9)

Todo:

- [x] test implementation
- [x] fix scattering process to conserve momentum
- [x] clean up code

---------

Signed-off-by: roelof-groenewald <[email protected]>
Co-authored-by: Remi Lehe <[email protected]>
Co-authored-by: Olga Shapoval <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

3 participants