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

Do not add locally owned dofs to ghost set during construction #14024

Merged
merged 2 commits into from Jun 24, 2022

Conversation

kronbichler
Copy link
Member

Similarly to what is done in

for (const auto local_dof_index : local_dof_indices)
if (!owned_level_dofs.is_element(local_dof_index))
ghosted_level_dofs.push_back(local_dof_index);
we should only add the dofs that end up being ghosts, to reduce the impact of the n log(n) sort costs and addition to the index set.

@kronbichler kronbichler changed the title Do not locally owned dofs to ghost set during construction Do not add locally owned dofs to ghost set during construction Jun 20, 2022
@kronbichler kronbichler force-pushed the setup_indices_global_coarsening branch from 66816f9 to f1ca0f9 Compare June 20, 2022 12:43
Copy link
Member

@drwells drwells left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor thing.

Co-authored-by: David Wells <drwells@email.unc.edu>
@tjhei tjhei merged commit 204fa04 into dealii:master Jun 24, 2022
@kronbichler kronbichler deleted the setup_indices_global_coarsening branch September 5, 2022 12:10
mkghadban pushed a commit to OpenFCST/dealii that referenced this pull request Sep 8, 2022
…_coarsening

Do not add locally owned dofs to ghost set during construction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants