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.

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

PABLO: fix evaluation of first layer of ghost on periodic boundaries #108

Merged
merged 9 commits into from
Jan 4, 2021

Conversation

andrea-iob
Copy link
Member

Ghost octants there are edge or node neighbours of internal octants were not detected properly on periodic boundaries.

Shuold fix #98.

@andrea-iob andrea-iob force-pushed the PABLO.fix.periodic.neighbours branch 2 times, most recently from 7bae643 to ad6e9a8 Compare November 6, 2020 17:53
@andrea-iob andrea-iob force-pushed the PABLO.fix.periodic.neighbours branch 2 times, most recently from 8b1f876 to 5261a1a Compare November 14, 2020 15:56
@andrea-iob
Copy link
Member Author

Faces associated with periodic edges/nodes are no longer marked as processor boundaries.

@edoardolombardi
Copy link
Member

I re-tested the pull and my tests worked well.

@andrea-iob andrea-iob force-pushed the PABLO.fix.periodic.neighbours branch from 5261a1a to f8b5ebb Compare January 4, 2021 10:36
@andrea-iob andrea-iob merged commit f8b5ebb into master Jan 4, 2021
@andrea-iob andrea-iob deleted the PABLO.fix.periodic.neighbours branch January 4, 2021 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PABLO: findNeighbours missing elements with periodic boundary conditions and MPI
2 participants