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.

Sign up for GitHub

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

Jump to bottom

Add MPI support for Periodic Boundaries and Cartesian Decompositions #77

Open
sslattery opened this issue Mar 4, 2019 · 3 comments · May be fixed by #263
Open

Add MPI support for Periodic Boundaries and Cartesian Decompositions #77

sslattery opened this issue Mar 4, 2019 · 3 comments · May be fixed by #263
Assignees
@sslattery @streeve @sjplimp
Labels
enhancement New feature or request

Comments

@sslattery
Copy link
Collaborator

Many applications in PIC and MD operate on a Cartesian Domain decomposition and often employ periodic boundary conditions. Our communication structures currently do not consider a Cartesian case and are instead more general. Because of this, we may miss opportunities for optimization. In addition, it makes support for periodic boundaries difficult.
@sslattery sslattery added the enhancement New feature or request label Mar 4, 2019
@sslattery
Copy link
Collaborator Author

Discussed this further with @streeve today - there is a clear implementation path forward in applications without this infrastructure. However, future analysis with ExaMiniMD to see if their Cartesian communication patterns and inline coordinate mapping is significantly more performant on Summit/Sierra

@sslattery sslattery mentioned this issue Oct 10, 2019
Closed
@sslattery
Copy link
Collaborator Author

After some further discussion with @streeve today it seems this would be best resolved with a Cabana/Cajita merger. This would allow for combined Cartesian structure with particles in the communication plan which naturally allows for partitioning and handling periodicity.

@streeve
Copy link
Member

streeve commented Jun 5, 2020

Noting this can now be done with Cajita as the last part of #197

@streeve streeve linked a pull request Jun 26, 2020 that will close this issue
Open
9 tasks
@streeve streeve mentioned this issue May 20, 2021
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sslattery sslattery

@streeve streeve

@sjplimp sjplimp

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Particle-grid periodic communication streeve/Cabana
Halo Owner Ranks sslattery/Cabana
3 participants
@sslattery @streeve @sjplimp