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

FLYonPIC v2.0 #4551

Conversation

BrianMarre
Copy link
Member

@BrianMarre BrianMarre commented May 4, 2023

Second version of FLYonPIC
now includes:

  • electronic collisional de-/excitation
  • spontaneous deexcitation
  • electronic collisional ionization
  • autonomous ionization
  • Stewart-Pyatt ionization potential depression(IPD)
  • pressure ionization based on the IPD implementation

as well as a reworked atomic rate solver with adaptive time stepping

still missing:

  • FLYonPIC input specification
  • compile test
  • user facing description

requires

@BrianMarre
Copy link
Member Author

BrianMarre commented May 4, 2023

@PrometheusPi and @steindev for now no review required unless you really want to, a detailed examination together with @psychocoderHPC is already planned

@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 2 times, most recently from 7efa915 to 8176990 Compare May 5, 2023 12:41
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 2 times, most recently from 0edcd5b to 319c2e2 Compare May 17, 2023 08:09
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 3 times, most recently from cd84064 to 95f2ced Compare May 31, 2023 12:57
Copy link
Member

@psychocoderHPC psychocoderHPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked mostly for division by zero and found some critical parts, please check where it can happen and if function parameter must be >0 to avoid a decision by zero please add it to the oxygen of the function that we do not forget about it.

@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 4 times, most recently from 027dda3 to ba163ba Compare June 9, 2023 14:44
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 2 times, most recently from c03665b to 028401c Compare June 12, 2023 18:44
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 3 times, most recently from 783462a to 57e0be8 Compare June 19, 2023 12:25
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch 8 times, most recently from 7c5d30d to 7591dd5 Compare June 26, 2023 08:18
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch from ff06725 to 372f595 Compare August 12, 2024 13:35
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch from 1f92595 to 28e8b98 Compare August 13, 2024 12:44
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch from 28e8b98 to d829da0 Compare August 13, 2024 12:46
@BrianMarre BrianMarre force-pushed the topic-refactorAtomicPhysics_v2.0 branch from f206239 to d8ed88a Compare August 13, 2024 13:02
- add traits to get, filterBy particle type
- add particle type tags for easy meta template programming
- add collection for attributes needed by atomicPhysics ions
- rename alias `atomicPhysics_` to `atomicPhysicsParticle`

Co-authored-by: BrianMarre <[email protected]>
@BrianMarre
Copy link
Member Author

I checked that 94f1aab still fulfills the atomicPhysicsCI test

@psychocoderHPC
Copy link
Member

@BrianMarre please add (seperate PR) the svg or source data of your big plot you added as png. The png can not be editied.

@psychocoderHPC psychocoderHPC merged commit 8581f76 into ComputationalRadiationPhysics:dev Aug 15, 2024
9 checks passed
@BrianMarre BrianMarre deleted the topic-refactorAtomicPhysics_v2.0 branch August 15, 2024 11:47
@BrianMarre BrianMarre restored the topic-refactorAtomicPhysics_v2.0 branch August 15, 2024 11:47
@psychocoderHPC psychocoderHPC added the changelog PR's marked with this label will be added to the changelog label Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog PR's marked with this label will be added to the changelog component: core in PIConGPU (core application)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants