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

Add ghost_mode shared facet to demos #2574

Closed
wants to merge 6 commits into from
Closed

Conversation

jorgensd
Copy link
Member

@jorgensd jorgensd commented Mar 7, 2023

Resolves: #2556

@jorgensd jorgensd added bug Something isn't working demo New demo or demo enhancements/improvements labels Mar 7, 2023
@jorgensd jorgensd requested a review from garth-wells March 7, 2023 14:28
@garth-wells
Copy link
Member

To fix the underlying bug I think we need a way to determine if we have the required ghost cells when executing a kernel over interior facets.

@jorgensd
Copy link
Member Author

jorgensd commented Mar 8, 2023

@garth-wells Added an error message that is hit if you run demo_axis.py pre this PR on two processes

@garth-wells
Copy link
Member

Open for some time, so closing.

The runtime check added in this PR introduces non-trivial computation, so a different approach is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working demo New demo or demo enhancements/improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EM demos wrong because Gmsh-created meshes are by default not ghosted
3 participants