JStewart28

1. I had to take another look at our code to remind myself what we are doing. Currently, `boundaryIndexSpace` returns the boundary index space (or an empty index space if...

> Looks close - can you remove the namespace commits (will make the update once we merge that much easier)? Will be easier to review Yes. Just reverted them.

@streeve all checks have passed. Do you have any more comments or suggestions?

`sharedIndexSpace` and `boundaryIndexSpace` look good. Question about `periodicIndexSpace`: If the offset is in a shared index space along a periodic boundary, `boundaryIndexSpace` returns, for example, for i min/max and j...

> > `sharedIndexSpace` and `boundaryIndexSpace` look good. Question about `periodicIndexSpace`: If the offset is in a shared index space along a periodic boundary, `boundaryIndexSpace` returns, for example, for i min/max...

Still trying to work this out. In Beatnik, we have the following code to give us the periodic index space for 2D partitioning: ``` template Cabana::Grid::IndexSpace periodicIndexSpace(LocalGridType local_grid, DecompositionType dt,...

Hi @kursatyurt, I am a developer of [Beatnik](https://github.com/CUP-ECS/beatnik/), which @streeve referenced, and I am currently implementing the FMM using hierarchal Cabana::Grid sparse meshes for research in optimizing its communication patterns....