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

P4est migration #3

Merged
merged 103 commits into from
Mar 15, 2023
Merged

P4est migration #3

merged 103 commits into from
Mar 15, 2023

Conversation

JordiManyer
Copy link
Member

This pull request is meant to incorporate all code migrations from GridapP4est.jl to GridapSolvers.jl. The planned migrations are listed in issue #1 .

@JordiManyer JordiManyer self-assigned this Oct 5, 2022
@JordiManyer JordiManyer marked this pull request as ready for review February 28, 2023 06:18
@JordiManyer
Copy link
Member Author

The current CI problems will be solved as part of #7

When deciding whether a facet belongs to the
patch boundary, the serial version was not
correctly treating interface facets (which
are neither marked as boundary nor have any
neighboring cell not belonging to the patch).
Distributed Patch based smoothers
The PatchFESpace does not have ghosts, which means that we do not
have to worry about the ghost structure of the matrix vs fespace.
We can thus remove some of the auxiliary vectors in the solver.
@amartinhuertas amartinhuertas merged commit a9edd1a into main Mar 15, 2023
@JordiManyer JordiManyer deleted the p4est-migration branch March 15, 2023 22:13
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.

2 participants