-
Notifications
You must be signed in to change notification settings - Fork 17
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
Separate grid for Laser #1106
Separate grid for Laser #1106
Conversation
@@ -1063,9 +1258,9 @@ MultiLaser::InitLaserSlice (const amrex::Geometry& geom, const int islice, const | |||
bx, | |||
[=] AMREX_GPU_DEVICE(int i, int j, int k) | |||
{ | |||
const amrex::Real z = plo[2] + (islice+0.5_rt)*dx_arr[2] - z0; |
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 changed how x, y, z is calculated here because the old way was wrong if the laser zeta axis was smaller compared to the full domain.
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.
Thanks for this PR, looks great! See comments below. This is non-backward-compatible, right? @huixingjian could you give it a try for your simulations?
This would be non-backwards compatible only because of the changes to |
In this PR, the grid of the laser is decoupled from the field's grid. To facilitate this, this PR...
Test of getting chi outside the field:
Chi on the laser grid (changed diagnostic) with a slightly wrong box to transition between chi inside and outside the fields grid.
Chi on the laser grid with the correct box
Chi on the fields grid
const
isconst
)