-
Notifications
You must be signed in to change notification settings - Fork 50
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
Periodic index spaces #765
base: master
Are you sure you want to change the base?
Conversation
@JStewart28 I still need to add tests, but wanted to push this and make sure this was in the right direction |
161c00e
to
2775e18
Compare
|
subset of sharedIndexSpace on periodic boundaries
2775e18
to
0c7aea3
Compare
I found a mistake here - initially I wrote returned the |
What I mean is for the same halo distance - in this case, 2 cells - and on a periodic boundary, in a |
Add local grid index space generation for periodic boundaries that matches the current boundary index space concept. This completes all current boundary types:
boundaryIndexSpace
periodicIndexSpace
sharedIndexSpace
TODO: add tests to avoid issues similar to #615 due to assumed "equivalent" boundary spaces
Closes #762