-
Notifications
You must be signed in to change notification settings - Fork 59
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
Use the eigenvalue estimation of PreconditionRelaxation #1064
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow I love this, this is so much cleaner and easier to read now.
@lpsaavedra Could you add a changelog entry?
I guess we need to drop deal.II 9.5 CI? |
I will check the deal.II version as we did here: https://github.com/lethe-cfd/lethe/blob/82095bab9286844d9f1cec0dab67cf3575c1791a/source/solvers/mf_navier_stokes.cc#L651 |
Description of the problem
The eigenvalue estimation for the smoother of the multigrid preconditioners was done through a
PreconditionChebyshev
.Description of the solution
Due to recent changes in deal.II (dealii/dealii#16742), now we are able to do it using the
PreconditionRelaxation
directly. This allows us to remove the estimate omega function since it is done internally by the smoother if the relaxation parameter is set to 1. This allows also to remove thedegree
parameter that was only used in the case of Chebyshev.