-
Notifications
You must be signed in to change notification settings - Fork 172
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
warpx.roundrobin_sfc: A runtime parameter to control distribution mapping #4909
base: development
Are you sure you want to change the base?
Conversation
99d9517
to
6f15ec5
Compare
…ping The default is false. If it's true, AMReX's RRSFS strategy will be used to override the default SFC strategy used by amrex::AmrCore. The motivation for this is that this might mitigate the load imbalance issue during initialization by avoiding putting neighboring boxes on the same process.
6f15ec5
to
275f45a
Compare
@@ -1157,6 +1159,8 @@ WarpX::ReadParameters () | |||
maxLevel() == 0 || !do_current_centering, | |||
"Finite-order centering of currents is not implemented with mesh refinement" | |||
); | |||
|
|||
pp_warpx.query("roundrobin_sfc", roundrobin_sfc); |
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.
This might need a user-facing entry in parameters.rst
.
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.
done
@@ -219,6 +219,8 @@ bool WarpX::do_device_synchronize = true; | |||
bool WarpX::do_device_synchronize = false; | |||
#endif | |||
|
|||
bool WarpX::roundrobin_sfc = false; |
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.
We need to add control for this from PICMI, too.
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.
done
Looks good. Would his affect the runtime, slowing the MPI communication since neighboring boxes will be far away from each other? |
The default is false. If it's true, AMReX's RRSFC strategy will be used to override the default SFC strategy used by amrex::AmrCore. The motivation for this is that this might mitigate the load imbalance issue during initialization by avoiding putting neighboring boxes on the same process.