-
Notifications
You must be signed in to change notification settings - Fork 218
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
FLYonPIC v2.0 #4551
Conversation
@PrometheusPi and @steindev for now no review required unless you really want to, a detailed examination together with @psychocoderHPC is already planned |
7efa915
to
8176990
Compare
include/picongpu/particles/atomicPhysics2/kernel/SpawnIonizationMacroElectrons.kernel
Outdated
Show resolved
Hide resolved
0edcd5b
to
319c2e2
Compare
cd84064
to
95f2ced
Compare
include/picongpu/particles/atomicPhysics2/rateCalculation/BoundBoundTransitionRates.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this 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.
include/picongpu/particles/atomicPhysics2/initElectrons/InitElectronAs.hpp
Outdated
Show resolved
Hide resolved
include/picongpu/particles/atomicPhysics2/initElectrons/InitElectronAs.hpp
Outdated
Show resolved
Hide resolved
include/picongpu/particles/atomicPhysics2/initElectrons/InitElectronAs.hpp
Outdated
Show resolved
Hide resolved
include/picongpu/particles/atomicPhysics2/initElectrons/InitElectronAs.hpp
Outdated
Show resolved
Hide resolved
include/picongpu/particles/atomicPhysics2/initElectrons/InitElectronAs.hpp
Outdated
Show resolved
Hide resolved
027dda3
to
ba163ba
Compare
c03665b
to
028401c
Compare
include/picongpu/particles/atomicPhysics2/atomicData/BoundFreeTransitionData.hpp
Outdated
Show resolved
Hide resolved
783462a
to
57e0be8
Compare
7c5d30d
to
7591dd5
Compare
Seperate control flow loop from sub stage calls. Co-authored-by: BrianMarre <[email protected]>
ff06725
to
372f595
Compare
1f92595
to
28e8b98
Compare
28e8b98
to
d829da0
Compare
f206239
to
d8ed88a
Compare
- 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]>
I checked that 94f1aab still fulfills the atomicPhysicsCI test |
@BrianMarre please add (seperate PR) the svg or source data of your big plot you added as png. The png can not be editied. |
8581f76
into
ComputationalRadiationPhysics:dev
Second version of FLYonPIC
now includes:
as well as a reworked atomic rate solver with adaptive time stepping
still missing:
requires