Skip to content

adjust DELTA_T_SI depending on SIMDIM to improve the CFL condition#774

Merged
ax3l merged 2 commits into
ComputationalRadiationPhysics:devfrom
PrometheusPi:improve_KHI_CFL_condition
Mar 24, 2015
Merged

adjust DELTA_T_SI depending on SIMDIM to improve the CFL condition#774
ax3l merged 2 commits into
ComputationalRadiationPhysics:devfrom
PrometheusPi:improve_KHI_CFL_condition

Conversation

@PrometheusPi
Copy link
Copy Markdown
Member

This pull request adjusts DELTA_T_SI to improve the Courant-Friedrichs-Lewy condition depending on the dimension of the simulation SIMDIM. It closes issue #763.

For 2D we now get:

PIConGPUVerbose PHYSICS(1) | Courant c*dt <= 1.00204 ? 1

For 3D we now get:

PIConGPUVerbose PHYSICS(1) | Courant c*dt <= 1.00556 ? 1

@PrometheusPi PrometheusPi force-pushed the improve_KHI_CFL_condition branch from b4bda17 to 21f78f8 Compare March 24, 2015 16:30
@PrometheusPi PrometheusPi added the component: examples PIConGPU or PMacc examples label Mar 24, 2015
@PrometheusPi PrometheusPi added this to the Open Beta milestone Mar 24, 2015
@ax3l ax3l self-assigned this Mar 24, 2015
@ax3l
Copy link
Copy Markdown
Member

ax3l commented Mar 24, 2015

As I proposed earlier, I would like to use the same time step for both 2D and 3D. That simplifies a lot of work, e.g., both simulations have the same time scale regarding to steps...

The "harm"/not-super-ideal CFL in 2D is ok for that case, it's an example that should be easy to read and to compare with, it is not a PhD thesis ;)

@ax3l ax3l added documentation regarding documentation or wiki discussions wait-4-author/WIP labels Mar 24, 2015
@PrometheusPi
Copy link
Copy Markdown
Member Author

I adjusted the set up (I missread your comment.):

now we get for 2D:

PIConGPUVerbose PHYSICS(1) | Courant c*dt <= 1.23155 ? 1

@ax3l
Copy link
Copy Markdown
Member

ax3l commented Mar 24, 2015

great! 👍 thank you for the fast turn-around!

ax3l added a commit that referenced this pull request Mar 24, 2015
adjust DELTA_T_SI depending on SIMDIM to improve the CFL condition
@ax3l ax3l merged commit de70a7c into ComputationalRadiationPhysics:dev Mar 24, 2015
@PrometheusPi PrometheusPi deleted the improve_KHI_CFL_condition branch March 24, 2015 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: examples PIConGPU or PMacc examples documentation regarding documentation or wiki discussions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants