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.

Sign up for GitHub

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

Jump to bottom

Remove parameter use_fast_hanging_node_algorithm from MF #13634

Merged
merged 1 commit into from Apr 24, 2022
Merged

Remove parameter use_fast_hanging_node_algorithm from MF #13634

merged 1 commit into from Apr 24, 2022

Conversation

peterrum
Copy link
Member

closes #12780

@peterrum peterrum added this to the Release 9.4 milestone Apr 23, 2022
@drwells
Copy link
Member

drwells commented Apr 23, 2022

It looks like tests/matrix_free/mixed_mesh_hn_algorithm_01.cc uses this interface.

Should this be deprecated first, or is it an unused enough feature that we can remove it outright?

@kronbichler
Copy link
Member

@drwells it was only introduced a few months ago, so there is no need to guarantee backward compatibility (and indeed it was introduced in case we do not trust in some aspects during the transition).

@peterrum peterrum force-pushed the use_fast_hanging_node_algorithm_remove branch from e8ffa1a to c06a6af Compare April 24, 2022 06:36
@peterrum peterrum merged commit bfae48a into dealii:master Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drwells drwells drwells approved these changes

@kronbichler kronbichler kronbichler approved these changes

Assignees
No one assigned
Labels
Matrix-free ready to test
Projects
None yet
Milestone
Release 9.4
Development

Successfully merging this pull request may close these issues.

Reevaluate MF::AdditionalData::use_fast_hanging_node_algorithm
3 participants
@peterrum @drwells @kronbichler